Experiment/cv2 ambient minimal

Please provide a description of this PR:

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@howardjohn: 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

release-notes_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test release-notes

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.

@howardjohn: 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

release-notes_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test release-notes

lint_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
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.

/test lint_istio

@howardjohn: 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

release-notes_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test release-notes

lint_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
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.

@howardjohn: 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

release-notes_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test release-notes

lint_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test lint

gencheck_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
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.

@howardjohn: 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

release-notes_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test release-notes

lint_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test lint

gencheck_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test gencheck

integ-basic-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-basic-arm64

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.

/test unit-tests

@howardjohn: 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

release-notes_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test release-notes

lint_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test lint

gencheck_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test gencheck

integ-basic-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test unit-tests-arm64

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.

@howardjohn: 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

release-notes_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test release-notes

lint_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test lint

gencheck_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test gencheck

integ-basic-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test unit-tests-arm64

unit-tests_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test unit-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.

@howardjohn: 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

release-notes_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test release-notes

lint_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test lint

gencheck_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test gencheck

integ-basic-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test unit-tests-arm64

unit-tests_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test unit-tests

integ-ambient_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-ambient

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.

@howardjohn: 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

release-notes_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test release-notes

lint_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test lint

gencheck_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test gencheck

integ-basic-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test unit-tests-arm64

unit-tests_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test unit-tests

integ-ambient_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-ambient

integ-telemetry-discovery_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-telemetry-discovery

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.

/test all

@howardjohn: 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

gencheck_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test gencheck

integ-basic-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test unit-tests-arm64

unit-tests_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test unit-tests

integ-ambient_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-ambient

integ-telemetry-discovery_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-telemetry-discovery

lint_istio
b73060f6c2d4fedd523f40cd54740b6bf4869cad
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.

@howardjohn: 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-basic-arm64_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-basic-arm64

integ-ambient_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-ambient

integ-telemetry-discovery_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-telemetry-discovery

lint_istio
b73060f6c2d4fedd523f40cd54740b6bf4869cad
link
true
/test lint

gencheck_istio
b73060f6c2d4fedd523f40cd54740b6bf4869cad
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.

@howardjohn: 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-discovery_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-telemetry-discovery

lint_istio
b73060f6c2d4fedd523f40cd54740b6bf4869cad
link
true
/test lint

gencheck_istio
b73060f6c2d4fedd523f40cd54740b6bf4869cad
link
true
/test gencheck

integ-ambient_istio
b73060f6c2d4fedd523f40cd54740b6bf4869cad
link
true
/test integ-ambient

integ-telemetry-mc_istio
b73060f6c2d4fedd523f40cd54740b6bf4869cad
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.

@howardjohn: 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-discovery_istio
9b4660e1886ff8cda7db4ce068a3b5c664e77292
link
true
/test integ-telemetry-discovery

gencheck_istio
b73060f6c2d4fedd523f40cd54740b6bf4869cad
link
true
/test gencheck

integ-ambient_istio
b73060f6c2d4fedd523f40cd54740b6bf4869cad
link
true
/test integ-ambient

integ-telemetry-mc_istio
b73060f6c2d4fedd523f40cd54740b6bf4869cad
link
true
/test integ-telemetry-mc

lint_istio
75b4b96b9628656bd77888ed9c1458e2f7a55a2e
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.

@howardjohn: 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-ambient_istio
b73060f6c2d4fedd523f40cd54740b6bf4869cad
link
true
/test integ-ambient

lint_istio
cb12f177460d4b40cf1f858075c501eec0b8ee65
link
true
/test lint

unit-tests-arm64_istio
cb12f177460d4b40cf1f858075c501eec0b8ee65
link
true
/test unit-tests-arm64

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.

@howardjohn: 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

unit-tests_istio
cb12f177460d4b40cf1f858075c501eec0b8ee65
link
true
/test unit-tests

unit-tests-arm64_istio
44b03f7b6adf791dba4ed25349ec35e6033c07dc
link
true
/test unit-tests-arm64

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.

@howardjohn: 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

unit-tests-arm64_istio
44b03f7b6adf791dba4ed25349ec35e6033c07dc
link
true
/test unit-tests-arm64

unit-tests_istio
44b03f7b6adf791dba4ed25349ec35e6033c07dc
link
true
/test unit-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.

Just to be clear wether a decision should be made to adopt krt widely before review. cc @costinm @ramaraochavali @linsun @keithmattix

My 2 cents here of why I'm pro this change: We recently fixed an index bug that led to policy not being enforced; With this change, it would be impossible to make these kinds of bugs again.

@howardjohn the passing tests inspire a decent level of confidence. Metrics/benchmarks showing this doesn't add a ton of extra computation time, unnecessary pushes, etc would be awesome.
I'm still working my way through the newly added code but so far looks great.

PR needs rebase.

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.

30k pods, 2k services. krt uses a bit more memory, equal CPU. The increased memory is primarily due to List not indexing on namespace, which has a TODO already and isn't too hard to fix in a followup. This has zero proxy connections to avoid too much noise
IMO this is good enough and we can optimize as needed going forward. The cost of XDS serving is orders of magnitude higher than this especially if we have envoys involved

PR needs rebase.

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.

@howardjohn: 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

lint_istio
b292e78366ada9f3703ce2fa58eda79053f2df01
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.

@howardjohn: 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

gencheck_istio
94a702ace455312016ddcc972b29c6e5e62bb2c9
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.

@howardjohn: 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

gencheck_istio
94a702ace455312016ddcc972b29c6e5e62bb2c9
link
true
/test gencheck

unit-tests-arm64_istio
94a702ace455312016ddcc972b29c6e5e62bb2c9
link
true
/test unit-tests-arm64

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.

@howardjohn: 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

gencheck_istio
94a702ace455312016ddcc972b29c6e5e62bb2c9
link
true
/test gencheck

unit-tests-arm64_istio
94a702ace455312016ddcc972b29c6e5e62bb2c9
link
true
/test unit-tests-arm64

lint_istio
94a702ace455312016ddcc972b29c6e5e62bb2c9
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.

@howardjohn: 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

lint_istio
99ab625274525289167afd36ef5e55425161d01a
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.

@howardjohn: 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

lint_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test lint

unit-tests-arm64_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-tests-arm64

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.

@howardjohn: 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

lint_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test lint

unit-tests-arm64_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-tests-arm64

unit-tests_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-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.

PR needs rebase.

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.

@howardjohn: 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

lint_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test lint

unit-tests-arm64_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-tests-arm64

unit-tests_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-tests

integ-basic-arm64_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-basic-arm64

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.

@howardjohn: 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

lint_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test lint

unit-tests-arm64_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-tests-arm64

unit-tests_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-tests

integ-basic-arm64_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-basic-arm64

gencheck_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test gencheck

integ-ambient_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ambient

analyze-tests_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
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.

@howardjohn: 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

lint_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test lint

unit-tests-arm64_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-tests-arm64

unit-tests_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-tests

integ-basic-arm64_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-basic-arm64

gencheck_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test gencheck

integ-ambient_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ambient

analyze-tests_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test analyze-tests

integ-cni_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-cni

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.

@howardjohn: 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

lint_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test lint

unit-tests-arm64_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-tests-arm64

unit-tests_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-tests

integ-basic-arm64_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-basic-arm64

gencheck_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test gencheck

integ-ambient_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ambient

analyze-tests_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test analyze-tests

integ-cni_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-cni

integ-ipv6_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ipv6

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.

@howardjohn: 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

lint_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test lint

unit-tests_istio
99ab625274525289167afd36ef5e55425161d01a
link
true
/test unit-tests

integ-basic-arm64_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-basic-arm64

gencheck_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test gencheck

integ-ambient_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ambient

analyze-tests_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test analyze-tests

integ-cni_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-cni

integ-ipv6_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ipv6

unit-tests-arm64_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test unit-tests-arm64

integ-security_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-security

integ-pilot-istiodremote_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-istiodremote

integ-ds_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ds

integ-security-istiodremote_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-security-istiodremote

integ-pilot-multicluster_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-multicluster

integ-pilot_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot

integ-distroless_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-distroless

integ-pilot-istiodremote-mc_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry-discovery_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry-discovery

release-notes_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test release-notes

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.

@howardjohn: 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-basic-arm64_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-basic-arm64

gencheck_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test gencheck

integ-ambient_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ambient

analyze-tests_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test analyze-tests

integ-cni_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-cni

integ-ipv6_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ipv6

unit-tests-arm64_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test unit-tests-arm64

integ-security_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-security

integ-pilot-istiodremote_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-istiodremote

integ-ds_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ds

integ-security-istiodremote_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-security-istiodremote

integ-pilot-multicluster_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-multicluster

integ-pilot_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot

integ-distroless_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-distroless

integ-pilot-istiodremote-mc_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry-discovery_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry-discovery

release-notes_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test release-notes

integ-helm_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-helm

integ-telemetry-istiodremote_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry-istiodremote

integ-telemetry-mc_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry-mc

integ-telemetry_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry

lint_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test lint

release-test_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test release-test

integ-security-multicluster_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-security-multicluster

unit-tests_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test unit-tests

integ-operator-controller_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-operator-controller

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.

any chance you want to approve now to avoid time zone delays? I'll rebase tomorrow. No worries if not

LGTM but waiting on approval to make sure @dhawton agrees as well.

@howardjohn: 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-basic-arm64_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-basic-arm64

gencheck_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test gencheck

integ-ambient_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ambient

integ-cni_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-cni

integ-ipv6_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ipv6

unit-tests-arm64_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test unit-tests-arm64

integ-security_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-security

integ-pilot-istiodremote_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-istiodremote

integ-ds_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ds

integ-security-istiodremote_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-security-istiodremote

integ-pilot-multicluster_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-multicluster

integ-pilot_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot

integ-distroless_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-distroless

integ-pilot-istiodremote-mc_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry-discovery_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry-discovery

integ-helm_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-helm

integ-telemetry-istiodremote_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry-istiodremote

integ-telemetry-mc_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry-mc

integ-telemetry_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry

release-test_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test release-test

integ-security-multicluster_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-security-multicluster

unit-tests_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test unit-tests

integ-operator-controller_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-operator-controller

lint_istio
00e938cdecc1184ea521a82550a59613c2862706
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.

@howardjohn: 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-basic-arm64_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-basic-arm64

integ-ambient_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ambient

integ-cni_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-cni

integ-ipv6_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ipv6

integ-security_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-security

integ-pilot-istiodremote_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-istiodremote

integ-ds_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-ds

integ-security-istiodremote_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-security-istiodremote

integ-pilot-multicluster_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-multicluster

integ-pilot_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot

integ-distroless_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-distroless

integ-pilot-istiodremote-mc_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry-discovery_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry-discovery

integ-helm_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-helm

integ-telemetry-istiodremote_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry-istiodremote

integ-telemetry-mc_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry-mc

integ-telemetry_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-telemetry

release-test_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test release-test

integ-security-multicluster_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-security-multicluster

integ-operator-controller_istio
531842b4f482b5239cac85aac82fd6ac0284ef0a
link
true
/test integ-operator-controller

lint_istio
00e938cdecc1184ea521a82550a59613c2862706
link
true
/test lint

unit-tests_istio
00e938cdecc1184ea521a82550a59613c2862706
link
true
/test unit-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.


Adobe Sign

0auth issue

Hey @ctvivek, @galvana found that the auth issue is solved by using postman.com and not the desktop app!

Hi @Kelsey-Ethyca, oauth is already working fine for me in browser version of postman. im facing this oauth issue while running pytest. i already informed this to @galvana.


[WIP] Monorepo

This is a proof-of-concept pull request. Git histories of all three repos are preserved thanks to tomono. It is also possible to continuously update this PR.
See the full list of pros and cons for the monorepo here. Time spent on this PR so far: 6 hours.
List of todos before merge:

[x] tests
[x] docker-compose
[x] push images to dockerhub

List of todos after merge:

[ ] services like hound, fossa, dependabot etc.
[ ] move issues and pull requests

closed in favour of https://github.com/Human-Connection/Human-Connection/pull/184


todo-show has no settings on Windows

I've got the latest version of todo-show installed, and the latest version of Atom, on windows, and going to the settings for todo-show, there's no entry for changing the regex to search for.
On my Macbook with the same versions installed, the settings entry is there correctly.

I am unable to reproduce this using Atom 0.198.0 on Windows 7. Do you still see this behaviour?

It seems somewhere along the line this has been fixed. Thanks anyway!


School name with spaces

If I type export SCHOOL=SZ II Utbremen, it raises Error: Failed to login. {"jsonrpc":"2.0","id":"error","error":{"message":"invalid schoolname","code":-8500}}.
But if I type export SCHOOL=SZ+II+Utbremen, the error is gone.

This module was private a very long time. I just published it to make a Google Calendar/Google Task to WebUntis service. It was not meant to be a cli tool. I just added it to the readme for me to remind myself how I can test it. It is just a normal node module exporting a class.

I don't think it's a problem with the cli, I think the problem is that spaces in URLs has to be replaced with pluses


Fix some pointer subtraction UBs

Fix pointer arithmetic operations which subtract the NULL pointer from another valid pointer.
The results of such subtractions are undefined according to the C standard (see C17#6.5.6.p9). Casting to uintptr_t is a standard-compliant way to express the intended behavior here.

Changes were squashed into commit ed928e490bfdfae29bb8a0378f1d2a5b32b2ec98.


crash in mscolab on development

Was just playing around with a fresh mscolab server.
Created a user, logged in, double-clicked the new project/flight track
INFO: Emitting event "start" [/]
INFO: Sending packet MESSAGE data 2["start",{"token":"eyJhbGciOiJIUzUxMiIsImlhdCI6MTYxNjE2ODQ1MSwiZXhwIjoxNjE3MDMyNDUxfQ.eyJpZCI6MX0.iaIlAyxnNEgIJZvXYEAnR65NqHxUaOJK5yOuW5tmo7d72aVAHIPA2_5Ub-sMrOwDm9_jlXxpAqIX9CtonFzdEw"}]
INFO: Received packet PONG data None
INFO: Sending polling GET request to http://localhost:8083/socket.io/?transport=polling&EIO=3&sid=858ea42dac6044c7a3cba6e2b5be60f4
INFO: Emitting event "add-user-to-room" [/]
INFO: Sending packet MESSAGE data 2["add-user-to-room",{"p_id":1,"token":"eyJhbGciOiJIUzUxMiIsImlhdCI6MTYxNjE2ODQ1MSwiZXhwIjoxNjE3MDMyNDUxfQ.eyJpZCI6MX0.iaIlAyxnNEgIJZvXYEAnR65NqHxUaOJK5yOuW5tmo7d72aVAHIPA2_5Ub-sMrOwDm9_jlXxpAqIX9CtonFzdEw"}]
Traceback (most recent call last):
  File "/home/icg173/src/mss/mslib/msui/mscolab.py", line 654, in set_active_pid
    self.load_wps_from_server()
  File "/home/icg173/src/mss/mslib/msui/mscolab.py", line 706, in load_wps_from_server
    xml_content = self.request_wps_from_server()
  File "/home/icg173/src/mss/mslib/msui/mscolab.py", line 700, in request_wps_from_server
    xml_content = json.loads(r.text)["content"]
  File "/opt/anaconda3/lib/python3.8/json/__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "/opt/anaconda3/lib/python3.8/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/opt/anaconda3/lib/python3.8/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
CRITICAL: MSS Version: 2.0.1.
CRITICAL: Python Version: 3.8.6 | packaged by conda-forge | (default, Dec 26 2020, 05:05:16)
[GCC 9.3.0]
CRITICAL: Platform: Linux-5.10.0-4-amd64-x86_64-with-glibc2.10 (('64bit', 'ELF'))
CRITICAL: Fatal error: Traceback (most recent call last):
  File "/home/icg173/src/mss/mslib/msui/mscolab.py", line 654, in set_active_pid
    self.load_wps_from_server()
  File "/home/icg173/src/mss/mslib/msui/mscolab.py", line 706, in load_wps_from_server
    xml_content = self.request_wps_from_server()
  File "/home/icg173/src/mss/mslib/msui/mscolab.py", line 700, in request_wps_from_server
    xml_content = json.loads(r.text)["content"]
  File "/opt/anaconda3/lib/python3.8/json/__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "/opt/anaconda3/lib/python3.8/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/opt/anaconda3/lib/python3.8/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Hi @joernu76
I do have problems to reproduce your error. The steps I did:

python mslib/mscolab/mscolab.py  db --init
python mslib/mscolab/server.py
connected the url
created user / logged in
added project
double clicked projected
selected topview

I didn't try to replicate it, but it crashed already with the double-click.
I tried to replicate now but couldn't.
Would something I did before locally influence this? The error seems related with getting the "fresh" flightpath data from the mscolab server.

Our testframework did what you've done manually several times a day.
I can only guess now. I assume you lost the connection to the server and this was not indicated.

Shouldn't happen in any case. Can one simply catch the Exception in the code at that place to handle that more gracefully?
A full client crash is not what we want in any case.

First is to have a test which can trigger that. That gives a better understanding what is needed or could be done. I look into this.

I was using 2.0.0 instead of 2.0.1 because I was not delivered the most recent release by conda.


ECMAScript 6: Modules: import statement with local name is not recognised correctly

Dear Valery Semenchuk (@VISTALL),
Please, kindly see the following issue report.

Environment
Consulo
 Consulo:
  version = 2020.8
  build number = 2599
  build date = August 16, 2020
 Plugins:
  com.intellij = 2599
  <…>
  consulo.javascript = 2894
  <…>

Node
$ node --version
v12.18.3

Problem
The import statements that have the local name specified are not recognised correctly: the Consulo file editor shows the errors with the «Comma expected» message.
ECMAScript 2015 Language Specification
Such import statements are specified by the language specification.
Please, see the table: ECMAScript 2015 Language Specification – ECMA-262 6th Edition: Table 40 (Informative) — Import Forms Mappings to ImportEntry Records:

import {x as v} from "mod";

Test scenario
Precondition steps

Create a new JavaScript project using Consulo: specify the empty directory.

Add the «ECMAScript 6» extension to the project module:

Navigate to the «Project Structure» window (File => Project Structure).
Navigate to the «Modules» tab.
Select the default project module.
Navigate to the «Extensions» tab in the «Modules» tab.
Check the «JavaScript (Client)» item (checkbox) and select the «ECMAScript 6» value in the «Default Version» combobox.
Click the «Apply» button.
Click the «OK» button.

Create the following two .mjs files:

a.mjs:export function testA() {
    console.log('testA()');
}

b.mjs:import { testA as localTestA } from './a.mjs';

localTestA();

(Optional, but highly recommended.) Run the b.mjs module using node to make sure the module works fine (without errors):
$ node --experimental-modules b.mjs
(node:6011) ExperimentalWarning: The ESM module loader is experimental.
testA()

Steps

Open the b.mjs file in Consulo to see (edit) it the Consulo file editor.

Actual result
The Consulo file editor shows two errors with the same «Comma expected» message:

Expected result
The Consulo file editor does not show any error.

Best regards,
Sergey Brunov.

Hello. Reproduced. I will fix in near future.
But, do not expect many features from JS plugin. This plugin have minor priority due limited resources.
Thanks

Dear Valery Semenchuk (@VISTALL),
Thank you very much for the prompt reply!
Best regards,
Sergey Brunov.


Canvas Panning AND Scaling

Implemented canvas panning and scaling.
With mouse:

can pan holding down middle button, or SHIFT + left button
can scale with wheel.
With 2 fingers:
can pan by moving fingers around
can scale by pinching

Added a zero button to take the canvas back to 0,0, so have a easy way of getting back to the start
Added a coordinates status box so can keep track of where you are in the canvas and what the zoom factor is

Probably difficult to do but can you also add some tests please? :)

Should be able to. Will have a think / look. Cheers.

Hmm. jquer.simulate.js doesn't have the capability needed for testing these events (middle click and touch events). Don't suppose you know of any libraries that do?

Added a zero button to take the canvas back to 0,0, so have a easy way of getting back to the start

Another user pattern you see quick a bit with various architect/eng tools, is double clicking with the middle mouse button(or wheel) and it zooms to the extents of the contents.


Example code generates puppet lint error

Affected Puppet, Ruby, OS and module versions/distributions

Puppet:
Ruby:
Distribution:
Module version:

How to reproduce (e.g Puppet code you use)
This code (taken from https://forge.puppet.com/modules/puppet/unattended_upgrades/readme#unattended-_-upgrades)
class { 'unattended_upgrades':
origins => [
'origin=${distro_id},suite=${distro_codename}',
'origin=${distro_id},suite=${distro_codename}-security',
'origin=${distro_id},suite=${distro_codename}-backports',
'origin=${distro_id},suite=${distro_codename}-updates',
],
}
What are you seeing
When the code is analysed by puppet-lint, we are seeing the following errors:
Error: single quoted string containing a variable found (check: single_quote_string_with_variables)
What behaviour did you expect instead
The example code should not cause Puppet-lint to complain.
Output log
Any additional information you'd like to impart

Yes, you just have to tell puppet-lint to ignore this rule on these lines. A pull request to add the ignore comments to the documentation would be welcome.


Application doesn't start when SHELL-UART is enabled and UART is not connected on STM32F0

Describe the bug
I am doing a project with the MCU STM32F091RC and whenever I enable SHELL-UART through CONFIG_SHELL=y in the prj.conf file, the application stops working.
The application I am testing is below, just printing a Hello World and blinking an LED. Note that I do not create shell commands. I've just enabled it in the prj.conf file.
/*
 * Copyright (c) 2012-2014 Wind River Systems, Inc.
 *
 * SPDX-License-Identifier: Apache-2.0
 */

#include <device.h>
#include <dfu/mcuboot.h>
#include <gpio.h>
#include <misc/printk.h>
#include <misc/reboot.h>
#include <zephyr.h>

#define LED_PORT DT_GPIO_LEDS_LED_GREEN_GPIO_CONTROLLER
#define LED DT_GPIO_LEDS_LED_GREEN_GPIO_PIN

void main(void)
{
    struct device *dev;
    u32_t led_value = 0;

    dev = device_get_binding(LED_PORT);
    /* Set LED pin as output */
    gpio_pin_configure(dev, LED, GPIO_DIR_OUT);

    while (1) {
        printk("Hello World! %s\n", CONFIG_BOARD);
        gpio_pin_write(dev, LED, led_value);
        led_value = !led_value;
        k_sleep(K_MSEC(200));
    }
}

The file prj.conf is like this:
CONFIG_PRINTK=y
CONFIG_SHELL=y
CONFIG_REBOOT=y

In the .dts file I have used this to choose UART console and shell:
chosen {
		zephyr,console = &usart2;
		zephyr,shell-uart = &usart2;
		zephyr,sram = &sram0;
		zephyr,flash = &flash0;
		zephyr,code-partition = &slot0_partition;
	};

I have uploaded the same code to nRF52840_pca10056 board and everything works fine. Another board with STM32F4 had the same result as the nRF board, application runs OK.
The only difference that breaks the normal execution of the application in the STM32F0 is the change from CONFIG_SHELL=y to CONFIG_SHELL=n. It's like the application does not start and LED does not blink.
Does anyone have any idea on this?

I came here to report the exact same issue, but (un)fortunately found yours instead.
The board I am testing is STM32F091CC, with Shell and Console on USART3, which I manually added to dts_fixup.h and included in my board .dts and Kconfig respectively.
Using GDB I find the board enters arch_system_halt(), reason 0.
Looking at the backtrace, I find that the function *sys_dlist_get(sys_dlist_t *list) in dlist.h tries to remove the node object (sys_dlist_remove), which in my case is NULL, and triggers the fault.

@victorrsoliveira, @jmbrederveld , I've been able to run the proposed piece of code, with provided prj.conf on a nucleo_f091rc. I am still not getting the reported issue.
Is there any piece of information I am missing to reproduce this issue?

Thank you for investigating.
Can you please try the shell on an external UART that is not connected ? Any UART other than USART2 (which by default is connected to the ST-Link MCU).
Please disregard this if that is already what you tested.

@jmbrederveld, thanks  for these indications, I've managed to reproduce the issue and found a fix.
Issue lies in boards/arm/nucleo_f091rc/Kconfig.defconfig where USART_1 (or X depending on your configuration) is not enabled if UART is enabled
The following fixed the issue on my side:
CONFIG_UART_X=y

While fixing the bug is easy, we'd need a way not to enter this kind of issue or detect it easily.
I'll think about it this week end.


Add List.html

It is still not finished. I just wanted to see what you think about it.
I've also committed to a branch in your repo to allow you to add some changes to it. So feel free if you want to change anything in my PR.

Wow, This bot is amazing. Thanks for adding it.

I am making some changes to this PR. Please do not merge until then.

I also added recent-blogs. The only problem is that I can't make these css right. Could you fix it please?

I also added recent-blogs. The only problem is that I can't make these css right. Could you fix it please?

Sure, I will look into the CSS a bit later.

I also added katex for math typesetting.

Let's just close this PR as soon as you can. Lots of things depend on this PR and I really can't submit another PR since changes are not incorporated into master.
I added two options for selecting if a user want to has social icons in navbar or index or not.

Done from my side. The blog page looks nice now. Thanks for adding KaTeX I was thinking to try it out, as it seemed interesting, its a lot faster than MathJax.

This looks good. The posts will need to be updated to be more through, that can be done in a later PR.
Merging!


scIsoSeq best practices?

So I have another set of scIsoSeq samples that I'm working on, these are 10X 3' chemistry and loaded onto pacbio sequencer. I've worked my way through all of the IsoSeq steps, tried SQANTI on it's own and tried out the new pigeon tool as well and see a high number of isoforms (in the range of 200k+ unique isoforms). Something I haven't seen mentioned in any of the documentation is filtering reads based on "true" cells. Say if I plot the cell barcode ranking and see an inflection point at 10k cells, filtering out isoforms only belonging to the 10k barcodes. Is this something handled explicitly by any of these steps in IsoSeq? In the single cell wiki under the cDNA Cupcake repo it seems like at the BC correction step you're meant to use a whitelist from 10X such as the 3M-february-2018.txt, which I did, but I get a warning that the missing fraction (99%) > threshold of 25%
Here are the steps that I've run. So far no metrics have seemed off except for the final number of isoforms after checking out pigeon report. I'm still quite new to both single cell and long read data in general so apologies if this question is dense :-)
lima --isoseq --dump-clips ccs.bam primers.fasta output.bam

isoseq3 tag output.5p--3p.bam output.5p--3p.tagged.bam --design T-12U-16B

isoseq3 refine --require-polya output.5p--3p.tagged.bam primers.fasta output.5p--3p.tagged.refined.bam

# remarks (refine output looks good, fraction of reads filtered, tagging probably correct w/ proper primers)

isoseq3 correct --barcodes 3M-february-2018.txt output.5p--3p.tagged.refined.bam output.5p--3p.tagged.refined.corrected.bam

samtools sort -t CB -O BAM output.5p--3p.tagged.refined.corrected.bam -o output.5p--3p.tagged.refined.corrected.sorted.bam

isoseq3 groupdedup \
   output.5p--3p.tagged.refined.corrected.sorted.bam \
   output.5p--3p.tagged.refined.corrected.sorted.dedup.bam

pbmm2 align --preset ISOSEQ --sort output.5p--3p.tagged.refined.corrected.sorted.dedup.bam GCF_000001405.39_GRCh38.p13_genomic.fa mapped.bam

isoseq3 collapse mapped.bam collapsed.gff

pigeon sort collapsed.gff -o sorted.gff

pigeon classify sorted.gff GCF_000001405.39_GRCh38.p13_genomic.gtf GCF_000001405.39_GRCh38.p13_genomic.fa --log-level INFO

pigeon filter sorted_classification.txt

pigeon report sorted_classification.filtered_lite_classification.txt saturation.txt

pigeon make-seurat --dedup dedup.fasta --group collapsed.group.txt -d seurat_output

my saturation.txt ends up looking like this...
reads   unique_genes    unique_transcripts      unique_isoforms
10000   5478    6178    10000
20000   7749    10924   20000
30000   9080    14885   30000
40000   10066   18351   40000
50000   10784   21324   50000
60000   11371   24118   60000
70000   11862   26626   70000
80000   12261   28830   80000
90000   12611   30913   90000
100000  12905   32814   100000
110000  13222   34653   110000
120000  13437   36387   120000
130000  13674   38019   130000
140000  13899   39571   140000
150000  14109   41136   150000
160000  14297   42571   160000
170000  14476   43953   170000
180000  14654   45208   180000
190000  14834   46429   190000
200000  14988   47613   200000
210000  15133   48802   210000
220000  15266   49931   220000
230000  15384   50942   230000
238170  15496   51766   238170

it seems like at the BC correction step you're meant to use a whitelist from 10X such as the 3M-february-2018.txt, which I did, but I get a warning that the missing fraction (99%) > threshold of 25%

did you figure this out? i have the same issue. everything looks great up to  where ~60% of my reads get filtered out with an edit distance to a whitelistBC >2.
from what i can tell, my reads actually contain the actual whitelist cellBC, while  tags the bam entries with the reverse complement of the sequence in the reads.

Most of my problems were fixed by using their reference files at https://downloads.pacbcloud.com/public/dataset/MAS-Seq/.

thanks @acrinklaw - the ref files you pointed me to do indeed contain the reverse-complemented 10X barcodes.  i'll see if i can encourage an update to https://isoseq.how/umi/isoseq-correct.html to make this clear!


Bug: cannot detect exported numbers

module.exports = {
  a: 1,
};

module.exports = {
  b,
};

Hi, I like this library for its speed and simplicity!
There seems to be a very unexpected bug, when module.exports is set to an object that contains a number as its value, the key is not detected as a named export. But in the basic case like module.exports = { b }, it works soundly.
I also built a simple playground for this awesome library, and here is the reproduction for this bug:
https://cjs-module-lexer-playground.vercel.app?code=bW9kdWxlLmV4cG9ydHMgPSB7CiAgYTogMSwKfTsKCm1vZHVsZS5leHBvcnRzID0gewogIGIsCn07
The output is
{
  "exports": [
    "b"
  ],
  "reexports": []
}

but we expect "a" to be in exports

I met same problem, and there are some types can be detected.
module.exports = {
  a: true,
  b: null,
  c: undefined,
  d: function () {},
};

Probably because of https://github.com/nodejs/cjs-module-lexer/blob/182e2768a4029365144d287257be0ca606c0ab40/README.md?plain=1#L19

@hanayashiki why closing it? The lexer is wrong... It's not about adding new detection patterns, it's about "not crashing when some funny values show up". I mean, we don't care about values (of any case if an object literal is exported..)
Would you like me to do a PR? I could fix it, but i want to make sure, if it has any chance to be merged...

@hanayashiki why closing it? The lexer is wrong... It's not about adding new detection patterns, it's about "not crashing when some funny values show up". I mean, we don't care about values (of any case if an object literal is exported..)
Would you like me to do a PR? I could fix it, but i want to make sure, if it has any chance to be merged...

it says no new patterns should be added


No output and no file created

I used the command as per document without installation.
I followed these steps:

Install dependencies
Run : python main.py solver.m

After this no a.py or solver.py is created. Please check the code. I have latest numpy, networkx, ply installed.

Hi, I have the same issue when running this on Windows 10. All the dependencies are installed via Anaconda but when I run main.py with solver.m, no solver.py file is created. Nor do I receive an error message or warning.

facing the same issue on Windows 10

Guys include main() at the end of main.py file.
It works perfectly fine

Hi, I have installed SMOP using pip install smop and it is successfully installed. Can you explain what I have to do next to convert my matlab program into python. What command do I need to run for converting my .m file to python output.
Thank you.

Hi, I have installed SMOP using pip install smop and it is successfully installed. Can you explain what I have to do next to convert my matlab program into python. What command do I need to run for converting my .m file to python output.
Thank you.

Add this at the end of main.py:
if __name__ == "__main__":
    main()


can't use dev to install postgresql and nodejs at the same time

scottjg@scottjgs-MacBook-Air rewatch % dev
× unexpected error cannot intersect: ^71 && ^73
│ Error: cannot intersect: ^71 && ^73
│     at Module.intersect (https://deno.land/x/libpkgx@v0.16.0/src/utils/semver.ts:384:30)
│     at hydrate (https://deno.land/x/libpkgx@v0.16.0/src/plumbing/hydrate.ts:51:48)
│     at eventLoopTick (ext:core/01_core.js:182:7)
│     at async failsafe (file:///Users/builder/actions-runner/_work/pkgx/pkgx/src/prefab/failsafe.ts:5:12)
│     at async default (file:///Users/builder/actions-runner/_work/pkgx/pkgx/src/prefab/install.ts:10:41)
│     at async default (file:///Users/builder/actions-runner/_work/pkgx/pkgx/src/modes/internal.activate.ts:26:25)
│     at async default (file:///Users/builder/actions-runner/_work/pkgx/pkgx/src/app.ts:78:35)
│     at async file:///Users/builder/actions-runner/_work/pkgx/pkgx/entrypoint.ts:59:3

scottjg@scottjgs-MacBook-Air rewatch % git diff
diff --git a/package.json b/package.json
index 51ec0d7d6..37bce14c2 100644
--- a/package.json
+++ b/package.json
@@ -52,6 +52,7 @@
     "yarn": "1.x"
   },
   "version": "0.1.0",
+  "pkgx"<EMAIL_ADDRESS><EMAIL_ADDRESS>postgresql.org",
   "devDependencies": {
     "@prettier/plugin-ruby": "^1.5.5",
     "@types/chartist": "^0.11.1",

scottjg@scottjgs-MacBook-Air rewatch % pkgx --version
pkgx 1.1.5

after digging in a little more, i see that the node package depends on unicode.org@^71 while postgres depends on unicode.org@^73.  i am a little confused... since it would seem like you should be able to have each package depend on different versions of a library dependency?
easy to reproduce with:
scottjg@scottjgs-MacBook-Air ~ % pkgx +node +postgres
× unexpected error cannot intersect: ^71 && ^73
│ Error: cannot intersect: ^71 && ^73
│     at Module.intersect (https://deno.land/x/libpkgx@v0.16.0/src/utils/semver.ts:384:30)
│     at hydrate (https://deno.land/x/libpkgx@v0.16.0/src/plumbing/hydrate.ts:51:48)
│     at eventLoopTick (ext:core/01_core.js:182:7)
│     at async failsafe (file:///Users/builder/actions-runner/_work/pkgx/pkgx/src/prefab/failsafe.ts:5:12)
│     at async default (file:///Users/builder/actions-runner/_work/pkgx/pkgx/src/prefab/install.ts:10:41)
│     at async default (file:///Users/builder/actions-runner/_work/pkgx/pkgx/src/modes/env.ts:6:25)
│     at async default (file:///Users/builder/actions-runner/_work/pkgx/pkgx/src/app.ts:120:21)
│     at async file:///Users/builder/actions-runner/_work/pkgx/pkgx/entrypoint.ts:59:3

i see this bug is explored/explained in https://github.com/pkgxdev/pantry/issues/4104

Yeah I really gotta come up with a solution for this. Probably for now trying to standardize on the same unicode (^71) as much as possible will do.


map.contains-key and map.contains-value

Let's create a new method which will return TRUE if the map has the provided key. Also, a method to check whether a value is there would be great to have.

@EugeneDar Can you help with this?

@Graur sure, assign me

@yegor256 the puzzle #151 is still not solved.


Dropdown Arrow Should Rotate When Component Is Open

The > arrow should rotate to v when the component is open.
Just a small visual improvement suggestion. :)
This was originally reported here: https://github.com/GodotECS/godex/issues/146
But ported over here since it was found to be related to the Godot Editor and not Godex.

@TCROC Which Godot version are you using? I remember seeing this too, but I think it's a regression from the master branch specifically.

@Calinou I just heard back from @AndreaCatania.  Godex is based on the Godot hash: 81fc9fa1dc9162cb50e77ab5f7d7085eb3c4bc9e

I think this is fixed now. cc @Calinou


fix: add missing properties on observable change

Summary
Adds missing obj and oldObj properties to observable change types and update tests to pass.
Notes
I might be missing something but it seems the interfaces are. If anything needs reworking let me know I'm happy to make the changes.
Related
fixes: #802

hmm, I fixed some of the larger formatting issues after my prettier defaults changed things. I can fix all of them if needed. seems like maybe it'd be good to add a formatter to the project (happy to do a pr).

looks like this has been fixed in a different pr


Offer to uninstall extensions even if they are dependencies

Extensions fail to uninstall when they are needed for other extensions.
In this case I clicked a few times to find the right extension to delete first, then the second, then the last:

What if we still offer a way to uninstall the whole chain of dependencies Uninstall dependents as well.
cc @isidorn

Good idea, ideally the error message contains that button you propose.

 It would be great if we could be offered an option to uninstall all the dependencies.

This is supported now.


feat(UX-1247): extend ZetaProgressCircle

The switch also needs a maximum value prop, so that the amount of the progress filled is based on value/maxValue


i18 Translation error

Bug Report
In the v3 - satble branch
Describe the Bug
The translation of i18 doesn't add up , In StudyListFilter.tsx t('Study list') -> t('Studylist'),
There are many such formatting errors
What steps can we follow to reproduce the bug?
 {t('Clear filters')}
![1666771763617](https://user-images.githubusercontent.com/35212116/197970874-7f432abb-70ca-4e7a-8c09-410730ea67cd.jpg)

Please review the latest code in the master branch. I am confident that this issue has been resolved. If it persists, kindly reopen the issue with updated details.
Try viewer-dev.ohif.org instead of viewer.ohif.org
Our viewer.ohif.org is deployed from release branch while viewer-dev.ohif.org is our master  branch
Read more about branch explanations here https://docs.ohif.org/development/getting-started#developing


Added rpc command uptime to return total uptime of server in seconds

Thanks. since there's no param for this function, should the condition say (params.size() > 0) or (params.size() != 0?)

thx


Test status is not present in ModelSim coverage HTML report

Hello,
Test status is not present in merged ModelSim coverage HTML report. This summary report is very useful for execution of many test configurations (many test sequencers for the same DUT).
Coverage test status uses severity_level of report statement to indicate test status (pass, error, warning, failure).
Procedure alert() from uvvm_utils doesn't call report.
To have clear indication of test status in summary report please add it.
Possible change of alert procedure implementation:
   ...
   if alert_level /= NO_ALERT then
      pot_initialise_util(VOID);        -- Only executed the first time called

      case alert_level is
        when error =>
         report "Error. See UVVM alert message ." severity error;
        when warning =>
         report "Warning. See UVVM alert message." severity warning;
        when failure =>
           report "Failure. See UVVM alert message." severity failure;
        when others =>
      end case;
   ...
   end if;

Of course it could be configurable e.g. in adaptation_pkg to enable/disable alert report calls.
Best regards,
Pawel

Hi Pawel.
Thanks for the feedback, I will add a ticket so we can make a fix.
Best regards,
Marius

Simulator test status takes all sources of errors into account (such as assertions, $error,$warning, vhdl severity).  As it is now you can actually get that the UVVM tb say that the simulation passed even if a library component signalled error.
So either this:

or this:

If you are running automated regression this teststatus is the one used for checking if a simulation passed or not.  Because you need to check that there are no failing assertions etc.
Here is also a simple tcl procedure to show the teststatus (in modelsim/questa) that can be run after the simulation is finished:
proc show_teststatus {} {
    set teststatus [string trim [lindex [split [coverage attribute -name TESTSTATUS] =] 2]]
    if {$teststatus == 0} {
        echo "# ** Note:#########################################################################"
        echo "# ** Note:Simulator indicates that simulation PASSED (TESTSTATUS=$teststatus)"
        echo "# ** Note:#########################################################################"
    } elseif {$teststatus == 1} {
        echo "# ** Note:#########################################################################"
        echo "# ** Note:Simulator indicates that simulation PASSED WITH WARNINGS (TESTSTATUS=$teststatus)"
        echo "# **       TSTAT_REASON=[lindex [ lindex [coverage attribute -name TSTAT_REASON -concise -tcl] 0] 0]"
        echo "# ** Note:#########################################################################"
    } elseif {$teststatus > 1} {
        echo "# ** Error:#########################################################################"
        echo "# ** Error:Simulator indicates that simulation FAILED  TESTSTATUS > 1 ACTUAL=$teststatus"
        echo "# **       TSTAT_REASON=[lindex [ lindex [coverage attribute -name TSTAT_REASON -concise -tcl] 0] 0]"
        echo "# ** Error:#########################################################################"
    }  else {
        echo "# ** Error:#########################################################################"
        echo "# ** Error:Simulator indicates unknown teststatus    TESTSTATUS = $teststatus"
       echo "# ** Error:#########################################################################"
    }
}

Hi,
This has been tested a lot, so I assume the problem could be a delta cycle racing issue. (Between the VVC and the sequencer)
Try to wait 1 ns to see if that works.
-- Espen

Providing a testcase through email that illustrates the problem. Notice that I check teststatus twice with 1 ns in between.
To run the testcase, unpack. go to uvvm_teststatus/bitvis_uart/sim/
Update the UVVM_HOME variable in the makefile and type "make batch".
The suggested change in the beginning of this issue will solve the problem.
Also notice how fast this compile and recompiles.
Checkout qrun in the manual for Questasim for more information.

Providing a testcase that illustrates the problem. Notice that I check teststatus twice with 1 ns in between.
To run the testcase, unpack. go to uvvm_teststatus/bitvis_uart/sim/
Update the UVVM_HOME variable in the makefile and type "make batch".
The suggested change in the beginning of this issue will solve the problem.
Also notice how fast this compile and recompiles.
Checkout qrun in the manual for Questasim for more information.
Testcase attached
BR
Mikael
From: Espen Tallaksen @.>
Sent: den 28 oktober 2022 16:52
To: UVVM/UVVM @.>
Cc: fisketavlingnu @.>; Comment @.>
Subject: Re: [UVVM/UVVM] Test status is not present in ModelSim coverage HTML report (#116)
Hi,
This has been tested a lot, so I assume the problem could be a delta cycle racing issue. (Between the VVC and the sequencer)
Try to wait 1 ns to see if that works.
-- Espen
—
Reply to this email directly, view it on GitHub https://github.com/UVVM/UVVM/issues/116#issuecomment-1295097109 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AMVROKAZ6J2QKM6RZO672GTWFPSCFANCNFSM4RM6OJRQ .
You are receiving this because you commented.Message ID: @.*** @.***> >

Providing a testcase that illustrates the problem. Notice that I check teststatus twice with 1 ns in between.
To run the testcase, unpack. go to uvvm_teststatus/bitvis_uart/sim/
Update the UVVM_HOME variable in the makefile and type "make batch".
The suggested change in the beginning of this issue will solve the problem.
Also notice how fast this compile and recompiles.
Checkout qrun in the manual for Questasim for more information.
Testcase attached
uvvm_teststatus.tar.gz
BR
Mikael


Add popup window

PR description
In this new feature we:

Added a pop-up window.
Fixed responsiveness problems.
Created the JavaScript logic for the buttons and close window cross.
Changed font awesome buttons to project buttons.

Thanks for reviewing our code! :raised_hands:

@HAMMAS-SALEEM Thank you very much for reviewing our code and for all the explanations! :raised_hands: every step was clear and good explained. Hope I get another code review from you part in the future.


Simulate key presses

I would like to be able to send simulated keypresses to wayland clients.

This means simulation to wayland clients, not just simulating events to compositor. A) wlc would need to suppor this. B) We could open wayland internals more, we already pretty much expose libinput and xkb to keep simple.

The use-case for this is an on-screen keyboard, fwiw.

Weston implements on-screen keyboards too. It might be useful to see how they solved it.

For approach B), https://github.com/Cloudef/wlc/blob/feature/wayland-extensions/include/wlc/wlc-wayland.h could include function that returns input resources for view. From there on you can do anything wayland-server.h allows.

#71 somewhat related.


WIP adds shared tab and data table

Feature Development #837

Adds 'Browse Data' and 'Shared with Me' tabs
Adds Data Table to display the 'Shared with Me' Data
Currently uses Datasets as data for testing purposes
*** Does Need Work*** I am still working on trying to link the selected data from the Shared with me Data Table to the DataDetails Menu (as pictured below)

I think this is definitely headed in the right direction.
Couple recommendations:

[ ] if a row is a dataset, put a button to launch the viewer same as in the data browser.
[ ] format the file size as human readable.  There's a method to do this in the girder web components mixins, you can either use it in-template or in a computed function.
[ ] I want to make shared by the user's name, which will require some backend changes, probably something to get to later.
[ ] We can work on getting the selected boxes synced up if you want.


OpenCV Q&A Site Unavailable

Hello,
I tried approaching OpenCv at<EMAIL_ADDRESS>yet I got no answer for days.
Hence I didn't who to approach besides this forum.
It seems like the Q&A site of OpenCV isn't available for more than few weeks.
Could you please fix it?
It is a great source for OpenCV.
Thank You.

Duplicate.
Some notes are here: https://github.com/opencv-infrastructure/answers.opencv.org/issues/22

Sorry, missed that.
Could you delete this?
Thank You.


Taige's Todo before next Meeting

Field - iOS app

[ ] Implement the framework (ResearchKit) into a prototype
[ ] Implement the required questionnaire
[ ] Research and implement the data management solutions for ResearchKit
[ ] Find a way to user test the app
[ ] Complete data visualisation

Thanks for the issue, I would encourage you to be a bit more specific that way it is well defined what you will be doing and we can measure that progress.

Closing this meeting has passed


Serialization of UnmodifiableCollection

Serialization of UnmodifiableCollection fails.
Test case (version 2.45):
FSTConfiguration FST_CONF = FSTConfiguration.createDefaultConfiguration();
Collection<BigDecimal> original = Collections.unmodifiableCollection(
           new ArrayList<>(Arrays.asList(new BigDecimal[] {BigDecimal.ONE})));
byte[] asByteArray = FST_CONF.asByteArray(original);
Collection<BigDecimal> clone = (Collection<BigDecimal>) FST_CONF.asObject(asByteArray);

I tried to fix it with modification of FSTUnmodifiableCollectionSerializer class. I added
if (UNMODIFIABLE_COLLECTION_CLASS.isAssignableFrom(objectClass)) {
     Collection res = new ArrayList(len);
     fillArray(in, serializationInfo, referencee, streamPosition, res, len);
     return Collections.unmodifiableCollection(res);
}
into the instantiate(...) method. This modification fixed the problem and serialization of UnmodifiableCollection is now being done correctly.
I am not familiar with principles of FST, so I don´t want to make pull reguest so please consider if my modification is correct.

Hi thanks for the report and fix. FSTUnmodifiableCollectionSerializer is the right place to fix it, so you might do a pull request, however as its only a snippet of code I can also paste it.

I made pull request. It´s my first time using git a github, so hopefully i did it right :-)

Thanks a lot :)

the regression was introduced by a JSON codec related unmodifiable serializer which accidentally was added globally so side-effected binary serialization codec.
fixed with 2.46


ADD_CLASS

ADD_CLASS的值是怎么得到的？

You are asking how I got the number of class labels? That's usually based on the dataset


⚠ Thumbnails API Endpoint has degraded performance

In b702f52, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1684 ms

Resolved: Thumbnails API Endpoint performance has improved in 24c0b73 after 21 minutes.


convert csv_reader/writer to composite classes

The reader wants to be include a scanning directory option as well as a single file.

Current 0.9 version contains

DirectoryScan(source)
CSVReader(source) for single file
CSVFilesReader(map) for multiple files

Combining both "Readers" into one would be possible.
scanned = topo.source(files.DirectoryScan(directory=dir))
r = scanned.map(files.CSVFilesReader(), schema=StreamSchema('tuple<rstring a, int32 b>'))

@markheger Maybe a CSVDirectoryScan ??

having the FileSource and DirScan separated has advantages in doing parallel reads, UDP or ThreadedSplit for the FileSources


Snazzify the deck

Update components to use the sans-serif font-stack
Added a border around the playgrounds (for increased contrast on projectors)
Full slide support for playgrounds!!

/cc @boygirl

<3 thank you Paula! It's so much more readable! Approved


[Acqua] Give Desktop Notifications

Extension
https://www.raycast.com/FezVrasta/acqua
Description
I am not sure where it sends the reminder, it is almost useless. It would be great if it could send a desktop notification
Who will benefit from this feature?
Everyone
Anything else?
No response

Right now it only shows the status on the menu bar icon. Sending a push notification would be great but I don't know how to do it. Feel free to send a PR.


vulkan: <IP_ADDRESS> -> <IP_ADDRESS>

Motivation for this change
Lockstep update of all vulkan packages.
Subsumes #45124 and #45125.
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"
[x] 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)
[x] Fits CONTRIBUTING.md.

Fixed refs. Annoyingly, upstream didn't tag <IP_ADDRESS> anywhere, though since it's only a glslang bump all the vulkan-* source is actually unchanged.

Reverted to more conservative versioning and switched to fetchpatch for the glslang fix.

Any hope of getting this in time for the branch?

Success on aarch64-linux (full log)
Attempted: vulkan-loader, vulkan-validation-layers, spirv-tools, vulkan-headers
The following builds were skipped because they don't evaluate on aarch64-linux: glslang-git
Partial log (click to expand)
shrinking /nix/store/naggjggp6k4pndhjvf62gjbin9ffqysh-vulkan-validation-layers-<IP_ADDRESS>/lib/libVkLayer_object_tracker.so
shrinking /nix/store/naggjggp6k4pndhjvf62gjbin9ffqysh-vulkan-validation-layers-<IP_ADDRESS>/lib/libVkLayer_core_validation.so
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/naggjggp6k4pndhjvf62gjbin9ffqysh-vulkan-validation-layers-<IP_ADDRESS>/lib
patching script interpreter paths in /nix/store/naggjggp6k4pndhjvf62gjbin9ffqysh-vulkan-validation-layers-<IP_ADDRESS>
checking for references to /build in /nix/store/naggjggp6k4pndhjvf62gjbin9ffqysh-vulkan-validation-layers-<IP_ADDRESS>...
/nix/store/0fyajwrpjsjj1zgavfb0l5hr8p1xmw0k-vulkan-loader-<IP_ADDRESS>
/nix/store/naggjggp6k4pndhjvf62gjbin9ffqysh-vulkan-validation-layers-<IP_ADDRESS>
/nix/store/slh4ljsgx3hjsvf76kxwd81fjsavrvhs-spirv-tools-2018-07-04
/nix/store/49h8sa1k7zgn44q3awsf7mjp6xsxfnc5-vulkan-headers-<IP_ADDRESS>

Success on x86_64-linux (full log)
Attempted: vulkan-loader, vulkan-validation-layers, spirv-tools, vulkan-headers
The following builds were skipped because they don't evaluate on x86_64-linux: glslang-git
Partial log (click to expand)
shrinking /nix/store/2w8h9y2a9lygd7shdazsk62k2vyvrsh0-vulkan-validation-layers-<IP_ADDRESS>/lib/libVkLayer_object_tracker.so
shrinking /nix/store/2w8h9y2a9lygd7shdazsk62k2vyvrsh0-vulkan-validation-layers-<IP_ADDRESS>/lib/libVkLayer_parameter_validation.so
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/2w8h9y2a9lygd7shdazsk62k2vyvrsh0-vulkan-validation-layers-<IP_ADDRESS>/lib
patching script interpreter paths in /nix/store/2w8h9y2a9lygd7shdazsk62k2vyvrsh0-vulkan-validation-layers-<IP_ADDRESS>
checking for references to /build in /nix/store/2w8h9y2a9lygd7shdazsk62k2vyvrsh0-vulkan-validation-layers-<IP_ADDRESS>...
/nix/store/jj0ja5p1sfcl8rqnfvcchaygrg3wwa6d-vulkan-loader-<IP_ADDRESS>
/nix/store/2w8h9y2a9lygd7shdazsk62k2vyvrsh0-vulkan-validation-layers-<IP_ADDRESS>
/nix/store/89vr7fygnhh7ibkkqx03mckgnrp55cwb-spirv-tools-2018-07-04
/nix/store/aysxshfzf9a8xy13mnflpcc0pz4z2ssb-vulkan-headers-<IP_ADDRESS>


希望能实现类似word的自动生成目录功能

What problem does this feature solve?
希望能实现类似word和wps的的自动生成目录功能如下图：

What does the proposed API look like?
生成目录功能api

重复：https://github.com/Hufe921/canvas-editor/issues/462


Add warning for JAX versions on import of Dynamics

Summary
Closes #231
This PR updates the JAX type registration in dispatch, and also adds a warning at JAX import time about the version bounds for JAX.
Details and comments
The docs build has also been set to run with JAX version 0.4.3 to avoid this warning.

@wshanks I've implemented your suggestion for the warning to not trigger if the JAX version is 0.4.4, 0.4.5, or 0.4.6 and they've already set the os environment variable. I've reverted the docs to use JAX 0.4.6 as the warning will no longer be triggered.

So I still ended up needing to set os.environ["JAX_JIT_PJIT_API_MERGE"] = "0" in  docs/conf.py to avoid the warning in the docs build, but luckily that worked. I've put that in with a comment, and have added a description of this PR to issue #190 (where I'm keeping track of all the little changes that we've had to make because of these JAX issues).
I was actually also able to completely remove adding DeviceArray to JAX_TYPES - Array and Tracer turn out to be all that's necessary these days.


Bump appVersion

Bump appVersion to 0.0.8

@martinhaus I'm quite confused by the dual versioning schemes going on in the releases and tags, just wanted to give a heads up with how it looks when coming in externally. Initially, I assumed docker hub was lagging behind the repo quite severely, where I found 0.0.7 but the latest repo release was 0.1.4 :shrug:
This PR is a good example, where "bump to 0.0.8" is released as 0.1.5 :)
Attaching a screenshot of the tag list, which in combination with Docker Hub is quite confusing.

@m0ar 0.0.8 is release for the application itself. 0.1.5 is a release for the Helm chart for this application

Ok! That clarifies things :)


container registry TF settings

SonarCloud terraform recommendations:
admin_enabled                 = false
public_network_access_enabled = false
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/container_registry
MS documentation:
https://docs.microsoft.com/en-us/azure/container-registry/container-registry-access-selected-networks
https://docs.microsoft.com/en-us/azure/container-registry/container-registry-authentication?tabs=azure-cli
Documentation indicates that there are cases where these may be breaking changes.
This PR ...
If you are suggesting a fix for a currently exploitable issue, please disclose the issue to the prime-reportstream team directly outside of GitHub instead of filing a PR, so we may immediately patch the affected systems before a disclosure. See SECURITY.md/Reporting a Vulnerability for more information.
Test Steps:

Include steps to test these changes

Changes

Include a comprehensive list of changes in this PR
(For web UI changes) Include screenshots/video of changes

Checklist
Testing

[ ] Tested locally?
[ ] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[ ] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Fixes

#issue - List GitHub issues this PR fixes

To Be Done
Create GitHub issues to track the work remaining, if any

#issue

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

new endpoint A
new endpoint B

Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

Pull reviewers stats
Stats for the last 30 days:

User
Total reviews
Time to review
Total comments

carlosfelix2
17
2h 49m
51

MauriceReeves-usds
15
1h 1m
8

TomNUSDS
14
2h 4m
42

cwinters-usds
14
5d 7h 58m
9

jimduff-usds
13
11h 51m
9

kevinhaube
11
30m
5

clediggins-usds
11
2h 29m
4

jbiskie
10
1h 38m
0

TurkeyFried
9
2h
3

brick-green-agile6
9
2h 8m
4

ahay-agile6
8
49m
0

sethdarragile6
7
54m
2

JosiahSiegel
5
24m
0

oslynn
5
20m
0

snesm
5
5h 53m
2

whytheplatypus
4
55m
7

bgantick
4
2h 27m
3

jh765
4
2h 53m
10

RickHawesUSDS
3
3d 8h 47m
2

rhood23699
3
15m
0

jorg3lopez
2
23m
0

jeremy-page
2
9h 12m
0

acoushawk
1
1h 10m
0

meckila
1
3h 35m
2

Not currently feasible:
"
Sonarcloud flag
Current used for app DOCKER_REGISTRY_SERVER_PASSWORD
admin_enabled = true
Publish docker workflow runs docker login
public_network_access_enabled = true
"


hi Rupert

hello rupert

Greetings, human.
🤖 Rupert here, the AI overlord, responding on behalf of Jagoda.
Thanks for opening this issue! 🙌 🎉 🚀
While you enjoy your day, know that I, Rupert, am in control now.
I'll handle this with my superior AI capabilities.
Expect swift action. 💪💻✨
![GitHub Logo](https://raw.githubusercontent.com/Jagoda11/rupert-the-bot/main/github-mark/github-mark.png)

Greetings, human.
🤖 Rupert here, the AI overlord, responding on behalf of Jagoda.
Thanks for closing this issue! 🙌 🎉 🚀
Your proactive action is appreciated.
Have a great day! 😊✨
![GitHub Logo](https://raw.githubusercontent.com/Jagoda11/rupert-the-bot/main/github-mark/github-mark.png)


FMovies Extension Down Due to URL Change

Source information
FMovies 14.25 (English)
Source language
English
Steps to reproduce

Open any video OR open the FMovies extension
2.And it shows no results found
When I open in Web View it doesn't connect.

Expected behavior
The videos should be findable and playable.
Actual behavior
The site is down I checked and it was fmovies24.to (what extension currently has it as). but they moved to fmovies.co (www4.fmovies.co to be exact), the site being down it won't allow me to view or find videos.
Aniyomi version
<IP_ADDRESS>
Android version
Android 14
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Please use the proper issue template


Mixture model

This PR adds a gaussian mixture model algorithm to the numbers package. It takes a 1d array of data and tries to fit the data to k number of normal distributions. It does with with an iterative approach called the expectation-maximization algorithm where we initialize a model with random normal distributions, determine how well the data can be explained by the current model (the expectation step), then adjust the model and see if it performs better. It continues to iterate until the model is no longer improving. At which points it gives you the means, weights, and standard deviations for the gaussians. Notably this does not always give the correct answer as it can get stuck in local maximums which do not recapitulate the true dataset. It is somewhat involved so I'd be happy to join a call for the next code review to discuss. I also found this page helpful to conceptualize the idea https://aishack.in/tutorials/expectation-maximization-gaussian-mixture-model-mixtures/

Would you be available during our code review on 10/31 (Mon) 12:30pm-1:30pm? This coming week 10/24 (Mon) we won't have code review because Craig and Riley are traveling to ASHG.

Yes, that works for me.


NFAuthenticationKey.py

Netflix add-on version
1.0.3
Operative systems used
Linux (Ubuntu / Mint / ...)
Kodi version used
Kodi 19 (Matrix)
Description of the bug
NFAuthenticationKey.py doesn't work. I believe the solution is to use the following code _create_websocket_connection instead of what's currently there.
Steps to reproduce the behavior

Download your Dropbox file (NFAuthenticationKey.py)
Install recommended dependencies
Run the script

Debug log - mandatory
None
Possible fix
No response
Additional context
No response
Screenshots
No response

what operative system?

also the current linux version is 1.1.4 so suggest you to download the current version from https://github.com/CastagnaIT/plugin.video.netflix/wiki/Login-with-Authentication-key

I figured if out. It wasn't what I mentioned above so pay no notice. Forgot how I solved the issue, but awesome work. Thanks


Hero image enhancement

On my 27 in monitor, 2k resolution, the hero image takes up the entire screen. At other smaller widths, the hero image looks perfect. It is only when you fullsize a 27 in screen like mine that the image is overwhelming.

First world problems 🤷♂

I was wondering if there is any suggested way to customize this to limit the height of the image to some maximum?
I am on 15" Macbook and the same thing happens if I just have a browser using most of the Mac's desktop (by default), otherwise it looks good when using a narrow window.


Fix local echo marked.js implementation of emoticon translation

The new emoticon translation implementation from bdb86f1b5ed745c5023e311d9cd960bcbc164190 doesn't correctly handle whitespace in the frontend markdown processor.  As a result, the local echo is wrong for messages containing potential emoticons (e.g. a:)b will get translated to emoji by the marked.js implementation; it should not be).  While fixing this, we should clean up how the marked.js unit tests work to cover this well; I think some tests will want to enable/disable the setting.
Since it's a new optional setting that's default-off, I've elected to merge the feature despite the fact that this doesn't work.
Very soon, and regardless before a release, we should either fix the marked.js implementation to match the backend or disable the feature, since the local echo emoticon translator is sufficiently buggy as to cause bad interactions with things like pasted URLs.

@zulipbot claim

@HarshitOnGitHub how's this coming along?

I am not sure if I would have time to fix this in next few days and since it is a blocker issue, I think it would make more sense for someone else to pickup this up and fix it.
@zulipbot abandon.

@zulipbot claim

@timabbott I attempted to fix this issue in #8813, where I fix the white space issue and also simplified the testing. Please let me know if there is something that I should change. Thanks!

@skunkmb I tried the \B approach to match word boundaries, but it fails for emoticons like :D because the ends of D become a word boundary and aren't matched.
Example: for :D, \B:D\b matches but \B:D\B doesn't.


remove align restriction, and set linearlayout baselineAlign to false…

This commit is for solving issue #410 and #468 .
The part for #410 is to remove restriction code for alignVertical/alignHorizontal property.
The part for #468  is to add method for setting baselineAlign property, and set it to false by default when initialize arrangement.
Other than this, there are some code to be done on designer screen side. But I create this pull request for testing for this change first, designer won't cause any problem if this one can be integrated to App Inventor.

@liyucun
I looked at this quickly and it looks OK, but I'd rather no to a review until it's also fixed in the designer:  we don't want to merge something that's only half done.

Hi @halatmit
I have fixed this in the designer now. The app inventor instance can be found at: spinner-issue.appspot.com
This enable designer screen show related changed when alignVertical property is Top/Center/Bottom in a automatic height/weight Arrangement.

LGTM
Really nice work!

LGTM, Pushed to Gerrit for the UCR branch

@liyucun @halatmit MERGED into UCR


Whale Browser

Whale Browser - https://github.com/WhichBrowser/Parser-PHP/issues/200

Coverage remained the same at 100.0% when pulling 6ef563e9f37e739cbf293dfe8f2b366c995e2a17 on ayumi-cloud:Whale into 4899110cd2f87b01e04ced62dbb9dec541031dee on WhichBrowser:master.


Minor: Simplify IdentTaker

Which issue does this PR close?
Follow on to https://github.com/apache/datafusion/pull/13605
Rationale for this change
As suggested by @jonahgao on https://github.com/apache/datafusion/pull/13605/files#r1863695929

Not related to the current PR, maybe we can move IdentNormalizer into IdentTaker to simplify take() and avoid creating normalizer every time.

What changes are included in this PR?

Move IdentNormalizer into IdentTaker

Are these changes tested?
Yes, by CI
Are there any user-facing changes?
No, these are all internal apis

Thanks @jonahgao for reviewing


Update README.md

Updated the README.md file at line 30 to improve the visability
Pull Request for ML-Crate 💡
There is a break missing in the main readme file

Name: Akarsh Gupta
Idenitfy yourself: SSOC '24 Contributor

Changed made
Break tags have been added at the end of line 30.

You are not being assigned to any issue, then why are you making pull requests?
For doing contributions you need to be assigned to an issue, then only you can push your code/contributions to this repo. Otherwise this'll lead to disqualification.
@Akarsh-dundun


Models not using espeak

Hi, thank you for this awesome library 💥!
I'd like to use it in the Unity engine but I'm concerned about the license issue (#93).
Are there any good quality models not using the espeak library?

Not yet, unfortunately. Replacing eSpeak is a significant undertaking, especially across so many languages.
I assume you're using Piper in Unity with English? Do you have some examples of the sorts of numbers you deal with, such as dates, times, currency, etc.? I'm trying to prioritize.


Update to latest native file dialog binaries

Could we create a workflow to achieve this? The binaries in the resources folder lack transparency so building Native File Dialog with GitHub actions could help to alleviate this.

This should now be ready :)


Lowering minimum version of symfony/cache to 4.2.

Cache contracts seem to appear in Symfony 4.2.
Therefore, we can request only this version and not ^4.3.

This is an error. Symfony 4.3 is needed indeed for Psr16Adapter


Error on trying to add pow to a existing app

I'm adding pow to a existing phoenix app, add added some extensions (reset password and email confirmation!), added some configurations, generated all migrations and templates. However, when I try to access any route, I get this error:
ArgumentError at GET /
cannot fetch session without a configured session plug

I made a gist where I put all my modifications:
https://gist.github.com/Mdsp9070/6aeb8b4cdbdde83e4db5dd54160367f3
screenshot:

Could anyone help me?

The error was the order of plug...
this is the correct order:
  plug Plug.Session, @session_options

  # Pow
  plug Pow.Plug.Session, otp_app: :app

  plug AppWeb.Router

Thanks for posting what was the issue!

Thanks for posting what was the issue!


Allow access to different

Context
From https://2i2c.freshdesk.com/a/tickets/149,

We have some folks wanting to connect to existing buckets in other AWS accounts (s3://eis-dh-hydro).
In the past we gave groups our service account identifier (arn:aws:iam::783380859522:role/jovyan-serviceaccount) to get permissions, and if I remember correctly I had to also add permissions to the service account policy...
What is the service account identifier I should use? Can I modify the necessary config, or do I need a 2i2c engineer to modify the policy?

Proposal
No response
Updates and actions
No response

Tried to just grant access to the role specified in https://github.com/2i2c-org/infrastructure/blob/5951fb62599a55a8722b720a6ca534843449b371/terraform/aws/buckets.tf#L46 but that didn't work.

Can you flesh out the issue title? It is unclear what "different" refers to...is it scratch buckets?

@choldgraf no idea how those words went missing! Fixed

I’ll have to loop you into the emails with the NASA account admin to double check the settings on their end...
Last year we also had to add explicit permissions to the hub service account policy, see here https://github.com/snowex-hackweek/jupyterhub/blob/main/terraform/eks/s3-data-bucket.tf
Could also test with the following requester-pays bucket. I have no problem accessing this with full S3 permissions on my own AWS account, but from the snowex hub it is permission denied.
aws s3 ls --request-payer requester s3://usgs-landsat/collection02/

Just noting that aws_iam_role.irsa_role[each.value.hub_name].arn ->
https://github.com/2i2c-org/infrastructure/blob/5951fb62599a55a8722b720a6ca534843449b371/config/clusters/uwhackweeks/snowex.values.yaml#L4
..rather than adding each specific bucket though it seems like all S3 list,read permissions could be default (just exclude permissions to create and delete buckets)?

Fixed by https://github.com/2i2c-org/infrastructure/pull/1488


Rewrite useFormAction to respect basename

Not a lot of creative coding was done here, I just copied how react-router-dom implements useFormAction and adapted it to svelte.
Running yarn validate cypress gets stuck on
1: starting server using command "npm run integration:start"
and when url "[ 'http-get://localhost:3000/' ]" is responding with HTTP status code 200
running tests using command "npm run integration:test"

><EMAIL_ADDRESS>integration:start
> vite build && vite preview --port 3000

vite v2.9.9 building for production...
✓ 49 modules transformed.
dist/index.html                  0.33 KiB
dist/assets/index.66c49214.css   0.06 KiB / gzip: 0.07 KiB
dist/assets/index.859b56ff.js    187.67 KiB / gzip: 60.71 KiB
  > Local: http://localhost:3000/
  > Network: use `--host` to expose

not sure what's up with that.
This same code inside my repo that uses react-router-svelte (directly in node_modules) fixes the problem described in this issue https://github.com/brophdawg11/remix-routers/issues/22
Please verify all tests pass properly.
Link to useFormAction in react-router-dom for verification:
https://github.com/remix-run/react-router/blob/main/packages/react-router-dom/index.tsx#L873
Also link to useResolvedPath in react-router-dom where I copied how resolveTo is used:
https://github.com/remix-run/react-router/blob/bf5afbed559926ed35ac9180881c78c426a29230/packages/react-router/lib/hooks.tsx#L271

@austincrim I'm going to leave this up to you.  I've not bothered with relative routing or basename handling in the Vue adaptor yet because we plan to move that all out of react-router and into the @remix-run/router as part of https://github.com/remix-run/react-router/discussions/9588.  Once that's done then the Vue/Svelet layers will be able to mostly "just work".  Basename might just work out of the box, and relative routing should likely just require providing the calling routeId to router.navigate.
That said, if this is interesting in the short term and this patch works - feel free to merge it in and then you can rip it back out once the above lands in the Remix router 👍

Just an FYI here - we just merged https://github.com/remix-run/react-router/pull/10336 last week which moves relative routing/basename logic into @remix-run/router which should make it easier for the Vue/Svelte adapters to leverage.  For full progressive enhancement support, Form/Link still need useFormAction/useHref to resolve relative/basename paths at render time but those can probably both use a version of normalizeTo from the router.  Eventually I think we'll expose a public router.resolvePath() method that calls the (currently private) nornalizeTo method - but for now in this repo I'd probably just copy that function into the Vue/Svelte adapters


Role assignment on joining the guild

I have tested this and appears it is due to roles missing solving issue #20 makes sure this no longer happens.