make preview-all failing

Cloning latest repo and running make preview-all  fails with:
[multicloud-gitops]$ make preview-all
make -f common/Makefile preview-all
make[1]: Entering directory '/home/user/repos/GitHub/validatedpatterns/multicloud-gitops'
Error: open /home/user/repos/GitHub/validatedpatterns/multicloud-gitops/overrides/values-Azure.yaml: no such file or directory
Error: open /home/user/repos/GitHub/validatedpatterns/multicloud-gitops/overrides/values-Azure.yaml: no such file or directory
Error: open /home/user/repos/GitHub/validatedpatterns/multicloud-gitops/overrides/values-Azure.yaml: no such file or directory
Error: open /home/user/repos/GitHub/validatedpatterns/multicloud-gitops/overrides/values-Azure.yaml: no such file or directory
Error: open /home/user/repos/GitHub/validatedpatterns/multicloud-gitops/overrides/values-Azure.yaml: no such file or directory
common/scripts/preview.sh: eval: line 79: unexpected EOF while looking for matching "' common/scripts/preview.sh: eval: line 80: syntax error: unexpected end of file common/scripts/preview.sh: eval: line 79: unexpected EOF while looking for matching "'
common/scripts/preview.sh: eval: line 80: syntax error: unexpected end of file
common/scripts/preview.sh: eval: line 79: unexpected EOF while looking for matching `"'
common/scripts/preview.sh: eval: line 80: syntax error: unexpected end of file
make[1]: *** [common/Makefile:59: preview-all] Error 2
make[1]: Leaving directory '/home/user/repos/GitHub/validatedpatterns/multicloud-gitops'
make: *** [Makefile:12: preview-all] Error 2

Hi @wadebee
try running it using out utility container like so
./pattern.sh make preview-all
I am seeing the looking at the last error in the script.  It's a bit cryptic but I will look into it.
Best,
Lester

Hi @wadebee
So the issue with the error common/scripts/preview.sh: eval: line 80: syntax error: unexpected end of file has to do with the comments in the sharedValueFiles: section of values-hub.yaml and values-group-one.yaml files.
Here's a sample output from my laptop once I remove the comments in that section:
claudiol@fedora:multicloud-gitops (main *$%|u= origin/main)
$ ./pattern.sh make preview-all
make -f common/Makefile preview-all
make[1]: Entering directory '/home/claudiol/work/gitrepos/multicloud-gitops'
helm template common/acm --name-template acm -n open-cluster-management  -f /home/claudiol/work/gitrepos/multicloud-gitops/overrides/values-AWS.yaml   --set global.pattern=multicloud-gitops --set global.repoURL=https://github.com/claudiol/multicloud-gitops.git --set main.git.repoURL=https://github.com/claudiol/multicloud-gitops.git --set main.git.revision=main --set global.namespace=open-cluster-management --set global.hubClusterDomain=apps.lc-claudiol-dillon-6c0f-1.blueprints.rhecoeng.com --set global.localClusterDomain=apps.lc-claudiol-dillon-6c0f-1.blueprints.rhecoeng.com --set global.clusterDomain=lc-claudiol-dillon-6c0f-1.blueprints.rhecoeng.com --set global.clusterVersion=4.14 --set global.clusterPlatform=AWS
WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: /home/claudiol/work/clusters/us-west-1-claudiol-dillon-6c0f-1/auth/kubeconfig
---
# Source: acm/templates/policies/acm-hub-ca-policy.yaml
# This pushes out the HUB's Certificate Authorities on to the imported clusters
---
# Source: acm/templates/policies/application-policies.yaml
# TODO: Also create a GitOpsCluster.apps.open-cluster-management.io
---
# Source: acm/templates/policies/private-repo-policies.yaml
# We copy the vp-private-repo-credentials from the "openshift-gitops" namespace
# to the "open-cluster-management" via the "private-hub-policy"
#
# Then we copy the secret from the "open-cluster-management" namespace to the
# managed clusters "openshift-gitops" instance
#
# And we also copy the same secret to the namespaced argo's namespace
---
# Source: acm/templates/multiclusterhub.yaml
apiVersion: operator.open-cluster-management.io/v1
kind: MultiClusterHub
metadata:
  name: multiclusterhub
  namespace: open-cluster-management
  annotations:
    argocd.argoproj.io/sync-wave: "-1"
    installer.open-cluster-management.io/mce-subscription-spec: '{"source": "redhat-operators" }'
spec: {}
---
<SNIP for brevity>
# Source: hello-world/templates/hello-world-route.yaml
apiVersion: route.openshift.io/v1
kind: Route
metadata:
  labels:
    app: hello-world
  name: hello-world
spec:
  port:
    targetPort: 8080-tcp
  to:
    kind: Service
    name: hello-world
    weight: 100
  wildcardPolicy: None
make[1]: Leaving directory '/home/claudiol/work/gitrepos/multicloud-gitops'

Let me know if this helps.
Best,
Lester

Hi Claudio - I wasn't aware of the need for pattern.sh so thanks for that.
Is its use documented anywhere? I ask because we have a company TLS CA that is needed and I am getting TLS cert errors that I don't know how to resolve:
./pattern.sh make preview-all
make -f common/Makefile preview-all
make[1]: Entering directory '/home/user/repos/EnterpriseKubernetes/multicloud-gitops'
E0322 18:46:08.601821      37 memcache.go:238] couldn't get current server API group list: Get "https://api.mycluster.aroapp.azure.us:6443/api?timeout=32s": x509: certificate signed by unknown authority
E0322 18:46:13.728943      37 memcache.go:238] couldn't get current server API group list: Get "https://api.mycluster.aroapp.azure.us:6443/api?timeout=32s": x509: certificate signed by unknown authority
E0322 18:46:18.864424      37 memcache.go:238] couldn't get current server API group list: Get "https://api.mycluster.aroapp.azure.us:6443/api?timeout=32s": x509: certificate signed by unknown authority
E0322 18:46:23.998418      37 memcache.go:238] couldn't get current server API group list: Get "https://api.mycluster.aroapp.azure.us:6443/api?timeout=32s": x509: certificate signed by unknown authority
^Cmake[1]: *** [common/Makefile:59: preview-all] Interrupt
make: *** [Makefile:12: preview-all] Interrupt
make: *** wait: No child processes.  Stop.

Hi @wadebee
Ok so the utility-container definition lives here: https://github.com/validatedpatterns/utility-container
I have never seen this before. Perhaps it's the oc cli that is failing?  I think that if you look under common/scripts/preview.sh you will see that the calls to oc are probably failing.
It should be using your kubeconfig  but I'm not sure how your environment is set up.  If you want send me an email at<EMAIL_ADDRESS>and maybe we can get on a zoom or gmeet to discuss.
Best,
Lester

Hi Claudio - I wasn't aware of the need for pattern.sh so thanks for that.
Is its use documented anywhere? I ask because we have a company TLS CA that is needed and I am getting TLS cert errors that I don't know how to resolve:
./pattern.sh make preview-all make -f common/Makefile preview-all make[1]: Entering directory '/home/user/repos/EnterpriseKubernetes/multicloud-gitops' E0322 18:46:08.601821 37 memcache.go:238] couldn't get current server API group list: Get "https://api.mycluster.aroapp.azure.us:6443/api?timeout=32s": x509: certificate signed by unknown authority E0322 18:46:13.728943 37 memcache.go:238] couldn't get current server API group list: Get "https://api.mycluster.aroapp.azure.us:6443/api?timeout=32s": x509: certificate signed by unknown authority E0322 18:46:18.864424 37 memcache.go:238] couldn't get current server API group list: Get "https://api.mycluster.aroapp.azure.us:6443/api?timeout=32s": x509: certificate signed by unknown authority E0322 18:46:23.998418 37 memcache.go:238] couldn't get current server API group list: Get "https://api.mycluster.aroapp.azure.us:6443/api?timeout=32s": x509: certificate signed by unknown authority ^Cmake[1]: *** [common/Makefile:59: preview-all] Interrupt make: *** [Makefile:12: preview-all] Interrupt make: *** wait: No child processes. Stop.

Hi Wade,
the custom PKI is an interesting case that we never thought of. Could you try the following patch (assuming your custom CA is installed system-wide under /etc/pki):
diff --git a/common/scripts/pattern-util.sh b/common/scripts/pattern-util.sh
index 9cec19fa..ea02b7ef 100755
--- a/common/scripts/pattern-util.sh
+++ b/common/scripts/pattern-util.sh
@@ -56,6 +56,7 @@ podman run -it --rm --pull=newer \
        --security-opt label=disable \
        -e EXTRA_HELM_OPTS \
        -e KUBECONFIG \
+       -v /etc/pki:/etc/pki:ro \
        -v "${HOME}":"${HOME}" \
        -v "${HOME}":/pattern-home \
        ${PODMAN_ARGS} \

and report back if that fixes the x509: certificate signed by unknown authority errors? I suspect this is entirely orthogonal to the preview-all target failing, which we'll look into

Thank you @mbaldessari and @claudiol for the helpful pointers. I have been able to get this working now. This "client-side validation tool" should help speed up the inner loop of my config/override dev process.
A couple points of feedback:

Since I am using a Fedora based client to run VP - adding a mount to /etc/pki corrected my TLS issue.  I image non-Fedora users might need a different path like /usr/local/share/ca-certificates/.

I needed to remove 1 line from values-hub.yaml to get output. It was the - '/overrides/values-{{ $.Values.global.clusterPlatform }}.yaml' that appears to be a default from your upstream repo. Could you either remove that line from upstream or provide default values for each major platform (Azure/AWS) so that this setting works by default?

I was surprised that I need to be logged into a cluster for the make preview parsing to work. Since this is (I believe) supposed to be a client-side parsing engine, why is there a need to be connected to a remote? The forced connection appears to be coming from these 3 lines of common/preview.sh
platform=$(oc get Infrastructure.config.openshift.io/cluster  -o jsonpath='{.spec.platformSpec.type}')
ocpversion=$(oc get clusterversion/version -o jsonpath='{.status.desired.version}' | awk -F. '{print $1"."$2}')
domain=$(oc get Ingress.config.openshift.io/cluster -o jsonpath='{.spec.domain}' | sed 's/^apps.//')

Could these be converted to local variables by default with a dynamic runtime override? This would allow for better automated testing.

Thanks again for the fast help with this.
Maybe with a little extra VP doc about the need for a pattern.sh launch of make preview-all - this issue can be closed?

FWIW I merged https://github.com/validatedpatterns/common/pull/444 to fix the pki issue you saw (thanks again for bringing that up)


Add cluster-autoscaler variables to Terraform configs

What this PR does / why we need it:
Add cluster_autoscaler_min_replicas and cluster_autoscaler_max_replicas variables to Terraform configs. Those variables control the minimum and the maximum number of replicas for MachineDeployments. cluster-autoscaler must be enabled for those variables to have an effect.
Those variables are needed to implement the cluster-autoscaler questionnaire as part of #2455.
Which issue(s) this PR fixes:
xref #2455
What type of PR is this?
/kind feature
Does this PR introduce a user-facing change? Then add your Release Note here:
Add `cluster_autoscaler_min_replicas` and `cluster_autoscaler_max_replicas` variables to Terraform configs. Those variables control the minimum and the maximum number of replicas for MachineDeployments. cluster-autoscaler must be enabled for those variables to have an effect

Documentation:
TBD

/lgtm
/approve


info • The import of 'package:image_cropper_platform_interface/image_cropper_platform_interface.dart' is unnecessary because all of the used elements are also provided by the import of 'package:image_cropper/image_cropper.dart' • packages/smooth_app/lib/pages/image_crop_page.dart:6:8 • unnecessary_import

The test fails because of this error: ...
However my IDE throws an error when removing the import.

Maybe because there was a recent change in the crop package version. Try to use the latest version.
Originally posted by @monsieurtanuki in https://github.com/openfoodfacts/smooth-app/issues/1765#issuecomment-1119478221

Note, this is happening in develop


ŗocks taxonomies misses data

Hi,
Reminiscent of #24, for some asteroids running rocks taxonomies again comes up with no results while rocks taxonomy works as expected. I suppose it is likely to again be on the SsODNet side?
Example case:
$ rocks taxonomies 2011 CG2
No taxonomies on record for (None) 2011 CG2.
$ rocks taxonomy 2011 CG2
X

This is with rocks 1.8,9.
Thanks!

Hi Lauri, yes this is a datacloud issue that we're working on. You can check if it's rocks or SsODNet using
$ rocks taxonomy.links 2011cg2
which gives you the link to the relevant datacloud json file. In this case, it says data:null.
I'll leave the issue open until we pushed the fixed datacloud version.

Issue in datacloud is fixed.


[Rancher2] Review getting started guide for usability for first-time users

Identify any areas that can be improved. This issue can be used for discussion.
cc @leodotcloud @spgardiner

Addressed by content restructure (Divio) in new site.


Try pip-tools

Reference: https://github.com/jazzband/pip-tools/.
Does work with Poetry?

Also: https://www.b-list.org/weblog/2023/dec/07/pip-install-safely/


Change the logo of the website in the Tab

I would like to change the logo the website as it is not clearly seen , add some CSS to it

Please assign me this issue. I can fix this issue

Assigned you the issue


Adding an eye icon to the password input field that allows users to toggle password visibility.

@GVishnudhasan assign this to me i will done it today


Fix code signing issue on MacOS

This closes https://github.com/phoenixframework/tailwind/issues/39
The problem stems from MacOS associating signature information with the vnode, and it isn't recomputed if the file already exists, leading to a cryptic error if you upgrade tailwind and overwrite the binary.
The solution is to first delete the file, which seems to force recomputation of the signature since a new inode/vnode is created when you download the file again.
It's not the cleanest thing, but without this upgrading tailwind on MacOS leads to super cryptic errors.
More color on the issue here: https://stackoverflow.com/questions/67378106/mac-m1-cping-binary-over-another-results-in-crash

❤❤❤🐥🔥


[EN] Feedback for: /updates/2015/04/cut-and-copy-commands.html

Font styling is broken on this.  Some headings are way to large.

This was fixed as part of a larger bunch of commits.


update url to currently playing ayah

Deployed to: http://staging.quran.com:33197

Deployed to: http://staging.quran.com:32777

Deployed to: http://staging.quran.com:32780

Deployed to: http://staging.quran.com:32801

@naveed-ahmad can i close this for now?

Sure, this was stalled anyway :)


Issue #49297 Help Button is now Obvious

Added more information in the Reset and Help buttons.
Used the design recommended in the issue: https://forum.freecodecamp.org/t/new-help-button-not-obvious/571552/7.

Checklist:

[ ] I have read and followed the contribution guidelines.
[ ] I have read and followed the how to open a pull request guide.
[ ] My pull request targets the main branch of freeCodeCamp.
[ ] I have tested these changes either locally on my machine, or GitPod.

Closes #49297

Added texts inside the Reset and Help buttons along with the SVG.
The code output for Reset and Help buttons :

No merge conflicts in this PR until now.

As a new contributor, we encourage you to read our contributing guidelines.
We expect our contributors to be aware of the process specific to this project. Following the guidelines religiously earns you the respect of fellow maintainers and saves everyone time.
Some examples of this are:

Do not edit files directly through GitHub – while you can, it's not a good idea.
Make sure you follow the PR checklist and not just tick things off; otherwise, we won't take you seriously.
Use the correct way to link issues in the description of the PR by updating the XXXXXX. Do not just add issue numbers everywhere and anywhere you feel like.
Keep mentions and review requests to a minimum. We understand you are excited about contributing, and our maintainers will get back to you as soon as they get a chance.
Do not work directly off your main branch - create a new branch for the changes you are working on.

We appreciate you taking the time to help us, and we hope to see more contributions from you.
Happy Contributing.


Cloud (Mie) scattering using Ackerman and Marley model

#431

Vapor pressures: (psat.py)

we should add more condensates at intermediate temperature.

This is kind of my memorandom rather than Issue, but in mieclouds branch, I found the default range of Mie_Lognormal is 1 - 1000 nm.  ref: https://pymiescatt.readthedocs.io/en/latest/forward.html#Mie_Lognormal
In mie.py or pardb.py we need to specify the optimal upper and lower when generating miegrid.

Finally, the Mie computation agree with the VIRGA results:

exojax/tests/integration/clouds/amclouds_comparison_virga.ipynb

In this case, the geometric approximation is not bad...


Expected behavior question

Quick question. I'm noticing that EPlusTV looks like it's passing the equivalent of each stream's master (adaptive bitrate) m3u8 for every channel now. I noticed it first because Fox Sports streams were ignoring the resolution limit and trying to play 4K in all of my players. Then, I noticed that the ESPN+ NHL feeds would play in 30fps in one player and 60fps in another. So, I opened each with FFMPEG and saw that all of the resolutions/bitrates are in the m3u8. I could select them using FFMPEG's -map flag and they all work. Is this expected behavior or did something in the code or with the streamers change? No hurry, since this isn't the end of the world. Just wondering.

Yeah, that's intended. I have to do the FOX resolution stuff because it won't grab the 4K HDR/SDR manifest if you don't explicitly ask for it


Extra File Storage for academic service plans

Academic service plans are not listed on the eligible list. If M365 A* SKUs are not eligible for the extra file storage described here, please add a note and if possible, provide a link to a page about extra file storage for academic plans.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b31aa550-3d84-95b8-6a74-b3e7ad1cbfe9
Version Independent ID: 795391d3-aced-e2f3-f52d-034e14fe1aee
Content: Add storage space for your subscription
Content Source: microsoft-365/commerce/add-storage-space.md
Service: o365-administration
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@redmondChris Thank you for your feedback. I have added  Office 365 A3 and A5(Faultly) to the document as i can see the option to Purchase extra file storage on them. This can be tracked via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/5454
Once the author verifies and approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @redmondChris Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Misleading license info

Near the bottom of gtk4-rs/gtk4/README.md it states that "gtk4 is available under the MIT License" which I find misleading, because gtk4 itself is LGPL.  I believe this should say "gtk-rs" or similar instead of "gtk4".

All the readmes needs an overall update, see #74


improve accesslog mode e2e tests

Please provide a description of this PR:
fixes: https://github.com/istio/istio/issues/44759

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-mc_istio
820c3b0bafe40ea0f204470e687ff6f624712920
link
true
/test integ-telemetry-mc

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-mc_istio
820c3b0bafe40ea0f204470e687ff6f624712920
link
true
/test integ-telemetry-mc

gencheck_istio
f1996dfdacbbcf4ff4ce4db3d7b2447d162b23cb
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-mc_istio
820c3b0bafe40ea0f204470e687ff6f624712920
link
true
/test integ-telemetry-mc

gencheck_istio
f1996dfdacbbcf4ff4ce4db3d7b2447d162b23cb
link
true
/test gencheck

lint_istio
f1996dfdacbbcf4ff4ce4db3d7b2447d162b23cb
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-mc_istio
820c3b0bafe40ea0f204470e687ff6f624712920
link
true
/test integ-telemetry-mc

gencheck_istio
f1996dfdacbbcf4ff4ce4db3d7b2447d162b23cb
link
true
/test gencheck

lint_istio
f1996dfdacbbcf4ff4ce4db3d7b2447d162b23cb
link
true
/test lint

analyze-tests_istio
f1996dfdacbbcf4ff4ce4db3d7b2447d162b23cb
link
true
/test analyze-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-mc_istio
820c3b0bafe40ea0f204470e687ff6f624712920
link
true
/test integ-telemetry-mc

gencheck_istio
f1996dfdacbbcf4ff4ce4db3d7b2447d162b23cb
link
true
/test gencheck

lint_istio
f1996dfdacbbcf4ff4ce4db3d7b2447d162b23cb
link
true
/test lint

analyze-tests_istio
f1996dfdacbbcf4ff4ce4db3d7b2447d162b23cb
link
true
/test analyze-tests

integ-telemetry_istio
f1996dfdacbbcf4ff4ce4db3d7b2447d162b23cb
link
true
/test integ-telemetry

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain Thanks for the fix! If this fix also applies to other branches, can you cherry-pick it to other branches?

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.15":
Applying: improve accesslog mode e2e tests
Using index info to reconstruct a base tree...
A	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/stats/prometheus/nullvm/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/stats/prometheus/nullvm/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 improve accesslog mode e2e tests
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45518

In response to a cherrypick label: new pull request created: #45519

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.16":
Applying: improve accesslog mode e2e tests
Using index info to reconstruct a base tree...
M	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/api/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/api/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 improve accesslog mode e2e tests
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45520

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.15":
Applying: improve accesslog mode e2e tests
Using index info to reconstruct a base tree...
A	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/stats/prometheus/nullvm/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/stats/prometheus/nullvm/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 improve accesslog mode e2e tests
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45521

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.17":
Applying: improve accesslog mode e2e tests
Applying: retry
Using index info to reconstruct a base tree...
M	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/api/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/api/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0002 retry
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45522

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.15":
Applying: improve accesslog mode e2e tests
Using index info to reconstruct a base tree...
A	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/stats/prometheus/nullvm/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/stats/prometheus/nullvm/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 improve accesslog mode e2e tests
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45523

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.16":
Applying: improve accesslog mode e2e tests
Using index info to reconstruct a base tree...
M	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/api/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/api/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 improve accesslog mode e2e tests
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45524

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.15":
Applying: improve accesslog mode e2e tests
Using index info to reconstruct a base tree...
A	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/stats/prometheus/nullvm/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/stats/prometheus/nullvm/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 improve accesslog mode e2e tests
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45525

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.16":
Applying: improve accesslog mode e2e tests
Using index info to reconstruct a base tree...
M	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/api/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/api/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 improve accesslog mode e2e tests
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45526

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.17":
Applying: improve accesslog mode e2e tests
Applying: retry
Using index info to reconstruct a base tree...
M	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/api/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/api/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0002 retry
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45527

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.16":
Applying: improve accesslog mode e2e tests
Using index info to reconstruct a base tree...
M	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/api/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/api/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 improve accesslog mode e2e tests
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45528

In response to a cherrypick label: new pull request could not be created: failed to create pull request against istio/istio#release-1.18 from head istio-testing:cherry-pick-45504-to-release-1.18: status code 422 not one of [201], body: {"message":"Validation Failed","errors":[{"resource":"PullRequest","code":"custom","message":"A pull request already exists for istio-testing:cherry-pick-45504-to-release-1.18."}],"documentation_url":"https://docs.github.com/rest/reference/pulls#create-a-pull-request"}

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.17":
Applying: improve accesslog mode e2e tests
Applying: retry
Using index info to reconstruct a base tree...
M	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/api/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/api/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0002 retry
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45529

In response to a cherrypick label: #45504 failed to apply on top of branch "release-1.17":
Applying: improve accesslog mode e2e tests
Applying: retry
Using index info to reconstruct a base tree...
M	tests/integration/telemetry/api/accesslogs_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/telemetry/api/accesslogs_test.go
CONFLICT (content): Merge conflict in tests/integration/telemetry/api/accesslogs_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0002 retry
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45530

In response to a cherrypick label: new pull request could not be created: failed to create pull request against istio/istio#release-1.18 from head istio-testing:cherry-pick-45504-to-release-1.18: status code 422 not one of [201], body: {"message":"Validation Failed","errors":[{"resource":"PullRequest","code":"custom","message":"A pull request already exists for istio-testing:cherry-pick-45504-to-release-1.18."}],"documentation_url":"https://docs.github.com/rest/reference/pulls#create-a-pull-request"}


WALLET NUMBER OF BLOCK STUCK

Hi there,
I upgraded my Verge wallet to the new version that came out last month, on my Win7 64 bit.
After I downloaded the upgraded version, I received a message that told me something (don't remember what) was corrupted. Pretty sure nothing was corrupted as the previous version was working just fine, but because of that, it has to download all the blocks back from 2014.
Now, it's like two days the block is stuck on May 26th 2019, at 19 weeks to be completed.
How do we solve this problem?
P.S. If I posted the issue in the wrong section then let me know where is the proper section for this issue.
Thanks

Hey @kevintenda, could you be so kind and tell me which version of Verge you are currently running?
Also, you can speed up your syncing by just downloading the correct block data from here:
https://verge-blockchain.com/
Please make sure to download the correct data for your version.

you need a new binary, not to just delete the blockchain data. check the releases tab here and get the newest executable for your operating system

@marpme Verge 5.3.1
I already had previous one Lumos, and upgraded to this new release of mid-Sept.
I don't think it's a problem of speed up the chain because I already have the fast speed downloaded along with Lumos wallet. In fact the speed was pretty fast even when I upgraded to 5.3.1.
However, it got stuck for 2 days on 26th May 2019.

Have you deleted your blockchain since you upgraded to v5.3.1.?
That's what you should do, re-index isn't solving the problem.
I think otherwise many other people (like miners etc) would have complaint way earlier.
Hopefully this helps you out.

By tomorrow I should be able to tell if that worked.


Make a note for yaml file that tabs can be problematic

Use spaces.

Has YAML been around for long enough that we can assume it's no longer likely to cause confusion?


feat: Implement basic auth for applications

I implemented basic auth handling for applications.
Thank you for the great work on this project. Would be happy if my feature will be reviewed and merged.

👍🏽 +1

Highly usefull feature, could also be cool to add Application Auth only on Preview Apps to prevent unwanted usage of that apps
+1


[question] transfromer of token after parsing?

the rule can be transformed, which is great. how about the token?
The closest I can find in the doc is recipe #1: lexer callback, but  the returned value have to be token in lexer callback, which is processed before the parsing I guess).
use cases: turn INT to int, NUMBER to float, etc.

Yes. It's a very new feature, so I hadn't had time to write documentation for it yet.
But if you initialize your transformer with visit_tokens=True, it will also process tokens.
Like
class T(Transformer):
    def NAME(self, name):
        return lookup_dict.get(name, name)

T(visit_tokens=True).transform(tree)

Thanks!
https://github.com/lark-parser/lark/pull/471/commits/10a09ebba86fb17c2c13dc3080d6aa1d9987211c
copied the example to docs.


Build and package limactl.ventura from inside the lima directory

The only functional change here is that the sha512sum file ends up with just the tarball filename without a relative path, the same way we generate the regular lima-and-qemu.tar.gz.sha512sum file.

For my test tag (on my fork), I get:
8b3d056ec27c3ae1c414552166ee3769430bedee2e96a2db6dcfa1373e7edbe02dedbb021e9bff34f60fe35068d8290a866ce86f5db53b039c3a92cf3b7a2c6a  lima-and-qemu.linux.tar.gz
d3e1034ae881ed8e51b9661f818cc9d3b61676e82387f8f3410ca54581c6545d36f49581f6ee173ccc4ff0438406be871b97ae33eb70c06f52075393ecd33a45  src/lima/lima-and-qemu.macos.tar.gz
273cce500a9aff8bfdca36afb92e9dd6798aa1dc69fba807f2dd8c8fa768ec76b3656de82b5fac6d38817496e99e2f9b71b8ff211c74460b4d3af491c6439e08  limactl.ventura.tar.gz

I don't understand, which file is this? There should only be a single sha512sum in each file.

That was all three files, concatenated, to show that the non-ventura macOS file still have paths in it.


Update: Remove jfrog scan

What type of PR is this?
(REQUIRED)

cleanup

What this PR does / why we need it:
(REQUIRED)

Its too much of a hassle to run the jfrog scan manually everytime for every PR.
Which issue(s) this PR fixes:
(REQUIRED)

Special notes for your reviewer:
(fill-in or delete this section)

Testing
(fill-in or delete this section)

Release Notes
(REQUIRED)

Done. Maybe will get more mellow quality control in the future.


assertion: "from->thread_delayed_free == NULL" triggered during heap deletion

Somewhat related to #202. This can trigger when an allocation a one non-backing heap is freed while that heap is being deleted. Below is a program that reproduces this behavior when an artificial sleep is introduced in mi_heap_absorb.
Note that the allocation p outlives the heap from which it was allocated.
Program:
#include <thread>
#include <mimalloc.h>
#include <unistd.h>

volatile void* p;

void t1main() {
    mi_heap_t* heap = mi_heap_new();
    p = mi_heap_malloc(heap, 1024);
    mi_heap_delete(heap);
}

int main() {
    auto t1 = std::thread(t1main);

    usleep(1000 * 150);
    assert(p);
    mi_free((void *)p);

    t1.join();
}

heap.c diff to cause the free to happen between _mi_heap_delayed_free and assertion.
 #include "mimalloc-atomic.h"

 #include <string.h>  // memset, memcpy
+#include <unistd.h>

 /* -----------------------------------------------------------
@@ -324,6 +325,8 @@ static void mi_heap_absorb(mi_heap_t* heap, mi_heap_t* from) {
   // free outstanding thread delayed free blocks
   _mi_heap_delayed_free(from);

+  usleep(1000 * 300);
+
   // transfer all pages by appending the queues; this will set
   // a new heap field which is ok as all pages are unfull'd and thus
   // other threads won't access this field anymore (see `mi_free_block_mt`)

Hi Sam, thanks again; I think this is indeed expected by the current design -- I commented in #202 for how we can perhaps improve on the current implementation.

@daanx, can you re-visit this? I don't think this is the same as #202. In this issue the allocation out lives the thread. The documentation and code all seem to suggest that this is supported. For example, mi_heap_delete() says:

This will release resources and migrate any still allocated blocks in this heap (efficienty) to the default heap.

@colesbury, yes you are right -- this should work; was thinking of mi_heap_destroy... I will look at it again and at your possible fix. I believe I should probably implement the local model as that is what the documentation implies as well.

Hi Sam; I think I fixed this issue -- the heap absorb was just wrong (out of date). Hope this works for you. Still considering how to handle #202 better ...

Here are the additional usleeps (on the dev branch 946a71c): https://gist.github.com/colesbury/f3487f9d4f89dbf3497bcff2ca454458

Ah, it is indeed not quite correct yet! We cannot call mi_heap_page_never_delayed_free as that could cause the page to never be freed. It works for abandoned pages as those get checked on reclamation if all blocks are free at that point. Since we absorb directly into a new heap here that is not the case now.
I see how to fix it though... we should wait before updating the new heap pointer in the page queues if it happens to be delayed_freeing at that time.

.. I think this should be right now. I like the sleeps that you added -- I will think of adding these at strategic places to enable better debugging of these concurrency errors.

Hi Daan, I'm still seeing assertion failures with the most recent change. (Same test script and same usleep calls). I'm on 67de254 so I think it includes the most recent fixes.
mimalloc: assertion failed: at "/Users/sgross/Projects/mimalloc/include/mimalloc-internal.h":455, mi_page_set_heap
  assertion: "mi_page_thread_free_flag(page) != MI_DELAYED_FREEING"

Hi Sam, ah right, in this case the assertion is wrong -- at the absorb time it is ok to be delayed freeing at that point; I will update that. Thanks!

Hope this fixes the issue. Thanks for your help getting this right :-)

Thanks, that seemed to fix the issue.


OCPTOOLS-280: Bump golang.org/x/crypto from 0.18.0 to 0.22.0

Fixes the following CVE(s):

CVE-2023-48795

/test images

/cherry-pick release-4.12

/cherry-pick release-4.15

@sayan-biswas  Raised #1815  to resolve merge conflicts

/label backport-risk-assessed
/label cherry-pick-approved
/label px-approved
/label docs-approved
/label qe-approved


Update Composer dependencies (2022-03-01-00-11)

Visual regression test passed!
View the visual regression test report


fix: outlier detection disabled by default

What type of PR is this?

"fix(translator): outlier detection need to be explicit enabled by a BTP"

What this PR does / why we need it:
disable passive health check if not explicitly enabled by a BTP. It's misleading for users that passive health check is enabled with default values without user knowing.
Use case where you don't need outlier detection:
If you users have tiers of proxies (envoy have clusters backends which is also proxies), outlier detection shouldn't be enabled.
Which issue(s) this PR fixes:
Fixes #4854

Release Notes: No

hey @lsjostro can you sign your commit/ commit amend and force push

can you also add a note in https://github.com/envoyproxy/gateway/blob/main/release-notes/current.yaml under the breaking changes section highlighting that this now disabled by default and If needed, needs to be explicitly set in https://gateway.envoyproxy.io/docs/api/extension_types/#backendtrafficpolicy

hey @lsjostro can you sign your commit/ commit amend and force push

trying out my workflow with jj so let me lock into that..  so bear with me 😄
excellent challenge though! 😄

for ref 😄  https://github.com/martinvonz/jj/issues/1399

While I agree that passive health checks does not make sense when EG is used as the first tier of a proxy chain, I believe they can be beneficial in most scenarios where EG sits directly in front of services. Considering this, I see two potential approaches:

Enable passive health checks by default and allow users to explicitly disable them using a BTP.

Disable passive health checks by default and include a best practices section in the documentation to guide users on enabling them when needed.

Personally, I lean slightly toward the first option, as it provides a more resilient out-of-the-box experience. My assumptions may not be correct, I’d like to hear more practical opinions from users who are actively using EG in their environments.

If 1. is picked, a user setting active health will need to explicitly disable passive health check

If 1. is picked, a user setting active health will need to explicitly disable passive health check

Yeah, that's a good point. An implicit health check could be hard to be noticed.
We can go ahead with option 2 and add a best practices section in the documentation to ask users to set it if needed.

agree option 2 is better otherwise we'll need to document why we choose to enable passive health by default and how that will benefit the user. I think it's better to let the user decide and don't have something enabled saying we know better than the user.

hey @lsjostro can you run make testdata as well ? and commit those changes


add metrics for V8 contexts usage

Scope & Purpose
Added metrics for V8 contexts usage:

arangodb_v8_context_alive: number of V8 contexts currently alive.
arangodb_v8_context_busy: number of V8 contexts currently busy.
arangodb_v8_context_dirty: number of V8 contexts currently dirty.
arangodb_v8_context_free: number of V8 contexts currently free.
arangodb_v8_context_max: maximum number of concurrent V8 contexts.
arangodb_v8_context_min: minimum number of concurrent V8 contexts.

Docs PR: https://github.com/arangodb/docs/pull/538

[ ] :hankey: Bugfix
[x] :pizza: New feature
[ ] :hammer: Refactoring
[x] :book: CHANGELOG entry made
[x] :muscle: The behavior in this PR was manually tested
[ ] :computer: The behavior change can be verified via automatic tests

Backports:

[ ] No backports required
[ ] Backports required for: 3.6

Testing & Verification

[x] This change is a trivial rework / code cleanup without any test coverage.

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11691/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11693/

Tests blue

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/11733/

Tests blue.


Allow (Api)Controllers to be marked internal

Very often we need to expose a lot more types than needed.
F.e. when a controller requires a service or repository injected through the constructor, the service/repository or at least their interface needs to be marked as public. The interface itself being public is not such a big problem, but the need to make things public cascades down through every type used in the interface (models, entities, etc.), which you don't want to make public. Even when an API controller returns a mapped viewmodel, you still have to inject a public interface and expose your internal models. Especially, in modular applications or public/shared libraries, this is a nightmare.
Therefore, it would be a good idea if controllers could be made internal and just work. Currently, there's no reason why anyone would make a controller internal (except if they do it by accident or because it feels logical), so removing this restriction should not be a breaking change.
I have a prototype working by overriding ControllerFeatureProvider by a class where the IsController also considers non-public types. While it is working, this causes additional maintenance that one doesn't want in an application.

I think that the current extensibility points, which work for exactly this scenario, are exactly what people should use if they have this requirement. I haven't seen this come up frequently, so I think that MVC has struck the right balance between default behaviors vs. extensibility.

The problem is that, if we ship modules with internal controllers, we need to tell the consumer to use a custom ControllerFeatureProvider, otherwise the endpoints won't work. If, on the other hand, everything is public, the consumer can start using services that are not intended to be used. Which means, if we change the interface of the service, we can potentially break the unwanted API contract.

Thanks for contacting us, @huysentruitw.
The well-established pattern for working with DI is to explose an extension method in your library, which will be the only thing called by consumers, and that'll take care of registering all the necessary services in the DI container. This will also register the ControllerFeatureProvider.
Given this, there is not really anything we will change.

Fair enough. Injecting IServiceProvider in the constructor of the controller might be a better option then.


Generic Type Converters

Is your feature request related to a problem? Please describe.

Type conversion is superbly done in Isar, as explained in the TypeConverters documentation.
However, say you have a lot of enums, you'd then need to write a converter for each one.
Other examples may apply as well not just enums, basically for any types with shared data-access behavior.

Describe the solution you'd like
We have three solutions here:

A. Ability to have generic type converter, in declaration and in annotation. I suppose the annotation part is tough?

class GenericIntTypeConverter<T> extends TypeConverter<T, int> { ... }
and then in the Isar object's property annotation:
@GenericIntTypeConverter<SomeType>()

Describe alternatives you've considered

B. Using TypeConverter over a superclass or interface class. This is a user's responsibility to have multiple classes extending a super class or implementing a shared interface class. This superclass or interface class can be used in the TypeConverter class definition.

class MyInterfaceTypeConverter extends TypeConverter<MyInterface, int> { ... }
and then in the Isar object's property annotation:
@MyInterfaceTypeConverter()
Action requested from Isar maintainer: I think this should be encouraged and mentioned in the  TypeConverters documentation.

C. This final method is related to enums and works only partially (toIsar works), which is not enough (compile error in fromIsar). Maybe someone can make it work! Now for enums, the solution looked good at first. It seems Enum is an abstract class representing all concrete enum classes! This was mentioned in two dart issues one and two. And was shipped recently with the stable dart release 2.14 as mentioned in the two issues linked here. But as I said, fromIsar fails miserably:

class EnumTypeConverter extends TypeConverter<Enum, int> {
  const EnumTypeConverter(); // Converters need to have an empty const constructor

  // This property can be assigned from constructor maybe?
  // Used for `fromIsar` -> Option B
  Enum? myEnum;

  // `fromIsar` does NOT work :( Neither options A nor B do.
  @override
  Enum fromIsar(int enumIndex) {
    Enum value;
    // Option A
    // Enum class does not expose `values`
    // compile error `The getter 'values' isn't defined for the type 'Enum'.`
    value = Enum.values[enumIndex];
    // Option B
    // An Enum instance (myEnum) does not expose `values`
    // compile error `The getter 'values' isn't defined for the type 'Enum'.`
    value = myEnum!.values;
    return value;
  }

  // `toIsar` WORKS!
  @override
  int toIsar(Enum myEunm) {
    return myEunm.index;
  }
}

Version

Platform: iOS, Android, Mac
Flutter version: 2.5.1 (stable)
Dart version: 2.14.2 (stable)
Isar version: 0.4.0

This is a nice-to-have of course... A developer can just repeat some code if no other solutions are available.

Hi,
I tried this and unfortunately is not recognized by the analyzer/generator:
@Collection
class Example {
...
 @RxBoolTypeConverter()
  RxBool? starred;
...
}

class RxBoolTypeConverter extends TypeConverter<RxBool, bool> {
  const RxBoolTypeConverter(); // Converters need to have an empty const constructor

  @override
  RxBool fromIsar(bool value) {
    return RxBool(value);
  }

  @override
  bool toIsar(RxBool value) {
    return value.value;
  }
}

This should work in the same way as for Enum ...
This would be a very appreciated to avoid using workaround like ignoring the current attribute and duplicate it with somewhat is expected stored value in the DB (e.g. bool dbStarred).

Type converters have been removed from the library.


Unknown 'torch.eq' expression type

Netron app and version: Netron-4.5.3.AppImage
OS and browser version: Arch Linux 5.8.10

Steps to Reproduce:

Change extension to .pt and open the file below in Netron, this model was traced and exported to torchscript from here
superpoint.zip

Netron presents this error

Please attach or link model files to reproduce the issue if necessary.

superpoint.pt.zip

netron_issue_609.pt.zip


Handling kubernetes auth config

Method to handle configuring Kubernetes auth method through configuration yaml. #46

Thanks for the submission @gowthamsubbu!
Could you please add an example (in a commented out form) to the sample YAML config?

@bonifaido Update the sample YAML config

@gowthamsubbu could you please fix the build? I think there is a syntax issue.

@bonifaido Fixed the syntax and spacing issue. Checks have passed. Please review.

@matyix Thank you ! Shall we look forward to 2.20 with these changes ?

Hi @gowthamsubbu, I have tagged 0.2.20, the builds are running on DockerHub already, the images should be available soon.


SUMo

The sumo package is out of date by quite a few versions.
I have contacted the maintainer see:
https://www.github.com/dtgm/chocolatey-packages/issues/330
chocolatey.org/packages/sumo

I use this sometimes to double check for program updates that lead to package updates that I maintain. I guess I'll pick it up.

I've got the package done (v<IP_ADDRESS>1) and ready to publish, but I haven't been added as maintainer yet.

@bcurran3 have you requested to be added as a maintainer of this package?  I don't see an email about it.

No. For some reason I always assume if I state it here, action will be taken.
I just requested maintainership via "normal" channels; i.e. e-mail generated to admins.

Done.
https://chocolatey.org/packages/sumo/<IP_ADDRESS>1

No. For some reason I always assume if I state it here, action will be taken.

While that "may" happen, it can't be relied upon.  The agreed process is to ask via the Contact Admins link, as that has the best chance to get actioned.

The agreed process is to ask via the Contact Admins link, as that has the best chance to get actioned.

This is also mentioned in the README.md for this repository as well:

Note: If you are taking over maintenance for a package, you also need to be added ot the list of maintainers. Use the contact admins link from the package page, point to the particular issue in this repository, and ask to be added.


[WIP] github-desktop: add GitHub Desktop to NixOS

Motivation for this change
Howdy folks,
This is a work in progress and I'm opening it early for feedback. It is not ready for merging. Whilst the expression builds successfully the application does not load yet. If anyone has some spare cycles to help out that would be k-rad. The upstream repository we are building from is https://github.com/shiftkey/desktop
Things done

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

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

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

@ghuntley any motivation to continue with this PR?

Closing as this has since been added to nixpkgs.


NullReferenceExpection in BaseSpatialObserver.CleanupMeshObject on Hololens

Does this affect the legacy HoloToolkit (master) or the Mixed Reality Toolkit (mrtk_release)?
Mixed Reality Toolkit
Describe the bug
We added the spatial awareness system to visualize the room mesh. It works as expected on the hololens, but in the log I receive lots of NullReferenceExceptions in a very short timespan (see screenshot).

To Reproduce
The setup of the configuration profile looks like this:

So it is initially active and will be set programatically to hidden by using following code, but as far as I found out this switch is not relevant as the error occurs on the InvokeSurfaceDataReadyEvent (see stacktrace):
private void SetSpatialMeshPulseVisible(bool visible) {
    bool isOpaque = MixedRealityToolkit.Instance.ActiveProfile.CameraProfile.IsOpaque;
    SpatialMeshDisplayOptions displayOptions = visible && !isOpaque ? SpatialMeshDisplayOptions.Visible : SpatialMeshDisplayOptions.None;
    MixedRealityToolkit.Instance.GetService<IMixedRealitySpatialAwarenessSystem>().MeshDisplayOption =
            displayOptions;
}

Unity Editor Version
2018.2.17f1
Mixed Reality Toolkit Release Version
mrtk_development 49f5c4c22b66a7e2c212f15b982056dcb31ff4cd

Thanks for the report

this should be addressed in #3399


Why adding ground truth bbbox during training?

❓ Questions and Help
Could anyone illustrate what is the intuition of add the ground truth bbox during training?
append ground-truth bboxes to proposals
    if self.training and targets is not None:
        boxlists = self.add_gt_proposals(boxlists, targets)

Thanks a lot!

I believe it is a hack to provide data to roi_heads classification task during training. These ground-truth bboxes are not taken into account in the rpn loss, as you can see here, buy they are taken into account in the roi_heads loss, as you can see here.
Once they are fed into the roi_heads loss evaluator, they will only generator meaningful information to the classification loss, since its bounding boxes must match perfectly to the targets, no regression loss is generated, then no gradients either.
Thus, this feeds your roi_heads with some useful information to train even in the early stages of the training process.
I'm not sure though this is the only reason.

Thanks a lot, that explanation makes sense to me!

This definitely can make the model converge faster. Otherwise,  the training of roi_heads would wait until RPN converge first.  Besides,  it can also make sure the input to the roi_heads is not empty. If the input is empty, it will cause some problems now and hard to solve it now.

I'm closing this issue, please let me know if you have any further question.

This definitely can make the model converge faster. Otherwise, the training of roi_heads would wait until RPN converge first. Besides, it can also make sure the input to the roi_heads is not empty. If the input is empty, it will cause some problems now and hard to solve it now.
I don't think it will cause the input to roi_heads become empty, rpn will select some proposals in any case


Combine Subscription and Node-Locked models in one product

Question

We would like to map the following license models in the same product:

Subscription model (7 days free, then 1 year for a license fee)
Node-locking (customer can install our program on 1 or 2 computers (e.g. identified by CPU serial number) during the subscription period).

Is it possible to map this model in NetLicensing? If so, is there a way to uninstall a node via API, for example, if the customer uninstalls our software on a computer?

The above scenario can be configured in NetLicensing using two licensing models - Node-Locked and Subscription.

where the Subscription model can specify the default package (e.g. 7 days) to be automatically assigned for every new customer:

and Node-Locked model can specify 1 default node:

Question 2

is there a way to uninstall a node via API, for example, if the customer uninstalls our software on a computer?

Option 1 - API
It is possible to release node locking via API - as described at #30
Option 2 - Customer Portal
It is also possible to allow customers to maintain their licenses by themselves via NetLicensing Customer Portal. So, customers can release paired nodes without involving the vendor's customer support.


Implement Scatter Plot for Feature Comparisons

Add scatter plot feature so that things like gene counts can be plotted against total assembly bases, scaffolds, etc.

This is solved for now


ceToCMSName

Hello,
I have a SiteName I would like to map to a CE, but I have noticed _siteresources() doesn't have the CE types anymore. So getAllCENames is an empty list and cmsNametoCE just throws an exception.
However, ceToCMSName seems to do the same as seToCMSName as they both just compare the fqdn without verifying the type (so, it's just compared against the SE FQDN since there are not CE FQDN types anymore). Is this a bug, am I miss-interpreting something or are SE/CE FQDNs the same on most CMS Sites? Any alternative to map a SiteName to a CE from WMCore without using condor_status?

As far as I remember we wanted to remove dealing with CE from all WMCore code and also want to remove CE from SiteDB. The only system that should care about CE is glideInWMS. As such it'll be
a) hard to find anyone that knows or cares about this code
b) dangerous to rely on this code to do anything in the long run (it could be removed)

This isn't going to help you, but basically we no longer use CE names except in GlideinWMS when it's deciding which queues to send pilots too. The code you point to is unused in WMAgent and presumably elsewhere.
Ah, I see Dirk has said much the same thing.

Ah! Thank you. I won't use that code then...


Which count matrix to use for creating custom cell profile matrix?

Hi, Thank you for such a wonderful tool. I am trying to create custom cell profile matrix from paired SC samples, yet I am quite confused which count matrix should I use. The options are
Using "raw" counts while setting the argument "normalize = TRUE"
Using RNA@data which are the normalized data
Using "integrated" count matrix, which is the batch corrected integrated count matrix
All of them give me different results, and for biological interpretation, they all seem valid (i.e. despite there is enrichment of another cell cluster, this cluster belongs to the same cell type)
Another question, which normalized GeoMx count matrix should I be using? I tried using the log_normalized count matrix and it doesn't work well as using the Q3_normalized count matrix that hasn't been log normalized yet.
I would really appreciate your help! Thank you!

Hi @aelhossiny,
We have not tested the best normalization method for the single-cell data in profile matrix generation, but I think it is safe to say that whatever method is best for the standalone single-cell data is best for creating a profile matrix. In your case, I would assume it is the batch corrected integrated count matrix.
As for the normalization method on the GeoMx side, SpatialDecon expects linear-scale cell profile matrices so don't use log normalization.
Maddy

just a follow up, I noted from the paper that described the algorithm that when optimizing
Let Xp∙K be the cell proﬁle matrix giving the linear-scale expression of p genes over K cell types.

just a follow up, I noted from the paper that described the algorithm that when optimizing
β.1 = argminβ.i||log(Y.i)−log(B.i+Xβ.i)||
Xp∙K is defined as the cell proﬁle matrix giving the linear-scale expression of p genes over K cell types.
But the scRNASeq data is log normalized already? Does that mean I raw counts?

If you log normalized your scRNAseq data, then yes you should use the raw counts and normalize in SpatialDecon using the argument "normalize = TRUE". But if you normalized the scRNAseq data not using log normalization I would use that.


Prevent GraphQL non breaking space

From https://github.com/Kong/insomnia/pull/2704/files#r505491859 it was found that the code to prevent GraphQL non breaking spaces (that cause a syntax error) is not working
This PR fixes the behaviour
Demo

Bottom Insomnia is latest version with the bug
Top Insomnia is the version with my PR code
Demo copies invalid text and shows it fails in the latest version but passes with the code change

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Merged with latest develop and added more irregular characters that might break the interface.


FilterUnloadCallback description

There's no comment "If a NULL is specified for this routine, then the filter can never be unloaded".

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 020480f3-b724-f282-cce8-43d7cde038d2
Version Independent ID: 84eeb4e3-4d88-862c-276e-7496c0f35f9b
Content: _FLT_REGISTRATION
Content Source: wdk-ddi-src/content/fltkernel/ns-fltkernel-_flt_registration.md
Service: unspecified
Product: windows-hardware
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

Hi @21street, I've add this this note to the topic, and it should be live shortly! Thanks for letting us know!


Migrate benchmark codes

We have some codes for ross benchmark in this repository: https://github.com/ross-rotordynamics/ross-benchmark
We should migrate this code to a 'benchmark' folder within the main ross repository and update the code so that it works with our current code base.

@raphaeltimbo , I started a PR #983 to migrate the files, but I have some doubts :

Do we create the "/Snakeviz_inputs" folders and the "Benchmarks" model outside of ross or inside the benchmark folder?
Can we include the snakeviz library in the general requirements.txt or just the .[dev]?
Should we create a Benchmark() class that contains a run method to make it easier for the user to execute?


class file has wrong version 55.0, should be 52.0

Hi,
I have the following error when running the demo in intellij:
Error:(1, 26) java: cannot access eu.hansolo.tilesfx.Tile
bad class file: /C:/Users/Sam Lim/Downloads/jar_files/tilesfx-11.36.jar!/eu/hansolo/tilesfx/Tile.class
class file has wrong version 55.0, should be 52.0
Please remove or make sure it appears in the correct subdirectory of the classpath.
is there a fix for this?
EDIT:
I was also wondering if the tiles can be dragged around to rearrange when  running ?

Could you please provide a code snippet that shows the problem? Make sure you use Java 11 or higher when using TilesFX. I just made a quick test and it works perfectly fine, you can check by simply execute "gradlew Demo" on the command line. And of course make sure you use at least Java 11.
TilesFX just provide the tiles and a layout container to create dashboards. If you would like to drag tiles around you sure can do that but that has to be managed by you, each tile is just a simple JavaFX controls like every button etc. So if you would like to drag controls around in your app you have to take care about that on your own.

Hi, thanks for the quick reply, I tried to use Java 11, but I has a similar issue to the one here:
https://github.com/HanSolo/tilesfx/issues/80
Should i use the versions you mentioned in the linked post? I downloaded my Java 11 from oracle, changed all my Javac and JDK to Java 11, and added JavaFX 11 (from the JavaFx website) as a library so I'm not sure if that is an issue.

Hmm...so I use AdoptOpenJDK and add JavaFX from the Gluon page as described on their website. This works on my Mac, Windows 10 and Linux Ubuntu 18.04 machine. I also use IntelliJ to develop TilesFX so I can assure it works fine with this combinations.

Ok, I will have a look at that. It might be a program setting on my end so I will look into it, as I don't think anyone else had this issue. Sorry for the trouble and thank you for developing this.

Well TilesFX version above 11 won’t run on JDK8 do it looks like you have the old stuff sitting around somewhere in IntelliJ...

My apologies, I was using the demo from https://github.com/HanSolo/tilesfxdemo which I think is the outdated version that works with Java 8 as I stumbled across an old post in your blog. You can close this issue for now since everything is working at the moment.

Ah ok...that explains it, no problem. Glad it works for you 👍🏻


Mingw64

A set of patches to:
-enable building in build dir different than source dir
-detect ws2_32, psapi and iphlpapi libraries on mingw32 and mingw64
-detect gethostbyname and getaddrinfo functions on windows
-on windows use gmtime_s instead of gmtime
-change pointer type to char to silence compiler

Merged!
Thanks a lot for your contribution!


Fix windows & Start graphics API

Graphics API plans got dragged into this accidentally, this was originally just for fixing this on Windows.

Hello myself how are we doing


Add error pages for 400 and 406 errors

Also add a redirect to handle the most common form of 406 error.
https://www.pivotaltracker.com/story/show/99489826

LGTM


Alternative install for zookeeper removed

The alternative install for ruby-zk no longer seems to be needed.  Running:
gem install zookeeper

from command line works on Raspberry Pi 2/Zero

https://ocd.appscale.com:8080/job/Daily Build/1150/
Thanks, @lmandres


Implement Fractal

Investigate the use of Fractal for retrieval of Eloquent models and implement if so deemed necessary.

Per discussion in Slack, this is being postponed to a future release.


Integrate handling of three-point water

After openmoltools is updated, update pipeline to allow handling of water as discussed in #13.

Waiting for https://github.com/choderalab/openmoltools/issues/134

The issue in openmoltools was resolved by https://github.com/choderalab/openmoltools/issues/143, so this is ready to be completed. In fact I think it doesn't require any changes to the existing code (except for non-three-point water models, which will be resolved in a separate issue).

@bannanc , @gduarter : I tested this and it seems to work properly now. Basically just make sure you are using the latest openmoltools and parmed and the usual start.py should now work properly with water. For example, I tried:
#solute names
solutes = ['phenol', 'toluene', 'benzene']
#Solvent names
solvents = ['cyclohexane', 'cyclohexane', 'water']

and verified that I do indeed get a benzene-water combination which uses TIP3P water which runs properly in GROMACS.
(This happens because openmoltools.amber.build_mixture_prmtop was updated to BY DEFAULT assume that TIP3P water is meant if you provide water molecules; if you want to go back to the old behavior - using flexible GAFF water - you have to specify it in an optional argument to build_mixture_prmtop.)


West Midlands | Samira Hekmati | Module-User-Focused-Data | Week 1

Learners, PR Template
Self checklist

[x] I have committed my files one by one, on purpose, and for a reason
[x] I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME | WEEK
[x] I have tested my changes
[x] My changes follow the style guide
[x] My changes meet the requirements of this task

Changelist
Briefly explain your PR.
Questions
Ask any questions you have for your reviewer.

Hi Russel, I really appreciate your feedback; special the one on the use of hexadecimal (hex) values for colours instead of rgb. I realized that they are different and cannot be used interchangeably.
I changed my code based on your review.
also, can you please label my PR as complete so that other volunteers understand that it has already been reviewed?
Thank you for your time.


feat: port MeasureTheory.Integral.LebesgueNormedSpace

bors merge

I'm not sure about the naming changes I made Zulip
bors r-

bors merge

bors merge


WIP: add yarn support for package install

Explain the motivation for making this change. What existing problem does the pull request solve?
Fixes #11122
Test plan (required)

Init / Install without Yarn installed on system (force npm). Ensure install does not mix Yarn / npm if Yarn is available after installing with npm.
Init / Install with Yarn installed on system. Ensure install continues to only use yarn.

Hi @olingern, I'm working on adding Yarn support for cli as well. please check it out #11174 :)

@n3tr Awesome! Sorry, I didn't see anything in #11122 mentioning you were on it. All yours :)

@olingern sorry, it was my mistake 😢. I haven't carefully checked for the issue and just realized when you mentioned to it.


Are recommender models getting better over time when using them?

I'm a real beginner in the area of ML so please bear with me: I searched the documentation, but couldn't find a real answer on whether the recommender models trained with Turi Create are getting better over time when I'm using them to recommend. Personally I doubt it, cause I'm not seeing any way to feed back information whether a recommendation was good or not. On the other hand there is a way to pass observation data to the recommend() function, so at least that could in theory be used to extend and improve the model, right?
So in the end the question would be: do I have to retrain the whole model on the old + new data to get an improved model?

Anyone who can answer that question? Really curious!

Awesome, thanks, that helped a lot!


[Russian] Fix gitter href for free-code-camp-official-chat-rooms article

Fix broken gitter href on the russian free-code-camp-official-chat-rooms article article

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for your contribution, but the guide has been moved here https://github.com/freeCodeCamp/freeCodeCamp/tree/master/guide and this cannot be merged. If you create a new PR targeting the new location we will do our best to get it merged.


Cannot work with 8 gpus, but work with 4. gpus

issue
Hi, thanks for your work!
when using the following command to run the code, I met a strange error: the code cannot work with 8 gpus even when changing batch size to 1 per gpu, but can work with 4 gpus.
python -m torch.distributed.launch --nproc_per_node=4 --master_port=25622 tools/train.py configs/detection/mv-det3d_8xb4_embodiedscan-3d-284class-9dof.py --launcher='pytorch' --work-dir='logs/det'

The machine I use is A6000, with 48G memory.
here is the logs
04/12 14:46:55 - mmengine - INFO - Epoch(train)  [1][  50/1946]  lr: 1.0000e-03  eta: 21:08:51  time: 3.2672  data_time: 0.3987  memory: 10248  grad_norm: 0.9408  loss: 2.3556  loss_center: 0.6253  loss_bbox: 0.7773  loss_cls: 0.9531
WARNING:torch.distributed.elastic.multiprocessing.api:Sending process 119878 closing signal SIGTERM
WARNING:torch.distributed.elastic.multiprocessing.api:Sending process 119879 closing signal SIGTERM
WARNING:torch.distributed.elastic.multiprocessing.api:Sending process 119880 closing signal SIGTERM
WARNING:torch.distributed.elastic.multiprocessing.api:Sending process 119881 closing signal SIGTERM
WARNING:torch.distributed.elastic.multiprocessing.api:Sending process 119882 closing signal SIGTERM
WARNING:torch.distributed.elastic.multiprocessing.api:Sending process 119883 closing signal SIGTERM
WARNING:torch.distributed.elastic.multiprocessing.api:Sending process 119885 closing signal SIGTERM
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: -9) local_rank: 0 (pid: 119877) of binary: /root/anaconda3/envs/embodiedscan/bin/python
Traceback (most recent call last):
  File "/root/anaconda3/envs/embodiedscan/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/root/anaconda3/envs/embodiedscan/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/root/anaconda3/envs/embodiedscan/lib/python3.8/site-packages/torch/distributed/launch.py", line 193, in <module>
    main()
  File "/root/anaconda3/envs/embodiedscan/lib/python3.8/site-packages/torch/distributed/launch.py", line 189, in main
    launch(args)
  File "/root/anaconda3/envs/embodiedscan/lib/python3.8/site-packages/torch/distributed/launch.py", line 174, in launch
    run(args)
  File "/root/anaconda3/envs/embodiedscan/lib/python3.8/site-packages/torch/distributed/run.py", line 715, in run
    elastic_launch(
  File "/root/anaconda3/envs/embodiedscan/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 131, in __call__
    return launch_agent(self._config, self._entrypoint, list(args))
  File "/root/anaconda3/envs/embodiedscan/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 245, in launch_agent
    raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
=======================================================
tools/train.py FAILED
-------------------------------------------------------
Failures:
  <NO_OTHER_FAILURES>
-------------------------------------------------------
Root Cause (first observed failure):
[0]:
  time      : 2024-04-12_14:48:53
  host      : b70316d392fe
  rank      : 0 (local_rank: 0)
  exitcode  : -9 (pid: 119877)
  error_file: <N/A>
  traceback : Signal 9 (SIGKILL) received by PID 119877
=======================================================

enviroment
System environment:
    sys.platform: linux
    Python: 3.8.19 (default, Mar 20 2024, 19:58:24) [GCC 11.2.0]
    CUDA available: True
    MUSA available: False
    numpy_random_seed:<PHONE_NUMBER>
    GPU 0,1,2,3,4,5,6,7: NVIDIA RTX A6000
    CUDA_HOME: /usr/local/cuda
    NVCC: Cuda compilation tools, release 11.8, V11.8.89
    GCC: gcc (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
    PyTorch: 1.11.0
    PyTorch compiling details: PyTorch built with:
  - GCC 7.3
  - C++ Version: 201402
  - Intel(R) oneAPI Math Kernel Library Version 2023.1-Product Build 20230303 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v2.5.2 (Git Hash a9302535553c73243c632ad3c4c80beec3d19a1e)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - LAPACK is enabled (usually provided by MKL)
  - NNPACK is enabled
  - CPU capability usage: AVX2
  - CUDA Runtime 11.3
  - NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_37,code=compute_37
  - CuDNN 8.2
  - Magma 2.5.2
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=11.3, CUDNN_VERSION=8.2.0, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -DEDGE_PROFILER_USE_KINETO -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.11.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=OFF, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON, USE_ROCM=OFF,

    TorchVision: 0.12.0
    OpenCV: 4.9.0
    MMEngine: 0.10.3

Do you encounter similar errors, or could you give me some ideas about this one?

For the training of 3D detection task, my environment is almost the same as yours (8 * A6000 with 48G), and I didn't encounter the same error. However, for the 3D grounding task, I encountered the error "Plane vertices are not coplanar." which is quite strange...

hi @EricLee0224, I also encountered the error "Plane vertices are not coplanar." when running the grounding task, but this issue is mentioned in this repo; you could have a check.
Could you please share your environment settings for 3D detection task?
I would appreciate it a lot!

hi @EricLee0224, I also encountered the error "Plane vertices are not coplanar." when running the grounding task, but this issue is mentioned in this repo; you could have a check. Could you please share your environment settings for 3D detection task? I would appreciate it a lot!

1）I have indeed noticed the discussions in issue #30 and #22. They suggest adjusting the value of eps (I have tried 1e-5/1e-4/1e-3/1e-2 but it still doesn't work) and using the --resume parameter (the error message tells me that no available ckpt is found), but none of them are effective. Have you successfully resolved this issue?
2）Sure. I run：
CUDA_VISIBLE_DEVICES=0,1,2,3,4,5,6,7 python -m torch.distributed.launch --nproc_per_node=8 tools/train.py configs/detection/mv-det3d_8xb4_embodiedscan-3d-284class-9dof.py --work-dir=work_dirs/mv-3ddet --launcher="pytorch"”
and you could find the environments in log:
2024/04/13 18:02:50 - mmengine - INFO -
System environment:
sys.platform: linux
Python: 3.8.19 (default, Mar 20 2024, 19:58:24) [GCC 11.2.0]
CUDA available: True
MUSA available: False
numpy_random_seed: 430976289
GPU 0,1,2,3,4,5,6,7: NVIDIA RTX A6000
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 11.3, V11.3.58
GCC: gcc (Ubuntu 9.4.0-1ubuntu1~20.04.2) 9.4.0
PyTorch: 1.11.0
PyTorch compiling details: PyTorch built with:

GCC 7.3

C++ Version: 201402

Intel(R) oneAPI Math Kernel Library Version 2023.1-Product Build 20230303 for Intel(R) 64 architecture applications

Intel(R) MKL-DNN v2.5.2 (Git Hash a9302535553c73243c632ad3c4c80beec3d19a1e)

OpenMP 201511 (a.k.a. OpenMP 4.5)

LAPACK is enabled (usually provided by MKL)

NNPACK is enabled

CPU capability usage: AVX2

CUDA Runtime 11.3

NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_37,code=compute_37

CuDNN 8.2

Magma 2.5.2

Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=11.3, CUDNN_VERSION=8.2.0, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -DEDGE_PROFILER_USE_KINETO -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.11.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=OFF, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON, USE_ROCM=OFF,
TorchVision: 0.12.0
OpenCV: 4.9.0
MMEngine: 0.10.3

Runtime environment:
cudnn_benchmark: False
mp_cfg: {'mp_start_method': 'fork', 'opencv_num_threads': 0}
dist_cfg: {'backend': 'nccl'}
seed: 430976289
Distributed launcher: pytorch
Distributed training: True
GPU number: 8

Thank you, @EricLee0224 , I think the reason why I cannot run train/3ddet is because of the RAM limitation (500G). Could you please show your RAM? ( the Mem part when usinghtop)

I figure it out, it's truly because of the RAM limitation (500G)


Want to add hover effect on the home page image

##Description
I want to add a hover effect on the home page image it will improve the UI of the website
##screenshot

Hello HimanshuNarware, thanks for opening a issue, your contribution is valuable to us. The maintainers will review this issue and provide feedback as soon as possible.

Closing this as not required


HDDS-4659. No cleanup in TestOzoneFSWithObjectStoreCreate

What changes were proposed in this pull request?
Mini cluster needs to be shutdown after each test.
https://issues.apache.org/jira/browse/HDDS-4659
How was this patch tested?
Checked size of hadoop-ozone/integration-test/target/test-dir after running TestOzoneFSWithObjectStoreCreate.

Thanks @bshashikant for the review.

Thanks @bshashikant for the review.


feetech_ros2_driver: 0.1.0-3 in 'jazzy/distribution.yaml' [bloom]

Increasing version of package(s) in repository feetech_ros2_driver to 0.1.0-3:

upstream repository: https://github.com/JafarAbdi/feetech_ros2_driver.git
release repository: https://github.com/ros2-gbp/feetech_ros2_driver-release.git
distro file: jazzy/distribution.yaml
bloom version: 0.12.0
previous version for package: null

feetech_ros2_driver
* Add feetech ros2 driver
* Contributors: Jafar Uruç

Holding for Jazzy sync


week-of-09-30-2024

Create a folder for week of 09/30/2024.
Notes for Reviewers
This PR fixes #
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 8cf0b1f84c96d4c9bfa09753db5f4301787e562d at: https://66faef2393f8150497816e6f--meshery-docs-preview.netlify.app


Getting error Error: the server could not find the requested resource (SQLSTATE HV000)

Describe the bug
Running the select * from .. query on some tables fails with an error: Error: the server could not find the requested resource (SQLSTATE HV000).
The following tables are impacted by the above error:

kubernetes_endpoint_slice
kubernetes_horizontal_pod_autoscaler
kubernetes_pod_disruption_budget
kubernetes_pod_security_policy

Steampipe version (steampipe -v)
Example: v0.19.3
Plugin version (steampipe plugin list)
Example: v0.18.0
To reproduce
Run any select * queries on the above mentioned tables.
For example:
select * from kubernetes_pod_security_policy

Expected behavior
If there is no resource of specific kind, the table should return zero result without any error.
Additional context
Add any other context about the problem here.

PodSecurityPolicy was deprecated in Kubernetes v1.21, and removed from Kubernetes in v1.25. Hence, the latest APIs (v1) no longer support this resource.


Got wrong day of the week when not generating indates

I'm currently using version 7.0.4.
I found that the first date of the month became Sunday when I used .off for parameter generateInDates.

I made a sample project here:
testProject.zip

This is expected.
Think about this, if you do not generate outdates, this means that the 1st will always be on the extreme left right.
Therefore, if the first is always on the extreme left, then under what column should the first be considering that you are the one that maintains the [Su | Mo |Tu | We| Th | Fr | Sa] header?
If this causes a design problem for you then let me know how you want you calendar to look, and then I will tell you what setup you should put in your configureCalendar function

Will close this, but if you have more questions, feel free to ask ok.

Would be possible to have the same location for the first day of the month however without the indates? As if the indates are hidden?

@ahmed-khalil could you rephrase that? I still do not understand the question. thanks


Lock error using existing database

Is there a way to initialize a database using an existing SQLite file? For instance, here's what it might look like if I combine your example repo with a sql.js example to load an existing database from a URL:
import initSqlJs from<EMAIL_ADDRESS>import { SQLiteFS } from 'absurd-sql';
import IndexedDBBackend from 'absurd-sql/dist/indexeddb-backend';

async function init() {
  let SQL = await initSqlJs({ locateFile: file => file });
  let sqlFS = new SQLiteFS(SQL.FS, new IndexedDBBackend());
  SQL.register_for_idb(sqlFS);

  SQL.FS.mkdir('/sql');
  SQL.FS.mount(sqlFS, {}, '/sql');

  // host an example file somewhere with permissive CORS settings
  const data = await fetch("https://example.com/chinook.db").then(res => res.arrayBuffer());
  const db = new SQL.Database(new Uint8Array(data));
  return db;
}

async function runQueries() {
  let db = await init();

  const stmt = db.prepare("SELECT Name FROM artists;");
  stmt.step(); // Execute the statement
  console.log(stmt.getAsObject());
}

runQueries();

This is the error that gets generated:
index.js:149 Uncaught (in promise) TypeError: Cannot read property 'lock' of undefined
    at SQLiteFS$1.lock (index.js:149)
    at eval (sql-wasm.js:93)
    at sql-wasm.wasm:0xf7cec
    at sql-wasm.wasm:0x9e651
    at sql-wasm.wasm:0x2a775
    at sql-wasm.wasm:0xd83d
    at sql-wasm.wasm:0xe7e5
    at sql-wasm.wasm:0xebb41
    at sql-wasm.wasm:0x986a
    at sql-wasm.wasm:0x27d84

Maybe it makes sense that it doesn't work. Your version of SQL.Database doesn't let you specify a filename if you give it a data array instead of a filename. There might be other limitations preventing this pattern from working, but I'm not sure.
This might be a trivial fix to your sql.js fork, but I'm unable to try it out since I've had all sorts of problems getting local versions of absurd-sql and sql.js to work with the example project.

I am so sorry for the delay here! Work got super busy right after I launched this because I had put off some stuff for this, but I'll be going through the issues soon. Thanks so much!

No worries, it's FOSS after all. Thanks again for making this library. Being able to import/export an existing SQLite database would basically make irrelevant the one thing PouchDB still has going for it (replicated sync), and I'd love to get out of PouchDB hell...

@dpmccabe , In case you're still looking for a solution, I've made a slight to change to @jlongster/sql.js and published it as @urdeveloper/sql.js. You'll be able to create the db from an existing one:
import initSqlJs from<EMAIL_ADDRESS>import { SQLiteFS } from 'absurd-sql';
import IndexedDBBackend from 'absurd-sql/dist/indexeddb-backend';

async function init() {
  let SQL = await initSqlJs({ locateFile: file => file });
  let sqlFS = new SQLiteFS(SQL.FS, new IndexedDBBackend());
  SQL.register_for_idb(sqlFS);

  SQL.FS.mkdir('/sql');
  SQL.FS.mount(sqlFS, {}, '/sql');

  // host an example file somewhere with permissive CORS settings
  const data = await fetch("https://example.com/chinook.db").then(res => res.arrayBuffer());
  const db = new SQL.Database(new Uint8Array(data), {filename: '/sql/chinook.db'});
  return db;
}

async function runQueries() {
  let db = await init();

  const stmt = db.prepare("SELECT Name FROM artists;");
  stmt.step(); // Execute the statement
  console.log(stmt.getAsObject());
}

runQueries();

Thanks for the fix. I was able to get that code to work with absurd-example-project, although only if I used 0.0.49 of absurd-sql or earlier. Starting with 0.0.50, I get this error:
sql-wasm.js:108 Uncaught (in promise) RuntimeError: abort(TypeError: Cannot read properties of undefined (reading 'constructor')). Build with -s ASSERTIONS=1 for more info.
    at Q (sql-wasm.js:108)
    at F (sql-wasm.js:187)
    at sql-wasm.wasm:0xf4a4c
    at sql-wasm.wasm:0xf1a64
    at sql-wasm.wasm:0x5c411
    at sql-wasm.wasm:0xf233a
    at Module._sqlite3_open (sql-wasm.js:202)
    at Ia (sql-wasm.js:102)
    at sql-wasm.js:205
    at new c (sql-wasm.js:75)

Maybe this is just a problem with the distributed wasm files on npm?

From which package you're using sql-wasm.wasm?

Hmm, never mind. When I copied a fresh copy of sql-wasm.wasm from @urdeveloper/sql.js/dist/ it worked this time. Feel free to close this issue.

@urdeveloper Are you planning on submitting your change in a PR? I find your changes very useful.

@awmuncy, I'd love to but I'm pretty sure, it's not going to go anywhere. jlongster himself also made a PR on the upstream sql.js and which never been merged. Th good thing is you have your own fork because both absurd-sql and sql.js authors were very generous with the licensing. Cheers!


GSSOC '24 : A Flask-Powered E-Commerce Website Built from Web-Scraped Data

Is your proposal related to a problem? Please describe.
ScrapToShop is an e-commerce website built using HTML, CSS, JavaScript, and Flask, designed to showcase products sourced from web-scraped data. Leveraging Flask's capabilities, the website offers a seamless browsing and shopping experience for users. The project addresses the challenge of aggregating product information from various websites and presenting it in a cohesive and user-friendly manner.
Add any other context or screenshots about the proposal request here.
ScrapToShop provides a solution for businesses or individuals looking to set up an online store without maintaining an extensive inventory. By scraping product data from external websites, the website ensures a diverse range of offerings without the need for manual input. With its minimalist design and efficient functionality, ScrapToShop offers a hassle-free e-commerce solution for both sellers and buyers.

please assign me this issue

@Kushal997-das You already created one issue work on that first .


Proposal: The News Update Bot of Telegram

Description

I created a bot to update daily news for users.
Every day when you submit /news, the bot will send you back the latest blockchain news of the day.

Approved for 300 KLAY payout for this telegram bot. Use CryptoPanic API please.

https://github.com/binaaaaaaaaa/TelegramBotNews


Image issue and widget child framework

The first issue happens when I tap on the carisle card. Here is my pubspec.yaml
asset:
    asset/background_pics/Tree_Logo.png

I thought I had this set up correctly but I still get an error message. This is the code:
return Center(
    child: Column(
      mainAxisSize: MainAxisSize.min,
      children: <Widget>[
        const SizedBox(height: 8.0),
        Opacity(
          opacity: 0.66,
          child: Image.asset(
            'asset/background_pics/Tree_Logo.png',
          ),
        ),

Also, here:
final initSettings = InitializationSettings(
      AndroidInitializationSettings('Tree_Logo'),
      IOSInitializationSettings(),
    );

I/flutter (13195): The following assertion was thrown resolving an image codec:
I/flutter (13195): Unable to load asset: asset/background_pics/Tree_Logo.png
I/flutter (13195):
I/flutter (13195): When the exception was thrown, this was the stack:
I/flutter (13195): #0 PlatformAssetBundle.load (package:flutter/src/services/asset_bundle.dart:221:7)
I/flutter (13195): <asynchronous suspenstion>
I/flutter (13195): #1 AssetBundleImageProvider._loadAsyc (package:flutter/src/painting/image_provider.dart:448:44)I/flutter (13195): <asynchronous suspenstion>
I/flutter (13195): #2 AssetBundleImageProvider._load (package:flutter/src/painting/image_provider.dart:433:14)
I/flutter (13195): #3 ImageProvider.resolve.<anonymous closure>.<anonymous closure>(package:flutter/src/painting/image_provider.dart:299:46)
I/flutter (13195): #4 ImageCache.putIfAbsent (package:flutter/src/painting/image_cache.dart:157:22)
I/flutter (13195): #5 ImageProvider.resolve.<anonymous closure>(package:flutter/src/painting/image_provider.dart:299:23)
I/flutter (13195): (elided 13 frames from package dart:async)
I/flutter (13195):
I/flutter (13195): Image provider: AssetImage(bundle: null, name: "asset/background_pics/Tree_Logo.png")
I/flutter (13195): Image key: AssetBundleImageKey(bundle: PlatformAssetBundle#8ff4a(), name:
I/flutter (13195): "asset/backgroung_pics/Tree_Logo.png", scale: 1.0)
I/flutter (13195):=============================================
I/flutter (13195): Another exception was thrown: 'package:flutter/src/widgets/framework/dart': Failed assertion: line 1655 pos 14: '!children.any((Widget child) => child == null)': is not true.

The second issue is when I tap on the drawer icon.

@tibbitslr do you have asset under flutter, in the list form?
flutter:
  assets:
    - asset/background_pics/Tree_Logo.png

about the second exception, please provide a code example that reproduces the exception, but it seems to be an implementation issue.

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.