can we manually decide that when a step will active

I tried to set a step active by using setActive() method within a block other than onCreate. But this is not working.
I am unable to show the next step active manually.
Is there a way to set active a step manually?
What i want is Set a step active at run time on some action.
This is the code where i want to change the active step within click listener.
sequenceLayout.start() works only in OnCreate method
step1.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View view) {
Toast.makeText(Main2Activity.this, "clicked", Toast.LENGTH_SHORT).show();
            if (sequenceStep2.isActive()){
                Toast.makeText(Main2Activity.this, "step 2 Active", Toast.LENGTH_SHORT).show();
                sequenceStep2.setActive(false);
                sequenceStep1.setActive(true);
                sequenceLayout.start();
            }
            if (sequenceStep3.isActive()){
                Toast.makeText(Main2Activity.this, "step 3 Active", Toast.LENGTH_SHORT).show();
                sequenceStep3.setActive(false);
                sequenceStep1.setActive(true);
                sequenceLayout.start();
            }
            if (sequenceStep4.isActive()){
                Toast.makeText(Main2Activity.this, "step 4 Active", Toast.LENGTH_SHORT).show();
                sequenceStep4.setActive(false);
                sequenceStep1.setActive(true);
                sequenceLayout.start();
            }

            if (sequenceStep5.isActive()){
                Toast.makeText(Main2Activity.this, "step 5 Active", Toast.LENGTH_SHORT).show();
                sequenceStep5.setActive(false);
                sequenceStep1.setActive(true);
                sequenceLayout.start();
            }
        }
    });

Hey, you're using sequenceStep1.setActive(true) and sequenceLayout.start() correctly actually. What  result do you see on screen when the click listener is triggered?

I want to see the results like If I active step1 statically then i want that step2 will be active when click listener is triggered.

Ok, so what does it do currently for you?

it shows any step active statically but doesn't reply on dynamic cases.
Please provide me a solution by which i will be able to change the active
step.
On Tue, 4 Feb 2020 at 17:58, Sascha Huth<EMAIL_ADDRESS>wrote:

I want to see the results like If I active step1 statically then i want
that step2 will be active when click listener is triggered.
Ok, so what does it do currently for you?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/transferwise/sequence-layout/issues/14?email_source=notifications&email_token=AOHLGAP4FMIQJOJSABFATS3RBFNQJA5CNFSM4KEUDENKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKXOSHI#issuecomment-581888285,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOHLGALVDM3PSEIOP3RSYP3RBFNQJANCNFSM4KEUDENA
.


Allow tmux HEAD install

When built from source tmux set versions like this.
ϟ tmux -V
tmux next-3.3

Thanks for the PR and the fix for the tmux develop version!

Thanks for the PR and the fix for the tmux develop version!


Update notepadplusplus-np.json

Add 'themes' folder as a persist to make themes available no matter the version.

Reserved for portable installations, closing.


Upsampling layer with bilinear interpolation fails in Torch 2.1.0

Hi, I installed the latest Torch version (2.1.0) and I noticed the Upsampling layer in Keras Core fails with the following error trace:
Traceback (most recent call last):
  File "/Users/tarban01/Desktop/SME-Hero/test.py", line 5, in <module>
    x = upsample_layer(torch.zeros((1, 1080, 1920, 3)))
  File "/Users/tarban01/Library/Python/3.9/lib/python/site-packages/keras_core/src/utils/traceback_utils.py", line 123, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/Users/tarban01/Library/Python/3.9/lib/python/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/Users/tarban01/Library/Python/3.9/lib/python/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/Users/tarban01/Library/Python/3.9/lib/python/site-packages/torchvision/transforms/functional.py", line 492, in resize
    return F_t.resize(img, size=output_size, interpolation=interpolation.value, antialias=antialias)
  File "/Users/tarban01/Library/Python/3.9/lib/python/site-packages/torchvision/transforms/_functional_tensor.py", line 467, in resize
    img = interpolate(img, size=size, mode=interpolation, align_corners=align_corners, antialias=antialias)
  File "/Users/tarban01/Library/Python/3.9/lib/python/site-packages/torch/nn/functional.py", line 3924, in interpolate
    raise TypeError(
TypeError: Exception encountered when calling UpSampling2D.call().

expected size to be one of int or Tuple[int] or Tuple[int, int] or Tuple[int, int, int], but got size with types [<class 'numpy.int64'>, <class 'numpy.int64'>]

Code to reproduce:
import torch
from keras_core import layers

upsample_layer = layers.UpSampling2D(size=(2, 2), interpolation="bilinear")
x = upsample_layer(torch.zeros((1, 1080, 1920, 3)))

torch 2.10 doesn't accept tuple of numpy ints and want python ints.  This is already fixed in Master branch.  If you install from github via -
pip uninstall -y keras
git clone https://github.com/keras-team/keras.git
pip install namex
python pip_build.py --install

Then use it as keras (its no longer keras-core in master).  We will release keras-nightly soon so you don't have to go through this build from github once the nightly is released.


Fix missing declarations of inet_to family for POSIX_NOPOLL

Fix #271

Could you please @roncapat solve the DCO issue?

Yep, in 30 minutes. I'm on mobile.


Provision for Optional Cancellation Token Parameter for KeysetPaginateAsync and OffsetPaginateAsync

I think there should be provision for passing down the optional cancellation token argument to these pagination methods most especially the KeysetPaginateAsync just like in EF Core linq extension methods.
Use case
For example while implementing a feature like infinite scroll on mobile or web apps, I want cancel request as the user is scrolling down fast or has moved away from the screen.
Thanks
Regards

Good idea. I'll add this as up-for-grabs for now. If no one takes it I'll probably do it at some point.


scale position.z by fudgeFactor only

While you're actual code is correct:
float zToDivideBy = 1.0 + position.z * u_fudgeFactor;
Your example is incorrect:
float zToDivideBy = (1.0 + position.z) * u_fudgeFactor;
We should scale position.z by u_fudgeFactor then add 1 to translate zToDivideBy from clipspace -1 1 to 0 2

doh! thanks!


[FontFamily] Custom FontFamily not work when publish nativeaot

When I use  fontcollection to add new custom fontfamily, It can work normal. But when I publish fot nativeaot, It can't take effect.What do I need to do for it?
To Reproduce
Steps to reproduce the behavior:

Add FontCollection
Debug or release
Publish NativeAot

OS: [ Windows, Mac, Linux]
Version [V11-Preview 6]

It's impossible to tell where your problem is.
NativeAOT by itself won't affect your embedded resources.
But if, for example, you have your resource/font in the separated assembly, which isn't directly referenced in the code, it might be removed by the compiler.
Can you provide more information?

this is my avalonia project, I add a font assembly and use ConfigureFonts add my custom fontfamily,  all the font files are AvaloniaResource, I add it for my desktop app, if i debug and release run this app, it work ok. But when i publish fot NativeAOT, the font is not work.

I add the font assembly in the rd.xml.

this is my app run with debug, the font is my custom font.

this is my app which published for NativeAOT, font is not the custom font.

You can get sourcecode from https://github.com/AvaloniaDeveloperOrg/AntDesign.Avalonia

I add this fontfamily for windows style and usercontrol style .

And  I also add it for window,but not invalid。

When I use this setting, it will crash


401 Error when using a OneDrive direct download link for license file

https://github.com/microsoft/navcontainerhelper/blob/4bb91201343df88fa11b9e3d8e59ba379afd4ee8/ContainerHandling/New-NavImage.ps1#L468
The line mentioned leads to a 401 error when using a one drive direct download link to provide a license file to the AL-Go pipeline.
I can reproduce the error in a local powershell session.
The same download URL works with Invoke-WebRequest though.
Is there a specific reason to use Download-File ?
I generated the link by adding &download=1 to the download link created in OneDrive.
See here:
https://techcommunity.microsoft.com/t5/onedrive-for-business/get-direct-download-link-of-a-file-in-onedrive-for-business/m-p/149766

Thanks, that would help very much :)

Any chance that you could place a file on a onedrive and send me a direct download URL in an email for that?
I cannot get the download=1 url to work on my machine at all.
Thanks

@StefanMaron - any chance you can help out here?

Oh, I have sent a link via mail. Did you not get it?

it was in my Junk email folder - sorry

I think that will work :) Thanks for implementing a solution, then I can just use my onedrive for those kind of static files and do not need to look into some other ways of storing them

fix Included in the latest prerelease

Shipped in 3.0.8

Having the same issue still when passing in a One Drive link to the code signing certificate
https://github.com/microsoft/navcontainerhelper/blob/74bd262fc0af2296c571c3c6428b9aa1b48f5ec3/AppHandling/Sign-NavContainerApp.ps1#L52


Improved coordinates

Made coordinate properties immutable
Made coordinate properties and methods public
Added convenience initializers
Added HorizontalCoordinates.northBasedAzimuth

Thanks. But did you also see #35 ?


files on OTG drive not scrobbled

I'm using Music Folder Player with the "Emulate Android Player" option as a source for the scrobbles. Songs  played from the phone's internal SD are detected by SLS without problems. However, music on a hard drive that I mount via OTG cable is played back fine, but does not get scrobbled.

Hi @christianl74, I am not sure I can recreate that environment to test, but I will do my best to help you!

Hi @christianl74 as long as the tracks are being played and Music Folder Player is broadcasting the track information, it should work.
It's probably something to do with Music Folder Player not broadcasting the information to SLS when OTG is enabled.

Hi @HumbleBeeBumbleBee, thanks, you might be right. I recently installed Lyrics Finder, and I see different behavior based on whether the song comes from internal storage ("Touch to see lyrics of ") vs. when it comes from the hard drive ("Touch to see lyrics of <01_full_file_name.mp3>"). Music Folder Player also seems to display mp3 tags only for local file, so maybe that's the problem...

I guess you could try to contact Music Folder Player about this<EMAIL_ADDRESS>I already did. I will report back to you their response and likely close this issue when they respond.
There isn't much we can do on SLS' end. It's for ZorillaSoft to decide what to do about this issue/bug.

Quote from Zorillasoft.

Thank you for the feedback.

The player gets the track information from the Android. It doesn't extract this information directly from the files.
Android only indexes local files. Therefore Android and the player don't have detailed information about the track.

I'm sorry, but currently I can't fix that without high efforts.

I am going to close this issue, but if it is fixed at some point could you leave a comment on the closed issue or send me a quick email<EMAIL_ADDRESS>?

Hi @HumbleBeeBumbleBee, I wrote to them as well, and their explanation actually helped.There are a few apps that let you do a media scan of the mounted directories - media.Re.Scan (https://play.google.com/store/apps/details?id=com.house.noranuko.mediarescanner) worked fine for me. That fixed my problems with scrobbling, lyrics, and ID3 tags. I guess it's OK as long as the contents of the external mounts don't change too often. Looks like it survives unmounting and reboots.
Thanks for your help!


table memory overflow

Hi everyone:
When I use table controls,and Create and delete repeatedly. The memory consumption of monitoring display is increasing（lv_sim_eclipse_sdl-release-v8.0). hope your answer
The code is as follows
static lv_obj_t * tableMain=NULL;

void creat_obj_1()
{
    if(tableMain!=NULL)
        return;
    tableMain = lv_table_create(lv_scr_act());
}

void destroy_obj_1()
{
    if(tableMain!=NULL)
    {
        lv_obj_del(tableMain);
        tableMain=NULL;
    }
}

static void btn_event_handle(lv_event_t *e)
{
    unsigned int i=0;
    lv_obj_t * btn = lv_event_get_target(e);
    if(btn==btn_mem_test3)
    {
        for(i=0; i<1000; i++)
        {
            creat_obj_1();
            destroy_obj_1();
        }
    }
}

I don't see any issue on my side.
Please attache a log where you can see the memory leak.

thinks, This problem has been solved through this web methodhttps://github.com/lvgl/lvgl/issues/2967


opt let keywords list same with legacy planner

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 50.46 seconds
stream load tsv:          546 seconds loaded<PHONE_NUMBER>9 Bytes, about 130 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>3 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 48.59 seconds
stream load tsv:          550 seconds loaded<PHONE_NUMBER>9 Bytes, about 129 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.9 seconds inserted 10000000 Rows, about 346K ops/s
storage size:<PHONE_NUMBER>9 Bytes


Users needed to find out if they are eligible to apply but were confused by the statement

Some users thought the first sentence implied eligibility if over 60.
Two of those we spoke two read “If you’re over 60 you could get a free bus pass” and then proceeded to start confident that they would be eligible.

Both of these users were 63 years old.

Fixed on live prototype:

@lizziebruce - I reread it 3 times until I spotted the difference in the text :D Do you think this will make the difference because it still says '60 or over', which is what drives user to proceed further without checking. Isn't it better if we change the text to '65 or over'?

@irinapencheva all the government and LA sites say 60 and over
https://www.gov.uk/apply-for-elderly-person-bus-pass

@joanneSouthern - agreed, but if this prototype is for the English scheme for which eligibility is the state pension age, why not change it and make it less confusing for the user (as was observed in the last round of UR)?

@irinapencheva I don't think 65 and over is correct. People can get an older person's bus pass from age 60 depending on when they were born. The female state pension age remember, which changed a few times.
The UR issue was that the word "could" made people think it was a likelihood rather than a possibility. Changing to "may be able to" though less concise is more clear.

@lizziebruce - OK, thanks for pointing out. Agree to changing to 'may be able to'


Adding support for CSP script-src safe inline, for SSR state transfer

VueJS is fully CSP compliant (script-wise) when using runtime-only builds [1].
Nuxt appears to follow suit, with the single exception that a strict locked-down CSP policy should not allow inline scripts. SSR state transfer relies on inline scripts in a non-avoidable way. The CSP spec allows for inline scripts to be whitelisted by nonce or checksum.
There appears to be an issue open on this topic already: #2422
My solution allows a user to opt in to CSP script compliance by setting the options.render.csp variable to either true or an object containing the key hashAlgorithm. The hashAlgorithm may be sha256, sha384 or sha512 per the CSP spec [2] and these three hash algorithms should also be supported in most builds of Node (it is technically dependent on the version of OpenSSL on the platform, but these are three very common algorithms [3]). The default value is sha256 if the user just sets the option to true.
[1] https://vuejs.org/v2/guide/installation.html#CSP-environments
[2] https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy/script-src
[3] https://nodejs.org/api/crypto.html#crypto_class_hash

Great news! @Atinux is this option available for 1.4 or will it be shipped in v2.?

It’s already here with 1.4
On Mon 23 Apr 2018 at 03:41, Dominic Garms<EMAIL_ADDRESS>wrote:

Great news! @Atinux https://github.com/Atinux is this option available
for 1.4 or will it be shipped in v2.?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nuxt/nuxt.js/pull/2549#issuecomment-383431427, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA3OFEjQPB2WTXVfrl7-Rdddg9MgwYUEks5trTEwgaJpZM4RY4Ca
.

great to hear, I think its just missing in the docs: https://nuxtjs.org/api/configuration-render


GRM: cronjob unknown conversion

How to categorize this issue?

/kind bug
What happened:
Cronjobs switched from apiVersion: batch/v1beta1 to apiVersion: batch/v1 with k8s version 1.21.
For a k8s/shoot cluster with version 1.20 the managed resource fails with
    message: 'Could not apply all new resources: 1 error occurred: error during apply
      of object "batch/v1beta1/CronJob/kube-system/hubble-generate-certs": converting
      (v1beta1.CronJob) to (v1.CronJob): unknown conversion'

when the follwing cronjob is trying to be created.
# Source: cilium/charts/hubble-relay/templates/cronjob.yaml
apiVersion: batch/v1beta1
kind: CronJob
metadata:
  name: hubble-generate-certs
  namespace: kube-system
  labels:
    k8s-app: hubble-generate-certs
    gardener.cloud/role: system-component
spec:
  schedule: "0 0 1 */4 *"
  concurrencyPolicy: Forbid
  jobTemplate:
    spec:
      template:
        metadata:
          labels:
            k8s-app: hubble-generate-certs
        spec:
          containers:
            - name: certgen
              image: quay.io/cilium/certgen:v0.1.5
              imagePullPolicy: IfNotPresent
              command:
                - "/usr/bin/cilium-certgen"
              # Because this is executed as a job, we pass the values as command
              # line args instead of via config map. This allows users to inspect
              # the values used in past runs by inspecting the completed pod.
              args:
                - "--cilium-namespace=kube-system"
                - "--hubble-ca-generate"
                - "--hubble-ca-reuse-secret"
                - "--hubble-server-cert-generate"
                - "--hubble-server-cert-common-name=*.default.hubble-grpc.cilium.io"
                - "--hubble-server-cert-validity-duration=94608000s"
                - "--hubble-relay-client-cert-generate"
                - "--hubble-relay-client-cert-validity-duration=94608000s"
          hostNetwork: true
          serviceAccount: hubble-generate-certs
          serviceAccountName: hubble-generate-certs
          restartPolicy: OnFailure
      ttlSecondsAfterFinished: 1800

What you expected to happen:
That the managed resource with the cronjob can be successfully applied.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Gardener version: 1.39
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
Others:

Yeah, conversion code is not exported to k8s.io/api, so we can't do client-side conversions between different API versions.
However, I don't get where this problematic conversion is done. This conversion here, should only convert between structured and unstructured, but in the same API version:
https://github.com/gardener/gardener/blob/e170b6d003bab4b74ac856d4c9c59fa8abe950f8/pkg/controllerutils/update.go#L62-L65

/assign


Upgrade whatwg-url dependency

Is your feature request related to a problem? Please describe.
Whenever I run a fern command on my local system, I get the following message in my terminal.

fern check
(node:96209) [DEP0040] DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.
(Use node --trace-deprecation ... to show where the warning was created)

Describe the solution you'd like
I'd like this to go away.
Describe alternatives you've considered
I suspect it's related to using the newest version of Node.js. v21.1.0
Additional context
The trace I see shows that the deprecation warning originates from the whatwg-url module, specifically from its usage of the punycode module.
The warning is triggered by code in the whatwg-url module, as indicated by the path whatwg-url/lib/url-state-machine.js and whatwg-url/lib/URL-impl.js.
The specific lines in the fern-api bundle where this occurs are mentioned (bundle.cjs:369261 and bundle.cjs:370331).
[ACTION]
Update Dependencies: Check if there is an updated version of whatwg-url that doesn't use the deprecated punycode module. Updating these dependencies might resolve the warning in the CLI.

This is related to two transitive dependencies (that we cannot remove) relying on node-fetch v2. For now, the best thing would be to not use node v21 to get rid of the error message. Thats what others are also advising: https://github.com/forcedotcom/cli/issues/2535
│
├─ @redocly/openapi-core@npm:1.4.0
│  └─ node-fetch@npm:2.6.9 [79d8e] (via npm:^2.6.1 [79d8e])
│
└─ swagger2openapi@npm:7.0.8
   └─ node-fetch@npm:2.6.9 [79d8e] (via npm:^2.6.1 [79d8e])

Downgraded from Node v21.1 to v20.9 and the error message went away.


finished the ape commands section documentation

also optimized some of the docs to be structured according to the requirements of sphinx.

awesome! thx


should add prefix on metrics

Hi,
...
Also maybe add some unique prefix to the identifier exported by prometheus - something like fbx_system_fan_rpm instead of system_fan ?

Originally posted by @ottobaer in https://github.com/shackerd/freebox-exporter-rs/issues/8#issuecomment-2258637665

If I add prefix to system_fan I must add this prefix also to the rest of the metrics for consistency purposes.
If I'm right, system prefix violates good practices or may be confused with others?

Hi,
Yes I meant to say add a prefix for all the metrics exported.
https://prometheus.io/docs/practices/naming/

..should have a (single-word) application prefix relevant to the domain the metric belongs to. The prefix is sometimes referred to as namespace by client libraries. For metrics specific to an application, the prefix is usually the application name itself. Sometimes, however, metrics are more generic, like standardized metrics exported by client libraries. Examples:

    prometheus_notifications_total (specific to the Prometheus server)
    process_cpu_seconds_total (exported by many client libraries)
    http_request_duration_seconds (for all HTTP requests)

Sounds legit, I will add a configurable prefix in config.toml then.


[Compose Extension] Cannot support compose action for LLM powered app

Type of issue
Typo
Feedback
We have developed an app powered by an LLM and want to integrate it as a plugin in Microsoft Copilot. While it works with the command box, we're facing a limitation with the compose action. Currently, every key press in the search box triggers an invoke call due to a debouncing mechanism. Since our app is not search-based but instead processes user queries using LLMs, which takes time, we want to avoid making an API call with every key press. Is there a way to trigger the API call only after the user has finished entering their entire query?
Page URL
https://learn.microsoft.com/en-us/microsoftteams/platform/messaging-extensions/how-to/search-commands/respond-to-search?tabs=dotnet#enable-and-handle-tap-actions
Content source URL
https://github.com/MicrosoftDocs/msteams-docs/blob/main/msteams-platform/messaging-extensions/how-to/search-commands/respond-to-search.md
Author
@surbhigupta
Document Id
a9546115-7c5d-39c9-ca8a-3cf3e02ff621

@Harshit-Samar - Thanks for reporting your issue. We will check this and update you soon.

@Harshit-Samar - Could you please share the repro steps and repro video so that we can try it at our end? Meanwhile, we are also checking this with the engineering team.

@Harshit-Samar - The engineering team wants to understand why there is a trigger on every key press. Copilot should only send the entire prompt to the plugin once the user hits send. Additionally, their plugin needs to respond to Copilot within a short period of time for the response to be considered.

Hi @Nivedipa-MSFT , the invoke on every key press is the issue related to search based functionality of message extension. To elaborate more: The manifest can have context to be "commandbox" or "compose". I want to support only "commandbox" so that the extension doesnt show up in outlook or other places where this invoke with every key press is observed.
The problem here is with the global search where I can search for the extension name and then type query which has the similar behaviour.


Failing ci tests involving attention on pt dev (2.6)

Disabling for now.
FAILED thunder/tests/test_grad.py::test_populate_grads_csa_torch_cuda_thunder.dtypes.float32 - AssertionError: Tensor-likes are not close!

Mismatched elements: 218295 / 589824 (37.0%)
Greatest absolute difference: 1030.24462890625 at index (473, 722) (up to 0.01 allowed)
Greatest relative difference: 2295.604248046875 at index (482, 518) (up to 0.01 allowed)

The failure occurred for item [2]
FAILED thunder/tests/test_grad.py::test_populate_grads_block_nvfuser_cuda_thunder.dtypes.float32 - AssertionError: Tensor-likes are not close!

Mismatched elements: 768 / 768 (100.0%)
Greatest absolute difference: 2461.907470703125 at index (275,) (up to 0.01 allowed)
Greatest relative difference: 15.106374740600586 at index (668,) (up to 0.01 allowed)

would be good to investigate, but I could not reproduce.
cc @borda

triage review: let's re-enable the test and see if this was transient or is a real issue to be looked into.


Convert YAML files into Service Models/objects

Added Service object model. Added tests to read various YAML files
and generate Service objects.
Fixes #36

Overall a good looking first pass!

Took care of the code review comments #36


Can you have multiple ColorAxis lined up together without overlapping?

I've been trying to animate a plot using bqplot but I need to have multiple scatters with separate ColorAxis for each of them lined up on the right-hand side of the figure.
Apparently, options like offset , label_color , are not working for ColorAxis which might enable me to adjust the positions of the color bars as I need. The docs mention these attributes and hence I had the question if these actually work or are they just meant for Axis (Note : offset and label_color seem to work for Axis)
Is it possible to have multiple ColorAxis in such a fashion?
Thanks in advance.
Here's a sample code (the original code implements multiple scatters and ColorAxis but they overlap):
from bqplot import *
x_sc = LinearScale()
y_sc = LinearScale()
c_sc = ColorScale()

ax_x = Axis(
    label='X Axis',
    scale=x_sc,
    grid_lines='solid',
    label_offset='50',
    label_location='start',
    label_color='blue'
)

ax_y = Axis(label='Y Axis', scale=y_sc, orientation='vertical', grid_lines='solid')

ax_c = ColorAxis(
    label='Color',
    label_location='middle',
    scale=c_sc,
    orientation='vertical',
    side='right',
    label_color = 'blue',
    offset = {'scale':x_sc,'value':0}
)

x_data = range(20)
y_data = [i**2 for i in range(20)]
c_data = [i**0.5 for i in range(20)]
scatter = bq.Scatter(x=x_data,
                     y=y_data,
                     color=color_data,
                     scales={'x': x_sc, 'y': y_sc, 'color': c_sc,})

fig = Figure(axes=[ax_x,ax_y,ax_c],marks =[scatter])
fig

Here's the output

@deeptavker  One way you can display two ColorAxis is by displaying one on the right and the other on the left etc. Right now, we cannot display both ColorAxis on the same side without overlapping. as there is a bug with the offset property. Although we cannot do an offset of the ColorAxis with respect to a scale, we should be able to offset it in pixels.
I will keep this issue open till we fix the offset. Thanks for reporting this.

There's an option in the Axis class _ipython_display_ which is set to None with a comment
" # We cannot display an axis outside of a figure."  Maybe changing that would help us set offset values such that different ColorAxis don't overlap. However, I'm new to this and I'm not sure what to do with this _ipython_display_.


Receiving junk values like OK, 1 or 0 in go-redis

Not enough details to do anything - such reports should come with a reproducer.


Write definition and version directly independently from the reader

Fixes #311
Fixes #221

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

7 of 7 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 77.907%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
2620

Relevant Lines:
3363

💛  - Coveralls


OMS Workspace resource Sku Parameter update

pricingTier parameter was not used. OMS Workspace resource had a hardcoded value for 100GB/day Capacity Reservation tier which would result in ~£5,500 commitment.

@maciekporebski : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@maciekporebski - I appreciate your input, but I'm rejecting this PR.  I've made it very clear in the article that you should review the pricing-related details and modify the template before using it. I decided to include that in there, along with other of the common workspace config options, so anyone reading this article had a template that served as a good reference. #please-close

Could you consider removing the unused parameter pricingTier. It does suggest it would be used as the value of Microsoft.OperationalInsights/workspaces/sku.name.

@maciekporebski - After re-reviewing this and evaluating further, I concur that with the 2015-11-01-preview API, the pricingTier parameter is no longer used based on what is specified in the reference article. Before I update the json template in the article, I'm going to have a discussion with engineering to verify a few things related to this beforehand to avoid any further confusion or inaccuracies since this was based on a previous version of the API and wasn't informed it changed.  Once I dot my "i"s, I'll release an updated version of the template and also the AzMon articles that include using ARM templates to create a workspace to ensure consistency.  Thanks.

@maciekporebski - I just realized what the real issue is here (brain fart). It isn't that I'm using a parameter that doesn't exist, it's that I didn't reference the value for the property named as "pricingTier" with Microsoft.OperationalInsights/workspaces/sku.name.  I'll fix this shortly and just refer to it as "sku" instead of following this example for deploying an LA workspace here.

Great, thanks Matt
On Wed, 25 Mar 2020, 15:29 Matt Goedtel<EMAIL_ADDRESS>wrote:

@maciekporebski https://github.com/maciekporebski - I just realized
what the real issue is here (brain fart). It isn't that I'm using a
parameter that doesn't exist, it's that I didn't reference the value for
the property named as "pricingTier" with
Microsoft.OperationalInsights/workspaces/sku.name. I'll fix this shortly
and just refer to it as "sku" instead of following this example for
deploying an LA workspace here
https://docs.microsoft.com/azure/azure-monitor/platform/template-workspace-configuration
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/pull/50429#issuecomment-603906923,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEV5CSHNUJMJKPAPZXHLSYDRJIPOFANCNFSM4LOXGAUA
.


[ENG-5214] Update core OSF syntax for py3.10

Purpose
Get rid of all syntax errors when upgrading to python 3.10
Changes
Fixed syntax errors
QA Notes
Documentation
Side Effects
Ticket
ENG-5214

Ace, thank you!


Hardware acceleration causing troubles in xbox

Hi, I've started to get numerous reports about green video the video freezes on xbox. I got a hint that the new D3D11 acceleration might be the cause....  I released an update not so long ago updating to the latest nuget packages and seems like something in the ffmpeg build is causing that.
Is it possible to disable it via configuration?
Thank you!

There are some config options to disable it. VideoDecoderMode i think it was.

Thank you 👍

Did you manage to reproduce the issue?

Yes! I did reproduce it using a xbox one X (it doesn't happen with the new series X).
Setting the Config.VideoDecoderMode = VideoDecoderMode.AutomaticSystemDecoder has fixed the problem.
Thanks

Well if hardware support wasn't available, then it should have switched to software decoding. I don't have an xbox one X, would it be possible for you to debug and see why it doesn't auto switch to software?

Unfortunately my c++ experience is very very limited...  😒

Hi, I have the same problem on another application that uses FFMpeg (but not this distribution) on UWP. I have video freezes on Xbox but not on Windows. So maybe a problem relative to FFmpeg itself?

Hardware acceleration is device dependent. Do you have a video file which reproduces the issues?

Unfortunately I don't have a video file since I'm streaming from a custom protocol that gives me H264 NAL. Also unfortunately since I'm not part of any program, UWP is the only solution for me for development on Xbox. I'm going to downgrade FFMpeg and let you know if this does resolve the issue
PS: I'm not using FFMpegInteropX, but I found this thread and would be useful to check out together what's causing this

This copying thing we had in the beginning, but we noticed it crashed the performance of 4K H265 videos, so we gave up on that and instead used the more streamlined device to device memory copy.
Unless I can get a way to reproduce this, I will unfortunately be unable to help you with this. But I think it might be worth reporting the problem to Microsoft. This really does sound like a problem with Xbox environment, rather than the framework.
If you can provide a PR with feature flag which toggles the system memory copy then we could integrate this into repository (Idealy by sub-classing the existing hardware sample provider). Unfortunately I will not be able to participate in large coding for this project for the next 2-3 months, maybe @lukasf can help. But I could take a look if someone else does most of the heavy lifting.

@TheElixZammuto I'm glad to hear you found a workaround that fixes your issues. In the first version using FFmpeg HW acceleration, we also used a copy approach (although I think it was direct GPU to GPU copy?). For some codecs, FFmpeg needs the hardware device before playback starts (and as you know, in UWP we can only get the device instance on playback start). But then we found ways to work around this issue on the PC, achieving much better performance on 4K material without any copy operations. It might be that our approach does not work on the older xbox due to some reason.
Without access to the actual xbox hardware, it is impossible for me to debug into this, to find out the real root cause. And I won't try to solve this blindly, without knowing what is really happening here and how the behavior is on xbox. If someone finds out how to properly fix this in FFmpegInteropX, feel free to create a PR.
Also, we have a good workaround of using system decoders, which should give excellent performance as well.

I think our original attempt is still somewhere in source control history.

Closing this.


Remove invalid return

Motivation:
JNI_OnUnload(...) does not return anything (has void in its signature) so we should not try to return something.
Modifications:
Remove return.
Result:
Fix incorrect but harmless code.

@Scottmitch @nmittler PTAL

thanks for the cleanup ... ship it!

@Scottmitch thx for review..
Cherry-picked into 4.0 (f4798edc07f046c32c56a632b8cd663604191dc6) and 4.1 (cd56f87ca12664fdec3aba8b44ee425cac5cd4be)


Make it work for new release 0.12.5

A couple of days ago wkhtmltopdf 0.12.5 has been released. It turns out the buildpack doesn't work out-of-the-box when setting the version in config (the release url seems to have changed). Can you make the buildpack work for the new version?

This should be fixed with https://github.com/dscout/wkhtmltopdf-buildpack/pull/36, though it doesn't appear to be in a mergeable state currently. Closing in favor of the PR.


⚠ Bofrost.de has degraded performance

In e684b96, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1829 ms

Resolved: Bofrost.de performance has improved in c3fea61.


Enhacement/warnings hascode

Here we go again! ;-)
Fixes:

Fixed #49 all .equals() by adding corresponding .hashcode(). This is important, otherwise Object.hascode() contract is broken and lots of problems arise with Collections
Fixed JRef: two Variables are the same not just because their textual name is the same; two  X coul be part of different queries!

Improvements:

Many in the documentation. Updated old information, and added quite a bit for the user to understand JRef and Lists better.

 Only 2 remains for generics that cannot be avoided.
Start moving some files to more standard file conventions.

Thanks.  Looks all pretty innocent :)  Merged.  Do you get these (they are there a long time)
[420/463] Building Java objects for jpl_jar.jar
Note: org/jpl7/fli/atom_t.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
[425/463] Building Java objects for jpltest.jar
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.

Thanks Jan. Yes all innocent except for .equals() with no .hashcode() that was simple but long-standing "glitch".
I don't get that warning. It is very strange, I cannot see atom_t using or overriding a deprecated API. It is indeed a very simple class with almost nothing....
I used to get many of that sort and other warnings, 30+. I cleaned them all up, now I got just 2 that are not possible to get rid of as they have to do with generics.
But, your compile and runner does have some magic. I remember when it used to load and run libraries that could never been loaded! ;-)
Strange...

BTW, I changed to OpenJDK 11 and yes I also get that:
Now why do I get progress as % and you get the ratio done/total?
100%] Built target test_certificates
[100%] Linking C shared module libjpl.so
[100%] Built target plugin_libjpl
Note: org/jpl7/fli/atom_t.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
[100%] Generating CMakeFiles/jpl_jar.dir/java_class_filelist
[100%] Creating Java archive jpl-7.6.0.jar
[100%] Built target jpl_jar
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
[100%] Generating CMakeFiles/jpltest.dir/java_class_filelist
[100%] Creating Java archive jpltest.jar
[100%] Built target jpltest

Now why do I get progress as % and you get the ratio done/total?

Because I use ninja to build instead of good old make.    Ninja gives a lot better build experience than make: faster, just gives progress on the same line unless there output of a command on stderr.  In that case it is printed, nicely separated from other concurrent commands.
Glad you can reproduce :)

Sorry, I've only just noticed how you've changed Variable equality: I maintain that "A Variable is equal to another if their names are the same and they are not anonymous" and fear that you are restoring one of Fred's original misconceptions ;-/
A Java-side Variable is just a lexical token; it is most definitely not any variable within the Prolog engine. The object identity of a Variable doesn't mean much at all, and it matters not whether two occurrences of a variable within a goal (say, p(A, A)) are represented by the same (identical) Variable object or by two same-named instances of Variable - they are just lexical tokens.
Comparing two Variables Java-side is a not-particularly-useful operation whose only consistent definition is based on equality of their names; your version of equality won't break anything important, but it is wrong! If necessary I can elaborate until you are convinced!

Hi @anionic ,
Thanks for commenting on that and happy to move it back.
I was completing the .hascode() for every class that had a .equals() and then I thought this was a glitch, seems not and this was thought before. As you can imagine I am not aware of all much discussion that has happened in the past  haha
I can be convinced very easy don't worry. I am not going to fight for something  you know much more than me. ;-)
You are right that comparing Variables is not useful operation and would almost never will be done. But of course we want still something that is as close to the "correct" meaning even if not used much. I agree one will never compare Variables.
So, can you explain me what is the intended behavior of this code (which I included in unit testing but I can change of course):
       Variable v1 = new Variable("X");
        Variable v2 = new Variable("X");

        Term s1 = new Query("? = 5", v1).oneSolution().get("X");
        Term s2 = new Query("? = 15", v2).oneSolution().get("X");

It appears to me that you want v1.equals(v2) to be True, despite the fact that they refer to different bindings (one 5 the other 150? Is that correct?
Seems you want to make them equal if they have the same textual name, full stop, is that correct?


Add pystac-client and update PyTMD on Sandbox

This PR adds pystac-client to our Sandbox docker image, as it is used in this STAC guide here:
https://docs.dea.ga.gov.au/setup/gis/stac.html
This also updates the pyTMD tidal modelling package to the latest version (as required for DEA Coastlines).

Hey @treefern @emmaai, I'd love to get these two changes made to the Sandbox image if they pass tests. Should I also make them to this branch too? https://github.com/GeoscienceAustralia/dea-sandbox/tree/update_odc_core

pystac-client is in the image I'll assume, since most stac functions will have dependencies on it. And I have made it explicitly in the conda build env on the upgrading the branch.
We don't tie any specific version of package any more, but rely on conda to resolve the conflicts, unless we have very specific reasons to do it.
The change can be merged to current build though. It'll take some time to roll out the new image.


Remove configuration sum_waveform_top_array from MergedS2s

Close: https://github.com/XENONnT/straxen/issues/1305
Before you submit this PR: make sure to put all operations-related information in a wiki-note, a PR should be about code and is publicly accessible
What does the code in this PR do / what does it improve?
Can you briefly describe how it works?
Can you give a minimal working example (or illustrate with a figure)?
Please include the following if applicable:

[ ] Update the docstring(s)
[ ] Update the documentation
[ ] Tests to check the (new) code is working as desired.
[ ] Does it solve one of the open issues on github?

Notes on testing

Until the automated tests pass, please mark the PR as a draft.
On the XENONnT fork we test with database access, on private forks there is no database access for security considerations.

All italic comments can be removed from this template.

coverage: 90.692% (-0.05%) from 90.739%
when pulling f15f5b30912a07f9e237e14bf4c22c2a4b1d00be on del_sum_waveform_top_array
into 007ede17dc181ca50ac1d928f2bd20b2b1f748ad on master.

Ah I saw you linked a PR.

Sorry, but I need a bit more context why are we removing this? Do we not want to store the top waveform any longer?

Whether store it or not depends on whether there is a field "data_top" of peaklets.


Hero Image is too big

hero image is too big and have to scroll to view complete image, that's bit annoing, will look more profession if make it visible in one scroll

I would like to take this up and fix it.

I would like to take this up and fix it.

Already fixed sir raised a pr


[iOS] Minimum deployment target 12.0

I noticed upon installing this library there is a minimum deployment target of 12 in the podspec. If this is needed (maybe it isn't?) then it should be mentioned in the readme since the default for projects at the moment is 9.

Readme updated


dbt 1.3.0 has changed raw_sql to raw_code

When compiling the manifest file with dbt 1.3.0 it is being created with 'raw_code' used instead of 'raw_sql' which appears to be breaking it.

@buremba is project still maintained?


Please clarify commercial usage of Read the Docs

The website currently mentions that it's free but it's not explicit about commercial usage. This suggests commercial usage is allowed but I'm wondering whether that's the case and what the conditions (if any) are.

We now have https://readthedocs.com for commercial use.

Hi,
Sorry to comment on a close issue but didn't find any better way to ask. We have a bit specific situation for which is hard to figure out the answer. We are working on a product which will be soon available in an open core mode (core part opensourced with some features commercial as add-ons).
The goal is to make a sustainable product like what you are trying to do with the rtd.com.
The product documentation will be mostly about the core part and we are preparing it to be available on a public repo for the whole product, including the smaller parts for addons, there is not much sense in dividing it. Now we are checking to use RTD but not sure if we can use the free option for our mostly but not completely open sourced product.
Could you confirm that we can use the free version? Or not :)


Mark highlights of the stream, to be viewable on the website.

Command
!add highlight
Optional arguments
--id ID of existing highlight to edit
--offset Amount of "back in time" to offset the highlight with
Positional argument:
Tag/Description
Table layout
tb_stream
id, auto increment, primary key, int
title, varchar, 256. max length for twitch titles right now seem to be 140 but yolo
stream_start, datetime, time for when the stream started. When the stream is started, this should be adjusted based on the initial "created_at" attribute of the API call which we use to check if the stream is live or not.
stream_end, datetime, time for when the stream is ended. This should be the timestamp when the stream hs first seen as "offline", not after the X amount of retries have gotten the "offline" mark in a row.
tb_stream_chunk
When a new chunk is detected (due to the change in _id), try to check if there's a video ID available here: https://api.twitch.tv/kraken/channels/strifecro/videos?limit=10&broadcasts=true
If there is no video with the same broadcast id there yet, set it to null.
Upon checking stream status, if the current chunk url is null, this API should be checked again.
id, auto increment, primary key, int
stream_id, foreign key linking back to a tb_stream
broadcast_id, unique ID, as seen in https://api.twitch.tv/kraken/streams/strifecro. use _id there
video_url, varchar, 128, nullable
chunk_start, datetime
chunk_end, datetime
tb_stream_chunk_highlight
id, auto increment, primary key, int
chunk_id, foreign key linking back to a tb_stream
created_at, datetime, timestamp of when the highlight was created
highlight_offset, datetime
description, text, nullable
Code structure
another model
should be StreamManager
which starts an execute_every for the stream_check interval (by default, this is currently 20).
Maybe StreamManager should not keep a permanent session, as the DB requests should very seldomly be made.
Web part
There should be an entire section for highlights.
How should we list them?
Right now we fetch all chunks, in descending order, and just list all highlights.
In the future, we should probably fetch, along with the video url, we fetch the preview image as well.
TODO for later

[x] Fetch preview image for each chunk as well
[x] Improve the listing on /highlights so it looks decent
[x] Add some sort of paging on the /highlights page
[x] Run bot before stream goes live, go live until video url is fetched, then go offline for a second, and go live again so a new chunk is started, make sure it can fetch the video url for that as well.
[x] ask ilisha how she wants the offset to work. Should the default be 40 seconds back in the vod?
[x] look into delaying the video_url fetching, by 10 minutes or something. should it be run in a separate thread? This will be apparent during the test mentioned above

Base feature complete. Ask the streams the bot runs in whether there's something missing or not.


setFullscreen not work

Thank you for your great plugin!
Almost works properly.
But setFullscreen() is not working on My environment(Unity2020.3.24f1).
VueCode:
Unity.setFullScreen(true);
Error:
"TypeError: Unity.setFullScreen is not a function"
Please, Can you help me ?
ittoh0819

Can you find the setFullscreen method on Unity object ? Please print the Unity on the console.


Update websql dependency to 1.0.0

This adds compatibility with Node 10 (sqlite3 3.x which is a dependency of websql won't compile; websql 1.0.0 uses sqlite3 4.0 which compiles fine)

websql 1.0.0 drops support for Node v0.10 and Node v0.12 but I guess it should not be a problem.

Yes, exactly, that would be the only incompatibility and we have the same opinion about it. Thanks for merging 😺


QiskitError: 'Label contains invalid characters.'

Informations

Qiskit version: 0.14.2
Python version: 3.7.8
Operating system: posix

What is the current behavior?
I'm trying to run a the below piece of code and it throws error stating "Label contains invalid characters"
Running on IBM Quantum (https://quantum-computing.ibm.com/)
---------------------------------------------------------------------------
QiskitError                               Traceback (most recent call last)
<ipython-input-101-db751b675485> in <module>
----> 1 qi.Operator.from_label('CX')
/opt/conda/lib/python3.7/site-packages/qiskit/quantum_info/operators/operator.py in from_label(cls, label)
    176         }
    177         if re.match(r'^[IXYZHST01rl\-+]+$', label) is None:
--> 178             raise QiskitError('Label contains invalid characters.')
    179         # Initialize an identity matrix and apply each gate
    180         num_qubits = len(label)
QiskitError: 'Label contains invalid characters.'
Steps to reproduce the problem
from qiskit.extensions import HGate, CXGate
import qiskit.quantum_info as qi
qc = QuantumCircuit(2)
qc.h(0)
qc.cx(0,1)
qc.draw('mpl')
# no issues with the below line
qi.Operator.from_label('HI')
# error at this line
qi.Operator.from_label('CX')
What is the expected behavior?
return the result as Operator object
Suggested solutions

From the documentation https://qiskit.org/documentation/stubs/qiskit.quantum_info.Operator.from_label.html, it seems like this from_label takes in single-qubit operators on N qubits as an N length string and returns a tensor product. CX doesn't satisfy that, yielding the error.

Use %matplotlib inline
Then try to correct  qc.draw('mpl') to qc.draw(output='mpl').
It works.

When I change it to
qi.Operator.from_label('H') + qi.Operator.from_label('X')
I got result
Operator([[ 0.70710678+0.j,  1.70710678+0.j],
          [ 1.70710678+0.j, -0.70710678+0.j]],
         input_dims=(2,), output_dims=(2,))

But when I only change it to
qi.Operator.from_label('HI')
I still got the result
Operator([[ 0.70710678+0.j,  0.        +0.j,  0.70710678+0.j,
            0.        +0.j],
          [ 0.        +0.j,  0.70710678+0.j,  0.        +0.j,
            0.70710678+0.j],
          [ 0.70710678+0.j,  0.        +0.j, -0.70710678+0.j,
            0.        +0.j],
          [ 0.        +0.j,  0.70710678+0.j,  0.        +0.j,
           -0.70710678+0.j]],
         input_dims=(2, 2), output_dims=(2, 2))

The problem is with the  qi.Operator.from_label('CX')

When I change it to
qi.Operator.from_label('H') + qi.Operator.from_label('X')
I got result
Operator([[ 0.70710678+0.j,  1.70710678+0.j],
          [ 1.70710678+0.j, -0.70710678+0.j]],
         input_dims=(2,), output_dims=(2,))

But when I only change it to
qi.Operator.from_label('HI')
I still got the result
Operator([[ 0.70710678+0.j,  0.        +0.j,  0.70710678+0.j,
            0.        +0.j],
          [ 0.        +0.j,  0.70710678+0.j,  0.        +0.j,
            0.70710678+0.j],
          [ 0.70710678+0.j,  0.        +0.j, -0.70710678+0.j,
            0.        +0.j],
          [ 0.        +0.j,  0.70710678+0.j,  0.        +0.j,
           -0.70710678+0.j]],
         input_dims=(2, 2), output_dims=(2, 2))

The problem is with the  qi.Operator.from_label('CX')

It's because from_label takes in single-qubit operator labels only. CX does not satisfy that. For further discussion, refer to https://qiskit.slack.com/archives/CMXDMFNN5/p1595336109201500


[ERROR] Unable convert

I have installed and added ebook-covert to my dependencies as well
Here is my code
var path = require('path')
var xtend = require('xtend')
var convert = require('ebook-convert')

// see more options at https://manual.calibre-ebook.com/generated/en/ebook-convert.html
var options = {
  input: path.join(__dirname, 'Downloads', 'example.html'),
  output: path.join(__dirname, 'Downloads', 'example.epub'),
}

/*
* create epub file
*/
convert(options, function (err) {
  if (err) console.log(err)
})

Dude, the problem is ebook-convert is not in your $PATH. This is addressed in the Calibre repo...
add /Applications/calibre.app/Contents/MacOS to your path

@MerovexI get the same error. Your mean is add this line export PATH=~/Applications/calibre.app/Contents/MacOS/:$PATH to .profile file?

@hungdev that should do it. But, first confirm the path I provide links you to the app.


Unify signatures of static and instance combination methods?

Kefir.combine(obss, [passiveObss], [combinator])

but

obs.combine(otherObs, [combinator])

So, if I want to combine more than two streams at once, I need to use only "static" version of combine.
Same for all combination methods.
Is this intentional, or we can make instance methods to behave same as static ones?

Also, doc shows example Kefir.combine([a, b], (a, b) => a + b);, but on practice it seems to work as Kefir.combine([a, b], ([ a, b ]) => a + b); (combinator receives an array). Is it error in doc?

Yeah, this was intentional. The logic is you either do combine(many) or combine(one + one), combine(one + many) seems kind of wrong to me.
Not sure if that makes sense, but I would't introduce a braking change like that for no strong reason.

I tried the first example in console, seems fine:

I tried the first example in console, seems fine

Yep, I "tested" in wrong way, I'm sorry.

The logic is you either do combine(many) or combine(one + one), combine(one + many) seems kind of wrong to me.

Not sure there is something wrong in doing combine(one + many)... Just combine streams, what's the difference how much of them there?
In fact, currently it makes sense to use only static methods everywhere, because the need to always keep in mind which version of API is allowed to be used in each concrete case looks like nonsense.
Anyway,

I would't introduce a braking change like that for no strong reason

Is this a breaking change? Besides passiveObss param, the only thing is required to do is this:
Observable.prototype.combine = function(other, combinator) {
  return combine([this].concat(other), combinator);
};

Am I wrong?

So you suggesting support Observable | Array<Observable> as the first argument? I'm not a big fun of such unnecessary polymorphisms to be honest. IMO, the less polymorphic the API the easier to read the code. For example we see foo.combine(bar) somewhere in the code, the question is what is bar? Is it array or an Observable?

Not sure there is something wrong in doing combine(one + many)

Sorry, it's probably just my intuition, it's just feels wrong to have that in API.

Also we have plans to support objects in static combine https://github.com/rpominov/kefir/issues/143, which may add more confusion in instance combine method if we add arrays support in it.

the less polymorphic the API the easier to read the code. For example we see foo.combine(bar) somewhere in the code, the question is what is bar? Is it array or an Observable?

Although in general I agreed, IMO the API consistence is more important than potential readability problems. After all, it's programmer's responsibility to keep his code readable.

Also we have plans to support objects in static combine #143

...and then instance methods will become even more useless?)
But ok, this is reasonable. With objects support, it will be impossible to provide a named key for instance on which combine was called, yes?

...and then instance methods will become even more useless?)

Probably, I tend to always use static alternative if there is one to be honest :)

But ok, this is reasonable. With objects support, it will be impossible to provide a named key for instance on which combine was called, yes?

Yep.

Understood. I prefer an opposite approach. Less coupling, as for me, easier to mock, etc.
But, your game – your rules :) Anyway your lib looks much more preferable for me in comparison with alternatives.
If you allow, one more question, besides the topic.
If I need a stream to which I want push events imperatively, from outside – is this a correct approach?
var dispatch;
var stream = Kefir.stream(emitter => dispatch = emitter.emit);
...
dispatch(...);

Or just call stream._emitValue(...) ? What is more proper way?

The first one with dispatch is more correct.
We had Kefir.emitter() before, it worked like this:
const em = Kefir.emitter()
em.map(x = > x + 1).onValue(...)
em.emit(1)

But people new to FRP tend to abuse this imperative API, for instance:
// wrong
const streamB = Kefir.emitter()
streamA.onValue(x => streamB.emit(x + 1))

// right
const streamB = streamA.map(x => x + 1)

So we decided to remove it from API, here is the original discussion #88

Also, although it looks more complicated with .fromBinder, now it more obvious what really happening under the hood, and if a developer write that code it means that he or she understand what going on. And it will be more obvious for other people who will read it.

:+1: Great
Thanks for your replies very much.


Localizations (translation)

Please add translations so you can configure a language in the options.
You could auto-generate these, for example with some tool that uses DeepL (I don't know which tool would be best for generating the translated strings) or simply by copy pasting all of them at once into DeepL.
It wouldn't be important to translate the instructions, it would be sufficient if the ingredients were translated.

That would be cool. I tried DeepL but my first impression was not that great.

The app would definitely get more popular if it was on F-Droid. With the user base growing, you'd also have people willing to do the translations, which is common in libre apps.

I just created a request for packaging on F-Droid. Hope the process is smooth :)

The app now uses i18n so all string values are stored in a translation.json file under a language directory. This will make it easy to add a new language.
Example here


Steps containing Cucumber expressions are displayed correctly

@laxersaz, I see that the under All Steps, the steps are displayed as:
I log in to the "{}" environment as "{}"

and not:
I log in to the "test" environment as "user1"

The steps appear correctly in the scenario details page.

Hi! This is expected behavior for steps with placeholders because seeing the concrete steps including their values would be pretty pointless since each one would most likely only have one match.

Thanks @laxersaz. I think I misunderstood what was being displayed under that tab.


[Help] Instruct switch to drop all packets from affected DDoS port

I am writing code to mitigate DDoS attacks. I can currently identify the switch ports for all switches from which DoS attacks originate. I can't figure out how I can, given the port and dpid of the switch, instruct the switch to drop all packages coming only from the affected port. So maybe I could use update the switch's flow table? but how I could that?
Finally, when the DDoS attack is over, how can I remove that rule?
Can you help me? Thanks!

My OpenFlow is rusty, but I think the core of it should be something like this...
import libopenflow_01 as of

# Add rule that drops
msg = of.ofp_flow_mod()
msg.match.in_port = BLOCK_PORT
core.openflow.sendToDPID(BLOCK_DPID, msg)

# Remove rule that drops
msg = of.ofp_flow_mod(command = of.OFPFC_DELETE_STRICT)
msg.match.in_port = BLOCK_PORT
core.openflow.sendToDPID(BLOCK_DPID, msg)

My OpenFlow is rusty, but I think the core of it should be something like this...

import libopenflow_01 as of

# Add rule that drops

msg = of.ofp_flow_mod()

msg.match.in_port = BLOCK_PORT

core.openflow.sendToDPID(BLOCK_DPID, msg)

# Remove rule that drops

msg = of.ofp_flow_mod(command = of.OFPFC_DELETE_STRICT)

msg.match.in_port = BLOCK_PORT

core.openflow.sendToDPID(BLOCK_DPID, msg)

Thank you very very much!! It works fine!
Where I can read some documentation about Openflow using Python with some example?
Thanks a lot!

A few possibilities:

The POX repository has a number of examples; the "forwarding" components in particular might be a decent place to start.
The POX documentation has a section on using OpenFlow with POX: https://noxrepo.github.io/pox-doc/html/
The Mininet OpenFlow Tutorial has a section: https://github.com/mininet/openflow-tutorial/wiki/Create-a-Learning-Switch
The mainline POX only has support for OpenFlow 1.0 and some Nicira (OVS) extensions.  The original OpenFlow 1.0 spec is not that long or complicated, and POX's OpenFlow corresponds to it pretty closely except that it's much easier to use since, e.g., it has reasonable defaults and computed values for many fields: https://opennetworking.org/wp-content/uploads/2013/04/openflow-spec-v1.0.0.pdf

Good luck!

Thank you very much, you have been very helpful! Thank you very much!


Change Textobj: cssColor

Change requested
support rgba
Sample Text
rabg(255,255,255,0.1)
Checklist

[X] The textobj would be useful to more users than just me. (You can use the API to create custom textobj.)
[X] This is a feature request to change an existing textobject, not a bug report. (Use the bug report form for those.)

Brother, it’s so fast NB


[add-welcome-mat] Add welcome mat

https://trello.com/c/YG9GLBf4

 I want to log in with a github account

I think this one is ready to review. Assigning @mirisuzanne and @duggiemitchell

This looks good to me. Any way of getting the welcome mat image to be be the full height? maybe adding an additional class on the same div as the grid column?

It is full height in my browser so I'm guessing whichever you are checking in that is ignoring some of my overrides. Can you send a screenshot please?
As far as the modal component, we have nothing to show under the modal. We thought it might be a bit odd to have the ability to close a modal to only have to re-open it right away to login. ¯_(ツ)_/¯

That makes sense.  Here's how it looks for me (on Chrome):

@duggiemitchell Can you confirm that the image-height issue is fixed?

@stacyk It's still not showing up full height for me in chrome, but :+1: in firefox.


[Analyzer] Dart code to AST

Analyzer*
I want to use analyzer to convert Dart code to AST, but I can't find the documentation (Google & github). Reading comments from its source code is very obscure.

You can find the documentation for the package in the doc directory, as is typical for most packages, specifically under tutorial (https://github.com/dart-lang/sdk/tree/master/pkg/analyzer/doc/tutorial).
If you have additional questions you can either open an issue or use the mailing list at https://groups.google.com/a/dartlang.org/g/analyzer-discuss.

thanks


Sample/mission replay

Refactored as MVC application with FXML

@john0005 Can you please review?

Some things to fix for the sample.

change color of 2D plane symbol to blue so it's not red on red
decrease the total zoom amount, goes out to far
(+/-) buttons don't work when the plane is being animated
need to default to the Grand canyon location when application first starts

Also on Mac when I try to run it the SceneView is completely black, will look more into that tomorrow.
Will look at MissionController tomorrow, only thing i have left to review.
Look way better btw.

Tried to find out what the problem with the black screen on Mac was today and through the process it stop doing it. I have heard of an issue were the SceneView will black screen sometimes when starting and that may have been the case. Will test it on linux to see how it runs.
Only on Mac the display for plane settings (lower right corner) is a little small and the words get cut off.

@john0005 Second review please?

@john0005 One thing I didn't change is the max/min zoom on the mini map. I don't think this is a big issue because users have control here. I did notice though that if I set max and min scales on the map, I can still bypass them by setting the scale programatically. Is this a bug?

Everything looks good. About the scale thing thats a good question. I would bring up that same question here, https://devtopia.esri.com/runtime/runtime-mapping-api-design/issues, and let them figure it out. Seems like a bug to me.

@tyle8552 few minor comments. Looks good overall.

Starting a new PR to address conflicts: https://github.com/Esri/arcgis-runtime-samples-java/pull/55


The Quick Action icons of 'extract method' and 'remove unused imports' are missing

Environment

Steps to Reproduce
1.Create a Python Application project
2.Input some code
3.Move the cursor to a one line
4.The Quick Action icon should appear in front of this line.
Expected behavior

Actual behavior

Sample Code
import array; import glob; import math
print(math.cos(10))

This is not python specific issue. It's being handled here: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1643677


[Question]  Is there any way I can customize the disabling of components?

In the editing modal there is an advanced option, for validation, data and display of the component, but I couldn't find it to enable or disable the component.
I would like to configure it to disable a specific component depending on the value in another component. But I tried different ways and I couldn't. I would like to configure via interface.

Hi @adhonay. You need to interact with instance object instead of component.
instance.disabled = true;

@RomanLetsuk
It doesn't work, I've tried several combinations
instance.component.disabled = true;
instance._parentDisabled = true;
instance.originalComponent.disabled = true;
instance._disabled=true;
instance.disabled=true;

I managed to disable it by rule using the logical tab.


Access denied page

Details in ticket -> HEC-1576
I found that the page renders correctly when accessed directly via the endpoint, but when it gets redirected via the EmailAllowedListFilter due to user not being in allow list, the style does nor render correctly, it looks something like this -

and I see these error in the console -
@nubz's thoughts

I think the email allow list filter is being applied to all requests except just this page/document (which then goes on to make multiple requests but each of those requests is being redirected to this page
so when I look at the header for a stylesheet request it is being redirected to /tax-check-for-licence/denied-access in the response
which returns text/html hence the console errors

Any ideas about why this might be so are welcome.

I think we may need to add a couple more things to isExcludedEndpoint here so that the page can retrieve what it needs to:
https://github.com/hmrc/hec-applicant-frontend/blob/main/app/uk/gov/hmrc/hecapplicantfrontend/filters/EmailAllowedListFilter.scala#L47
it might be that you need to allow requests such as these through the filter:
->         /hmrc-frontend           hmrcfrontend.Routes
GET        /assets/*file            controllers.Assets.versioned(path = "/public", file: controllers.Assets.Asset)

If that's the case then you'd need to have some rules like if(rh.uri.cotnains("/hmrc-frontend") or if(rh.uri.contains("/asset") I think in isExcludedEndpoint`.

I think we may need to add a couple more things to isExcludedEndpoint here so that the page can retrieve what it needs to: https://github.com/hmrc/hec-applicant-frontend/blob/main/app/uk/gov/hmrc/hecapplicantfrontend/filters/EmailAllowedListFilter.scala#L47
it might be that you need to allow requests such as these through the filter:
->         /hmrc-frontend           hmrcfrontend.Routes
GET        /assets/*file            controllers.Assets.versioned(path = "/public", file: controllers.Assets.Asset)

If that's the case then you'd need to have some rules like if(rh.uri.cotnains("/hmrc-frontend") or if(rh.uri.contains("/asset") I think in isExcludedEndpoint`.

That worked @AndyHWChung, updated https://github.com/hmrc/hec-applicant-frontend/pull/99/commits/2a266efc27f26dc7e4b9ee8a353e20659fe5f8c1

Shouldn't we have test case for this page? atleast for asserting the title

Shouldn't we have test case for this page? atleast for asserting the title

It's a static page, so didn't think we needed tests, but I can add quickly just to have it in place if the content changes 👍


added Machine Learning contents in Korean language, Author of the con…

What does this PR do?
Add resource(s)
For resources
Description
Adding contents in machine learning of courses-korean.
Adding authors to links that previously existed in ko - ML courses
Why is this valuable (or not)?
How do we know it's really free?
One is youtube playlist, and the other one is the site that provides free courses in Korean. There are various of courses.
For book lists, is it a book? For course lists, is it a course? etc.
It is a course.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


[Feat] Avoid Adding _jgo.jwt.head for Root JWTs with -rec

When using the -rec option for root JWTs, it print _jgo.jwt.head also for the ROOT Header fields.
This feature request is to make it possible to omit _jgo.jwt.head for root JWTs, simplifying the reading

"_jgo.jwt.head": {     "alg": "RS256",     "kid": "68e99fa9-7013-49dd-852e-b47187fd8da7",     "typ": "entity-statement+jwt" }

This is fixed by https://github.com/thomaschiozzi-tndigit/jgo/pull/11


Aws cross account

How is this different than #120? Can you provide a short description of the problem this change solves?

@brharrington Both #120 and #121 are same. Only differences between these two pull requests are AwsClient.scala. Using InstanceProfileCredentialsProvider to get credential details, if edda is running on EC2 instance.

I started trying to review this but I don't think it's a reasonable approach.  The AssumeRole functionality shouldn't bleed into every crawl implementation IMO.


Guess My Number

A fun game to guess a random number, along with a helper to let you know if your guessed number is too high or too low!
It is made using HTML, CSS and JS.
Please assign it the hactoberfest label.

@gantavyamalviya , please assign it the hactoberfest label.


Cannot step through font size values in the text style panel

Bug Description
A user cannot use the arrow keys to step through the font size values as expected
Expected Behaviour
Arrow keys should allow the user to step through the values up/down
Steps to Reproduce

Screenshots

Additional Context

Plugin Version:
WordPress Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Related: #6828 (Again -- probably we should do this in the NumericInput directly)


UIEXPMGR-94: Update Node.js to v18 in GitHub Actions

UIEXPMGR-94
Purpose

Approach

TODOS and Open Questions

Learning

Pre-Merge Checklist
Before merging this PR, please go through the following list and take appropriate actions.

[ ] I've added appropriate record to the CHANGELOG.md
Does this PR meet or exceed the expected quality standards?

[ ] Code coverage on new code is 80% or greater
[ ] Duplications on new code is 3% or less
[ ] There are no major code smells or security issues

Does this introduce breaking changes?

[ ] If any API-related changes - okapi interfaces and permissions are reviewed/changed correspondingly
[ ] There are no breaking changes in this PR.

If there are breaking changes, please STOP and consider the following:

What other modules will these changes impact?
Do JIRAs exist to update the impacted modules?

[ ] If not, please create them
[ ] Do they contain the appropriate level of detail?  Which endpoints/schemas changed, etc.
[ ] Do they have all they appropriate links to blocked/related issues?

Are the JIRAs under active development?

[ ] If not, contact the project's PO and make sure they're aware of the urgency.

Do PRs exist for these changes?

[ ] If so, have they been approved?

Ideally all of the PRs involved in breaking changes would be merged in the same day to avoid breaking the folio-testing environment.  Communication is paramount if that is to be achieved, especially as the number of intermodule and inter-team dependencies increase.
While it's helpful for reviewers to help identify potential problems, ensuring that it's safe to merge is ultimately the responsibility of the PR assignee.

@folio-org/fe-tl-reviewers @vashjs In scope of that PR I updated eslint version and eslint file since we updated @folio/eslint-config-stripes. We had a conflict between eslint versions and config


Assertion failures in mruby-pack on big endian

The good news is mruby 2.0.0 works on OpenBSD/sparc64 (previous versions failed to build).  However, running the tests shows the following failures:
Fail: pack float (mrbgems: mruby-pack)
 - Assertion[1] Failed: Expected to be equal
    Expected: "\x00\x00@@"
      Actual: "@@\x00\x00"
 - Assertion[2] Failed: Expected to be equal
    Expected: [3]
      Actual: [2.304855714121459e-41]
 - Assertion[3] Failed: Expected to be equal
    Expected: "@@\x00\x00"
      Actual: "\x00\x00@@"
 - Assertion[4] Failed: Expected to be equal
    Expected: [3]
      Actual: [2.304855714121459e-41]
 - Assertion[5] Failed: Expected to be equal
    Expected: "@@\x00\x00"
      Actual: "\x00\x00@@"
 - Assertion[6] Failed: Expected to be equal
    Expected: [3]
      Actual: [2.304855714121459e-41]
 - Assertion[7] Failed: Expected to be equal
    Expected: "@@\x00\x00"
      Actual: "\x00\x00@@"
 - Assertion[8] Failed: Expected to be equal
    Expected: [3]
      Actual: [2.304855714121459e-41]
Fail: pack double (mrbgems: mruby-pack)
 - Assertion[1] Failed: Expected to be equal
    Expected: "\x00\x00\x00\x00\x00\x00\b@"
      Actual: "@\b\x00\x00\x00\x00\x00\x00"
 - Assertion[2] Failed: Expected to be equal
    Expected: [3]
      Actual: [1.043466644016713e-320]
 - Assertion[3] Failed: Expected to be equal
    Expected: "@\b\x00\x00\x00\x00\x00\x00"
      Actual: "\x00\x00\x00\x00\x00\x00\b@"
 - Assertion[4] Failed: Expected to be equal
    Expected: [3]
      Actual: [1.043466644016713e-320]
 - Assertion[5] Failed: Expected to be equal
    Expected: "@\b\x00\x00\x00\x00\x00\x00"
      Actual: "\x00\x00\x00\x00\x00\x00\b@"
 - Assertion[6] Failed: Expected to be equal
    Expected: [3]
      Actual: [1.043466644016713e-320]
 - Assertion[7] Failed: Expected to be equal
    Expected: "@\b\x00\x00\x00\x00\x00\x00"
      Actual: "\x00\x00\x00\x00\x00\x00\b@"
 - Assertion[8] Failed: Expected to be equal
    Expected: [3]
      Actual: [1.043466644016713e-320]

These failures do not occur on OpenBSD/amd64, and the test output leads me to believe this may be a general mruby-pack issue on big endian systems.

Do you have MRB_ENDIAN_BIG defined?

I'm guessing it isn't explicitly defined.  I'm going to close this now, and if there are still assertion failures with it defined, I'll reopen.

I just wanted to confirm that defining MRB_ENDIAN_BIG did fix the problem.  We ended up doing something similar to this so the same code works on both big endian and little endian:
 /* define on big endian machines; used by MRB_NAN_BOXING */
-//#define MRB_ENDIAN_BIG
+#include <endian.h>
+#if (BYTE_ORDER == BIG_ENDIAN)
+#define MRB_ENDIAN_BIG
+#endif

That would be a nice idea. I was working on a different solution that relies on the dynamic endian check.
But this one is better.


feat(i18n): adding i18n for signin page

I'll work on the i18n of the pages. Should I sending the PRs on demand in order to update the project for non-English speaking users, or finish the whole work to open the PR?

@GabrielNSilva awesome work! it would be great to open one PR per section of the app going forwards in case we have other team members working on translations as well. I will hopefully be adding some more support for translations soon, probably continuing on the recipient journey side first, so please do let us know. I have started a discord channel here, if you would like to join: https://discord.gg/3QkV7ny2EW


Make Identity Javadocs JDK20 Compatible

Updates Identity Javadocs to ensure they compile against JDK 20.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-identity


Modified .gitignore to include Visual Studio Code workplace settings folder

Modified the project .gitignore to include the Visual Studio Code (vs code) workplace settings folder.
Vs code uses the .vscode directory to store settings such files as tasks.json and launch.json. See vs code documentation:
https://code.visualstudio.com/Docs/customization/userandworkspace

Thanks, @mkenne11.

No worries @shockone.


Provide UserClaimsPrincipalFactory

Currently, IUserClaimsPrincipalFactory<TUser> has an implementation as UserClaimsPrincipalFactory<TUser, TRole>. However, this is not convinent for implementations which don't support IRoleStore<TRole>. Providing UserClaimsPrincipalFactory<TUser> would be great.

Happy to PR for this if this sounds reasonable.

Feel free to submit a PR for this sure :)

https://github.com/aspnet/Identity/issues/581

Any progress on this? We should also have IServiceCollection.AddIdentity<TUser> without having to specify TRole.

@blowdart we should discuss this in triage again

This was done in 2.0 to support roles being optional

Done via https://github.com/aspnet/Identity/issues/1269


Typed metrics endpoint dict

Adds typing annotation for COSAgentProvider's metrics_endpoints init arg.
Fixes #106

It kind-of doesn't, but OF uses pyright, not mypy, and we're trying to slowly converge a little. pyright doesn't like dict. It could be ok with Dict[str, Any] or something, but it may even allow us to do a little enforcement someday. Mostly this is for pyright and a little sanity.


Fix PolicySet::link to avoid mutating policy set on link fail

If linking failed because the template id corresponded to a static policy, PolicySet::link would correctly return an error, but the error was returned after calling ast::PolicySet::link. When given a static policy id, this function updated the links HashMap to add an entry for the target policy id which was not removed when returning an error.
Description of changes
Issue #, if available
Checklist for requesting a review
The change in this PR is (choose one, and delete the other options):

[ ] A breaking change requiring a major version bump to cedar-policy (e.g., changes to the signature of an existing API).
[ ] A backwards-compatible change requiring a minor version bump to cedar-policy (e.g., addition of a new API).
[ ] A bug fix or other functionality change requiring a patch to cedar-policy.
[ ] A change "invisible" to users (e.g., documentation, changes to "internal" crates like cedar-policy-core, cedar-validator, etc.)
[ ] A change (breaking or otherwise) that only impacts unreleased or experimental code.

I confirm that this PR (choose one, and delete the other options):

[ ] Updates the "Unreleased" section of the CHANGELOG with a description of my change (required for major/minor version bumps).
[ ] Does not update the CHANGELOG because my change does not significantly impact released code.

I confirm that cedar-spec (choose one, and delete the other options):

[ ] Does not require updates because my change does not impact the Cedar Dafny model or DRT infrastructure.
[ ] Requires updates, and I have made / will make these updates myself. (Please include in your description a timeline or link to the relevant PR in cedar-spec, and how you have tested that your updates are correct.)
[ ] Requires updates, but I do not plan to make them in the near future. (Make sure that your changes are hidden behind a feature flag to mark them as experimental.)
[ ] I'm not sure how my change impacts cedar-spec. (Post your PR anyways, and we'll discuss in the comments.)

Disclaimer
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Just out of curiosity, how was this bug discovered?

Basically just looking at the code


Fix Typos in Developer Website's tbDEX SDK Section on Front Page

Fix Typos in Developer Website's tbDEX SDK Section on Front Page
🚀 Goal
We want to ensure our website's different pages don't have any mistakes.
🤔 Background
This issue focuses specifically on fixing typos on the front page of our website. Your contributions here make a difference!
🤝 How You Can Contribute
We've broken it down into specific typos for you to address. Every contribution counts!
🔑 Tasks and Acceptance Criteria

Front Page

[ ] Fix "instutition" typo to "institution" in tbDEX SDK text box.

Make sure all tests are passing.

🌟 Resources

Developer Website

Getting Started

Comment ".take" on this issue to get assigned
Fork the repository and create a new branch for this task
Follow the tasks outlined above
Submit a pull request with your changes
Respond to any feedback during the review process

Questions?
If you have any questions or need clarification, please comment on this issue or join our Discord community.
Happy coding! 🎉

.take

.take


gnome3.gnome-desktop: fix thumbnailing with missing fontconfig cache

Motivation for this change
#56174 updated gnome-desktop which broke thumbnailing due to the sandbox trying to mount a non-existing directory (/var/cache/fontconfig). We patch the script so it won't try mounting the directory when it doesn't exist.
Things done
Tested thumbnailing in eog and gnome-desktop.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Can this be upstreamed?

I would think so.

upstream MR: https://gitlab.gnome.org/GNOME/gnome-desktop/merge_requests/34. Not sure if it's worth it to pull the patch from there?

MR landed upstream :) so changed things to fetchpatch so we'll easily catch it when upgrading.


Yield yearly

I miss the yield yearly value in your Zonneplan One integration. Is it possible to add this value?
The Zonneplan integration is very nice!

Not sure if the value is available. But the idea is to use the HA energy dashboard for this

Thank you for your comment. For future energy measurements this is true, but for past measurements not. I use the yearly yield to calculate the Specific yield (kWh/kWp) on my location. As this is calculated per year it is very useful to get the yearly yield of Zonneplan.

You can use the Utility Meter - Home Assistant to create a year sensor


[BEAM-8651] Guard pickling operations with a lock to prevent race condition in module imports.

Portable pipeline execution assumes an execution model where Python SDK workers can launch multiple threads that can process the delegated by the runner. Up to 2.17.0 the model assumed 12 worker threads by default, possibly customized by pipeline options. As of 2.18.0, we will scale these threads dynamically, see: https://issues.apache.org/jira/browse/BEAM-8151
Part of work execution involves unserializing (unpicking) payloads, such as serialized DoFns. Pickling   depends on a pickler library (dill), and Python pickling mechanisms (pickle module). As a part of unpickling, python interpreter will dynamically load some modules referenced in users' code.
It appears that concurrent unpickling of data causes races during module imports, which causes deadlocks and/or corrupted imports, see: BEAM-8651. The error is very common on Python 3.x, including latest releases of Python 3.7.
This PR guards picking operations with a lock to mitigate these errors in Beam.

Run Python 3.5 PostCommit

Thanks, @ibzib ! All tests besides Direct Runner tests passed in the previous run: https://scans.gradle.com/s/eptpl337kz4ck.

Run Python 3.5 PostCommit


Incorrectly show coordinate axis of SequnceTube and annotation data

Hi, I'm trying to load my data to MoMI-G.
The vg file is converted from rgfa (minigraph) file. And xg file is generated using vg index. There is no problem visualizing the xg file on SequenceTubeMap (vgteam).

But when I loaded the xg file to MoMI-G, some problems arose. The coordinate axis of SequnceTubeMap is disordered and it's unable to show the annotation data on the website.

Additionally, the same problem arose when I converted the example data chm1_chr21.pcf to xg file using vg (v1.14.0):
pcf2gfa.sh chm1_chr21.pcf hg38_chr1.fa > chm1_chr21.gfa
vg view -Fv chm1_chr21.gfa > chm1_chr21.old.vg
vg mod -X 1024  chm1_chr21.old.vg >  chm1_chr21.vg
vg index -x  chm1_chr21.xg  chm1_chr21.vg

I checked the errors. It indicated that some wrong happens around GraphWrapper.tsx:

And it seems that the cause of the problems is unable to resolve the coordinate information in xg file.

Could you please help me with my problems?
Thanks!

Thank you for using MoMI-G. I fixed the error "Cannot convert undefined or null to object" in the latest master.
For the incorrect coordinate issue, our development team is currently investigating this. Would it be possible for you to wait for a while?

I cannot reproduce the

Thanks for your prompt reply! It's very strange that this problem can't be reproduced.
I cloned the last master of MoMI-G and tried again. I used the pcf2gfa.sh (the last master) and vg (v1.23.0) to convert chm1_chr21.pcf to xg file:
#reference genome is chromosome 21 of hg38
pcf2gfa.sh chm1_chr21.pcf hg38_chr21.fa > chm1_chr21.gfa
vg view -Fv chm1_chr21.gfa > chm1_chr21.old.vg
vg mod -X 1024  chm1_chr21.old.vg >  chm1_chr21.vg
vg index -x  chm1_chr21.xg  chm1_chr21.vg

This is the configuration file of docker I used:
FROM quay.io/vgteam/vg:v1.23.0 as build
# frontend container
FROM momigteam/momig-backend:latest

COPY --from=build /vg/bin/vg /vg/bin/
COPY static/chm1_chr21.xg /vg/static/
COPY static/config.yaml /vg/static/
EXPOSE 8081

CMD ["./graph-genome-browser-backend", "--config=static/config.yaml", "--interval=1500000", "--http=<IP_ADDRESS>:8081", "--api=/api/v2/"]

This is the configuration file of MoMI-G I used:
bin:
  vg: "vg"
  vg_tmp: "vg"
  vg_volume_prefix: ""
  graphviz: "dot"
  fa22bit: "faToTwoBit"
  bigbed: "bedToBigBed"
reference:
  chroms: "static/GRCh.json"
  data:
    - name: "hg38"
      features:
        - name: "gene_annotation"
          url: "static/gencode.v27.basic.annotation.gff3"
          chr_prefix: "chr"
data:
  - name: "chm1"
    desc: "2017/12/18"
    chr_prefix: "chr"
    ref_id: "hg38"
    source:
      xg: "/vg/static/chm1_chr21.xg"
      csv: "static/chm1_chr21.pcf"
    features:
      - name: "gene_annotation"
        url: "static/ensGene.bb"
        chr_prefix: "chr"
    static_files: []

But the incorrect coordinate issue still exists.

I'm wondering what commands you used to make xg file. And this is my xg file, you can download and test it.
https://www.dropbox.com/s/k217q15o03ro5gl/chm1_chr21.xg?dl=0

Thank you for sharing the detail. The configuration seems good to me. Is it possible to tell me the genomic range of the incorrect coordinate so that I can see the same view?

The genomic range: chr21:45,839,448-46,840,448

Still I failed to reproduce the same view.
Is it possible to use  momigteam/momig-backend:prod instead of momigteam/momig-backend:latest?
Is the frontend running the latest master of MoMI-G?

This is the my chm1 chr21 xg file.
https://www.dropbox.com/s/g9je846z0aka54x/chm1_chr21.xg?dl=0

I used momigteam/momig-backend:prod and it worked!

Although annotation path can be shown in SequenceTubeMap, an error occurred:
[2021-12-21T14:40:13Z DEBUG graph_genome_browser_backend::annotations] Unsupported format "static/ensGene.bed"

And why does annotation path miss as the genome range becomes longer?
Additionally, if I want to use MoMI-G without using Docker, which softwares should I install?

annotation path can be shown in SequenceTubeMap, an error occurred: Unsupported format "static/ensGene.bed"

Please use bigbed file instead of bed file.
    features:
      - name: "gene_annotation"
        url: "static/ensGene.bb"
        chr_prefix: "chr"

And why does annotation path miss as the genome range becomes longer?

This might be a bug, I fixed in #93. If annotation paths are still missing, please tell me.

Additionally, if I want to use MoMI-G without using Docker, which softwares should I install?

MoMI-G backend is available on github.

MoMI-G works perfectly now!

Thank you so much for your help!

Awesome! Please contact us if you have any questions.


Check the CI

A draft PR to check the status of the CI.

Manually git bisected my way to success

Manually git bisected my way to success


Allow visitor to set his/her name

Visitor can now set their name with setname $name
name can be long and has space between them

Hi @LinhNguyenLe2109 please rebase/merge the main branch to your branch to clear the merge conflicts.


how can I adjust the linewidth the diagnostics occupy? the diagnostics is beyond the window?

Hey there! I'm not actually the maintainer of this; this is a fork of a repository in SourceHut. I recommend you ask this question in upstream.


unexpected argument '--sqlite-path' found

I am trying to use the denokv standalone and am getting
error: unexpected argument '--sqlite-path' found

Usage: denokv --sqlite-path <SQLITE_PATH> serve [OPTIONS] --access-token <ACCESS_TOKEN>

For more information, try '--help'.

This is when running with either installing with cargo or the docker container, reproduce with:
export DENO_KV_ACCESS_TOKEN=$(openssl rand -base64 15)
docker run -it --init -p 4512:4512 -v ./data:/data ghcr.io/denoland/denokv serve --sqlite-path /data/denokv.sqlite serve --access-token $DENO_KV_ACCESS_TOKEN

Whoops... just needs to be fixed in the readme:
https://github.com/denoland/denokv/blob/main/README.md?plain=1#L20
https://github.com/denoland/denokv/blob/main/README.md?plain=1#L77
And the blog post:
https://deno.com/blog/kv-is-open-source-with-continuous-backup (first example)


Add github action to run unit test

Thanks @DannyFeliz for your support. This GitHub action that you've proposed will enhance our current workflow since we won't have to manually test all the changes that go to our develop or master branches.
Merging this PR, right away.


LCM-Calculator

Takes 2 numbers as input from the user and calculates the LCM of the entered 2 numbers and gives it as the output

@RuanAragao, I would like to work on this calculator. Kindly assign this issue to me.


Packaging the tests

Package up the tests so they can be installed via conda/pip and versioned.
It is possible to pass a package to pytest pytest --pyargs my-package which runs test code inside that package: https://docs.pytest.org/en/7.1.x/explanation/goodpractices.html#tests-as-part-of-application-code.
However, will need to test that custom command line args such as --control-path , --output-path etc still work (see https://stackoverflow.com/questions/41270604/using-command-line-parameters-with-pytest-pyargs). Otherwise will need to create wrapper commands that call pytest. (Will also need to restructure code into src and test, and add pyproject.toml etc.)
It will also need a think of how it is deployed in CI tests that is run on NCI. As the tests are currently run using the deployed payu module (https://github.com/ACCESS-NRI/payu-condaenv).  This package can be added to this conda environment but then the version in the payu conda environment is fixed - which could be fine when the package is in a stable state or the environment can be redeployed with an updated version.

Versioning tests were discussed in this issue https://github.com/ACCESS-NRI/access-om2-configs/issues/61, so older configurations can run earlier versions of test code.

Also add CI to test packaging, and also CI to run any local tests

Could use pip install PACKAGE_NAME as part of CI tests in access-om2-configs and reproducibility. A potential issue is in repro tests run on NCI, if there were two concurrent repro tests which were installing different versions


CaseStudies SharedState example update.

Repetitive fields from Counter.State and Profile.State have been moved into a separate CounterData struct. This makes the code more DRY and concise.

Going to close this for now, but if you revisit a slightly more complex example that you'd like to propose, feel free to open another PR!
And thanks again for taking the time to contribute :smile:

1


Create feature tests with coverage

I'll maybe try to mount the jacoco jar on a volume, instead of using a new image


closes #10 added ephemeral drive support

I started up a cluster using this branch & a multi-node zookeeper and found a bug where the myid file was not being created in the right location.  I created a PR to your branch.

found a bug where the myid file was not being created in the right location

Good catch.  I overlooked that when rebasing master to pull in your multiple zookeepers support.

Discussed this PR w/ @mikewalch on chat.   Came to the following conclusion.

Use map of instance type to # ephemeral.   Always setup and configure all ephemeral drives.
Only support instance types that have at least one ephemeral drive.  Always store all data on ephemeral.
In a follow on issue only support two instance types.  One for nodes that run datanodes, tservers, etc.  And one for nodes that run Accumulo master, zookeepers, namenode, etc.

@mikewalch did I miss anything?

No, you didn't miss anything

Are you going to drop support for declaring instance type in the [nodes] section of fluo-deploy.props?  Or should that be another issue?

Are you going to drop support for declaring instance type in the [nodes] section of fluo-deploy.props?

My thought was that could be done in the follow on issue for supporting just two instance types.   I could just drop it here if you think that makes sense.  I suppose since we don't know when that issue would be done, it may make sense for this commit to move things to a good state.

Up to you.. I think I can work on supporting two instance types this week so no problem if you want to leave it for later.

I tested 11f1d2 w/ m1.large and m1.xlarge instance types.  I saw that 2 and 4 ephemeral drives were setup and that the datanodes used them.

Up to you.. I think I can work on supporting two instance types this week so no problem if you want to leave it for later.

Ok, I will not remove it then.

+1


[Task]: Polish UI: Add on iOS css -webkit-user-select: none

Testing iOS is a nightmare without this simple change


SHAKE is Made Half its Security Level by Default in OpenSSL DGST

128 or 256 in SHAKE128 or SHAKE256 is not the length, it's the security level, i.e. SHA256 or SHA512 are 128 and 256 accordingly. SHAKE, when not double the size its said security level, gives half the promised security. Quote, "SHAKE256, with an output length of at least 64 bytes, provides 256-bit security against all attacks." Therefore the default of OpenSSL DGST is not properly set and makes a hash function twice as weak. This is essentially a bug, a defect, in the default length setting, that came from misreading of the actual specification.
Many people may trust OpenSSL and might use the default, please fix.

OTC: Because there might be already users depending on the existing default length we cannot change it. We just need to document that the defaults doesn't give the expected security level.

OTC: Because there might be already users depending on the existing default length we cannot change it.

That's a really suboptimal decision IMO. Users might just as well be depending on the fact that it does what it says on the tin, and likely in larger (though silent) numbers than those who depend on an unintentional degradation of the security level.

The decision was made because the EVP_DigestFinal API does not allow for passing output buffer length. That means the change could potentially trigger a security issue by overflowing the existing buffer. So no, we cannot change the decision.


[SQL Index and Data Source Cache] Some queries of TPC-DS failed if the table is partitioned.

When we test data source cache with partitioned table generated by tpcds-kits, some queries(q26, q27, q68 ...) failed thanks to the reason:

If the tables is not partitioned, all queries work well.

Have fixed.