Implement ESC/POS QR code rendering

This pull request implements the ESC/POS command for QR code rendering: GS ( k. The added code is a direct port of the same feature in in mike42/escpos-php (see Printer.php, EscposTest.php, mike42/escpos-php#19). As in the original code, the error correction level, code model and block size are now user-selectable, and test cases have also been ported. This was tested on an Epson TM-T20II.
On printers that don't support the command, existing code wont break, but some new (optional) parameters are now added-
epson.qr("http://www.example.com/", size=10)

$ time python -m escpos.cli test

real	0m1.852s
user	0m1.056s
sys	0m0.700s

But on printers with GS ( k, the QR code can be rendered on the printer instead. The escpos.qr method simply needs native=True to render the same output on the printer:
epson.qr("http://www.example.com/", native=True, size=10)

$ time python -m escpos.cli test

real	0m0.073s
user	0m0.052s
sys	0m0.008s

Because image processing is not used, the second example executed some 25x faster, probably fixing #22.
--
Notes-

I've locally had to adjust cli.py to print a newline: epson.text("Scan to recall TODO list\n"), so that the printer's print buffer is empty before image commands arrive. I'm not including this in the pull request to avoid conflicting with #107
Feedback on code style/quality and/or adherence to project conventions would be appreciated, as I'll be porting some more substantial code in the next few weeks.

Thank you! Unfortunately I can't test it on my TM-T88II, but the code looks good.
I should probably put down some code-style-guidelines. (#114)
However, your style looks good to me and is easy to read. I very much like that you provide comments. :-)
I would say go on like this.
Just one thing: you should adapt the comment-section in the top of the tests file.

Cheers, I'll close this and open a new pull request with the changes, since there's a rebase in there.


HBASE-27434 Use ${revision} as placeholder for maven version to make it easier t…

…o control the version from command line

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 21s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 21s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m  3s
master passed

+1 :green_heart:
compile
9m 40s
master passed

+1 :green_heart:
checkstyle
1m 20s
master passed

+1 :green_heart:
spotless
1m  2s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 16s
the patch passed

+1 :green_heart:
compile
9m 13s
the patch passed

+1 :green_heart:
javac
9m 13s
the patch passed

+1 :green_heart:
checkstyle
1m 44s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
1m  2s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
11m  5s
Patch does not cause any errors with Hadoop 3.2.4 3.3.4.

+1 :green_heart:
spotless
0m 58s
patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
8m 15s
The patch does not generate ASF License warnings.

64m 49s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4836

Optional Tests
dupname asflicense javac hadoopcheck spotless xml compile checkstyle

uname
Linux bbae5d55656b 5.4.0-1085-aws #92~18.04.1-Ubuntu SMP Wed Aug 31 17:21:08 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5cacece790

Default Java
Temurin-1.8.0_345-b01

Max. process+thread count
138 (vs. ulimit of 30000)

modules
C: hbase-checkstyle hbase-annotations hbase-build-configuration hbase-logging hbase-protocol-shaded hbase-common hbase-metrics-api hbase-metrics hbase-hadoop-compat hbase-client hbase-zookeeper hbase-replication hbase-balancer hbase-resource-bundle hbase-http hbase-asyncfs hbase-procedure hbase-server hbase-mapreduce hbase-testing-util hbase-thrift hbase-shell hbase-endpoint hbase-backup hbase-it hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-shaded/hbase-shaded-mapreduce hbase-external-blockcache hbase-hbtop hbase-compression hbase-compression/hbase-compression-aircompressor hbase-compression/hbase-compression-brotli hbase-compression/hbase-compression-lz4 hbase-compression/hbase-compression-snappy hbase-compression/hbase-compression-xz hbase-compression/hbase-compression-zstd hbase-assembly hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-testing-util-tester hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes hbase-archetypes/hbase-client-project hbase-archetypes/hbase-shaded-client-project hbase-archetypes/hbase-archetype-builder . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 59s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 18s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 42s
master passed

+1 :green_heart:
compile
1m 53s
master passed

+1 :green_heart:
shadedjars
3m 46s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
11m 31s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 29s
the patch passed

+1 :green_heart:
compile
1m 52s
the patch passed

+1 :green_heart:
javac
1m 52s
the patch passed

+1 :green_heart:
shadedjars
3m 43s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
11m 37s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
266m 34s
root in the patch passed.

315m 22s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4836

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 140c93f9e2ff 5.4.0-122-generic #138-Ubuntu SMP Wed Jun 22 15:00:31 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5cacece790

Default Java
Eclipse Adoptium-<IP_ADDRESS>+1

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/1/testReport/

Max. process+thread count
4603 (vs. ulimit of 30000)

modules
C: hbase-checkstyle hbase-annotations hbase-build-configuration hbase-logging hbase-protocol-shaded hbase-common hbase-metrics-api hbase-metrics hbase-hadoop-compat hbase-client hbase-zookeeper hbase-replication hbase-balancer hbase-resource-bundle hbase-http hbase-asyncfs hbase-procedure hbase-server hbase-mapreduce hbase-testing-util hbase-thrift hbase-shell hbase-endpoint hbase-backup hbase-it hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-shaded/hbase-shaded-mapreduce hbase-external-blockcache hbase-hbtop hbase-compression hbase-compression/hbase-compression-aircompressor hbase-compression/hbase-compression-brotli hbase-compression/hbase-compression-lz4 hbase-compression/hbase-compression-snappy hbase-compression/hbase-compression-xz hbase-compression/hbase-compression-zstd hbase-assembly hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-testing-util-tester hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes hbase-archetypes/hbase-client-project hbase-archetypes/hbase-shaded-client-project hbase-archetypes/hbase-archetype-builder . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 44s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 20s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 18s
master passed

+1 :green_heart:
compile
1m 29s
master passed

+1 :green_heart:
shadedjars
4m  5s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
8m 33s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m  3s
the patch passed

+1 :green_heart:
compile
1m 27s
the patch passed

+1 :green_heart:
javac
1m 27s
the patch passed

+1 :green_heart:
shadedjars
4m  3s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
8m 31s
the patch passed

_ Other Tests _

-1 :x:
unit
337m 43s
root in the patch failed.

377m 58s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4836

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 92485a9f87de 5.4.0-1083-aws #90~18.04.1-Ubuntu SMP Fri Aug 5 08:12:44 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5cacece790

Default Java
Temurin-1.8.0_345-b01

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/1/testReport/

Max. process+thread count
2217 (vs. ulimit of 30000)

modules
C: hbase-checkstyle hbase-annotations hbase-build-configuration hbase-logging hbase-protocol-shaded hbase-common hbase-metrics-api hbase-metrics hbase-hadoop-compat hbase-client hbase-zookeeper hbase-replication hbase-balancer hbase-resource-bundle hbase-http hbase-asyncfs hbase-procedure hbase-server hbase-mapreduce hbase-testing-util hbase-thrift hbase-shell hbase-endpoint hbase-backup hbase-it hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-shaded/hbase-shaded-mapreduce hbase-external-blockcache hbase-hbtop hbase-compression hbase-compression/hbase-compression-aircompressor hbase-compression/hbase-compression-brotli hbase-compression/hbase-compression-lz4 hbase-compression/hbase-compression-snappy hbase-compression/hbase-compression-xz hbase-compression/hbase-compression-zstd hbase-assembly hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-testing-util-tester hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes hbase-archetypes/hbase-client-project hbase-archetypes/hbase-shaded-client-project hbase-archetypes/hbase-archetype-builder . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 41s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 19s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 19s
master passed

-1 :x:
compile
0m 55s
root in master failed.

+1 :green_heart:
checkstyle
0m 58s
master passed

+1 :green_heart:
spotless
0m 38s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m  8s
the patch passed

+1 :green_heart:
compile
6m  4s
the patch passed

-0 :warning:
javac
6m  4s
root generated 703 new + 0 unchanged - 0 fixed = 703 total (was 0)

+1 :green_heart:
checkstyle
0m 56s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m 31s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
7m 54s
Patch does not cause any errors with Hadoop 3.2.4 3.3.4.

+1 :green_heart:
spotless
0m 36s
patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
5m 30s
The patch does not generate ASF License warnings.

38m 38s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4836

Optional Tests
dupname asflicense javac hadoopcheck spotless xml compile checkstyle

uname
Linux f9607d1774ac 5.4.0-1085-aws #92~18.04.1-Ubuntu SMP Wed Aug 31 17:21:08 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dffc8e0fbe

Default Java
Temurin-1.8.0_345-b01

compile
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/2/artifact/yetus-general-check/output/branch-compile-root.txt

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/2/artifact/yetus-general-check/output/diff-compile-javac-root.txt

Max. process+thread count
139 (vs. ulimit of 30000)

modules
C: hbase-checkstyle hbase-annotations hbase-build-configuration hbase-logging hbase-protocol-shaded hbase-common hbase-metrics-api hbase-metrics hbase-hadoop-compat hbase-client hbase-zookeeper hbase-replication hbase-balancer hbase-resource-bundle hbase-http hbase-asyncfs hbase-procedure hbase-server hbase-mapreduce hbase-testing-util hbase-thrift hbase-shell hbase-endpoint hbase-backup hbase-it hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-shaded/hbase-shaded-mapreduce hbase-external-blockcache hbase-hbtop hbase-compression hbase-compression/hbase-compression-aircompressor hbase-compression/hbase-compression-brotli hbase-compression/hbase-compression-lz4 hbase-compression/hbase-compression-snappy hbase-compression/hbase-compression-xz hbase-compression/hbase-compression-zstd hbase-assembly hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-testing-util-tester hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes hbase-archetypes/hbase-client-project hbase-archetypes/hbase-shaded-client-project hbase-archetypes/hbase-archetype-builder . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 40s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 35s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m  7s
master passed

+1 :green_heart:
compile
6m  8s
master passed

+1 :green_heart:
checkstyle
0m 57s
master passed

+1 :green_heart:
spotless
0m 35s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m  6s
the patch passed

+1 :green_heart:
compile
6m  7s
the patch passed

+1 :green_heart:
javac
6m  7s
the patch passed

+1 :green_heart:
checkstyle
0m 58s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m 32s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
7m 55s
Patch does not cause any errors with Hadoop 3.2.4 3.3.4.

+1 :green_heart:
spotless
0m 37s
patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
5m 37s
The patch does not generate ASF License warnings.

44m 23s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4836

Optional Tests
dupname asflicense javac hadoopcheck spotless xml compile checkstyle

uname
Linux 146a0cedf74d 5.4.0-1085-aws #92~18.04.1-Ubuntu SMP Wed Aug 31 17:21:08 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dffc8e0fbe

Default Java
Temurin-1.8.0_345-b01

Max. process+thread count
140 (vs. ulimit of 30000)

modules
C: hbase-checkstyle hbase-annotations hbase-build-configuration hbase-logging hbase-protocol-shaded hbase-common hbase-metrics-api hbase-metrics hbase-hadoop-compat hbase-client hbase-zookeeper hbase-replication hbase-balancer hbase-resource-bundle hbase-http hbase-asyncfs hbase-procedure hbase-server hbase-mapreduce hbase-testing-util hbase-thrift hbase-shell hbase-endpoint hbase-backup hbase-it hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-shaded/hbase-shaded-mapreduce hbase-external-blockcache hbase-hbtop hbase-compression hbase-compression/hbase-compression-aircompressor hbase-compression/hbase-compression-brotli hbase-compression/hbase-compression-lz4 hbase-compression/hbase-compression-snappy hbase-compression/hbase-compression-xz hbase-compression/hbase-compression-zstd hbase-assembly hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-testing-util-tester hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes hbase-archetypes/hbase-client-project hbase-archetypes/hbase-shaded-client-project hbase-archetypes/hbase-archetype-builder . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/3/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 46s
Docker mode activated.

-0 :warning:
yetus
0m  2s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 18s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m 25s
master passed

+1 :green_heart:
compile
1m 31s
master passed

+1 :green_heart:
shadedjars
4m  9s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
8m 35s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m  7s
the patch passed

+1 :green_heart:
compile
1m 29s
the patch passed

+1 :green_heart:
javac
1m 29s
the patch passed

+1 :green_heart:
shadedjars
4m  5s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
1m 22s
root generated 15 new + 69 unchanged - 15 fixed = 84 total (was 84)

_ Other Tests _

+1 :green_heart:
unit
403m 50s
root in the patch passed.

445m 34s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4836

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 871fdda3176c 5.4.0-1083-aws #90~18.04.1-Ubuntu SMP Fri Aug 5 08:12:44 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dffc8e0fbe

Default Java
Temurin-1.8.0_345-b01

javadoc
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/3/artifact/yetus-jdk8-hadoop3-check/output/diff-javadoc-javadoc-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/3/testReport/

Max. process+thread count
4785 (vs. ulimit of 30000)

modules
C: hbase-checkstyle hbase-annotations hbase-build-configuration hbase-logging hbase-protocol-shaded hbase-common hbase-metrics-api hbase-metrics hbase-hadoop-compat hbase-client hbase-zookeeper hbase-replication hbase-balancer hbase-resource-bundle hbase-http hbase-asyncfs hbase-procedure hbase-server hbase-mapreduce hbase-testing-util hbase-thrift hbase-shell hbase-endpoint hbase-backup hbase-it hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-shaded/hbase-shaded-mapreduce hbase-external-blockcache hbase-hbtop hbase-compression hbase-compression/hbase-compression-aircompressor hbase-compression/hbase-compression-brotli hbase-compression/hbase-compression-lz4 hbase-compression/hbase-compression-snappy hbase-compression/hbase-compression-xz hbase-compression/hbase-compression-zstd hbase-assembly hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-testing-util-tester hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes hbase-archetypes/hbase-client-project hbase-archetypes/hbase-shaded-client-project hbase-archetypes/hbase-archetype-builder . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/3/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 41s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 32s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
2m  9s
master passed

+1 :green_heart:
compile
6m  0s
master passed

+1 :green_heart:
checkstyle
0m 56s
master passed

+1 :green_heart:
spotless
0m 35s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m  4s
the patch passed

+1 :green_heart:
compile
5m 58s
the patch passed

-0 :warning:
javac
5m 58s
root generated 1 new + 701 unchanged - 2 fixed = 702 total (was 703)

+1 :green_heart:
checkstyle
0m 56s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m 31s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
7m 52s
Patch does not cause any errors with Hadoop 3.2.4 3.3.4.

+1 :green_heart:
spotless
0m 38s
patch has no errors when running spotless:check.

 U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4836/4/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


[bug] Github actions all platform builds fail with MODULE NOT FIND

Describe the bug
Using the GitHub action provided on the website (with just a change to use NPM instead of Yarn) here:
name: "publish"
on:
  push:
    branches:
      - release

jobs:
  publish-tauri:
    strategy:
      fail-fast: false
      matrix:
        platform: [macos-latest, ubuntu-latest, windows-latest]

    runs-on: ${{ matrix.platform }}
    steps:
    - name: Checkout repository
      uses: actions/checkout@v2

    - name: Node.js setup
      uses: actions/setup-node@v1
      with:
        node-version: 16

    - name: Rust setup
      uses: actions-rs/toolchain@v1
      with:
        toolchain: stable

    - name: Install dependencies (ubuntu only)
      if: matrix.platform == 'ubuntu-latest'
      run: |
        sudo apt-get update
        sudo apt-get install -y libgtk-3-dev webkit2gtk-4.0 libappindicator3-dev librsvg2-dev patchelf

    - name: Install app dependencies and build it
      run: npm install && npm run build

    - name: Build the app
      uses: tauri-apps/tauri-action@v0
      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      with:
        tagName: app-v__VERSION__ # the action automatically replaces \_\_VERSION\_\_ with the app version
        releaseName: "App v__VERSION__"
        releaseBody: "See the assets to download this version and install."
        releaseDraft: true
        prerelease: false

I've left it running and initially I got an error on the windows pipeline due to prettier that got solved by adding a .gitattributes file to the root of the project with * text=auto eol=lf. (I would be willing to make an MR to get this info in the docs if you judge it necessary - I know it is specific to prettier, but it could avoid other problems due to line-endings)
All other pipelines seemed ok but I later found out they all get stuck (and consume minutes) on the build step with the following error:
Run tauri-apps/tauri-action@v0

running npx [ 'run', 'tauri:build' ]
npm WARN config global `--global`, `--local` are deprecated. Use `--location=global` instead.
npm WARN exec The following package was not found and will be installed: run
Watching D:\a\test-tauri-app\test-tauri-app and all sub-directories not excluded by your .gitignore. Will not monitor dotfiles.
Found & ignored ./node_modules ; is listed in .gitignore
Found & ignored ./dist\css\app.6cd52001.css ; has ignored extension
Found & ignored ./src\assets\style\tailwind.css ; has ignored extension
Starting: tauri:build
node:internal/modules/cjs/loader:936
  throw err;
  ^
Error: Cannot find module 'D:\a\test-tauri-app\test-tauri-app\tauri:build'
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)
    at Function.Module._load (node:internal/modules/cjs/loader:778:27)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
    at node:internal/main/run_main_module:17:47 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}

It is my first experience with Github Actions and I have no idea what this error is pointing to.
Reproduction
No response
Expected behavior
No response
Platform and versions
The Platform and version is from the gitlab pipelines

Stack trace
No response
Additional context
No response

Are you using vue-cli-plugin-tauri?


codegen: Fix whitelisting inside namespaces.

fixes #287
r? @fitzgen

@bors-servo r+

:pushpin: Commit a95d9d5 has been approved by fitzgen

:zap: Test exempted - status


Various minor improvements

This PR contains just three minor improvements in the build, mostly related to cmake. They should not harm in any way, and have been tested in different environments and on different platforms.

[ ] No ticket for trivial changes
[ ] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[ ] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

The build errors in Travis are unrelated. Since this PR should almost certainly not have a negative impact, merging despite a few failing builds in Travis.


Bump versions of scim2, oauth and frameword

Purpose
$subject

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/8482032385
Status: success


Configurator Wont Open

Ive Download every possible free version and almost all work by centering the apps but none of them open the configurator. Im running Windows 2h21. Please help, thanks in advance

@DannyRepublica I will try to assist you.
It can be somewhat missleading, but the actual "Taskbar.exe" file does not include configuration. A seperate file exists for this.
Therefore please follow these instructions:

Open Windows search (Win + S)
You're looking for file called "TaskbarX Configurator.exe"

If it doesn't show up check your installation folder. It has to contain "TaskbarX.exe" and "TaskbarX Configurator.exe". If this is not the case, download the latest version and unzip it again.

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


allow rebinding of shortcuts for moving around the graph (scroll/grab/etc)

Is this issue related to an issue?.
When using the SI UI on a non-mac, the default binding of CMD+scroll for Zoom is problematic on other OSes which have that keycode bound (e.g., it's my "Meta" key for my window manager, so this keybind does zoom the graph, but also cycles through my active window layouts :) )
Describe the outcome you'd like from us
Allow for rebinding of keyboard/mouse shortcuts
Describe alternatives you've considered
Support for non-MacOS scrolling using e.g. Ctrl+Scroll would also solve my immediate pain
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Additional context
No response

Bump that, I cant use spacebar for moving around as its binded for shift for me if I hold it


Deploying static web app fails

Our pipelines for static web apps stopped working.
Here's my output with verbose logging when we try to deploy a static site;
We're using Deploy Azure Static Web App (Preview) task (0.* Preview version)
******************************************************************************
Starting: deploy static web app
******************************************************************************
==============================================================================
Task         : Deploy Azure Static Web App
Description  : [PREVIEW] Build and deploy an Azure Static Web App
Version      : 0.203.0
Author       : Microsoft Corporation
Help         : https://aka.ms/swadocs
==============================================================================
[command]/usr/bin/bash /home/azureuser/myagent/_work/_tasks/AzureStaticWebApp_18aad896-e191-4720-88d6-8ced4806941a/0.203.0/launch-docker.sh
stable: Pulling from appsvc/staticappsclient
Digest: sha256:c1fac5b3c048bc93533acf15079a926497d091e6a9e961e7ac878eb2679fe677
Status: Image is up to date for mcr.microsoft.com/appsvc/staticappsclient:stable
/opt/oryx/benv: line 32:     8 Done                    set
         9 Segmentation fault      | grep -i '^dynamic_install_root_dir='

Hi @detaybey Can you please provide more details of your site or if you could share your repo so that we could try repro the same on our end. Thanks

Hi @detaybey, thanks for the info. Is this the only repository for which you are facing this issue? Were the deployments working fine before? Also, could you please let me know if you still facing this issue after retrying.

@detaybey  Can you please let us know if you are still facing the issue?


I am a bit confused, do the wherez have a static port? what is being advertised?

I was reading the code and I have seen the example but I am unsure if I understood correctly.
The wherez service listens on one port and the service is in another, right?
Then the wherez advertises it's service which is on let say port 8080?
Thanks,

Yes, you're right. Wherez itself listens on port 40000 by default:
https://github.com/nictuku/wherez/blob/master/wherez/wherez.go#L13
Hope this helps.

Thanks!


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_09c866e8-c5b7-4e6d-a4b9-5be0224bffe4/pull/1 is merged into live branch.
Auto archive content to https://github.com/huangmin-ms/docs-archive-test-target_09c866e8-c5b7-4e6d-a4b9-5be0224bffe4.git

Docs Build status updates of commit 5d7cac2:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://aka.ms/SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Refactor macros and add warnings for common issues

Splits pickling macros into separate functions. There are
now warnings for the following three cases:

Trait is unsealed
Trait does not have any sub-classes
No companion symbol found for case classes

We are unfortunate to be affected by the two compiler bugs
SI-7046 and SI-7567 which are the cause for 2) and 3). Yet,
the developer can apply simple workarounds as described in
the warnings.
The current suggestion for 2) is to use integer constants:
type Base = Int  // sealed trait Base
object Base {
  val Child1 = 0 // case object Child1 extends Base
  val Child2 = 1 // case object Child2 extends Base
}

A type-safe variant would be to use an approach similar
to https://github.com/lloydmeta/enumeratum/.
As for 3) there may be even a workaround that could be
implemented in uPickle. A possible solution is described
here:
http://stackoverflow.com/questions/16440124/get-the-companion-object-instance-of-a-inner-modul-with-the-scala-reflection-api

Merged, with tweaks that provide a better error message on scala 2.11 using whitebox macro. Thanks a lot! This will go out with Scala.js 0.6.0 when that's released


Design and implement character name to Unicode mapping

Stretch goal: Implement fallback system for font not supporting certain characters.

Implemented code-wise


cppcheck returns different error when run in ale and when its run through terminal on the same file

Information
I started using cppcheck for linting c files and noticed it has some issues when run with ale related to
macro expanding. I have other linters disabled and also no extra options set for cppcheck.
VIM version
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Nov 29 2017 04:24:51)
Included patches: 1-1358
Operating System: Debian Testing
:ALEInfo
 Current Filetype: c
Available Linters: ['clang', 'clangtidy', 'cppcheck', 'gcc']
  Enabled Linters: ['cppcheck']
 Linter Variables:

let g:ale_c_cppcheck_executable = 'cppcheck'
let g:ale_c_cppcheck_options = ''
 Global Variables:

let g:ale_echo_cursor = 1
let g:ale_echo_msg_error_str = 'Error'
let g:ale_echo_msg_format = '%s'
let g:ale_echo_msg_warning_str = 'Warning'
let g:ale_enabled = 1
let g:ale_fix_on_save = 0
let g:ale_fixers = {}
let g:ale_keep_list_window_open = 0
let g:ale_lint_delay = '400'
let g:ale_lint_on_enter = 1
let g:ale_lint_on_save = 1
let g:ale_lint_on_text_changed = 'always'
let g:ale_linter_aliases = {}
let g:ale_linters = {'c': ['cppcheck'], 'tex': [], 'bash': ['shellcheck'], 'cpp': ['cppcheck'], 'text': [], 'python': ['pylint'], 'css': []}
let g:ale_open_list = 0
let g:ale_set_highlights = 1
let g:ale_set_loclist = 1
let g:ale_set_quickfix = 0
let g:ale_set_signs = 1
let g:ale_sign_column_always = 0
let g:ale_sign_error = '>>'
let g:ale_sign_offset = 1000000
let g:ale_sign_warning = '--'
let g:ale_statusline_format = ['%d error(s)', '%d warning(s)', 'OK']
let g:ale_warn_about_trailing_whitespace = 1
  Command History:

(executable check - success) cppcheck
(started) ['/bin/zsh', '-c', 'cd ''/home/szeraton/Gites/crc32'' && ''cppcheck'' -q --language=c --project=compile_commands.json  ''/tmp/vUxhq5F/2/crc.c''']
(finished - exit code 0) ['/bin/zsh', '-c', 'cd ''/home/szeraton/Gites/crc32'' && ''cppcheck'' -q --language=c --project=compile_commands.json  ''/tmp/vUxhq5F/3/crc.c''']

<<<OUTPUT STARTS>>>
[/tmp/vUxhq5F/3/crc.c:177]: (error) syntax error
<<<OUTPUT ENDS>>>

But when we run it in command line like this:
cppcheck -q --project=compile_commands.json --language=c crc.c
The error is:
[crc.c:165]: (error) Deallocating a deallocated pointer: output
What went wrong
cppcheck reports syntax error while it should only report excess pointer deallocation. The syntax error
appear right after the macro ARGBEGIN and since ale works on buffers and not files by default,
that makes me think that the macro is just not expanded in vim buffer.
Reproducing the bug

Install cppcheck with your favorite package manager.
Add this two variables to your vimrc:

let g:ale_c_cppcheck_options = ''
let g:ale_linters = {
        \   'c' : ['cppcheck'],
        \}

Clone this repository: https://github.com/Szero/crc
Open crc.c file in vim. It should report syntax error on 178th line.
Running cppcheck -q --project=compile_commands.json --language=c crc.c in terminal should report Deallocating a deallocated pointer: outputon line 166.

The issue seems to be that, for some reason, ALE is checking a copy of the source file that lives in /tmp somewhere, instead of the one in the source tree. This way, it doesn't include headers or anything and just checks that one file in isolation.

The command should be updated to include the arguments necessary to tell the program where the headers are, so you can check for errors while you type with the right type information.

Are you talking about compile_commands.json? Because that doesn’t work for out-of-source builds, at least for me. Looks like it traverses up the project tree only.
If I copy my source somewhere by itself and run the command line cppcheck on it, I get the same output as ALE gives me.

If cppcheck is incapable of being run on anything except the exact location of the file on disk, then we could get rid of the ability to run it while you type and only run it against the files instead.

Do you mean just on write/save?
Help me understand. The normal way to check a file while typing is to write the buffer to /tmp somewhere, and then run cppcheck with --project= on it?
If so, would it be possible to adjust the search for compile_commands.json by specifying a build directory variable? So if that variable were build/, and we are editing file a/b/c/d.cpp from project dir, it could look in a/b/c, a/b/c/build/, a/b, a/b/build, etc. If the variable were /some/other/path, it could just look in that absolute dir.

Yes, check only the file on disk, not while you edit it, and pass the program the path to the file. If that's the only way it will work properly, then that's what we should do.

That would be unfortunate. Searching for compile_commands.json would not work?
How do other linters do this?

If we can fix this by looking for compile_commands.json in build directories, then I'll do that. I'm going to do that for other linters too, and it can be done for this. See here: https://github.com/w0rp/ale/commit/fbe90f07b66a9944bf304458fd4c15bad6f6a4e4

That sounds great! When looking for the project dir, remember that .git is not always a good indicator for it when submodules are present. You could check for .git being a file (which it is in a submodule) vs a directory (which it will be in a root).

Yeah, .git is just a decent guess when there's nothing else available.

I just tested it with compile_commands.json, and it checks every file listed there. I don't know if that scales for large projects.
It looks like the options are either just check one file in isolation and miss stuff, or use compile_commands.json and check everything.
A possible third option could be to create the temporary file in the same directory as the original file.

I wouldn't choose to create a temporary file in the same directory. That causes files to be left behind from time-to-time, that end up in your source tree.

Makes sense. Here's another idea: Run cppcheck to include the original directory. Something like this for bar.cpp inside ${project_root}/src/foo:
cppcheck -q -I${project_root}/src/foo /tmp/nvim123/1/bar.cpp
That produces the correct result in my brief test.

That might work.

Would you like to create a pull request that does that?

Hm I just realized that using --project wouldn't check the file currently being edited anyway, so I don't think that's a solution for "as-you-type" feedback. It would still work for "check-on-write".

I can give that a shot, but am not familiar with your codebase or VimL.

See #2506. I'll need some help with one of the tests.


Update/fuzz crashers fix

Codecov Report

Merging #28 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #28   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          30     30
  Lines        3271   3352   +81
=====================================
+ Hits         3271   3352   +81

Impacted Files
Coverage Δ

decode_stream.go
100% <100%> (ø)
:arrow_up:

decode_number_int.go
100% <100%> (ø)
:arrow_up:

decode_string_unicode.go
100% <100%> (ø)
:arrow_up:

decode_stream_pool.go
100% <100%> (ø)
:arrow_up:

decode_number.go
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Make Documentation Website for React-Conflux

Problem
We need a documentation website to demonstrate to users how to install, use, and test the React-Conflux package.
Solution
Harness the power of Gatsby to craft a documentation website.
This will include:

A "Getting Started" page
A "Testing" page
Documentation for the APIs (StateProvider and useStateValue)
Summaries of implementing reducers and a bit about the Context API

Screenshot

@dustinmyers There's mostly content left to do on this PR.


Add support for Moodle Course Completion Webhooks

The only thing that remains is a way to easily define course_id => group_id.
I don't think the source of truth for this map should be stored here and maybe inside Moodle itself


The project created by bun create brisa my-app does not work correctly

When I change the jsx code, the page loses its CSS
The counter component in the interaction has no effect

System information:

win11
Section 1.1.30

It's duplicated of this one https://github.com/brisa-build/brisa/issues/538. We hope for the next Brisa version to be fixed. We will prioritize it before any other.

yes , i like bresa,it's fast!

@liuhuapiaoyuan we did some improvements in 0.1.2-canary.2, can you check it? thanks

ok，I'll test it this evening.

Oh no... still the same error

@liuhuapiaoyuan Thanks for the feedback, that's the thing about not having Windows, I just got one and so it will be much better to finish fixing the issue, anything better to talk through the other open task.


action: Add issue to project and move to "In progress" on linked PR

Add GitHub actions to:

Add newly-created issues to the issue backlog project.
Move issues with a linked PR into the "In progress" column
of the issue backlog project.

Related: https://github.com/kata-containers/kata-containers/issues/512
Fixes: #825.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

/test

/test


Link Touch Event documentation to Examples/ #869

The example offers a deep dive, which is useful for understanding the API more deeply.

I think it might be best to add a small example in the documentation instead as it might be slightly more understandable, the documentation examples are tested for static errors and the example may change without us keeping track for it.

I'm closing this PR since we now recommend people to use Gesture Handler


Update embed.mongo from 3.4.6 to 3.4.7

Description

Change the version of de.flapdoodle.embed.mongo from 3.4.6 to 3.4.7 because of failures in the Server side build in CI.

Fixes # (issue)

Will create an issue once the CI passes.

Type of change

Please delete options that are not relevant.

Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Ran few individual tests on Local setup and they ran successfully.

Checklist:

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

/ok-to-test sha=ed32ff9

#17609 has similar change with the latest version though.


[Need help] Replay Video Automatically

Hello Everyone
Am trying to make video autoreply after finishing, I added that to the player options and it's not working, if anyone could help me solve this, please

Try to swap the order of setOptions and setDataSource?
Of course, this may not be the problem. But options are only available if they are set before setDataSource.


🔐 Postgres Minor Upgrade for operations-engineering-metadata-poc

User Need
As an Operations Engineering Team
I want to upgrade postgres for operations-engineering-metadata-poc from 15.3 to 15.5
so that our app continues to function and keep up to date with security and other fixes.
Value
By keeping up to date with mitigate operational and security risks.
Functional Requirements:

Upgrade postgres version from 15.3 to 15.5.

Cloud Platform User Guide - https://user-guide.cloud-platform.service.justice.gov.uk/documentation/deploying-an-app/relational-databases/upgrade.html#upgrading-to-a-new-minor-database-version
Non-Functional Requirements:

Work must be completed by 29th March or earlier

Acceptance Criteria:

Postgres minor version upgraded.
Automated tests completed successfully.
Application tested and running correctly

Notes:
Email from Cloud Platform Team

🆙 Update

PR to explicitly upgrade to v15.5 https://github.com/ministryofjustice/cloud-platform-environments/pull/18561 (there were some other PRs around this one, following the CP user guidance)
Connected to DB and checked the version which is now showing as PostgreSQL 15.5

👀 Remaining Work

All done ✅

📝 Notes

I also added automatic updates of minor versions for the DB, although this didn't update from 15.3 to 15.5, apparently that setting is for the patch version (if we're talking SEMVAR terms)


lyne-timetable-occupancy

state: in progress
figma: https://www.figma.com/file/oA9B5W6w0449gtItDzLS86/00%3A-Components?node-id=360%3A735437
dev: https://lyne.sbb.ch/components/lyne-timetable-occupancy
ux quality-check
code quality-check
refactoring (optionaly)
testing

Feature Description
We want to review/refactor the lyne-button implementation.
Figma Spec

dev: https://lyne.sbb.ch/components/lyne-timetable-occupancy

References

Design Spec
Technical Spec

Definition of Done

[ ]  ux quality-check
[ ]  Code quality-check
[ ]  Refactoring (optionaly)
[ ]  Testing
[ ]  Component is implemented
[ ]  Tests are implemented
[ ]  E2E Tests are implemented
[ ]  Storybook stories are implemented
[ ]  Navigation via keyboard is tested
[ ]  Screen reader output is tested
[ ]  High-contrast is tested
[ ]  Remaining accessibility is tested
[ ]  UX approved

Will be tracked/decided in #1901


External select not refreshing

(Filling out the following with as much detail as you can provide will help us solve your issue sooner.)
Reproducible in:
The Slack SDK version
^3.17.1
Node.js runtime version
v20.11.0
Hi! I have an external select inside a modal with placeholder text of "Search for a place...". When a user selects a place, and presses the "Add place" button, I add the selection to a list by updating the modal. However, when I update the modal, the external select doesn't revert back to the placeholder text, I think because the state object gets passed to the updated modal. How can I refresh the state so that every time I update the modal I see the placeholder text?
https://github.com/slackapi/bolt-js/assets/10735162/8d767972-913a-493e-b49b-c359771c4b00

Hi, @jacklein! Thanks so much for submitting this question and for the video demonstrating your issue! 🙌
After doing some research on the select menu with external data types, it seems like the placeholder is only intended to surface when an option is not selected. In this case, an option is considered "selected" once you select it from the menu and from that point onwards you have no choice but to select another option to change the value in the field.
Something you could try is calling a views.update for your modal after hitting the "Add place" button that completely updates  / replaces the form so it's fresh and new, but demonstrates all of your selected options below it like you've shown in the video.
I hope this helps! ✨

@hello-ashleyintech Thank you for the response! Sorry if I wasn't exactly clear up top, but I am calling views.update with a fresh external select block. However, the state object which has a selected_value doesn't get reset and gets passed to the new modal. Because of this, the new external select block shows the previously selected value. I tried to pass a fresh state into the views.update like:
views.update({
   ...
   state: {},
   ...
})

but it doesn't seem like I can do this.

Ah, I see! Thanks for the clarification @jacklein! 🙌
This may not be exactly what you want since it's not directly updating that state, but take a look at this suggestion and let me know if it would work in your use case!

@hello-ashleyintech ahhh ok thanks I will try this workaround 🙏


Regression: cannot resubmit a message

When attempting to resubmit a message an exception is thrown:

Exception: The message body cannot be read multiple times. To reuse it store the value after reading.

I suppose this is due to the way the message is cloned, right? And I suppose this is due to the old code that we recently re-introduced in SBE...

I suppose this is due to the way the message is cloned, right? And I suppose this is due to the old code that we recently re-introduced in SBE...

I was unable to reproduce this using the latest develop branch. Was it one message or was it the batch resubmit?

I was unable to reproduce this using the latest develop branch. Was it one message or was it the batch resubmit?

Single message

Single message

I have tried that both from deadletter and normal queue. Also tried peek and received. Please state repro steps including message content.

I have tried that both from deadletter and normal queue. Also tried peek and received. Please state repro steps including message content.

I'm experiencing this issue as well. after editing, the original message is still sent.

@RascoApps, this issue occurs in the latest released build. Is that what you are using? The issue you are describing has been fixed in the current develop branch with PR #496.
@SeanFeldman, which build are you using when this occurs?

@SeanFeldman, which build are you using when this occurs?

I believe it was the latest, 5.0.1. Not 100% sure. I'll raise a separate issue when/if it happens.

My steps to reproduce:

Enter either queue or topic subscription
Click Deadletter and Peek some messages
Click Repair And Resubmit Selected Message (single message)
Click Submit
Choose Send To queue/topic
Click Ok - works fine, message send successfully
Again - in Repair and Resubmit Message window - Click Submit
Again - Choose Send To queue/topic
Again - Click Ok  - Exception: The message body cannot be read multiple times. To reuse it store the value after reading.

<10:17:43> [1] messages were sent to [my-local-sb] in [130] milliseconds.
<10:18:40> Exception: The message body cannot be read multiple times. To reuse it store the value after reading.

Version: 5.0.1

@lkurzyniec do you experience the issue with the latest version and/or release?

@paolosalvatori yes, all the time.

@SeanFeldman @ErikMogensen are you planning to fix this issue? Otherwise, I'll have to reserve some time in the coming weeks to fix this issue.

This issue should be fixed by PR #496. @lkurzyniec, is it possible for you to verify that by building and testing the develop branch?
Since this is a critical functionality and another bug #492, also solved by #496 is worse I think we should aim for making a new release.

@ErikMogensen @SeanFeldman is #496 solves this issue + #492, let's ship a new release 😄

@SeanFeldman , I'd would appreciate it if you created the release.

Closed via https://github.com/paolosalvatori/ServiceBusExplorer/commit/f87537a5ded12501a9e33fc3edd2a29487246425

Thanks @SeanFeldman @ErikMogensen 😄

@ErikMogensen downloaded the newest version (5.0.2) with chocolatey and confirmed that the fix made the job, well done, thank you!

Thanks @lkurzyniec! I was a bit surprised that you were able to download it from Chocolatey since there were an issue with an icon url in the latest packages but I see now that you may download packages even though they are not approved. That icon url should be correct in the next release, 5.0.3.


Memory Leak w/ NextJS HMR: JS Heap out of Memory

Description
This has been happening since last few weeks with me. I'm using NextJS w/ Xstate, specifically using Fabric JS library in Xstate. Xstate doesn't work well with HMR, but these days I'm getting the error mentioned below. After this the CPU usage skyrockets quickly, sometimes browser crashes and sometimes I just have to restart my PC to do anything. I wasn't at first entirely sure whether this is a bug with Xstate, but I noticed a few times that this occurs only when I make some change in the machine and refresh the browser (but not everytime) so this is probably not a bug with NextJS HMR:
Expected Result
This should not happen.
Actual Result
event - compiled successfully
wait  - compiling...
event - compiled successfully
wait  - compiling...
event - compiled successfully
wait  - compiling...
event - compiled successfully
wait  - compiling...

<--- Last few GCs --->

[84402:0x5623dd217bc0]  1208945 ms: Scavenge 1951.0 (1995.0) -> 1949.8 (1996.0) MB, 6.2 / 0.0 ms  (average mu = 0.870, current mu = 0.562) allocation failure
[84402:0x5623dd217bc0]  1210092 ms: Mark-sweep (reduce) 2089.0 (2135.3) -> 2079.7 (2119.9) MB, 723.9 / 0.0 ms  (+ 0.0 ms in 0 steps since start of marking, biggest step 0.0 ms, walltime since start of marking 729 ms) (average mu = 0.839, current mu = 0.80

<--- JS stacktrace --->

FATAL ERROR: MarkCompactCollector: young object promotion failed Allocation failed - JavaScript heap out of memory
 1: 0x5623db01dfd1 node::Abort() [/usr/bin/node]
 2: 0x5623daf3c144 node::FatalError(char const*, char const*) [/usr/bin/node]
 3: 0x5623db1fdd62 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/usr/bin/node]
 4: 0x5623db1fdfc8 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/usr/bin/node]
 5: 0x5623db3bdc36  [/usr/bin/node]
 6: 0x5623db3ea0bd v8::internal::EvacuateNewSpaceVisitor::Visit(v8::internal::HeapObject, int) [/usr/bin/node]
 7: 0x5623db3f4b2e void v8::internal::LiveObjectVisitor::VisitBlackObjectsNoFail<v8::internal::EvacuateNewSpaceVisitor, v8::internal::MajorNonAtomicMarkingState>(v8::internal::MemoryChunk*, v8::internal::MajorNonAtomicMarkingState*, v8::internal::EvacuateNewSpaceVisitor*, v8::internal::LiveObjectVisitor::IterationMode) [/usr/bin/node]
 8: 0x5623db414d21 v8::internal::FullEvacuator::RawEvacuatePage(v8::internal::MemoryChunk*, long*) [/usr/bin/node]
 9: 0x5623db3e5405 v8::internal::Evacuator::EvacuatePage(v8::internal::MemoryChunk*) [/usr/bin/node]
10: 0x5623db3e56d7 v8::internal::PageEvacuationTask::RunInParallel(v8::internal::ItemParallelJob::Task::Runner) [/usr/bin/node]
11: 0x5623db3d71a9 v8::internal::ItemParallelJob::Run() [/usr/bin/node]
12: 0x5623db3f2749 void v8::internal::MarkCompactCollectorBase::CreateAndExecuteEvacuationTasks<v8::internal::FullEvacuator, v8::internal::MarkCompactCollector>(v8::internal::MarkCompactCollector*, v8::internal::ItemParallelJob*, v8::internal::MigrationObserver*, long) [/usr/bin/node]
13: 0x5623db413225 v8::internal::MarkCompactCollector::EvacuatePagesInParallel() [/usr/bin/node]
14: 0x5623db413539 v8::internal::MarkCompactCollector::Evacuate() [/usr/bin/node]
15: 0x5623db413ed3 v8::internal::MarkCompactCollector::CollectGarbage() [/usr/bin/node]
16: 0x5623db3cb47f v8::internal::Heap::MarkCompact() [/usr/bin/node]
17: 0x5623db3cbe88 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::GCCallbackFlags) [/usr/bin/node]
18: 0x5623db3cc487 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/usr/bin/node]
19: 0x5623db3cfb2c v8::internal::Heap::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/usr/bin/node]
20: 0x5623db3cfb94 v8::internal::Heap::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/usr/bin/node]
21: 0x5623db394a9e v8::internal::Factory::AllocateRaw(int, v8::internal::AllocationType, v8::internal::AllocationAlignment) [/usr/bin/node]
22: 0x5623db38ef24 v8::internal::FactoryBase<v8::internal::Factory>::AllocateRawWithImmortalMap(int, v8::internal::AllocationType, v8::internal::Map, v8::internal::AllocationAlignment) [/usr/bin/node]
23: 0x5623db390ec0 v8::internal::FactoryBase<v8::internal::Factory>::NewRawTwoByteString(int, v8::internal::AllocationType) [/usr/bin/node]
24: 0x5623db5ecb56 v8::internal::String::SlowFlatten(v8::internal::Isolate*, v8::internal::Handle<v8::internal::ConsString>, v8::internal::AllocationType) [/usr/bin/node]
25: 0x5623db2144f9 v8::String::Utf8Length(v8::Isolate*) const [/usr/bin/node]
26: 0x5623daff0363  [/usr/bin/node]
27: 0x5623db267257 v8::internal::FunctionCallbackArguments::Call(v8::internal::CallHandlerInfo) [/usr/bin/node]
28: 0x5623db26803e  [/usr/bin/node]
29: 0x5623db268532  [/usr/bin/node]
30: 0x5623db2687e6 v8::internal::Builtin_HandleApiCall(int, unsigned long*, v8::internal::Isolate*) [/usr/bin/node]
31: 0x5623dba35859  [/usr/bin/node]
^C^C

Reproduction
Cannot reproduce on Codesandbox. Can't share the code as well because of NDA, but I'm 100% sure there's no memory leak in my code. At first I thought it could be a leak in Fabric.JS but it happened again w/ NextJS without me using a machine which uses Fabric. I think it should happen in any NextJS project w/ Xstate, so I'll update this issue soon in a while with a Github repo.
Additional context
Xstate version: ^4.16.2
Xstate React version: ^1.2.2

Whenever u provide a repro case i will jump into investigating this immediately. Until then its rather a wild-goose chase that i'd like to avoid.

Closing until there is a reproduction.

Hi, @a-y-u-s-h! I'm also using the FabricJs in my NextJs project and randomly getting "heap out of memory" errors while on development machine with HMR. However, I'm not using Xstate. Have you been able to resolve your issue?

Hi, @a-y-u-s-h! I'm also using the FabricJs in my NextJs project and randomly getting "heap out of memory" errors while on development machine with HMR. However, I'm not using Xstate. Have you been able to resolve your issue?

I haven't really resolved it. Couldn't even reproduce it after that, and then never got around to it. It must be the issue with FabricJS then with HMR.


Refactor create method contents into multiple descriptive methods

The create method itself is 49 lines long.  This is much too long to be viewable at a glance.  Break it up into other methods whose names are clear enough to be self-documenting.

Doing some investigation, what I've found is that the following query
select distinct(tree_folders.name) from tree_folders;

produces the following output
+-------------------+
| name              |
+-------------------+
| Questionnaires    |
| Courses           |
| Assignments       |
| Review            |
| Metareview        |
| Author Feedback   |
| Teammate Review   |
| Assignment Survey |
| Global Survey     |
| Course Survey     |
| Bookmark Rating   |
| Quiz              |
+-------------------+

and the following query
select distinct(display_type) from questionnaires;

produces the following output
+------------------+
| display_type     |
+------------------+
| Review           |
| Metareview       |
| Author Feedback  |
| AuthorFeedback   |
| Survey           |
| CourseEvaluation |
| TeammateReview   |
| GlobalSurvey     |
| Teammate Review  |
| Quiz             |
| Teammate%Review  |
| Author%Feedback  |
+------------------+

It would appear that the tables aren't consistent (as noted in the comments above the line about setting the display_type in the create method). Instead of setting the display type we should probably be using a foreign key to the tree_folders table or we should be using a questionnaire_display_types table. However, this means that we need to make a migration. I would suggest that for the purposes of this refactor. We should refactor from to use the tree_folders table and a foreign key.

However, before I can properly write a migration script, I need them to run these SQL scripts on the production data.


Update Docusaurus to 1.3.2

Summary

Update Docusaurus to 1.3.2 based on https://github.com/facebook/jest/issues/6640#issuecomment-402938248. This version has few bug fixes which is important imho. In additions:

Being able to easily debug commit that cause 404 / broken build in gh-pages branch

Consider this commit https://github.com/endiliey/endiliey.github.io/commit/4c6abe0786e2be11cda2ae7c8d847e957d6ba89d

Now the deploy commit on gh-pages shows the correct triggering commit
https://github.com/endiliey/endiliey.github.io/commit/936bbc3e2fa8d647aa824e138d52f7b8fbf17b2e

Make debugging life easier 👍

Search docs depending on current version of docs
(refer to test plan below)

Expose algolia api key for easy debugging

cc @yangshun @JoelMarcey
Test plan

Static website build works without non-zero exit code

cd website
yarn build

Test locally

cd website
yarn start

Search according to current version.
Example I'm at 22.2, my search will be scoped to 22.2 docs.

Example I'm at docs/en/next, my search will be scoped to docs/en/next docs.

Seeing as the api key is already in the URL when performing searches, I see no reason to just have it in source.
@cpojer @thymikee @rickhanlonii thoughts on that?

If you notice it in this PR preview, https://deploy-preview-6644--jest-preview.netlify.com/ now can search because process.env.ALGOLIA_JEST_API_KEY is not set in netlify
We at Docusaurus expose it as well
https://github.com/facebook/Docusaurus/blob/936725c94d156a832cadd971bd64497e20e16b80/website/siteConfig.js#L37
But I'll follow Jest team decision on this 👍

Loving the preview :D So good

@SimenB even Docusaurus didn't search according to version 😭.
@yangshun @JoelMarcey
Should we search according to version as well ? I have an index on algolia ready for that

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6644   +/-   ##
=======================================
  Coverage   63.73%   63.73%
=======================================
  Files         235      235
  Lines        8935     8935
  Branches        4        3    -1
=======================================
  Hits         5695     5695
  Misses       3239     3239
  Partials        1        1

Continue to review full report at Codecov.

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

As @SimenB already mentioned, this API key doesn't look like a secret, so I'm ok exposing it.

In addition, I think we can close all the 404 jest website error. I've been monitoring the website for few days and there is no 404 so far. This version also includes a non-zero exit code for any build error (https://github.com/facebook/Docusaurus/pull/809) so the 404 won't happen easily.
Let us know if it does happen though. Will come running to fix it 😎

In addition, I think we can close all the 404 jest website error. I've been monitoring the website for few days and there is no 404 so far.

We haven't had a merge spree either, but I agree we could close them as hopefully fixed. Easy enough to reopen or file new ones

We haven't had a merge spree either, but I agree we could close them as hopefully fixed. Easy enough to reopen or file new ones

I'm quite confident that at least the website going down won't happen easily. Few reasons:

We use run step on circleci for website deploy now instead of deploy previously. Deploy step can cause unwanted behavior because it will fails on PR with even one failing test (e.g test-node-6 fails)
We only deploy the website if docs or website files are changed now. Previously all the merge will deploy the website and they are all running in parallel 😢
We properly test website build on every PR now. Previously it was just Skipping deploy ... and maybe it breaks during push to master ? 😄 Even got netlify preview now 🙌
Docusaurus 1.3.2 is better, etc.

Nevertheless, we all hope it won't happen 😍

Is it possible to add the ALGOLIA_JEST_API_KEY key to a Netlify config?

@rickhanlonii
Only those with access to netlify setting of Jest could add it :)

@yangshun or I should be able to do this. I am on vacation right now, but can try to find time to do it before I get back if @yangshun can't do this before me.

Have set the key in Netlify:


Context propagation

We need a way to interface with the Propagators API.  Joining a distributed trace is currently an ad hoc process.

#125 is rough around the edges, but works.


Revert "Add webhook secret bootstrap and verification for Gitlab (#4492)"

Summary
We've seen minder startup errors with the latest webhook patch.
This reverts commit 052daf18f4cf73172f07feb66b09e453f42ca24a.
Change Type

[x] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
N/A
Review Checklist:

[ ] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 52.819% (+0.08%) from 52.742%
when pulling 8a6e248063760f3e27eabcb55eef78c2785cac48 on jhrozek:revert
into 052daf18f4cf73172f07feb66b09e453f42ca24a on stacklok:main.


mpirun doesn't exit when exception is thrown in some process

Hi,
I'm encountering some strange behavior. When running chainermn scripts, I would like them to exit immediately if any process fails, and for mpirun to exit as well. Instead, I've run into two problems:

In some cases, even if execution fails in some processes, execution continues on other processes until they exit normally.
(Even more pernicious): even if execution fails in some processes, execution never finishes (mpirun process never exits)

I think something is going wrong when communicators are made.
Running the following with OpenMPI 2.1.2, Chainer 3.5.0, and ChainerMN 1.2.0 reproduces problem 2 (and problem 1):
mpiexec -mca orte_abort_on_non_zero_status 1 -n 4 python my_script.py

my_script.py:
import chainermn

def main():
    comm = chainermn.create_communicator('naive')
    if comm.mpi_comm.rank != 0:
        raise ValueError('failure!')
    comm = chainermn.create_communicator('naive')

if __name__ == '__main__':
    main()

The desired behavior is for mpirun to exit (and cause all spawned processes to exit) when any failure occurs. Is there any way I can achieve this? And what could be happening in create_communicator to cause this? While I admit this example is contrived, my own use case isn't, but faces the same problem, even when I create only one communicator (and use another condition to force a failure on certain processes).
Thank you

Yes, as you said, the expected behaviour of MPI runtime is to kill all child processes and shutdown with an error code.
We sometimes observe the issues as well. It's a known issue for Open MPI <= 2.1.
In the script you showed, the second create_communicator is supposed to raise an error.
I will investigate if there's any way or option to avoid the issue.
It seems that the failed process detection and subprocess shutdown feature were improved in Open MPI 3.0,  but the version brings another issue to ChainerMN (see #221 for details)
We are trying to solve #221, but it will take time because it's not a ChainerMN issue.
In the long-term roadmap, we are working hard to remove fault tolerance to ChainerMN.
Thanks

@keisukefukuda ,
Thank you for looking into this!
Unfortunately, I tried it with Open MPI 3.0.1, but I encountered the same behavior.
I should note that even if I do not create two communicators, this issue still occurs:
import chainermn

def main():
    import mpi4py.MPI
    mpi_comm = mpi4py.MPI.COMM_WORLD
    if mpi_comm.rank == 0:
        raise ValueError('failure!')
    comm = chainermn.create_communicator('naive', mpi_comm)

if __name__ == '__main__':
    main()

I looked into this some more. It seems like mpi4py isn't handling python exceptions correctly. I tried calling init_ranks with some timeout code that sends a SIGALRM signal after some time, but the handler never gets called.
mpirun -n 2 python -m trace --trace repro.py
this trace shows that mpi_comm operations (gather and scatter) cause execution to hang:
_communication_utility.py(32):          global_names=mpi_comm.gather(mpi4py.MPI.Get_processor_name())

I've just found a hack to solve the issue.
It works for the tiny script, but we need to check if it works for real-world applications.
Note that the problem happens without chainermn.
import sys

# Global error handler
def global_except_hook(exctype, value, traceback):
    sys.stderr.write("except_hook. Calling MPI_Abort().\n")
    import mpi4py.MPI
    mpi4py.MPI.COMM_WORLD.Abort(1)
    sys.__excepthook__(exctype, value, traceback)
sys.excepthook = global_except_hook

def func1():
    import chainermn
    import mpi4py.MPI
    mpi_comm = mpi4py.MPI.COMM_WORLD
    if mpi_comm.rank == 0:
        raise ValueError('failure!')
    comm = chainermn.create_communicator('naive', mpi_comm)

def func2():
    import mpi4py.MPI
    mpi_comm = mpi4py.MPI.COMM_WORLD
    if mpi_comm.rank == 0:
        raise ValueError('failure!')

    mpi4py.MPI.COMM_WORLD.Barrier()

if __name__ == '__main__':
    d = {'func1' : func1,
         'func2' : func2}

    fname = sys.argv[1] if len(sys.argv) >= 2 else 'func1'
    d[fname]()

Improved version of the error handler:
import sys

# Global error handler
def global_except_hook(exctype, value, traceback):
    import sys
    try:
        import mpi4py.MPI
        sys.stderr.write("\n*****************************************************\n")
        sys.stderr.write("Uncaught exception was detected on rank {}. \n".format(
            mpi4py.MPI.COMM_WORLD.Get_rank()))
        from traceback import print_exception
        print_exception(exctype, value, traceback)
        sys.stderr.write("*****************************************************\n\n\n")
        sys.stderr.write("\n")
        sys.stderr.write("Calling MPI_Abort() to shut down MPI processes...\n")
        sys.stderr.flush()
    finally:
        try:
            import mpi4py.MPI
            mpi4py.MPI.COMM_WORLD.Abort(1)
        except Exception as e:
            sys.stderr.write("*****************************************************\n")
            sys.stderr.write("Sorry, we failed to stop MPI, this process will hang.\n")
            sys.stderr.write("*****************************************************\n")
            sys.stderr.flush()
            raise e

sys.excepthook = global_except_hook

def func():
    import mpi4py.MPI
    mpi_comm = mpi4py.MPI.COMM_WORLD
    if mpi_comm.rank == 0:
        raise ValueError('failure!')

    mpi4py.MPI.COMM_WORLD.Barrier()

if __name__ == '__main__':
    func()

Thanks. I will also test the mpi4py 's recommended way if it works with ChainerMN.
I think I can now close the issue. thanks for your contribution!


Confusion on inputClass

My assumption was that [inputClass]="form-control" would apply that class to the input field and this was not the case.  Instead I extended .form-control using the completer class and SCSS (not an option for everyone).
Even after getting the desired styles applied like form-control I then was fighting with the width of the dropdown.  This is also unclear in documentation (although classes are listed) what can be done to style the dropdown/input.

try using [inputClass]="'form-control'"

Ok that worked out.  Closing issue.  Thank you.


Support extract syntax

SQL defines the extract expression that can be used to access components of temporal datatypes. Calcite already supports it, so when an expression includes an extract function it parses. But once Pinot tries to convert it to its own expression AST, it fails with:
ProcessingException(errorCode:150, message:PQLParsingError:
java.lang.ClassCastException: class org.apache.calcite.sql.SqlIntervalQualifier cannot be cast to class org.apache.calcite.sql.SqlBasicCall (org.apache.calcite.sql.SqlIntervalQualifier and org.apache.calcite.sql.SqlBasicCall are in unnamed module of loader 'app')
	at org.apache.pinot.sql.parsers.CalciteSqlParser.toExpression(CalciteSqlParser.java:682)
	at org.apache.pinot.sql.parsers.CalciteSqlParser.compileFunctionExpression(CalciteSqlParser.java:737)
	at org.apache.pinot.sql.parsers.CalciteSqlParser.toExpression(CalciteSqlParser.java:682)
	at org.apache.pinot.sql.parsers.CalciteSqlParser.toExpression(CalciteSqlParser.java:619))

Pinot already have functions like minute, year, etc so this seems to be a matter of syntax.

I am planning to pick this up.

I have added the progress so far in the above PR. May I get some eyes on this, to ensure that I am on the right track?###

@npawar Update -- PR is merged.

I have added the documentation in the PR above ^. Please review it.

Merged. Thank you so much for your contribution!


can requests user:memberof:... scope from organization that doesn't exists

when creating a JWT using client credential flow, I can create a jwt token that has some scope that I shouldn't have
Test script to generate token:
def generatetoken(clientid, clientsecret, organization, scopes):
    params = {
        'grant_type': 'client_credentials',
        'client_id': clientid,
        'client_secret': clientsecret,
        'response_type': 'id_token',
        'scope': scopes,
    }

    url = 'https://itsyou.online/v1/oauth/access_token'
    resp = requests.post(url, params=params)
    resp.raise_for_status()
    token = resp.content.decode('utf8')
    print("export JWT='{}'".format(token))

Wen asking a scope like user:memberof:nonexistingorg the token created contains an empty scopes filed. Which is ok.
But if I ask a scope like user:memberof:myorg.nonexistingorg, the token created contains the asked scope, even if myorg.nonexistingorg is not a sub organization that exists.

The issue here is the following:
Database checks if the user is member of myorg.nonexistingsuborg, which is obviously not the case since this org does not exist. So the logic then checks ifs the user is a member of myorg, as this is a parent org of nonexistingsuborg, and by membership inheritance rules, being a member of myorg then makes you a member of nonexistingsuborg as well. A simple check to see if myorg.nonexistingsuborg exists first should fix this AFAIC

I've been told that you can be member of a sub-organisation without being member of the root organization. Is this assumption correct ? Cause if not we are changing a rule that could impact other use cases no ?

Not sure i fully understand your question, however:
It is possible to be a member of a suborg without being in the rootorg: say i have an organization A with suborganization B (so full name is A.B), then i can be a member of B without being a member in A. If i were to be a member of A, then I am also a member of A.B, since membership is inherited to suborganizations.
The fix does not change any functionality, it simply prevents the memberof scope from being given for a suborganization that does not exist if you are a member of the parent organization. If you have the structure myorg.nonexistingsuborg, you got the memberof scope because the system recognized you as a member of myorg, and this implied membership of nonexistingsuborg. The db logic has simply been made aware that it should check if a suborg exists first before confirming (implicit) membershp (because you obviously can't be a member of an nonexisting organization).

Ok so my assumptions were correct. all good :+1:

Moved to bugfix release branch


Unable to pip install fairseq.

I have been constantly trying to resolve this for 5 days now (only today decided to make this post). I finally managed to install fairseq. For anyone with a similiar problem, this is how I did it:
Starting from the venv inside my cmd.exe
git clone https://github.com/pytorch/fairseq
cd fairseq
set LIB=%LIB%;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.40.33807\ATLMFC\lib\x86;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.40.33807\lib\x86;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.40.33807\lib\onecore\x64;C:\Program Files (x86)\Windows Kits\10\Lib\10.0.26100.0\um\x64;C:\Program Files (x86)\Windows Kits\10\Lib\10.0.26100.0\ucrt\x64
set LIBPATH=%LIBPATH%;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.40.33807\ATLMFC\lib\x86;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.40.33807\lib\x86;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.40.33807\lib\x86\store\references;C:\Windows\Microsoft.NET\Framework\v4.0.30319
set INCLUDE=%INCLUDE%;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.40.33807\include;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.40.33807\ATLMFC\include;C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Auxiliary\VS\include;C:\Program Files (x86)\Windows Kits\10\Include\10.0.26100.0\ucrt;C:\Program Files (x86)\Windows Kits\10\Include\10.0.26100.0\shared;C:\Program Files (x86)\Microsoft
set Path=%Path%;C:\Program Files (x86)\Windows Kits\10\bin\x64
then I copied rc.exe and rcdll.dll from here C:\Program Files (x86)\Windows Kits\10\bin\10.0.22621.0\x64 to here C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.40.33807\bin\Hostx86\x64 (which is where the link.exe is sitting)
pip install --editable .\
Now, fairseq is finally installed at last.

I ran this to install fairseq:
git clone https://github.com/pytorch/fairseq
cd fairseq
pip install fairseq
But this error keeps popping up and I have been unable to find a solution for it:
Installing build dependencies ... done
Getting requirements to build wheel ... error
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> [16 lines of output]
Traceback (most recent call last):
File "C:\Users\v\PycharmProjects\CausalAI\lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 353, in
main()
File "C:\Users\v\PycharmProjects\CausalAI\lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 335, in main
json_out['return_val'] = hook(**hook_input['kwargs'])
File "C:\Users\v\PycharmProjects\CausalAI\lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 118, in get_requires_for_build_wheel
return hook(config_settings)
File "C:\Users\v\AppData\Local\Temp\pip-build-env-xw_hib4s\overlay\Lib\site-packages\setuptools\build_meta.py", line 327, in get_requires_for_build_wheel
return self._get_build_requires(config_settings, requirements=[])
File "C:\Users\v\AppData\Local\Temp\pip-build-env-xw_hib4s\overlay\Lib\site-packages\setuptools\build_meta.py", line 297, in _get_build_requires
self.run_setup()
File "C:\Users\v\AppData\Local\Temp\pip-build-env-xw_hib4s\overlay\Lib\site-packages\setuptools\build_meta.py", line 313, in run_setup
exec(code, locals())
File "", line 27, in
File "", line 18, in write_version_py
FileNotFoundError: [Errno 2] No such file or directory: 'fairseq\version.txt'
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
I have found the MANIFEST.in which is supposed to resolve this via include fairseq/version.txt, but for me it does nothing. I also tried changing the path to fairseq\version.txt and several other path variations (even absolute from root). No results.
Environment:
Windows 11
Python 3.9
pytorch  2.3.1+cu118
I have tried to install fairseq 0.10.2, 0.11.1 and 0.12.2 - all to no avail. Same error. I would be really thankfull for help.
EDIT 1: I keep trying to find a workaround:
using  pip install git+https://github.com/pytorch/fairseq runs into this:
ERROR: Cannot install fairseq and fairseq==0.12.2 because these package versions have conflicting dependencies.
The conflict is caused by:
fairseq 0.12.2 depends on omegaconf<2.1
hydra-core 1.0.7 depends on omegaconf<2.1 and >=2.0.5
EDIT 2:
I switched over to Microsoft Visual Studio and their Developer Command Prompt. Now pip install fairseq results in :
LINK : fatal error LNK1104: Cannot open file "kernel32.lib".
error: command 'C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.40.33807\bin\HostX86\x86\link.exe' failed with exit code 1104
No matter which route I take, I find no way to install this package. At this point, I would gladly take a precompiled wheel for windows 11.

Same problem.


[api] Increment the API version to 6.29.0 after approval

@gyorb Yes, I know about this problem. It was fixed in #2901 and merged yesterday. I rebased this commit, so now it should work.


Editor's update button appears to be un-clickable on iPhone Safari

Issue Summary
I attempted to edit a blog post on my Ghost blog this afternoon from my iPhone 7, and was unable to actually submit the updates using the update button at the top of the page. It seems that I'm unable to click the button.
Steps to Reproduce

Open a published story.
Make a change to it.
Attempt to tap update.

Actual results: Nothing appears to happen.
Expected results: I expected the update button to be clicked and the blog post to be changed.
Technical details:

Ghost Version: 1.0.0-rc1
Node Version: 6.11.1
Browser/OS: Safari / iOS 10.3.3
Database: MySQL

Hey @farmdawgnation 👋 Closing as a duplicate of https://github.com/TryGhost/Ghost/issues/8657, it's fixed in master and will be part of the next release.

:rofl: I promise I looked for other occurrences of this ticket before filing it. Thank you!


[HYDRAR-115] adding accuracy tests for coefficients from step.lm

Adding tests for coefficients from step.lm with/without intercept, comparing the results with step() from R.
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Build triggered.

Build success.
All unit tests passed.


Resolve #207

Improvements for #119

/cc @AlekseyMartynov

Here are my comments and suggestions:

The _buildColors function became too long. We need to split it.
rgbToHex: too many repeats of toString(16), could be rewritten
Seems there's no need to nest hue2rgb inside hslToRgb
What is the reason of assigning an id for each color cell (id + "_color_" + color, applies to the both strategies)
public enum PaletteMode: I suggest using a more specific name ColorPickerPalette.... Also we need to be consistent ("style" vs "mode")
Not sure "RGB" is the adequate term. Comparing to the current default style, I'd name it "continuous".

Also I think we should demonstrate both styles in the ColorPicker page of SampleSite.

/cc @AlekseyMartynov

LGTM
(merge this without squashing please)


Refactor

Codecov Report
Attention: Patch coverage is 92.96875% with 9 lines in your changes missing coverage. Please review.

Project coverage is 91.05%. Comparing base (6a57ae9) to head (94e92c9).

Files
Patch %
Lines

src/pydantic_avro/type_handlers.py
94.05%
6 Missing :warning:

src/pydantic_avro/class_registery.py
83.33%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #107      +/-   ##
==========================================
+ Coverage   90.27%   91.05%   +0.77%
==========================================
  Files           2        4       +2
  Lines         216      257      +41
==========================================
+ Hits          195      234      +39
- Misses         21       23       +2

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


Jesse: Watch romtronic.com

Jesse requests the watch of the watch_keyword romtronic\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

romtronic\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Host is not receiving the eject disk event from the machine

When ejecting a disk via GS/OS (and possibly vice-versa), the host doesn't remove the disk from the drive (it may be stuck on Ejecting...)
Need to audit/retest the 3.5" drive ejection logic and see how the host responds to it.
This only applies to 3.5" Disk and SmartPort devices.

Currently SmartPort devices can be ejected but cannot be remounted until a reboot.   Will have to think of a better interface to account for this.


Program abnormal terminated

commit: f2cbb2e7d8d2be2f1083d7832914d68d8724bd9e
CPU: i7 6700K
error message:
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
Freshly pulled from git and running on my Arch machine, and the program exit immediately.
Last build (few months ago, though) the issue wasn't exit.

It seems you don't have enough memory to run this program.


why always open terminal when start the output binary

As the title.
why always open terminal when start the output binary? Is there any setting to avoid that?
Thanks
David

I'm not sure what exactly you are facing on your machine so could you elaborate?
On my machine there is running terminal inside VCS but just for compilation and linking processes. When actual application starts then terminal is not running anymore.
Terminal has to start because all compilation and linking tasks are executed inside it. But still this terminal is running as part of VSC.

No response for few months - closing this thread.


Too many false detections

I'm using pigo in my realtime app and observing many false detections. How can i reduce amount of them? Does pigo have some parameters to adjust?

You have to play with the various parameters provided by the library. In general to reduce the false positive rates you have to consider the detection window over you are running the face detector. Also there are three core components responsible for the detection score: iou, shift and scale. I would recommend first to try modifying the first two options. Read the documentation and also this blog post: https://www.esimov.com/2019/07/pigo-face-detection-library.
If the obtained results are still not satisfying please provide me a sample image with the applied parameters and i will try to help you.

On this image with params iou=0.2, shift=0.1, scale=1.1 pigo detected face in rectangle (171,331)-(271,431).
This kind of false detections occurs regularly on random frames when i'm processing live stream with pigo. Gocv with lbpcascade_frontalface_improved.xml under the same conditions (same video streams) gives no false detections.
I read documentation but didn't find any hints how to adjust this params. I can try to adjust them randomly but maybe you can speed up this process.
Another problem i faced: rarely pigo detects face with too large scale (more larger than actual face).

I have tried with the default parameters and also with the parameters you have provided but no face have been detected. If you download the attached image and run over it pigo with the default parameters it should not detect any face.
Regarding the last issue, the scale param has nothing to do with the face dimension. To detect large faces, I mean faces with the rectangular area greater than the min and max values, you have to adjust these values.

Since I haven't received any feedback i'm closing this issue. You can reopen at any time.


Write compliance guidance for customers

In order for our cloud.gov customers to better understand how they can maximize their security and compliance (and for us to show compliance reviewers that we are helping customers with these things), we need to integrate a few points into our user documentation (docs.cloud.gov) about buildpacks, environment variables, and DNSSEC.
See this notes spreadsheet and this notes doc for details. These have numbered items, and this story is about taking care of the user-facing documentation tasks in items 5, 6, and 21.
Acceptance criteria
The acceptance criteria for this story is that we've added guidance somewhere appropriate in docs.cloud.gov for the following items (see notes above for details/context):

[ ] Item 5, buildpacks - write guidance for customers about buildpacks.
[x] Item 6, environment variables - document for users that they shouldn’t leak their own credentials.
[x] Item 21, DNSSEC - document that customers can implement DNSSEC if they want to.

For all of these items (especially item 21), after putting these things on docs.cloud.gov, we should document in some appropriate place in the SSP that this guidance exists (by linking to it).

/cc https://github.com/18F/cg-product/issues/63

This looks done to me, thanks @cnelson! I added notes to Noah's doc about these additions, so he can integrate mentions of them into SSP work if appropriate.


WRKLDS-657: add PDBUnhealthyPodEvictionPolicy to tech preview

adding UnhealthyPodEvictionPolicy for PodDisruptionBudgets which was added as an alpha in kube 1.26

please create a fake bump in kas-o and see if this passes testing.

ok, testing in https://github.com/openshift/cluster-kube-apiserver-operator/pull/1441

proof PR in https://github.com/openshift/cluster-kube-apiserver-operator/pull/1441 was good
/lgtm
/approve

this will be tested once api gets bumped in KAS operator
/label qe-approved
/label px-approved
/label docs-approved


Video Models

Hi can we use this library to use GRAD-CAM on 3D CNN models?

Hi there @kamalesh0406,
The visualization script is not adapted for 3D tensors, as they can't be mapped to PIL images. But the CAM extractor itself is compatible yes. Here is a small example:
import torch
import torch.nn as nn
from torchcam.cams import SmoothGradCAMpp

# Define a 3D CNN model
model = nn.Sequential(
    nn.Conv3d(3, 8, 3, 1),
    nn.BatchNorm3d(8),
    nn.LeakyReLU(0.1),
    nn.MaxPool3d(2),
    nn.Conv3d(8, 16, 3, 1),
    nn.BatchNorm3d(16),
    nn.LeakyReLU(0.1),
    nn.AdaptiveAvgPool3d(1),
    nn.Flatten(1),
    nn.Linear(16, 2))

input_t = torch.rand((1, 3, 32, 32, 32))
# Parametrize the CAM extractor
extractor = SmoothGradCAMpp(model, '6', '0')
scores = model(input_t)
class_idx = scores.squeeze(0).argmax().item()
activation_map = extractor(class_idx, scores).cpu()
extractor.clear_hooks()
print(activation_map.shape)

which yields:
torch.Size([13, 13, 13])

I would advise against using CAM methods that are computationally intensive with 3D CNN depending on your system specs (Score CAM & SS CAM for instance).
If you feel like submitting a PR to add a visualization option, I'd be happy to look into it :+1:
Let me know if that helped, cheers!

Yup this helps thanks!

Glad it helped @kamalesh0406!
I'll close the issue then, feel free to reopen if you get some troubles along the way :ok_hand:


Using alternate directory for moulinette store with symlinks doesn't work on windows

Hello,
Running on foundry 9.269, with moulinette core 4.7.0
I've been extensively using moulinette on a MAC system where all my assets are stored under the /sharedassets/ directory, beneath this directory sits de moulinette directory. This setup was done using symbolic links
I had to setup the same environment on a windows machine, and I used a windows logical link . When I try to generate my assets index,  I am getting the following error
moulinette/tiles/custom does not exist or is not accessible in this storage location
```

If I remove the link and launch an indexation, moulinette creates an "empty" directory tree and indexes the game icons.

I am almost certain that it worked before.

Is there any way to solve this ?

Hello
I kind of solved by removing the link and replacing it with a junction using mklink /d /j destination source

solved by creating a junction instead of a shortcut

Hi!
For your information, the recent versions of Moulinette provides a UI to let you pick and choose a folder and it to the source to be indexed. It is much easier than the old way of doing it (that you use).


Wrong alignment when using a different landmark detector

hello, eos is a good job! thks~
I meet some problems when reduce the points num by change a  landmark detector, the result is bad。

can you give some advice? thx~

Hi,
My guess is that you've got the landmark mappings wrong. Make sure to get the 2D landmark -> 3D vertex indices mapping correct.
Please debug and understand your code. Thanks!

Hi,
My guess is that you've got the landmark mappings wrong. Make sure to get the 2D landmark -> 3D vertex indices mapping correct.
Please debug and understand your code. Thanks!

solved it, thx~


Song crashes with dadbattle spotlight

Describe your bug here.
I used the dadbattle spotlight inside of guns in week 7 and whenever I load up the song it crashes.

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No

Not a bug

Dadbattle Spotlight only works with the week 1 stage, same with how Philly Glow only works in the week 3 stage. You could remake the event to work on all stages, but it won't work otherwise.

Dadbattle Spotlight only works with the week 1 stage, same with how Philly Glow only works in the week 3 stage. You could remake the event to work on all stages, but it won't work otherwise.

yea, too bad no one made a dadbattle spotlight recreation event on lua

its purposely hardcoded into week 1

Dadbattle Spotlight only works with the week 1 stage, same with how Philly Glow only works in the week 3 stage. You could remake the event to work on all stages, but it won't work otherwise.

yea, too bad no one made a dadbattle spotlight recreation event on lua yet

https://gamebanana.com/tools/11052

Dadbattle Spotlight only works with the week 1 stage, same with how Philly Glow only works in the week 3 stage. You could remake the event to work on all stages, but it won't work otherwise.

yea, too bad no one made a dadbattle spotlight recreation event on lua yet

https://gamebanana.com/tools/11052

welp i spoke too soon

close this issue


Add French and Hungarian style guides

Docs Build status updates of commit c66d4d3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

globalization/localization/ministyleguides/mini-style-guide-french.md
:white_check_mark:Succeeded
View

globalization/localization/ministyleguides/mini-style-guide-hungarian.md
:white_check_mark:Succeeded
View

globalization/localization/styleguides.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.