Terminology: master --> control-plane

Kubernetes seems to being transitioning away from referring to control nodes as master in favor of control-plane.  We should confirm this and then do the same before we ship:

[x] Documentation should refactor: master --> control-plane and masters -> control-plane nodes
[x] Rename NodeRole.Master -> ControlPlane (value is control-plane)
[x] Rename APIs as well
[x] Update cluster definitions:

[x] neonKUBE
[x] neonCLOUD

[x] Rename ClusterDefinition.AllowPodsOnMasters --> AllowPodsOnControlPlane
[x] Update label selectors too
[x] rename:

[x] firstMaster --> firstControlPlane
[x] master -> controlPlane
[x] ClusterProxy methods and properties

[ ] Testing

[ ] Hyper-V cluster
[ ] neon-desktop built-in cluster
[ ] AWS
[ ] Azure
[ ] XenServer

@marcusbooyah: Here's the logs from kubeadm init
root@neon-desktop:/home/sysadmin# bash init.sh
W0723 00:51:54.954543    8614 common.go:83] your configuration file uses a deprecated API spec: "kubeadm.k8s.io/v1beta2". Please use 'kubeadm config migrate --old-config old.yaml --new-config new.yaml', which will write the new, similar spec using a newer API version.
W0723 00:51:54.955250    8614 initconfiguration.go:120] Usage of CRI endpoints without URL scheme is deprecated and can cause kubelet errors in the future. Automatically prepending scheme "unix" to the "criSocket" with value "/var/run/crio/crio.sock". Please update your configuration!
[init] Using Kubernetes version: v1.24.0
[preflight] Running pre-flight checks
        [WARNING SystemVerification]: missing optional cgroups: blkio
[preflight] Pulling images required for setting up a Kubernetes cluster
[preflight] This might take a minute or two, depending on the speed of your internet connection
[preflight] You can also perform this action in beforehand using 'kubeadm config images pull'
[certs] Using certificateDir folder "/etc/kubernetes/pki"
[certs] Generating "ca" certificate and key
[certs] Generating "apiserver" certificate and key
[certs] apiserver serving cert is signed for DNS names [e4c84e5d1bc5d492.neoncluster.io kubernetes kubernetes-controlplane kubernetes.default kubernetes.default.svc kubernetes.default.svc.cluster.local neon-desktop work] and IPs [<IP_ADDRESS> <IP_ADDRESS>]
[certs] Generating "apiserver-kubelet-client" certificate and key
[certs] Generating "front-proxy-ca" certificate and key
[certs] Generating "front-proxy-client" certificate and key
[certs] Generating "etcd/ca" certificate and key
[certs] Generating "etcd/server" certificate and key
[certs] etcd/server serving cert is signed for DNS names [localhost neon-desktop] and IPs [<IP_ADDRESS> <IP_ADDRESS> ::1]
[certs] Generating "etcd/peer" certificate and key
[certs] etcd/peer serving cert is signed for DNS names [localhost neon-desktop] and IPs [<IP_ADDRESS> <IP_ADDRESS> ::1]
[certs] Generating "etcd/healthcheck-client" certificate and key
[certs] Generating "apiserver-etcd-client" certificate and key
[certs] Generating "sa" key and public key
[kubeconfig] Using kubeconfig folder "/etc/kubernetes"
W0723 00:51:56.623976    8614 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Writing "admin.conf" kubeconfig file
W0723 00:51:56.734209    8614 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Writing "kubelet.conf" kubeconfig file
W0723 00:51:57.048966    8614 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Writing "controller-manager.conf" kubeconfig file
W0723 00:51:57.268167    8614 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Writing "scheduler.conf" kubeconfig file
[kubelet-start] Writing kubelet environment file with flags to file "/var/lib/kubelet/kubeadm-flags.env"
[kubelet-start] Writing kubelet configuration to file "/var/lib/kubelet/config.yaml"
[kubelet-start] Starting the kubelet
[control-plane] Using manifest folder "/etc/kubernetes/manifests"
[control-plane] Creating static Pod manifest for "kube-apiserver"
[control-plane] Creating static Pod manifest for "kube-controller-manager"
[control-plane] Creating static Pod manifest for "kube-scheduler"
[etcd] Creating static Pod manifest for local etcd in "/etc/kubernetes/manifests"
[wait-control-plane] Waiting for the kubelet to boot up the control plane as static Pods from directory "/etc/kubernetes/manifests". This can take up to 4m0s
[kubelet-check] Initial timeout of 40s passed.

Unfortunately, an error has occurred:
        timed out waiting for the condition

This error is likely caused by:
        - The kubelet is not running
        - The kubelet is unhealthy due to a misconfiguration of the node in some way (required cgroups disabled)

If you are on a systemd-powered system, you can try to troubleshoot the error with the following commands:
        - 'systemctl status kubelet'
        - 'journalctl -xeu kubelet'

Additionally, a control plane component may have crashed or exited when started by the container runtime.
To troubleshoot, list all containers using your preferred container runtimes CLI.
Here is one example how you may list all running Kubernetes containers by using crictl:
        - 'crictl --runtime-endpoint unix:///var/run/crio/crio.sock ps -a | grep kube | grep -v pause'
        Once you have found the failing container, you can inspect its logs with:
        - 'crictl --runtime-endpoint unix:///var/run/crio/crio.sock logs CONTAINERID'
error execution phase wait-control-plane: couldn't initialize a Kubernetes cluster
To see the stack trace of this error execute with --v=5 or higher
W0723 00:55:57.698013    9059 common.go:83] your configuration file uses a deprecated API spec: "kubeadm.k8s.io/v1beta2". Please use 'kubeadm config migrate --old-config old.yaml --new-config new.yaml', which will write the new, similar spec using a newer API version.
W0723 00:55:57.699782    9059 initconfiguration.go:120] Usage of CRI endpoints without URL scheme is deprecated and can cause kubelet errors in the future. Automatically prepending scheme "unix" to the "criSocket" with value "/var/run/crio/crio.sock". Please update your configuration!
[init] Using Kubernetes version: v1.24.0
[preflight] Running pre-flight checks
        [WARNING Port-6443]: Port 6443 is in use
        [WARNING Port-10259]: Port 10259 is in use
        [WARNING Port-10257]: Port 10257 is in use
        [WARNING FileAvailable--etc-kubernetes-manifests-kube-apiserver.yaml]: /etc/kubernetes/manifests/kube-apiserver.yaml already exists
        [WARNING FileAvailable--etc-kubernetes-manifests-kube-controller-manager.yaml]: /etc/kubernetes/manifests/kube-controller-manager.yaml already exists
        [WARNING FileAvailable--etc-kubernetes-manifests-kube-scheduler.yaml]: /etc/kubernetes/manifests/kube-scheduler.yaml already exists
        [WARNING FileAvailable--etc-kubernetes-manifests-etcd.yaml]: /etc/kubernetes/manifests/etcd.yaml already exists
        [WARNING SystemVerification]: missing optional cgroups: blkio
        [WARNING Port-10250]: Port 10250 is in use
        [WARNING Port-2379]: Port 2379 is in use
        [WARNING Port-2380]: Port 2380 is in use
        [WARNING DirAvailable--var-lib-etcd]: /var/lib/etcd is not empty
[preflight] Pulling images required for setting up a Kubernetes cluster
[preflight] This might take a minute or two, depending on the speed of your internet connection
[preflight] You can also perform this action in beforehand using 'kubeadm config images pull'
[certs] Using certificateDir folder "/etc/kubernetes/pki"
[certs] Using existing ca certificate authority
[certs] Using existing apiserver certificate and key on disk
[certs] Using existing apiserver-kubelet-client certificate and key on disk
[certs] Using existing front-proxy-ca certificate authority
[certs] Using existing front-proxy-client certificate and key on disk
[certs] Using existing etcd/ca certificate authority
[certs] Using existing etcd/server certificate and key on disk
[certs] Using existing etcd/peer certificate and key on disk
[certs] Using existing etcd/healthcheck-client certificate and key on disk
[certs] Using existing apiserver-etcd-client certificate and key on disk
[certs] Using the existing "sa" key
[kubeconfig] Using kubeconfig folder "/etc/kubernetes"
W0723 00:55:57.838298    9059 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Using existing kubeconfig file: "/etc/kubernetes/admin.conf"
W0723 00:55:57.993131    9059 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Using existing kubeconfig file: "/etc/kubernetes/kubelet.conf"
W0723 00:55:58.043327    9059 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Using existing kubeconfig file: "/etc/kubernetes/controller-manager.conf"
W0723 00:55:58.127389    9059 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Using existing kubeconfig file: "/etc/kubernetes/scheduler.conf"
[kubelet-start] Writing kubelet environment file with flags to file "/var/lib/kubelet/kubeadm-flags.env"
[kubelet-start] Writing kubelet configuration to file "/var/lib/kubelet/config.yaml"
[kubelet-start] Starting the kubelet
[control-plane] Using manifest folder "/etc/kubernetes/manifests"
[control-plane] Creating static Pod manifest for "kube-apiserver"
[control-plane] Creating static Pod manifest for "kube-controller-manager"
[control-plane] Creating static Pod manifest for "kube-scheduler"
[etcd] Creating static Pod manifest for local etcd in "/etc/kubernetes/manifests"
[wait-control-plane] Waiting for the kubelet to boot up the control plane as static Pods from directory "/etc/kubernetes/manifests". This can take up to 4m0s
[kubelet-check] Initial timeout of 40s passed.

Unfortunately, an error has occurred:
        timed out waiting for the condition

This error is likely caused by:
        - The kubelet is not running
        - The kubelet is unhealthy due to a misconfiguration of the node in some way (required cgroups disabled)

If you are on a systemd-powered system, you can try to troubleshoot the error with the following commands:
        - 'systemctl status kubelet'
        - 'journalctl -xeu kubelet'

Additionally, a control plane component may have crashed or exited when started by the container runtime.
To troubleshoot, list all containers using your preferred container runtimes CLI.
Here is one example how you may list all running Kubernetes containers by using crictl:
        - 'crictl --runtime-endpoint unix:///var/run/crio/crio.sock ps -a | grep kube | grep -v pause'
        Once you have found the failing container, you can inspect its logs with:
        - 'crictl --runtime-endpoint unix:///var/run/crio/crio.sock logs CONTAINERID'
error execution phase wait-control-plane: couldn't initialize a Kubernetes cluster
To see the stack trace of this error execute with --v=5 or higher
W0723 00:59:58.631494    9221 common.go:83] your configuration file uses a deprecated API spec: "kubeadm.k8s.io/v1beta2". Please use 'kubeadm config migrate --old-config old.yaml --new-config new.yaml', which will write the new, similar spec using a newer API version.
W0723 00:59:58.632315    9221 initconfiguration.go:120] Usage of CRI endpoints without URL scheme is deprecated and can cause kubelet errors in the future. Automatically prepending scheme "unix" to the "criSocket" with value "/var/run/crio/crio.sock". Please update your configuration!
[init] Using Kubernetes version: v1.24.0
[preflight] Running pre-flight checks
        [WARNING Port-6443]: Port 6443 is in use
        [WARNING Port-10259]: Port 10259 is in use
        [WARNING Port-10257]: Port 10257 is in use
        [WARNING FileAvailable--etc-kubernetes-manifests-kube-apiserver.yaml]: /etc/kubernetes/manifests/kube-apiserver.yaml already exists
        [WARNING FileAvailable--etc-kubernetes-manifests-kube-controller-manager.yaml]: /etc/kubernetes/manifests/kube-controller-manager.yaml already exists
        [WARNING FileAvailable--etc-kubernetes-manifests-kube-scheduler.yaml]: /etc/kubernetes/manifests/kube-scheduler.yaml already exists
        [WARNING FileAvailable--etc-kubernetes-manifests-etcd.yaml]: /etc/kubernetes/manifests/etcd.yaml already exists
        [WARNING SystemVerification]: missing optional cgroups: blkio
        [WARNING Port-10250]: Port 10250 is in use
        [WARNING Port-2379]: Port 2379 is in use
        [WARNING Port-2380]: Port 2380 is in use
        [WARNING DirAvailable--var-lib-etcd]: /var/lib/etcd is not empty
[preflight] Pulling images required for setting up a Kubernetes cluster
[preflight] This might take a minute or two, depending on the speed of your internet connection
[preflight] You can also perform this action in beforehand using 'kubeadm config images pull'
[certs] Using certificateDir folder "/etc/kubernetes/pki"
[certs] Using existing ca certificate authority
[certs] Using existing apiserver certificate and key on disk
[certs] Using existing apiserver-kubelet-client certificate and key on disk
[certs] Using existing front-proxy-ca certificate authority
[certs] Using existing front-proxy-client certificate and key on disk
[certs] Using existing etcd/ca certificate authority
[certs] Using existing etcd/server certificate and key on disk
[certs] Using existing etcd/peer certificate and key on disk
[certs] Using existing etcd/healthcheck-client certificate and key on disk
[certs] Using existing apiserver-etcd-client certificate and key on disk
[certs] Using the existing "sa" key
[kubeconfig] Using kubeconfig folder "/etc/kubernetes"
W0723 00:59:58.783924    9221 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Using existing kubeconfig file: "/etc/kubernetes/admin.conf"
W0723 00:59:58.927588    9221 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Using existing kubeconfig file: "/etc/kubernetes/kubelet.conf"
W0723 00:59:59.077837    9221 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Using existing kubeconfig file: "/etc/kubernetes/controller-manager.conf"
W0723 00:59:59.184082    9221 endpoint.go:57] [endpoint] WARNING: port specified in controlPlaneEndpoint overrides bindPort in the controlplane address
[kubeconfig] Using existing kubeconfig file: "/etc/kubernetes/scheduler.conf"
[kubelet-start] Writing kubelet environment file with flags to file "/var/lib/kubelet/kubeadm-flags.env"
[kubelet-start] Writing kubelet configuration to file "/var/lib/kubelet/config.yaml"
[kubelet-start] Starting the kubelet
[control-plane] Using manifest folder "/etc/kubernetes/manifests"
[control-plane] Creating static Pod manifest for "kube-apiserver"
[control-plane] Creating static Pod manifest for "kube-controller-manager"
[control-plane] Creating static Pod manifest for "kube-scheduler"
[etcd] Creating static Pod manifest for local etcd in "/etc/kubernetes/manifests"
[wait-control-plane] Waiting for the kubelet to boot up the control plane as static Pods from directory "/etc/kubernetes/manifests". This can take up to 4m0s
[control-plane] Creating static Pod manifest for "kube-apiserver"
[control-plane] Creating static Pod manifest for "kube-controller-manager"
[control-plane] Creating static Pod manifest for "kube-scheduler"
[kubelet-check] Initial timeout of 40s passed.
^C
root@neon-desktop:/home/sysadmin# ^C
root@neon-desktop:/home/sysadmin# ^C

Here's the debug code:

I fixed the "deprecated API spec" warning, but this is still failing.  systemctl status kubelet is reporting:
![Image](https://user-images.githubusercontent.com/4908431/180617914-854e3c1b-b323-410b-8fe2-27b00d9bbbe6.png)

I also noticed this in the cluster.yaml config file:

[ ] Cluster IP address is duplicated
[ ] Node hostname is duplicated
[ ] Does it make sense to include workstation name in the cert too?

It looks like kube-controller-manager isn't authorized to access the API server for some reason:

I think this is a problem with the control-plane-endpoint config which must have a DNS name which is not resolving, as this guy describes:
https://github.com/kubernetes/kubeadm/issues/2370
I did mess with this, so I'll have a look.

Hmmmm: Looks like HAProxy isn't running yet when we're calling kubeadm init... and the cluster.yaml file is specifying controlPlaneEndpoint: "kubernetes-control-plane:6442", but nobody is listening on that port.
We're starting HAProxy as a static pod, right?  I see the /etc/kubernetes/manifests/neon-etcd-proxy.yaml static pod manifest, but Kubelet doesn't appear to be starting that for some reason.

Ahhh, I think I found it!
kubelet logs:
Jul 25 17:24:34 neon-desktop kubelet[13458]: {"ts":1658769874853.2131,"caller":"config/file.go:187","msg":"Could not process manifest file","path":"/etc/kubernetes/manifests/neon-etcd-proxy.yaml","err":"invalid pod: [spec.containers[0].ports[0].name: Invalid value: \"k8s-controlplane\": must be no more than 15 characters]"}

value: "k8s-controlplane": must be no more than 15 characters]
This used to be "k8s-masters" which was smaller than 15 chars.

@marcusbooyah: Looks like we have another problem:

I don't think this is caused by my renaming:

@marcusbooyah: I rebuilt the setup container images before rebuilding the Hyper-V node image but it looks like I don't have the latest neonCLOUD changes:
The Helm chart for metrics-server specifies the image tag as 0.6.1 but the .version file for this image in neonCLOUD is set to v0.4.2.
Do you have any neonCLOUD changes that need to be pushed to master?

DONE!


medium .net exe not running when attaching to another process

I have an executable that is around 2.7 MB when I run the DonutTest without any process it executes the exe, but when I give another Process ID it does not work. I have tried with multiple processes.

This could be a number of issues such as incorrect process architectures, blocks by defenses, or something in the logic of the payload. I will close this issue for now. If you have more details you can post them here as a comment and reopen.


Updates to latest wabt and pares back to 1.0

Run make build.examples
# command-line-arguments
examples/testdata/cat.go:11:20: ReadFile not declared by package os
make: *** [Makefile:22: build.examples] Error 123
Error: Process completed with exit code 2.

oh maybe we should ping Go version (which is implicitly used by TinyGo) in example workflow

do we even need examples.yaml in the first place though? :D

if we stop compiling examples they won't build when we drift changes, so I think we have to

Thanks for the help.
to clarify, for some reason our examples and tests weren't re-compiling.. not sure why. Two things drifted:

since whenever last, wabt started using bulk-memory by default. I turned off all >1.0 flags defensively.
tinygo doesn't have one os.ReadFile, so I switched to ioutil

Should be ok again and thanks for the help!


Indefinite ProgressBar status

Sometimes it is useful to have indefinite progress.
Thank you

Hi, and thanks for the report!
Indeed, a spinner/waiting bar could be useful. You can have a look at https://github.com/deinstapel/cursive-async-view, they have some nice progress bars!

Thank you. Looks exactly like something I could use.

After looking at their examples more carefully, I actually do think, that this is not what I had in mind.
What I'm talking about is simply ProgressBarView with different style of "progress". One just like it is at the moment, and the other one "indefinite".


Configuration options for SSL validation

This request stems from #318.
I think the "target" moved slightly since this was opened.  It doesn't restore the ability to use HTTP instead of HTTPS, but a default reddit install using the install script is now HTTPS enabled using a self-signed certificate.  This allows you to ignore the certificate validation errors and successfully interact with an out of the box installation of reddit.
I did what testing I could, I wasn't familiar enough to generate new test cases for with and without valid certificates.

I'm not sure why Travis failed... It looks like on the image tests, one of them got a HTTP 502 and one got a HTTP 503 and the other doesn't say what the error is.
Are the JPG tests one of the flakey ones?  I've been unable to recreate the failure in python 2.7

Thanks for the PR. I'm having travis re-run the image tests. Yes, they are currently flakey (they're the only tests that still hit reddit servers while testing).

Awesome thanks.  It does look like it passed the second time around.
I'm still not sure what the best way to generate a test case for CI to test success with self signed certificates or not and the ini option... but for now I'll take the changes being in there and not breaking standard usage as success.

I'm still not sure what the best way to generate a test case for CI to test success with self signed...

I would suggest simply adding a test for when validate_certificate is false. That at least tests that nothing broke during the process. Then I'll simply take your word that it works on your local machine :).

All the comments you made should be integrated now.  I moved a couple extra things in praw.ini in to alphabet order (logging, store_json)  and put everything in the same order in configuration_files.rst

It also looks like Python 3.2 got a 503 again on the tests to cause the CI failure.

I restarted the build for you, so it's passing now. I'll let bboe look it over and decide whether to merge.

Looks great. Thanks!


Identify wsf with package header

Codecov Report

Merging #1011 (1563e4b) into dev (319938d) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #1011      +/-   ##
==========================================
- Coverage   76.96%   76.90%   -0.07%
==========================================
  Files         134      134
  Lines       12938    12915      -23
==========================================
- Hits         9958     9932      -26
- Misses       2980     2983       +3

Impacted Files
Coverage Δ

assemblyline/common/identify_defaults.py
100.00% <ø> (ø)

assemblyline/datasource/alert.py
88.46% <0.00%> (-3.85%)
:arrow_down:

assemblyline/odm/randomizer.py
94.39% <0.00%> (-0.87%)
:arrow_down:

assemblyline/odm/models/service_delta.py
100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


A11Y: WebView2 Sample is completely inaccessible with all the screen readers

Describe the bug

The WebView2 example in both (XAML Controls Gallery and WinUI3 Controls Gallery) is completely inaccessible with all the screen readers.
Steps to reproduce the bug

Steps to reproduce the behavior:

Go to 'Start menu'
Search for "XAML Controls Gallery or WinUI3 Controls Gallery (Either of the app should be installed)
As soon as the app opens up, search for WebView2 and click the search result. Now navigate via tab inside the WebView2 Sample.
Observe with any Screen reader, nothing will be read out.
Now press arrow keys to read the web content.
Observe, the situation is still same.
Try to close the app via Alt+F4.
App won't close.
Please see the attached app too for demo.

Expected behavior

WebView2/Sample should be accessible and all the web content should be read by the screen reader.
We should be able to close the app/window via Alt+F4
Actual behavior

As soon as we tab inside the WebView2 control, our focus gets trapped, screen reader doesn't read anything via tab or arrow keys and we are unable to close the app with Alt+F4.
Screenshots

Version Info

[ ] Xaml Controls Gallery - Downloaded from Store

If so, which version? (1.2.x -- Found on the Settings page of the app.)

[x] WinUI 3 Controls Gallery - Downloaded from Store

If so, which version? (1.3.x -- Found on the Settings page of the app.)
Microsoft.WinUI3ControlsGallery version <IP_ADDRESS>

[ ] App built locally from source

If so, which branch? (WinUI 3 versions are built off the winui3 branch)

Additional context

Windows version:

[x] Insider Build (22581)
[ ] May 2021 Update (19043)
[ ] October 2020 Update (19042)
[ ] May 2020 Update (19041)
[ ] November 2019 Update (18363)
[ ] May 2019 Update (18362)
[ ] October 2018 Update (17763)
[ ] April 2018 Update (17134)
[ ] Fall Creators Update (16299)
[ ] Creators Update (15063)
[ ] Anniversary Update (14393)

Device form factor:

[x] Desktop
[ ] Mobile
[ ] Xbox
[ ] Surface Hub
[ ] IoT

https://user-images.githubusercontent.com/6381747/161434117-343681bc-a6f4-4c79-a19d-0d443842ab71.mp4
Scarlett.zip

No, not at all.
It is not related

Hi I'm an AI powered bot that finds similar issues based off the issue title.
Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one. Thank you!
Closed similar issues:

Possible bug on webview2. (#5410),  similarity score: 0.73

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

Any update @gregwoo-microsoft

Thanks for filing this bug @akash07k. We are currently looking into solving this issue. I'll close this issue when this is fixed.

@akash07k thanks for following up. I'll defer to @karkarl who might have more insights on the progress made here.

@akash07k does this still repro with latest WinAppSDK ? There are several fixes in this space over the past year. The Alt+f4 issue is a known issue.

Does not repro in current version, webview is accessible by screen readers. Alt+F4 not working is a core webview issue tracked outside of this repo (https://github.com/MicrosoftEdge/WebView2Feedback/issues/2961)


code-format: fix up downstream clang issues

1050 branch didn't have CI running so some issues got in. Enable CI and fix issues.

This fixes the formatting issue but there's still a CI unit test failure. I'll let someone with more expertise look into that. I think we should at least merge this to get the formatting fixes in.


runtimeError:Errors in loading state_dict for ChatUniViLlamaForCausalLM

I have this problem when I run the video understanding api code. But I see the strict you set is false. Thus, I do not know the reason why I have this problem.
Before this error, I have the following error:

I solve this error through downloading the mm_projector.bin in this website https://huggingface.co/liuhaotian/LLaVA-13b-delta-v0/tree/main. Then I place it in a suitable flod.
I guess the first error maybe caused by the unsuitable mm_projector.bin. Can you help me to slove this problem. Thank you very much!

Firstly, there is no need to provide mm_projector.bin separately, as the mm_projector weights are already integrated into the model weights (see link).
I suspect this error may stem from incomplete weight downloads. You can try manually downloading the files locally:
pip install -U huggingface_hub
huggingface-cli download --resume-download Chat-UniVi/Chat-UniVi --local-dir ${model path}

then setting the model path as follows:
model_path = ${model path}

Thank you very much! But I have tried to manualy downloading the files locally, the following is my downloading bin files:

I use the xftp to transfer the bin files to Server, the size of bin files as the same as the local downloading bin files. I have face the same promblem: no such file mm_projector.bin

can you tell me what the problem about this ?

Did you set parameter model_base? model_base should be set to None during inference:
model_base = None

you are right. the model_base should be None. Thank you very much !


FAQ: Menu cut off

Where to find the issue
DE: https://www.coronawarn.app/de/faq/
EN: https://www.coronawarn.app/en/faq/
Describe the issue
The FAQ menu meanwhile got so long, it does not fit on my screen anymore:

The point "Datenschutz und Sicherheit" is out off and there is no option to scroll down further. The point "Geklärt" is thus not accessibly anymore from the FAQ menu.
Suggested change
Either make the FAQ menu scrollable, shorten the titles or find another solution.
Additional information
This is closely related to #882. If @MikeMcC399 would change his title and make the OP more general, I would agree to merge this issue and #882.

@Ein-Tim
Your problem has the same root cause as #882 which occurs on a mobile device, or on when Google Chrome is set to the minimum width for https://www.coronawarn.app/en/faq.
I can reproduce your issue under Microsoft Windows with Google Chrome viewing https://www.coronawarn.app/de/faq if I reduce the height of the browser window.
Unfortunately the implementation is not scalable to adding more and more sections to the list (EU Zertifikate was just newly added). We need the Open Source Team to acknowledge the issue and to provide a solution.
The issue #882 is a special case of the more generic one you are reporting here. I would like to leave the special case dedicated to that specific topic, so I suggest leaving both issues open.

@MikeMcC399

[...] so I suggest leaving both issues open.

Agreed! Thanks for your feedback!

@larswmh
Would you maybe like to take a look at this?

@Ein-Tim @MikeMcC399 @larswmh
I this been proposed by @larswmh to reduce the entries in the menu. E.g. keep EU certificates and remove from the menu all other certificate sections. Clicken on EU certificates with bring you to a fist entry in this section, which could be a kind of index of the following style:

EU certificates

FAQ entry 1
FAQ entry 2 ...

Vaccination certificates

FAQ entry 1
FAQ entry 2 ...

Recovy certificates

FAQ entry 1
FAQ entry 2 ...

Test certificates

FAQ entry 1
FAQ entry 2 ...

That way we would reduce the length of the menu.
Any other suggestions most welcome.

@dsarkar
Introducing a further level in the menu structure would shorten the menu, but unless the menu is scrollable, I'm not sure it would solve the issue on all devices (especially small mobile ones).

Internal Tracking ID: EXPOSUREAPP-8974


Deployments failing on App GW NSG rule validation after initial deployment

This is autogenerated. Please review and update as needed.
Describe the bug
After initial greenfield deployment, future deployments fail on Application Gateway NSG validation, previously reported in #21256 and was seemingly working again in a previous version.
Command Name
az deployment group create
Errors:
{"status":"Failed","error":{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"Conflict","message":"{\r\n  \"status\": \"Failed\",\r\n  \"error\": {\r\n    \"code\": \"ResourceDeploymentFailure\",\r\n    \"message\": \"The resource operation completed with terminal provisioning state 'Failed'.\",\r\n    \"details\": [\r\n      {\r\n        \"code\": \"DeploymentFailed\",\r\n        \"message\": \"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.\",\r\n        \"details\": [\r\n          {\r\n            \"code\": \"BadRequest\",\r\n            \"message\": \"{\\r\\n  \\\"error\\\": {\\r\\n    \\\"code\\\": \\\"ApplicationGatewaySubnetInboundTrafficBlockedByNetworkSecurityGroup\\\",\\r\\n    \\\"message\\\": \\\"Network security group /subscriptions/xxx/resourceGroups/rg-weu-aksaccelerator-1/providers/Microsoft.Network/networkSecurityGroups/appgw blocks incoming internet traffic on ports 65200 - 65535 to subnet /subscriptions/xxxx/resourceGroups/rg-weu-aksaccelerator-1/providers/Microsoft.Network/virtualNetworks/vnet-weu-aksaccel/subnets/appgw, associated with Application Gateway /subscriptions/xxx/resourceGroups/rg-weu-aksaccelerator-1/providers/Microsoft.Network/applicationGateways/appgw-weu-aksaccel. This is not permitted for Application Gateways that have V2 Sku.\\\",\\r\\n    \\\"details\\\": []\\r\\n  }\\r\\n}\"\r\n          }\r\n        ]\r\n      }\r\n    ]\r\n  }\r\n}"}]}}

To Reproduce:
Steps to reproduce the behaviour. Note that argument values have been redacted, as they may contain sensitive information.
Run the deployment a second time
Expected Behaviour
Deployment should succeed
Environment Summary
Windows-10-10.0.19043-SP0
Python 3.10.4
Installer: MSI

azure-cli 2.36.0

Additional Context

ARM

Network security group /subscriptions/xxx/resourceGroups/rg-weu-aksaccelerator-1/providers/Microsoft.Network/networkSecurityGroups/appgw blocks incoming internet traffic on ports 65200 - 65535 to subnet /subscriptions/xxxx/resourceGroups/rg-weu-aksaccelerator-1/providers/Microsoft.Network/virtualNetworks/vnet-weu-aksaccel/subnets/appgw, associated with Application Gateway /subscriptions/xxx/resourceGroups/rg-weu-aksaccelerator-1/providers/Microsoft.Network/applicationGateways/appgw-weu-aksaccel. This is not permitted for Application Gateways that have V2 Sku.

@bigglesuk69 Could you see if this error message is helpful to you?

@zhoxing-ms the rules are on the NSG, I have validated they are correct, so the error message is wrong. The issue is somewhere in the validation. To test I did the following.
Deleted the App GW, ran the deployment, successful
Run the deployment a second time, error is shown.
At all times the NSG has the correct rule.
NSG is deployed with no rules, then we inject in the pre-requisite rules using the Microsoft.Network/networkSecurityGroups/securityRules resource. If the validation only checking the Microsoft.Network/networkSecurityGroups resource for the rules? If so, that is a bug.

Some further information on this issue. I have a master bicep template that calls child modules for the deployments. So the vnet/NSG is deployed in one deployment, and the app GW in another. So what is happening is:

vnet deployment runs first time and succeeds (no AppGW deployed)
AppGW deployment runs successfully once the vnet deployment has completed
vnet deployment fails on all further runs

This error message is returned by the REST service, so this is a service issue


Add attributes for metacoq subpackages

Description of changes
The attributes for the relevant subpackages of metacoq have been forgotten during the initial release.
This PR re-export the corresponding derivations in the coqPackages.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg eval


Stuff

Must have

[ ] This app won't reply to ARP or anything so the router that sends traffic to us currently needs to have a static ARP entry to our MAC address.
[ ] On egress we don't currently rewrite the MAC address nor do we implement any form of routing. Perhaps "default" will do, ie we really just send to a pre-configured MAC address? regardless, we need to rewrite the destination MAC in the packet to that address before sending.
[ ] Configuration is currently completely static. We need to be able to reconfigure while the program is running. Other programs are doing this. I suppose we can get some inspiration from them. Just re-read config file every Nth second or so?
[ ] Support mitigation for multiple destination IPs with different rules!? At the very least I suppose we should differentiate so that if a source IP is blocked for attacking X it can still communicate with Y.

Nice to have

[x] Egress traffic back over the same physical interface it came in on but with a VLAN. This way we can halve the number of required 10G ports. (#3)
[ ] Egress traffic to actual egress PE router in the network. For this the traffic needs to be encapsulated with something like GRE or L2TPv3. We would also need a proper routing table so we know which PE to send the traffic to.

So I assumed there was an ARP app readily available. I was wrong. I really don't want to focus on writing one before the hackday so I suggest we stick with static ARP entry on the router and in SnabbDDoS we use static destination MAC for egress traffic.

Multiple mitigations is fixed through 42ce74a353d83c915aef3b213330f369aa0bbe82.

Mitigation configuration is now (after 95a07303b90451ae8e6f47a590d553ce0b819756 ) in a separate file but it's still not reloaded continuously.

I implemented reloading of the config in a9a0b7f1a43eb3fa16f9d57f964285f8b2af0cd9
It's rather ugly with re-reading the content of the file every second (called from periodic()) and if a change is detected we actually load the new config. self.sources is reset on loading of config so current state of sources is lost. We should obviously do this in a more graceful manner.

I'm closing this as I've created separate issues for all individual items.


Composer doesn't proxy websocket connection in the runtime env

Platformatic composer uses an http-proxy with a websocket option set to true. Although it doesn't work in the runtime env with a service domain names, because ws server doesn't use a fastify-undici-dispatcher. Not sure we can do anything here.
It's hard to catch this bug, we should log a warn message at least. If we can't solve this of course.
@mcollina wdyt?

Let's aim to fix this. It's not going to be easy tbh, we should aim to do it.


database is locked

数据存到远端smb共享的群晖文件夹，已经开启everyone读取/写入，docker-compose也给了权限，但还是提示database is locked，求解
docker-compose：
services:
bili-sync-rs:
image: amtoaer/bili-sync-rs:v2.0.0
restart: unless-stopped
network_mode: bridge
tty: true  # 该选项请仅在日志终端支持彩色输出时启用，否则日志中可能会出现乱码
hostname: bili-sync-rs
container_name: bili-sync-rs
environment:
- PUID=0
- PGID=0
volumes:
- /volume1/docker/bilibili/config:/app/.config/bili-sync
- /volume1/video/bilibili:/Videos/Bilibilis/
- /volume1/docker/emby:/emby
logging:
driver: "local"
日志：
[2024-04-18T12:27:11Z INFO  bili_sync_rs::config] 配置加载完毕，覆盖刷新原有配置
[2024-04-18T12:27:11Z INFO  bili_sync_rs::config] 校验配置文件内容...
thread 'main' panicked at src/main.rs:25:41:
called Result::unwrap() on an Err value: Execution Error: error returned from database: (code: 5) database is locked
Caused by:
0: error returned from database: (code: 5) database is locked
1: error returned from database: (code: 5) database is locked
2: (code: 5) database is locked
Stack backtrace:
0:
1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
stack backtrace:
note: Some details are omitted, run with RUST_BACKTRACE=full for a verbose backtrace.
[2024-04-18T12:27:17Z INFO  bili_sync_rs::config] 配置加载完毕，覆盖刷新原有配置
[2024-04-18T12:27:17Z INFO  bili_sync_rs::config] 校验配置文件内容...
之后一直循环

我试了试在群晖本地container manager部署又可以了！那我把问题关了吧


Doesn't run in Safari

In case you're unaware, http://wizgrav.github.io/clubber doesn't run in Safari (or Safari TP, or WebKit Nightly).
Web Inspector Console shows the following issues:
[Error] IndexSizeError (DOM Exception 1): The index is not in the allowed range.
Clubber (clubber.js:57)
Global Code (common.js:19)
[Error] TypeError: undefined is not an object (evaluating 'clubber.smoothing = [0.66, 0.8, 0.92][id]')
threshold (main.js:16)
Global Code (main.js:60)

Thanks, I was indeed unaware. It seems that requesting 4096 samples for the FFT may be too much for webkit. I put it down to 2048, I think this should fix it. Please test and tell so we can close this.

That has fixed it, although the performance is quite poor compared to Chrome.
Looks good to close though.


As an HR advisor I want to view and employee's details so that the employee's promotion request can be supported.

Implemented in: 9fb9a9276f3a0886c0474a0bf06f79cd389238d0


When user post a qutoes so it should be show at the top

When someone post a qutoes so it show in the bottom means last index.. i want that quotes should be show at the top 0 index... When someone post...

@danishzayan I want to work on this. Please assign it to me.

I want to work on this. Please assign it to me.

I want to work on this. Please assign it to me.

Done

@danishzayan I want to work on this. Please assign it to me.

Done... Raised some issue and tell me


Binder Kafka: unexpected consumer retries when max-attemps: 1 & enable-dlq: false

Describe the issue
If error is thrown from listener function, there are 9 unexpected retries without delays, although I explicitly set max-attempts: 1 in generic consumer configuration in yml. It's confusing, not covered in docs for max-attempts property.
To Reproduce
Message receiving class
@Service
class TestEventKafkaMessageReceiver {
    @Bean
    fun testEvent(): Consumer<String> {
        return Consumer { rawMessage ->
            throw RuntimeException("something happened with $rawMessage")
        }
    }
}

application.yml config
spring:
  cloud:
    function:
      definition: testEvent
    stream:
      kafka:
        bindings:
          testEvent-in-0:
            consumer:
              ack-mode: RECORD
              enable-dlq: false
      bindings:
        testEvent-in-0:
          binder: my-kafka
          destination: test-topic
          group: test-group
          consumer:
            max-attempts: 1
            start-offset: latest
      binders:
        my-kafka:
          type: kafka
          environment:
            spring:
              kafka:
                ### kafka cluster settings

Version of the framework
Spring Boot 2.7.5
Spring Cloud Dependencies 2021.0.5
Expected behavior
There is only one attempt to consume message, then listener commits offset and reads the next message.
Additional context
Linked Issues:
https://github.com/spring-cloud/spring-cloud-stream-binder-kafka/issues/1135
https://github.com/spring-cloud/spring-cloud-stream/issues/2205
PR which reduced number of tries from 10 * max-attempts to max-attempts, but not working for max-attempts=1:
https://github.com/spring-cloud/spring-cloud-stream-binder-kafka/pull/1136

If you want to disable retries altogether, add a ListenerContainerCustomizer and set the commonErrorHandler to a DefaultErrorHandler with a suitably configured BackOff and recoverer.

My team is experiencing the same issue so I created a sample project that is similar to what we have. I have application logs a Kafka DLQ consumer table available. You'll see that item with value 195 gets repeated.
Setting new FixedBackOff(0, 0) in the error handler does resolve this issue but the problem was unexpected and took us some time for us to figure out that was the issue.


Not able to start the Server(Wildfly 18 ) while updating the Dozer-Core 6.5.2 jars

Are you in the right place?

For issues or feature requests related to the code in this repository file a Github issue.
For general technical questions, post a question on Google Groups.

Whats your runtime?

Dozer version: 6.5.2
Dozer-Spring4: 6.5.2
OS version: Windows 10
JDK version: JDK 11

Whats the problem?
Steps to reproduce:

Caused by: org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type 'com.github.dozermapper.core.config.BeanContainer' available: expected at least 1 bean which qualifies as autowire candidate. Dependency annotations: {}"}}
11:43:01,797 INFO  [org.jboss.as.server] (ServerService Thread Pool -- 35) WFLYSRV0010: Deployed "napier-his-web.war" (runtime-name : "napier-his-web.war")
11:43:01,800 INFO  [org.jboss.as.controller] (Controller Boot Thread) WFLYCTL0183: Service status report
WFLYCTL0186:   Services which failed to start:      service jboss.deployment.unit."napier-his-web.war".undertow-deployment: java.lang.RuntimeException: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'enecoreservice': Unsatisfied dependency expressed through field 'registrationService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'com.napier.core.registration.service.external.RegistrationService': Unsatisfied dependency expressed through field 'regCoreService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'registrationCoreService': Unsatisfied dependency expressed through field 'customDozerBeanMapper'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'CustomDozerBeanMapper' defined in class path resource [dozer-spring.xml]: Cannot resolve reference to bean 'javaBeanMapper' while setting bean property 'inboundMapperBean'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'javaBeanMapper' defined in class path resource [dozer-spring.xml]: Unsatisfied dependency expressed through constructor parameter 1; nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type 'com.github.dozermapper.core.config.BeanContainer' available: expected at least 1 bean which qualifies as autowire candidate. Dependency annotations: {}

Observed Results:
What happened?  This could be a description, log output, etc.
Expected Results:
What did you expect to happen?
Link to GitHub repo with Unit test
The easiest way for us to fix issues, is if you provide a GitHub repo, using the latest release and a simple unit test.

Dozer is only tested against wildfly10

https://github.com/DozerMapper/dozer/blob/master/tests/dozer-wildfly-tests/pom.xml#L40

if you want it tested against 18, you'll need to update as the project isn't active.


Adding identity exchange

Adds identity exchange

Adds jshint
Adds request library
Added spec document

This pull request is now managed with CodeReviewHub. Refresh.

I'm going to close this for now. Please  re-submit when the tests are ready. We have PRs open for way too long around here.


Page Unresponsive/Frozen/Game Crash when...

When you intentionally do "Mario Star Get" animation from the taunt wheel it can sometimes freeze the website and/or crash the game; you cannot reset the game or reload the website. If you try to reload the page it will say "Page Unresponsive" forcing you to open SM64JS.com in another tab. This is more common if you do the taunt repeatedly without completing it.
This was proven in Cool, Cool, Mountain.
(https://user-images.githubusercontent.com/77451531/104614397-37e71780-5656-11eb-8696-b9ca9c4df1d1.png)

ok

ok

hmm, this doesn't happen to me. I can't reproduce this.

just spam the star taunt

hmm, this doesn't happen to me. I can't reproduce this.

just spam the star taunt

This issue happened to me, interesting, I'll check it out

This issue happened to me, interesting, I'll check it out

This issue happened to me, interesting, I'll check it out

its easier if you hold t and spam the star animation

There are more important bugs to fix

Think I got a fix working locally.


How to provide function

When i provide function or something that cannot be parcelized it becomes lost when the application kills the system, what should i do?

@T8RIN Sorry, I do not understand what exactly do you mean. Can you provide an explanation or an example?

@olshevski
  @Parcelize
     class Exit(
         @IgnoredOnParcel val onExit: () -> Unit = {}
     ) : Dialog()

    BackHandler {
         dialogController.show(Dialog.Exit(onExit = { this?.finishAffinity() }))
         onBack()
     }

    DialogNavHost(controller = controller) { dialog ->
         when (dialog) {
             is Dialog.Exit -> {
                 ExitDialog(onExit = dialog.onExit)
             }
         }
}

Something like this, i cannot parcel function, so i added IgnoredOnParcel annotation

@T8RIN You are doing something very tricky here 😀
Why don't you want just to set onExit listener inside DialogNavHost:
@Parcelize
     object Exit() : Dialog()

    BackHandler {
         dialogController.show(Dialog.Exit)
     }

    val activity = LocalContext.current as Activity
    DialogNavHost(controller = controller) { dialog ->
         when (dialog) {
             is Dialog.Exit -> {
                 ExitDialog(onExit = { activity.finishAffinity() })
             }
         }
     }

Something like this.

@T8RIN in this case it is possible, but when there are comes callback functions i cant just do that in nav host

@T8RIN Ok, I've looked through your code in the app and I see what you are doing.
Unfortunately, there is no way to serialize functions, because it is not data.
I wonder, why do you need DialogNavHost in your code in the first place? Do you switch between these dialogs anywhere in the app? It seems to me that it would be much easier to define simple composable dialogs in the place of use, thus the callback could be defined there. And if you have a set of dialogs that should be switched between, I also suggest you to define a smaller local DialogNavHosts with all callbacks defined in the place of use.
If it is not an option to you, and you really need to keep all dialogs in a single place, you could potentially think about some shared data holder + event dispatcher. But it will make things even more complicated.

@olshevski i did this in thoughts of making app more structured, to shift the call of dialogs to another element, function transfer works, even with repeated screen rotation or other global recompositions, but when not in the application for a long time, it is erased. will it be ok if i create a viewmodel to save this?

@T8RIN No, do not rely on view models. The whole process together with ViewModels may be destroyed and recreated later. Your callbacks however will be reset to the default ones {} in this case.
Try to rethink what you are actually doing here. If you  still want to go on with your current structure you need to decouple dialogs and screens that are awaiting callbacks. It could be done with some kind of events shooting from DialogHost through some channel/event bus and being consumed by each screen.


Spelling fixes in comments

Changed "Juse" to "Use".
Changed "miliseconds" to "milliseconds"

Thanks!@


Add GitHub workflow to create release and upload to PyPI when version number is bumped

This PR adds two workflows to replace our current Jenkins job to tag new versions of this package, and also adds a new digitalmarketplace-test-utils PyPI package.
The hope is that by uploading packages to PyPI we will be able to get Dependabot to manage our library versions across our apps, and save us a lot of work updating packages.

I think the only way to test this is to merge it and try making a release, so I'm going to merge this once it gets an approval and if we decide later we don't like it I will revert the changes.

🤞


Minor style updates and fixes for readthedocs issues

Proposed changes
Made some minor style updates for compatability with the new beginners guide notebooks.
Also fixed up some issues that are affecting correct rendering on the readthedocs page (headings order, spaces between bullet points and text, multiple formatting types).
Finally, re-wrote some of the instructional code so that it makes sense even if the text is left in the notebook rather than removed (as happens reasonably regularly.
Checklist (replace [ ] with [x] to check off)

[x] Notebook created using the DEA-notebooks template
[x] Remove any unused Python packages from Load packages
[x] Remove any unused/empty code cells
[x] Remove any guidance cells (e.g. General advice)
[x] Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
[x] Include relevant tags in the final notebook cell (refer to the DEA Tags Index, and re-use tags if possible)
[x] Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated
[x] Test notebook on both the NCI and DEA Sandbox (flag if not working as part of PR and ask for help to solve if needed)
[ ] If applicable, update the Notebook currently compatible with the NCI|DEA Sandbox environment only line below the notebook title to reflect the environments the notebook is compatible with

Note that the DEA logo doesn't render in the template until it's moved to a subdirectory due to the relative file link


Auxes not loading

When trying to create any action in companion for an aux in eventmaster the drop down selector for auxes reports "no auxes loaded yet and is therefore unusable.

Did you try a beta?

Just confirmed this works correctly in beta. Thanks
On Tue, Oct 3, 2023 at 11:31 PM Jeffrey Davidsz @.***>
wrote:

Did you try a beta?
—
Reply to this email directly, view it on GitHub
https://github.com/bitfocus/companion-module-barco-eventmaster/issues/64#issuecomment-1746116762,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACWQWA2YLCPYMTNQZEV6BKDX5TRBJAVCNFSM6AAAAAA46I4G66VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBWGEYTMNZWGI
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


ci: publish major and minor tags

Make sure major and minor tags are up to date on a patch release.
This would allow the common practice to use major tag on the action:
- name: Create bump and changelog
  uses: commitizen-tools/commitizen-action@0

(Ideally you may want to prefix version with v 🤷)
This what this third party action does under the hood:
$ TAG="0.21.0"
$ MINOR="${TAG%.*}"
$ echo $MINOR
0.21
$ MAJOR="${MINOR%.*}"
$ echo $MAJOR
0

This is used here for example: https://github.com/astral-sh/setup-uv/blob/main/.github/workflows/update-major-minor-tags.yml

Since you are using the default token with ${{ secrets.GITHUB_TOKEN }} it may not be triggered automatically, though :-/
I added the workflow_dispatch trigger to allow to run it manually on tags.


fixing pydantic schema to exclude parent class name

See https://github.com/lennartpollvogt/ollama-instructor/issues/6

See #6 why this PR was closed and not merged


Switch to MW 37 and PHP 8

I saw you will need this for micro-PKC. Not tested at all yet

Merged a squashed version in https://github.com/FantasticoFox/docker-aqua-mediawiki/commit/7bc895153fe990220e62c8c6b063073ded384d08


Update to packer 1.8.2

Updating to Packer 1.8.2

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.

@DocEmmetBrown could you please sign the SLA 🙏
We also need this update

🙏🏻 Need this one too lol

cli signed!

tagging @ksatirli @sylviamoss @lomar92 as y'all seem to be the maintainers of the repo ;)


Pro cloth-mask section needed.

In the last two days:
US health services group Deaconess Health Systems shares a cloth mask design as part of their appeal for cloth masks (March 20):
PDF: https://www.deaconess.com/How-to-make-a-Face-Mask/Documents-Mask/Mask-Information Video: https://youtu.be/9tBg0Os5FWQ
US health services group Providence shares a cloth mask design video (March 21) - part of their "100 million mask" initiative:
Video: https://vimeo.com/399324367/13cd93f150
flattenthecurve.com comes across as anti-mask, a little? A lot?  Site needs to change to mirror what's happening out there re cloth masks, and the home made potential of that.

And because there are going to be those that argue back, the studies are on my site https://cv-masks.github.io

Related: https://github.com/flattenthecurve/guide/pull/242

This issue can be closed now.
@steesdale, @mverzilli


Api audit changes

@shaojingli @gnavvy @Pessimistress @heshan0131 @apercu @abmai
Covers API audit comments on GeoJson, Polygon and Path layers

GeoJsonLayer getHeight => getElevation

GeoJsonLayer Docs: Clarified that elevation is always in meters for cartographic projection modes

PolygonLayer getHeight => getElevation

PolygonLayer Docs: Clarified that elevation is always in meters for cartographic projection modes

PolygonLayer Docs: documented missing props

PolygonLayer Docs: Marked lightSettings as experimental

PathLayer getWidth => getStrokeWidth

PathLayer strokeWidth => strokeWidthScale

PathLayer strokeMinPixels => strokeWidthMinPixels

PathLayer strokeMaxPixels => strokeWidthMaxPixels

HexagonLayer Docs: Marked lightSettings as experimental

HexagonLayer Docs: Marked selectedPickingColor as experimental

GridLayer Docs: Marked lightSettings as experimental

Cleanup: More layers now only imports the get utility function instead
of the full Container object from utils - this provides most of the
benefits with smaller impact on the code.

DOCS

Moved docs into 4.0 subfolder to support publishing multiple doc versions
from a single tree.

The new PathLayer includes all the prop name changes except strokeWidthMinPixels and strokeWidthMaxPixels.

The new PathLayer includes all the prop name changes except strokeWidthMinPixels and strokeWidthMaxPixels.

I believe they are in there:
@Pessimistress https://github.com/uber/deck.gl/pull/343/files#diff-86470456a0256bf322c15977dd5ec9c6R15


[Feature Request]: Implementing Bellman Ford algorithm in C++ under DSA folder.

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I am requesting the implementation of the Bellman-Ford algorithm in C++ for the Data Structures and Algorithms (DSA) folder. The Bellman-Ford algorithm computes the shortest paths from a single source vertex to all other vertices in a weighted graph. It can handle graphs with negative weight edges, which many other shortest path algorithms cannot.
Use Case
This feature would be a valuable addition to the DSA folder as it provides an essential algorithm for handling graphs with negative weights, a common requirement in various applications. A specific use case for this feature would be in educational settings where students learn about graph algorithms, particularly those dealing with negative weights.
Benefits
Implementing the Bellman-Ford algorithm in the DSA folder would enhance the repository by adding a crucial graph algorithm, aiding in educational purposes, and providing developers with a reliable solution for shortest path computations in graphs with negative weights.
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Ajay-Dhangar sir, kindly assign me this issue under GSSOC'24.

Hi @sjain1909
Here are some Guidelines for Writing DSA Problem Solutions for your reference. Please ensure that all your contents follow these guidelines so that we can maintain the quality of the solutions at a high level.
Feel free to ask if you have any questions.


Refactor ProfileViewModel to remove code that will not execute

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #[issue number] or Closes #[issue number]
Engineer Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the FHIRCore app to verify my change fixes the issue and/or does not break the app

Code Reviewer Checklist

[ ] I have verified Unit tests have been written for any new feature(s) and edge cases
[ ] I have verified any strings visible on UI components are in the strings.xml file
[ ] I have verifed the CHANGELOG.md file has any notable changes to the codebase
[ ] I have verified the solution has been implemented in a configurable and generic way for reuseable components
[ ] I have built and run the FHIRCore app to verify the change fixes the issue and/or does not break the app

Have you moved the code to a new destination? or just deleted it here?

Have you moved the code to a new destination? or just deleted it here?


control-service: update DataJobsController with account deployment data

what: Updated the DataJobsController and and associated service code, so that when a deployment is made or updated, the relevant data is written to the DB.
why:
Part of the Data Job configuration persistence initiative.
testing:
added test

I don't see the usage of the new Feature flag that you have introduced.

Does it make sense to create or update deployment when a job is created or modified?

Does it make sense to create or update deployment when a job is created or modified?

To me it does make sense to update the deployment when a job is modified - otherwise I think we would override it with a null value. Maybe it doesn't make sense to create an empty deployment when a job is created though.

Does it make sense to create or update deployment when a job is created or modified?

To me it does make sense to update the deployment when a job is modified - otherwise I think we would override it with a null value. Maybe it doesn't make sense to create an empty deployment when a job is created though.

Which fields will be updated when the job is modified?

Does it make sense to create or update deployment when a job is created or modified?

To me it does make sense to update the deployment when a job is modified - otherwise I think we would override it with a null value. Maybe it doesn't make sense to create an empty deployment when a job is created though.

Which fields will be updated when the job is modified?

When the update methods of the DataJobController class are called through the API there isn't any deployment information, that's why we only make sure not to override any existing deployment info.

Closing since with recent changes DataJobs don't have an associated deployment entity anymore.


(fix) Fix tip amount scaling in fulfillOrders

Motivation

Fixes https://github.com/ProjectOpenSea/seaport-js/issues/558
Solution

Codecov Report
Attention: Patch coverage is 99.17355% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 98.41%. Comparing base (f618d19) to head (4d32ae4).
Report is 62 commits behind head on main.

Files
Patch %
Lines

src/seaport.ts
97.14%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #559      +/-   ##
==========================================
+ Coverage   98.27%   98.41%   +0.13%
==========================================
  Files          35       35
  Lines       14526    14956     +430
  Branches      660      671      +11
==========================================
+ Hits        14276    14719     +443
+ Misses        245      232      -13
  Partials        5        5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


use more subtle styling for current line highlight

removes border
uses same colour as editor.selectionBackground for current line background

vscode removes current line highlight when making a selection, so this shouldn't get confusing

Hmm should these changes also be added here https://github.com/drcmda/poimandres-theme/blob/main/themes/poimandres-noitalics-color-theme.json ?

i scriptified it

Nice. Good call


update shuttle bus info

Types of changes
Please put an x in the box that applies

[ ] Bugfix
[ ] New feature
[ ] Refactoring
[ ] Breaking change (any change that would cause existing functionality to not work as expected)
[ ] Documentation Update
[X] Other (please describe)

Description
Describe what the change is
Update the shuttle bus departure time for the community track.
Steps to Test This Pull Request
Steps to reproduce the behavior:

Go to localhost:8000/conference/community-track

Expected behavior
The correct departure time.

Codecov Report

Merging #937 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #937   +/-   ##
=======================================
  Coverage   75.04%   75.04%
=======================================
  Files          80       80
  Lines        3082     3082
=======================================
  Hits         2313     2313
  Misses        769      769

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 40994e2...60f3f67. Read the comment docs.


doc feedback for client generation

Been reading through the docs and this PR has some minor feedback on things I've stumbled on.

No changes needing a change description found.

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs

Thank you so much for getting this merged!! Really appreciate it!


e2e tests failing on 2.9 because of extension version that shouldn't be available to install

Setup

Rancher version: 2.9
Rancher UI Extensions: any
Browser type & version:

Describe the bug

E2E tests on release-2.9 are failing because the extension tests are trying to install the clock extension 0.3.0, which is not compliant with 2.9, despite the presence of the annotation https://github.com/rancher/ui-plugin-examples/blob/main/index.yaml#L10

Elemental QA automated tests are failing because Elemental 2.0.0 is not available to install anymore on Rancher 2.9, which should be compliant and available

To Reproduce

Result

clock extension 0.3.0 is wrongfully available to install
elemental extension 2.0.0 is not available to install

Expected Result

clock extension 0.3.0 should not be available to install
elemental extension 2.0.0 should available to install

Screenshots

Additional context

Problem lies with semver.satisfies not considering suffixed version strings when doing a comparison, ex:

semver.satifies('2.9-4535jda324sff324-head, '>= 2.9') will return false

semver.satifies('2.0.2-rc.1', '>= 2.0.0') will return false

Note: Will need forward-porting to master

/forwardport v2.10.0

Reviewed and this LGTM since we have existing automation around extensions and there were some additional validations done with the PR.


Per page manifest settings

Is it possible to use a different start_url and scope parameters per page? A web-games website does this so that each game can be installed separately. Is this possible to do with this plugin?
Thanks in advance.

Hi @soylomass , this is not possible at the moment but we may think about an implementation that would make it possible :)

Thanks for your answer @kevinmarrec . Could you guide me a bit on where to start if I'd try to implement this myself?

@soylomass I would tell you first to fully understand the base implementation of the module here.
Especially the manifest (creation of manifest json) & meta (injection of manifest in meta tags) parts.
For now, the manifest json meta is added at root level, using the property app.head of Nuxt configuration to configure root head. By root I mean it will be added on all pages.
Then, the idea is to to use useHead Nuxt 3 composable to inject the manifest meta tag.
But to generate a manifest json with different start_url and scope parameters per page, I think it will need https://v3.nuxtjs.org/guide/features/server-routes#server-middleware to serve a on-demand manifest.json based on the page.
That said, AFAIK it's possible, but complex.

Hey @soylomass
I've implemented on-demand manifest.json through https://github.com/kevinmarrec/nuxt-pwa-module/commit/fa800723f86de346343a4ea434346d79beda6989
The server route is simple as serving a Javascript object : https://github.com/kevinmarrec/nuxt-pwa-module/blob/main/src/parts/manifest/handler.ts
We could think about using something like this for your case, but we'll also need to find out how to determine which part of your app is requesting manifest.json.
But for web-games website, it would need specific assets (icons/splash) for each game in addition to custom manifest.json
It would make things complex :)

Hi @soylomass,
I've been thinking couple of hours on this feature, and my opion is that this is too much complex.
More than generate a manifest.json on demand, it also means generating icons for each possible result, but also other things...
This has complexity on multiple levels.
So, unfortunately, I'm gonna close this issue as not planned.
Thank you in advance for your understanding :)


API function to verify password

Is your feature request related to a problem? Please describe.
Currently, I cannot verify the password on the server side to authenticate actions. For example, "Enter your password to delete this item".
Describe the solution you'd like
const isCorrect: boolean = await auth.api.verifyPassword({
    password,
});

Describe alternatives you've considered
Manually getting the data from the database and verifying it. This is somewhat hacky and I would not prefer it.

Nice would be an option to add it as a password entry try to limit requests. For example:
const isCorrect: boolean = await auth.api.verifyPassword({
    password,
    limited: true
});

With limited having false as a default value.

you can use the ctx api (which isn't documented yet)
const ctx = await auth.$context;
await ctx.password.verify({password: "", hash: ""});


Investigate build failures with Bazel 1.0.0

The best way to reproduce this is by using the ci/ scripts:
cd ci
sudo docker build .   --file kokoro/docker/Dockerfile.ubuntu   --tag gcpp-ci-ubuntu-18.04:latest
kokoro/docker/build.sh asan

Error is:
ERROR: /h/.cache/bazel/_bazel_aryan/a7ecd8237744645c5d189c197108d6d2/external/com_google_googleapis/BUILD.bazel:24:1: in srcs attribute of _generate_cc rule @com_google_googleapis//:_grpc_bigquerystorage_proto_grpc_codegen: '@com_google_googleapis//google/cloud/bigquery/storage/v1beta1:bigquerystorage_proto' does not have mandatory providers: 'proto'. Since this rule was created by the macro 'cc_grpc_library', the error might have been caused by the macro implementation in /h/.cache/bazel/_bazel_aryan/a7ecd8237744645c5d189c197108d6d2/external/com_google_googleapis/BUILD.bazel:24:1

I will take a stab at this.


wip: alchemy nft metadata

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #4563

PR #4574   👈

This comment was auto-generated by Graphite.


[HUDI-1468] Support custom clustering strategies and preserve commit …

…metadata as part of clustering
This PR is a re-work of #3211 with some minor changes and tests.
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure


Incorrect API call to list roles during group creation in organization

Describe the issue:
This API call is used to list roles in organization onboarding.
/t/carbon.super/o/api/server/v1/organizations/f87640bc-3a32-4253-85d4-7efc04465bac/roles
This needs to be changed into /o/scim2/v2/Roles
https://github.com/wso2/product-is/assets/25483865/64604201-a4a8-4fb6-815a-b6a61780d432

This issue is fixed from following PR.

https://github.com/wso2/identity-apps/pull/4760


Redesign: colours

This issue is for @melnikga who is already familiar with the work.
Goal:

Update colors, typography, button backgrounds etc to match the visual style of the new design here: https://www.figma.com/design/xFkTkTInz5pEiNEWd6tyQL/Cairo---Design-System?node-id=0-1&t=FnxRrPBnRCkriF4f-1
Update the dark mode according to Figma design. Also update the light mode to be consistent with the dark mode. We do not have the light mode design so use your design skills to make it look good

Do not do:

Do not move buttons or change layout as in the Figma design. Keep the app as is

I'll work on it.


Add Jenkinsfile

I would like to build this on my Jenkins server, but it doesn't have a Jenkinsfile, so I can't use BlueOcean.
Here is a sample Jenkinsfile that I use in one of my projects:
pipeline {
  agent any
  stages {
    stage('Build') {
      steps {
        sh 'bash ./gradlew build'
      }
    }

    stage('ProcessMod') {
      steps {
        sh 'mv build/libs/modid-1.0.jar build/libs/JosephCore.jar'
      }
    }

    stage('Archive') {
      steps {
        archiveArtifacts(artifacts: 'build/libs/*', fingerprint: true)
      }
    }

  }
}

I am not really sure why you would want to build this project with jenkins, as this repository is just a template with no actual functionality and only meant to be forked. If you want to build your own fork you could always just add the Jenkinsfile in your own repository only, or did I miss something?

I suppose, but it would be nice to provide the Jenkinsfile so others can (extremely) easily build our forks of the project on Jenkins.

Since I don't want to clutter the project with build system files I will just provide this simple pipeline script. Be warned i have no experience with the Jenkins pipeline, but it worked fine in my test runs. Just replace  and  and you should be good to go.
pipeline {
   agent any

   stages {
      stage('Setup') {
          steps {
            git 'https://github.com/<USER-NAME>/<FORK-NAME>.git'

            sh './gradlew ilbluInitApply'
          }
      }
      stage('Build') {
         steps {
            sh './gradlew jar sourcesJar javadocJar shadowJar ilbluPaperclip'

            archiveArtifacts 'ilblu-paperclip.jar, <FORK-NAME>-API/build/libs/*.jar'
         }
      }
   }
}


[11.x] Only create framework/maintenance.php file when using file driver

Change
When you're using the cache based maintenance mode, Laravel still tries to create the framework/maintenance.php file. However, this file does nothing when there's no down-file, so basically it is useless. I moved the creation of this file to the file based implementation, so it's only created when relevant.
N.B. I deliberately left the deletion in the up command to make sure it is always deleted.
Context
We're running our apps in multiple read-only containers and thus need to use the cache based maintenance mode. This works fine, but we always get an error because Laravel tries to create this framework/maintenance.php file on a read-only filesystem.

This is the file that actually handles the request though, as you can see in the default public/index.php. So, if this file doesn't exist, all of the maintenance mode features like bypass, except URIs, etc. would be broken when using the cache driver.

That maintenance file does nothing when there's no down-file, which is the case when you're using the cache driver. Also, when running a multi server environment, only the server where the down command is ran, will have the maintenance file and the others don't. Isn't this use case the whole point of the cache driver?
We're currently using features like bypass etc. using the cache driver on a read-only filesystem, so the maintenance file isn't present, and it is working fine. Please reconsider. I'm happy to help with another solution!

I don't think that file is even required for the file based maintenance driver any longer.
Since the real checks are done in Middleware/PreventRequestsDuringMaintenance.php
So maybe your PR should be to remove it completely, not only for the cache based driver.

@johanrosenson that middleware is not reach unless the standalone maintenance file allows the request to proceed to the framework with a bypass etc. as the middleware is also responsible for setting the bypass cookie, double checking everything etc..


Chore/migrate to dep and circle 2

This PR:

Migrates from Glide to Dep
Migrates from Circle 1 to Circle 2

Ahh...good point. I’ll do that this morning.
On Tue, Aug 21, 2018 at 3:27 AM Yevgeniy Brikman<EMAIL_ADDRESS>wrote:

@brikis98 commented on this pull request.
Thx! Could you search the docs for a mention of glide? I'm sure it's in
there and those instructions need to be updated to use dep.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gruntwork-io/terragrunt/pull/545#pullrequestreview-147933507,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ9zROresOTtwT4aYLi7tJ9MLjoeD67Oks5uS7Z4gaJpZM4WEjO5
.

--
Matt Calhoun
web http://gruntwork.io/ | in http://www.linkedin.com/in/mattcalhoun | @
https://twitter.com/mdcalhoun | m:<PHONE_NUMBER> <201-988-3187>


 I like the informal style this application has, but using slang like dafuq isn't that professional. Consider ditching 'dafuq' from Public Keys by default.
Will try to make some actually useful contributions in the future;).

Fin. Thank you! 💓


[WIP] Conditionally disable self signup

Motivation and context

How has this been tested?

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

We will close this PR because there is not any updates from your side.
Please, do not hesitate to notify us when you make some progress here and we will reopen it.
Thank you .


white list black list share sheet items

[[call before getAutoInstance()]]
examples:
Branch.includeInShareSheet("com.twitter.android");
Branch.excludeFromShareSheet("com.Slack");
first swing. @agrimn please test this out.

@EvangelosG  is there a  need for calling including /excluding before Branch.getInstace()? Can't we call this any time with the builder?

Other than the above  suggestions this LGTM . great work 👍

No need for calling including/excluding before getInstance, that was the original design before I refactored into the builder from the static Branch function.


Test ticket for Github FS Integration

 FreshService Ticket ID: 118 FreshService Ticket Agent: Sai Madhan K FreshService Ticket Agent Email<EMAIL_ADDRESS>Ticket Priority: Low FreshService Ticket Description: This is just a test ticket for Github FS Integration.Ticket attachments : 1.Freshservice-logo.png

Agent Sai Madhan K linked FreshService ticket 118 to this issue

testing a commment

Agent Sai Madhan K unlinked FreshService ticket 118 from this issue


Remove player.prepare() of prepare() in android native module

android.media.MediaPlayer.prepare() is not necessary because create() method call it automatically.
Also,  player.prepare() code in RNSoundModule.java throws an exception with { code: -1, message: null } with my device (Galaxy S7 edge, Android 7.0) and avd (Android 6.0 Google APIs, x86).
Since I removed player.prepare(), the error has gone.

reference

+1

I tested with MIUI and it works.

tks, the same problems cause,

These changes have been added to #169, so if that one gets merged we can close this PR.

Thanks for this - these changes have been merged in via #169 (that included your changes) so I'm closing.

Hi, this change actually the new URL support for Android, so I can no longer load sounds from the network. prepare must be called in this section, because we are not always using the create() method to create a new MediaPlayer.


Screenshots and examples

Sounds amazing. Please consider adding a few screenshots and working examples.

@justinlawrence There is live demo, you can see it here. Adding screenshots is a good idea as well.


Fix readme issue 21

So I am not sure that all this is really necessary and was expecting a apt-get install jekyll to do it but this is not the case? If this is not the case then this is good doc but I am not sure this is the right place. We have many lessons and why should this particular lesson be the place to document Jekyll installation and not others? Definitely we want to avoid duplicating information across lessons as well. So perhaps a separate repo?

I wanted to get a preview on my laptop before committing lessons and as I have a fresh Ubuntu it took me some time to figure out all the dependencies, so though might save some one else time .  In an ideal world "apt-get install jekyll" should work, unfortunately we do not live in one.

Why closing this? Why deleting the branch? I did not want this useful doc to disappear. Will it be applied to some other repo? What was the error that you got with apt-get install jekyll? I am asking this because I use standard jekyll package (not on Ubuntu) and did not have to go through the pain of installing "from sources".


Fix LIBFUZZER warning

Fix: #1734

This actually works? Why is the function is even there?

This actually works? Why is the function is even there?

Yes, LLVMFuzzerInitialize is no longer needed and does not affect


Cannot upload files from Safari, Chrome

When attempting to upload a main file to drafts, I can open the uploadcare modal and select a file, but when I confirm the file the modal closes and nothing happens.
https://user-images.githubusercontent.com/28573875/144205227-b092b178-1b32-4f82-8f2a-15168fb1a182.mov

Which browser is this happening in for you?
I cannot reproduce in Firefox, but was able to reproduce in Safari and Chrome.

Which browser is this happening in for you?
I cannot reproduce in Firefox, but was able to reproduce in Safari and Chrome.

Good question, I was using Safari when this error happened to me.

Debugging a bit right now and it seems like this is in part the result of the secure uploading.
The page asks the server for the security signature to upload the file with, but then when it is being uploaded it asks for it again, which updates the signature and then makes the widget close as a result.
I will disable the secure uploading for now and open up a new issue to make sure to re-enable it. This is primarily because it is taking me more time than I have for the bughunt tomorrow 😅


MaterialProduct class; in which package should this class belong?

in v0.8.0, MaterialProduct class belongs to the com.kazurayam.materialstore.reduce package as
https://github.com/kazurayam/materialstore/blob/main/src/main/java/com/kazurayam/materialstore/reduce/MaterialProduct.java
Is it the best choice?
Why not com.kazurayam.materialstore.filesystem.MaterialProduct?
Or why not com.kazurayam.materialstore.differ.MaterialProduct?

worked out at v0.8.3-SHAPSHOT


Migration to jackc/pgx for Postgresql ?

Hi,
I've seen that github.com/lib/pq is not in active development anymore, and the usage of pgx is recommended by maintainers.

This package is effectively in maintenance mode and is not actively developed. Small patches and features are only rarely reviewed and merged. We recommend using pgx which is actively maintained.

In the pgx's readme, a comparison is made with lib/pg :

For prepared queries with small sets of simple data types, all drivers will have have similar performance. However, if prepared statements aren't being explicitly used, pgx can have a significant performance advantage due to automatic statement preparation. pgx also can perform better when using PostgreSQL-specific data types or query batching. See go_db_bench for some database driver benchmarks.

Is pgx already compatible with ent ?

If no, is it planed to migrate to pgx as main postgres driver ?

Hey @sundowndev,
This issue already raised a few months ago - see https://github.com/facebook/ent/issues/605 and entgo.io/pgx.
There should be no problems working with pgx. If you face any issues with it, please open an issue.


ci: merge staging to master

This is an automatic PR generated by the pipeline CI/CD. This will be automatically fast-forward merged if successful.

Pipeline Attempt on 973932402 for 7cb9fd4de3f93da08711b3e58c47357898c33129
https://gitlab.com/MatrixAI/open-source/js-quic/-/pipelines/973932402

Pipeline Succeeded on 973932402 for 7cb9fd4de3f93da08711b3e58c47357898c33129
https://gitlab.com/MatrixAI/open-source/js-quic/-/pipelines/973932402


Fix yp_snprintf

Fixes #1057.

Perfect, thanks. I'm hoping this doesn't break old Visual Studio builds (we'll find out when it mirrors into CRuby). I never know when I'm using a function that old non-compliant Windows builds don't support...


Score view: filters get reset and middle click not working

Describe the bug
In the score view, you can't middle click on the button Online editor. Therefore, you have to open the editor in the same tab. The problem is that the filters (e.g. Show Build Failed) get reset if you navigate back.
To Reproduce

Go to the Score view of an exercise.
Select Show Build Failed
Click on Online Editor for a submission
Go back.

Expected behavior

Show Build Failed is still selected
Online editor is middle-clickable

Environment

 - OS: Linux
 - Browser: Firefox
 - Version: 72.0.1

The buttons on this page now support clicking them with the middle mouse button to open the link in a new tab.


avoid nested else ifs createFormData

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
If this is a new feature please add a description of what was added and why below:
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Unit tests

Checklist:

[x] My code follows the guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings or errors
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

closing in favor of https://github.com/forge42dev/remix-hook-form/pull/112
I don't want to use reduce due to performance impact, for loops are faster, although the separate function simplifies it and makes the code leaner I like to have it in once place as it's simple enough as it is


Dates fix

Thanks. ;)

Coverage remained the same at 72.414% when pulling b292f45e783c1f1382083daf928b29fc941f4982 on jontorrado:patch-1 into 61d463cf24fb6a4bafcba76aadcd4b7e350f7a7d on jkphl:master.


Fetching the highest value from CosmosDb table

Library name and version
Azure.Data.Tables 12.4.0
Query/Question
With the SDK of Azure.Data.Tables I’m trying to write a query that groups the data and fetches the highest value from each group. Is there a way to achieve this?
Currently I’m fetching all the data and executing the following LINQ query:
    public class SomeClass
    {
        public string CompanyName { get; set; }
        public long SomeValue { get; set; }
        public string ProperyAttribute1 { get; set; }
        public string ProperyAttribute2 { get; set; }
        public long ProperyAttribute3 { get; set; }
    }

    List<SomeClass> someList = FetchingDataFromCosmosDbTableStorage(); //fetching all the data
    var result = someList.GroupBy(x => x.CompanyName)
                         .Select(y => y.OrderByDescending(i => i.SomeValue).First())
                         .ToList();

Instead of filtering all the data in my application I would prefer to write a query to get the same result from CosmosDb Table.
Environment
Hosting platform: netcoreapp3.1
IDE: Rider 2022.1.2

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

List<SomeClass> someList = FetchingDataFromCosmosDbTableStorage(); //fetching all the data
var result = someList.GroupBy(x => x.CompanyName)
                     .Select(y => y.OrderByDescending(i => i.SomeValue).First())
                     .ToList();

Instead of filtering all the data in my application I would prefer to write a query to get the same result from CosmosDb Table.

The Tables API doesn't offer any kind of query functionality around grouping or ordering. If you go to full Cosmos DB though, I believe you can do these kinds of queries.


Login/register form styling

To test:

Try to login and register. Everything should work the same, it should just be colourful and prettier

Notes

Can't work out how to make the login box around it stop at the bottom of the form. Suggestions appreciated.
Register form needs more work but is still kinda nicer/a little less boring than before?

 Add CSS styling


Get types of arguments from event

I subscribe to events and i don't know exact format of data that will come.
How can i get type of argument in event?

solved http://stackoverflow.com/questions/42864239/autobahn-msgmapck-unpack-complicated-type-c/42883363#42883363

Thanks for linking SO answer - yeah, with complex payloads, figuring out static types in C++ manually can be annoying ..
Which reminds me that we should finally make progress on reflection (an advanced router feature), so people can come up with code generators. Will only work when payload types are non-polymorphic of course ..


fix @click.command example by adding parens

Otherwise the example fails with the following error:
"name = name or cmd.name AttributeError: 'function' object has no attribute 'name'".
More details: https://stackoverflow.com/a/51923415/4619705

Cherry picked to 1.0-maintenance as well: 363205bdc3abcfeed08c7e85e397fb7a7dc9ef07


Sync eng/common directory with azure-sdk-tools for PR 1498

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/1498  See eng/common workflow

/check-enforcer override

/check-enforcer override


Explicitly change the rds module to build a simple database

Context

Changes proposed in this pull request

Guidance to review

Relevant links
Things to check

[ ] I have added any new ENV vars in all deployed environments
[ ] I have tested any code added or changed
[ ] I have run integration tests

This forces a db rebuild, so we'll have to do the backup and restore explicitly


ng serve compiles sucessfully even if some assets are missing

Bug Report or Feature Request (mark with an x)
- [ x ] bug report -> please search issues before submitting
- [ ] feature request

Versions.

@angular/cli: 1.1.0
node: 7.8.0
os: win32 x64                          //(Windows 7)
@angular/animations: 4.1.3
@angular/common: 4.1.3
@angular/compiler: 4.1.3
@angular/core: 4.1.3
@angular/forms: 4.1.3
@angular/http: 4.1.3
@angular/platform-browser: 4.1.3
@angular/platform-browser-dynamic: 4.1.3
@angular/platform-server: 4.1.3
@angular/router: 4.1.3
@angular/cli: 1.1.0
@angular/compiler-cli: 4.1.3
Repro steps.

In angular-cli.json, add a non-existant file in the assets array.

The log given by the failure.

webpack: Compiled successfully when doing ng serve
Desired functionality.

I expected some sort of "file not found" error to appear.
Mention any other details that might be useful.

I'm trying to use a JavaScript file as a Service Worker. I cannot put it in /assets/ folder because that would change the scope of the SW to /assets/ instead of / (I tried doing that, and I had to use serviceWorker.register('/assets/sw.js') or Angular couldn't find the file). I'd like to put the file on the root, so I can have a root scope for my SW.
Working with the angular-cli.json file I realized there is no way of testing if the file I'm referencing to is recognized somehow by Angular CLI when doing ng serve so I tried to add a non-existant file in order to purposefully generate an error while compiling, which did not happen. So I have no way of checking if my asset references are correct or not. I tried doing <script src="sw.js"></script> at the index.html file and adding a console.log in order to check if the file is loaded or not. The sw.js file is in the src folder, and some relevant lines from the angular-cli.json file are the following:
"apps": [
    {
      "root": "src",
      "outDir": "dist",
      "assets": [
        "assets",
        "sws.js", //Non-existant file! It should be sw.js instead
        "favicon.ico"
      ],
      "index": "index.html",

And the SW registration script in my index.html:
<script>
      if('serviceWorker' in navigator){
      navigator.serviceWorker
        .register('/sw.js')
        .then(() => {
          console.log("Service Worker Registered. ");
        })
        .catch((err) => {
          console.log("Service Worker failed to register: " + err);
        })
      } else {
        console.log("Service what?");
      }
</script>

So I cannot find a way to include sw.js as an asset. I heard of Angular Mobile, but I want to start with something simpler first.
I used as reference to implement my SW the following:

https://coryrylan.com/blog/fast-offline-angular-apps-with-service-workers
https://www.youtube.com/watch?v=BfL3pprhnms

Thank you.

Looks like a problem with old version of the CLI. Let us close the issue for now. If it's still relevant, please open a new ticket with steps to reproduce and a minimal demo. Thanks!


Using Primeng File Upload in Formly Form component, not able to use NG_VALUE_ACCESSOR.

Question
When trying to use primengFile upload with Formly form, unable to use the NG_VALUE_ACCESSOR
where trying to append the formControl throws the error as shown below

I want this to be part of the Form, by including the formControl as shown below

but upon adding that am getting that error.
I have the below code but that works only for input type=file

I want to utilize the capability of primeng file upload as part of formly form.
Please suggest a solution here.

if p-fileUpload doesn't support Angular reactive form, try to write value manually using formControl.setValue:
@Component({
  selector: 'formly-field-file',
  template: `
    <p-fileUpload
--     [formControl]="formControl"
      ...
  `,

})
export class FormlyFieldFile extends FieldType<FieldTypeConfig> {
+ onChange(file) {
+   this.formControl.setValue(file);
+  }
}


Fixes undefined authbase breaking auth URLs

Fixes Error: Invalid URI "undefined/v1/authenticate/extensionApp.

Thank you for your pull request and welcome to our community! We require contributors to sign a Contributor License Agreement and we don't seem to have CLAs on file for these contributors to the Pull Request: (@jlowry). In order for your PR to be reviewed and merged, please follow the directions at the link above.
Project team: please do not merge this Pull Request until Foundation staff have confirmed that a CLA is in place for the new contributor(s) listed above.
If there are any questions, please don't hesitate to get in touch with our Infrastructure Support team. /CC @finos-admin


[5752] Amended cf to be kubectl

Context
Old cf
Changes proposed in this pull request
Use kubectl
Guidance to review
It works
Important business

[x] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[x] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[x] Do we need to send any updates to DQT as part of the work in this PR?
[x] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Support playbook: Update command to set feature flags


[cudamapper] Perf improvements for cudamapper

PR from @vellamike . Grabbing highlights from commit messages.

Multi-GPU support
Index caching
Improved multi-threaded overlap writing

rerun tests

I spotted two potential bugs (not sure if the second one is actually a bug):

read_ids of both query and target start from 0. This means that if we're not doing all-to-all cache could return query as target and target as query. I've been thinking if we should fix this in general (by setting a starting read_id for each index), but I'm still not sure if that's necessary

Agree. I have disabled caching for non-all-to-all (it isn't useful for this anyway)

get_index and evict_index are not thread-safe. I have a feeling you made sure that only one thread is active for each device at a time with the way you're dealing with ThreadPool at the end of the file, but that part is poorly documented so I'm not sure if I got it right. I think we should think about it a bit more, especially if we want to ready for using streams/multiple thread per device

I have documented the thread-per-device behaviour and also added mutexes to guard the cache.

Other comments are mainly about const-correctness and namings, so nothing crucial.

all addressed

I would suggest documenting the code better, especially the multi-threading part

rerun tests