docs: Make Aside (In this article) scrollable

🙋 Documentation Request
In the DataGrid Page, the aside is too long, and difficult to see everything without scrolling the entire page.

💁 Possible Solution
to the aside tag add the following CSS:
max-height: calc(100dvh - 90px); //get from body-content CSS
overflow-y: auto;

This will be rolled out to the demo site with next release.


HBASE-24133 Temporary issue... ignore

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

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

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
5m 35s
branch-2 passed

+1 :green_heart:
compile
0m 23s
branch-2 passed

-1 :x:
shadedjars
0m 14s
branch has 7 errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 22s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 11s
the patch passed

+1 :green_heart:
compile
0m 22s
the patch passed

+1 :green_heart:
javac
0m 22s
the patch passed

-1 :x:
shadedjars
0m 12s
patch has 7 errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 21s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 16s
hbase-common in the patch passed.

15m 48s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 64c5be84fec3 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 74a85e26ee

Default Java
1.8.0_232

shadedjars
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/artifact/yetus-jdk8-hadoop2-check/output/branch-shadedjars.txt

shadedjars
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/artifact/yetus-jdk8-hadoop2-check/output/patch-shadedjars.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/testReport/

Max. process+thread count
376 (vs. ulimit of 10000)

modules
C: hbase-common U: hbase-common

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

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

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
7m 10s
branch-2 passed

+1 :green_heart:
compile
0m 24s
branch-2 passed

-1 :x:
shadedjars
0m 12s
branch has 7 errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 18s
hbase-common in branch-2 failed.

_ Patch Compile Tests _

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

+1 :green_heart:
compile
0m 25s
the patch passed

+1 :green_heart:
javac
0m 25s
the patch passed

-1 :x:
shadedjars
0m 11s
patch has 7 errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 16s
hbase-common in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
1m 58s
hbase-common in the patch passed.

20m  7s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 5edb16644f9e 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 74a85e26ee

Default Java
2020-01-14

shadedjars
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/artifact/yetus-jdk11-hadoop3-check/output/branch-shadedjars.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-common.txt

shadedjars
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/artifact/yetus-jdk11-hadoop3-check/output/patch-shadedjars.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-common.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/testReport/

Max. process+thread count
272 (vs. ulimit of 10000)

modules
C: hbase-common U: hbase-common

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

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

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
5m 44s
branch-2 passed

+1 :green_heart:
checkstyle
0m 25s
branch-2 passed

+1 :green_heart:
spotbugs
0m 44s
branch-2 passed

_ Patch Compile Tests _

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

-0 :warning:
checkstyle
0m 23s
hbase-common: The patch generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0)

-0 :warning:
whitespace
0m  0s
The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

+1 :green_heart:
hadoopcheck
10m 31s
Patch does not cause any errors with Hadoop 2.10.0 or 3.1.2.

+1 :green_heart:
spotbugs
0m 50s
the patch passed

_ Other Tests _

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

31m 23s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/artifact/yetus-general-check/output/Dockerfile

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

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux ea44c971a3eb 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 74a85e26ee

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-common.txt

whitespace
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/artifact/yetus-general-check/output/whitespace-eol.txt

Max. process+thread count
93 (vs. ulimit of 10000)

modules
C: hbase-common U: hbase-common

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12

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

This message was automatically generated.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/console in case of problems.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/console in case of problems.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/console in case of problems.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 42s
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 _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
5m 10s
branch-2 passed

+1 :green_heart:
compile
0m 22s
branch-2 passed

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

+1 :green_heart:
javadoc
0m 23s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m  9s
the patch passed

+1 :green_heart:
compile
0m 23s
the patch passed

+1 :green_heart:
javac
0m 23s
the patch passed

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

+1 :green_heart:
javadoc
0m 21s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 19s
hbase-common in the patch passed.

23m 51s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 07a1e368d7ee 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 9da7f95fa7

Default Java
1.8.0_232

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/testReport/

Max. process+thread count
363 (vs. ulimit of 12500)

modules
C: hbase-common U: hbase-common

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 34s
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 _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
6m  4s
branch-2 passed

+1 :green_heart:
compile
0m 24s
branch-2 passed

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

-0 :warning:
javadoc
0m 17s
hbase-common in branch-2 failed.

_ Patch Compile Tests _

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

+1 :green_heart:
compile
0m 25s
the patch passed

+1 :green_heart:
javac
0m 25s
the patch passed

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

-0 :warning:
javadoc
0m 18s
hbase-common in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
1m 30s
hbase-common in the patch passed.

26m 40s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux eef0e3daa2ff 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 9da7f95fa7

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-common.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-common.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/testReport/

Max. process+thread count
361 (vs. ulimit of 12500)

modules
C: hbase-common U: hbase-common

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 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  0s
No case conflicting files found.

+0 :ok:
shelldocs
0m  0s
Shelldocs was not available.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

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

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
5m 43s
branch-2 passed

+1 :green_heart:
checkstyle
0m 25s
branch-2 passed

+1 :green_heart:
spotbugs
0m 47s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 13s
the patch passed

-0 :warning:
checkstyle
0m 23s
hbase-common: The patch generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0)

+1 :green_heart:
shellcheck
0m  2s
There were no new shellcheck issues.

-0 :warning:
whitespace
0m  0s
The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

+1 :green_heart:
hadoopcheck
10m 55s
Patch does not cause any errors with Hadoop 2.10.0 or 3.1.2.

+1 :green_heart:
spotbugs
0m 50s
the patch passed

_ Other Tests _

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

32m  9s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/artifact/yetus-general-check/output/Dockerfile

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

Optional Tests
dupname asflicense shellcheck shelldocs spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux f6856d42dff4 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 9da7f95fa7

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-common.txt

whitespace
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/artifact/yetus-general-check/output/whitespace-eol.txt

Max. process+thread count
93 (vs. ulimit of 12500)

modules
C: hbase-common U: hbase-common

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1447/2/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) shellcheck=0.4.6 spotbugs=3.1.12

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

This message was automatically generated.


range(N) does not work on android

Operating system or device - Godot version:
android 5.0.2, android 5.1.1 - Godot 2.1 stable
compiled  at windows 10 x64
works fine on other platforms
Issue description (what happened, and what was expected):
range(N) does not work
Steps to reproduce:
func test():     print("range")     var test2= range(0,2)     print(test2)     print("after range")
code above never reaches  print(test) and after range
Link to minimal example project (optional but very welcome):

Why doesn't it reach the print statements? What does it do? Crash, or something else?

it does nothing, real code looks like this
func _ready():      for i in range(AMOUNT_TO_SPAWN):                spawn()     set_bot_pos()     pass
functions spawn() and set_bot_pos() are never called, so it looks like entire scene is not working.
no crash, no debug output, all other scenes, inputs, working as they should

If you put a print at the top of the ready statement, does it reach there? Ranges work fine for me on android. Curious if it's just failing to call ready.

Bump, can we get an example project, or a confirmation it no longer happens?

First of all thank you for your report and sorry for the delay.
We released Godot 3.0 in January 2018 after 18 months of work, fixing many old issues either directly, or by obsoleting/replacing the features they were referring to.
We still have hundreds of issues whose relevance/reproducibility needs to be checked against the current stable version, and that's where you can help us.
Could you check if the issue that you described initially is still relevant/reproducible in Godot 3.0 or any newer version, and comment about it here?
For bug reports, please also make sure that the issue contains detailed steps to reproduce the bug and, if possible, a zipped project that can be used to reproduce it right away. This greatly speeds up debugging and bugfixing tasks for our contributors.
Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.
Thanks in advance.
Note: This message is being copy-pasted to many "stale" issues (90+ days without activity). It might happen that it is not meaningful for this specific issue or appears oblivious of the issue's context, if so please comment to notify the Bugsquad about it.

Looks like the original reporter gave up long before @bojidar-bg made the bump request, so this is probably dead. Nothing similar has been reported, has it?


DDS_DURATION_INFINITE is a RTI specific DDS type

Feature request
Feature description
DDS_DURATION_INFINITE is a RTI specific macro, to initialize a Duration_t to infinite sec/nsec should be set separately
Implementation considerations
DDS_DURATION_INFINITE (or DDS::DURATION_INFINITE) is RTI specific so should not be used to simplify porting rmw code between vendors

Please assign to me


Travis CI build failure in dubbo-metadata-report-zookeeper

[ ] I have searched the issues of this repository and believe that this is not a duplicate.
[ ] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment
Travis CI build environment

Dubbo version: xxx
Operating System version: xxx
Java version: xxx

Steps to reproduce this issue
Run build in travis CI from latest master branch

xxx
xxx
xxx

Pls. provide [GitHub address] to reproduce this issue.
Expected Result
Build should not failed because of dubbo-metadata-report-zookeeper UT. below are current error log while running UT on dubbo-metadata-report-zookeeper
Running org.apache.dubbo.metadata.store.zookeeper.ZookeeperMetadataReportTest
log4j:WARN No appenders could be found for logger (org.apache.dubbo.common.logger.LoggerFactory).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.
Tests run: 2, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 4.305 sec <<< FAILURE! - in org.apache.dubbo.metadata.store.zookeeper.ZookeeperMetadataReportTest
testStoreProvider(org.apache.dubbo.metadata.store.zookeeper.ZookeeperMetadataReportTest)  Time elapsed: 2.461 sec  <<< FAILURE!
java.lang.AssertionError
at org.apache.dubbo.metadata.store.zookeeper.ZookeeperMetadataReportTest.testStoreProvider(ZookeeperMetadataReportTest.java:59)
Results :
Failed tests:
ZookeeperMetadataReportTest.testStoreProvider:59
Actual Result
What actually happens?
If there is an exception, please attach the exception trace:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.19.1:test (default-test) on project dubbo-metadata-report-zookeeper: There are test failures.
[ERROR]
[ERROR] Please refer to /home/travis/build/apache/incubator-dubbo/dubbo-metadata-report/dubbo-metadata-report-zookeeper/target/surefire-reports for the individual test results.
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn <goals> -rf :dubbo-metadata-report-zookeeper
/home/travis/.travis/job_stages: line 523:  4690 Terminated              travis_jigger "${!}" "${timeout}" "${cmd[@]}"

Now ci succeed . close via https://github.com/apache/incubator-dubbo/commit/5ea4fa531f9ec5ae446d4cb1ef2417b53712bd46


[Snyk] Security upgrade protobuf from 3.19.1 to 3.19.5

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

cmd/suggestion/optuna/v1beta1/requirements.txt

⚠ Warning
optuna 2.10.1 requires numpy, which is not installed.
optuna 2.10.1 requires sqlalchemy, which is not installed.
optuna 2.10.1 requires scipy, which is not installed.
Mako 1.2.4 requires MarkupSafe, which is not installed.
cmaes 0.9.0 requires numpy, which is not installed.
alembic 1.8.1 requires SQLAlchemy, which is not installed.

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

499/1000  Why? Has a fix available, CVSS 5.7
Denial of Service (DoS) SNYK-PYTHON-PROTOBUF-3031740
protobuf:  3.19.1 -> 3.19.5
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 72.857%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
3111

Relevant Lines:
4270

💛  - Coveralls


Fix typings for stats app

Before the typings file would be empty, now it will contain the proper typings

@wolfy1339 wanna pull in those changes and merge master for this?

Sure thing.

Done.

@hiimbex Please re-review this as I added some fixes for some stuff that popped up in the Travis CI build log

Just a gentle ping, so you can look at the new changes and get this merged

Hey there. Is there anything holding back this PR?

@wolfy1339 Sorry for the delay on this. I fixed the conflicts we caused from some other PRs that landed. Merging now. Thanks for the contribution!


Chess: Move C8 to A6

Please do not change the title. Just click "Submit new issue".

@MGBoulware88 done! Successfully played move c8a6 for current game.
Thanks for playing!


feat: allow customizing the background colors

There are quite a few options that can be overridden here, but there isn't an API. However, we can re-define the variable in Sass as done in this Stackblitz.
This is a more advanced feature that probably should be hidden behind some kind of checkbox/option.

This is nice!
Advanced menu sounds like a good place to group such configs.


⚠ Bofrost.at has degraded performance

In acedbe6, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 858 ms

Resolved: Bofrost.at performance has improved in 0efdb9d after 15 minutes.


Support EndpointSlice in Antrea proxy

Support EndpointSlice in Antrea proxy, ServiceTopology is not supported in this commit. #1310
Signed-off-by: Hongliang Liu<EMAIL_ADDRESS>

Codecov Report

Merging #1694 (e0d794c) into master (9d3d10b) will decrease coverage by 23.31%.
The diff coverage is 31.05%.

@@             Coverage Diff             @@
##           master    #1694       +/-   ##
===========================================
- Coverage   63.31%   40.00%   -23.32%
===========================================
  Files         170      108       -62
  Lines       14250    13489      -761
===========================================
- Hits         9023     5396     -3627
- Misses       4292     7611     +3319
+ Partials      935      482      -453

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
40.00% <31.05%> (-1.28%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cmd/antrea-agent/agent.go
0.00% <0.00%> (ø)

pkg/agent/agent.go
12.20% <0.00%> (-36.51%)
:arrow_down:

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/agent/cniserver/interface_configuration_linux.go
5.79% <0.00%> (-14.44%)
:arrow_down:

pkg/agent/cniserver/pod_configuration.go
34.52% <0.00%> (-19.24%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

...gent/controller/noderoute/node_route_controller.go
32.57% <0.00%> (-13.90%)
:arrow_down:

...agent/controller/traceflow/traceflow_controller.go
0.00% <0.00%> (-82.70%)
:arrow_down:

pkg/agent/proxy/endpointslicecache.go
0.00% <0.00%> (ø)

pkg/agent/querier/querier.go
66.66% <ø> (-21.50%)
:arrow_down:

... and 170 more

If this PR replaces https://github.com/vmware-tanzu/antrea/pull/1310, please close that one


MNML

Great app looks like Google's homegrown apps.Also a dark mode and a separate black theme would be great

The app does have dark mode. Do you mean a dark mode where it is more greyish? Then keep the current dark and name it black or amoled?

Yup.
On Fri, Jan 10, 2020, 8:23 AM Keegan Given<EMAIL_ADDRESS>wrote:

The app does have dark mode. Do you mean a dark mode where it is more
greyish? Then keep the current dark and name it black or amoled?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/afollestad/mnml/issues/81?email_source=notifications&email_token=AOHTLH63W35LE5X7NFASESTQ47PLNA5CNFSM4KFBNSNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEISQBCQ#issuecomment-572850314,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOHTLH6XQNVFHO7HG36HERDQ47PLNANCNFSM4KFBNSNA
.


Optimal way to use SQLAlchemy with dataclasses

How can i use SQLAlchemy with Blacksheep
i am using SQLAlchemy MappedAsDataclass this is an example:
class Clip(Base):
	__tablename__ = "clip"
	name: Mapped[str] = mapped_column(String(128))
	filename: Mapped[str] = mapped_column(String(128))
	clip_type: Mapped[ClipType]

	tags: Mapped[List["Tag"]] = relationship(secondary=ClipTag,back_populates="clips")

and every time i try to convert it to a dict or to json i get and Error:
RecursionError: maximum recursion depth exceeded
What is The best solution for this should i use Pydantic or should i add a to_dict to every class

Hey @Mrdhaaw.
Blacksheep supports Pydantic models and dataclasses to build OpenAPI schema. It has data binders that support Pydantic models as well. So, I decided to use Pydantic with BlackSheep in my project in general


153 config parser

Capture last species, tolerance or reaction JSON.
Add to the error message when any return status is not SUCCESS.

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (aabe856) 96.07% compared to head (1ae1ec0) 96.25%.

Files
Patch %
Lines

include/micm/configure/solver_config.hpp
80.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #369      +/-   ##
==========================================
+ Coverage   96.07%   96.25%   +0.18%
==========================================
  Files          37       37
  Lines        2852     2856       +4
==========================================
+ Hits         2740     2749       +9
+ Misses        112      107       -5

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


Support for half2 type convolution

Hi, I'm checking mix precision programming: https://developer.nvidia.com/blog/mixed-precision-programming-cuda-8/ Does cudnn frontend has any support for half2 type convolution?

Hi infloop777, thank you for your questions! Inside the library do have kernels that runs HFMA2 instructions for compute, however in the latest generations of GPUs, the focus has been on tensor cores which is much faster than HFMA2. cuDNN API (include the frontend) hides the implementation details of the actual kernels, so you just need to specify half datatype, and we will dispatch it to the best kernel in the library.

@YangXu1990uiuc Thank you for help!

I have another question regarding to fp16 vs fp32. Do you think for the same convolution fp16 is always better than fp32, or it's worth trying fp32 algorithms as well(though need some graph level modification)?

Hi infloop777, it depends on what's your situation, and what's your definition of better.

If your model has high numerical precision requirement, then FP32 will work better for you in terms of convergence and accuracy.
If your goal is to achieve good performance but not sacrifice on convergence on most models, then FP16 mixed precision (i.e. FP16 I/O, FP32 compute) will be the better choice.
Of course this also depends on what GPU you are using. Only the newer ones (Volta, Turing, Ampere) can run FP16 mixed precision efficiently.

Thanks! This is helpful.


Update documentation with minimum packer version

As per https://github.com/actions/runner-images/issues/5911#issuecomment-1185231772

Thanks!


Don't write the ref file unless necessary

Updating snapshot_download so that it doesn't re-write the commit hash every single time.
This has been incredibly problematic if we're keeping a repo saved to disk, and snapshot_download can't write to that disk.
This should at least partially resolve the issue.

What's the underlying issue you are facing? Is it a permission error? In that case I'd be more inclined to add a try/except instead of additional logic to read before writing (more error-prone)

If the model is on a read-only drive, this will break. My logic fixes that.
Ideally, you should probably have both. Only write when you need to, and
also wrap it all in a try catch so it doesn't break the whole program if
something fails.
On Fri, Nov 8, 2024, 3:33 AM Lucain @.***> wrote:

What's the underlying issue you are facing? Is it a permission error? In
that case I'd be more inclined to add a try/except instead of additional
logic to read before writing (more error-prone)
—
Reply to this email directly, view it on GitHub
https://github.com/huggingface/huggingface_hub/pull/2657#issuecomment-2464233018,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMO4NGT6RFB2AS4FHSLQCTZ7SAM3AVCNFSM6AAAAABRL6BTZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRUGIZTGMBRHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

What's the underlying issue you are facing? Is it a permission error? In that case I'd be more inclined to add a try/except instead of additional logic to read before writing (more error-prone)

See my additional commit - I added exception handling for both parts of dealing with the ref file.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Thanks @d8ahazard!

Thank you! You have no idea how many headaches this is going to solve for me. ;)


iOS build fails after removing last plugin

Description
After removing the last plugin, ios can't build anymore. I had to manually remove all pod files and run pod init and pod install to get things running again.
Steps to Reproduce

Create project

flutter create march18
cd march 18
flutter run

Add plugin

dependencies:
  device_info: "^0.0.5" // <-- new/only dependency
  flutter:
    sdk: flutter

Run app

flutter packages get
flutter run

Remove plugin

dependencies:
  # device_info: "^0.0.5"
  flutter:
    sdk: flutter

Run app

flutter packages get
flutter run

Error:
Launching lib/main.dart on iPhone X in debug mode...
Running Xcode clean...
Xcode build done
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    === BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Debug ===
    diff: /Users/pascalwelsch/Projects/march18/ios/Pods/Manifest.lock: No such file or directory
    error: The sandbox is not in sync with the Podfile.lock. Run 'pod install' or update your CocoaPods installation.
Could not build the application for the simulator.
Error launching application on iPhone X.

Logs
run_error.log (verbose)
Flutter Doctor
[✓] Flutter (Channel master, v0.2.5-pre.61, on Mac OS X 10.13.3 17D102, locale en-DE)
    • Flutter version 0.2.5-pre.61 at /Users/pascalwelsch/Projects/flutter
    • Framework revision bffae21571 (3 hours ago), 2018-03-28 12:07:12 -0700
    • Engine revision dd6f46c485
    • Dart version 2.0.0-dev.41.0.flutter-2f68e82526

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /usr/local/share/android-sdk
    • Android NDK at /usr/local/share/android-sdk/ndk-bundle
    • Platform android-27, build-tools 27.0.3
    • ANDROID_HOME = /usr/local/share/android-sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 1.9.2
    • CocoaPods version 1.3.1

[✓] Android Studio
    • Android Studio at /Applications/Android Studio 3.1 Preview.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] IntelliJ IDEA Community Edition (version 2017.3.4)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 22.0.2
    • Dart plugin version 173.4548.30

[!] VS Code (version 1.17.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Dart Code extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.dart-code

[✓] Connected devices (1 available)
    • iPhone X • E8880D2A-08D0-47D8-8E09-AD3A1C8B527A • ios • iOS 11.2 (simulator)

Tentatively duping into #11819


Updated Export Coreml Docs Style Transfer

Updated our export to CoreML docs for Style Transfer to not use the Vision Framework

Fixes #840


Update service domain for soundtouch from 'media_player' to 'soundtouch'

Breaking Change:
This change breaks existing service call references to the media_player.soundtouch_* services by changing the service calls to be soundtouch.*.
Description:
Update the domain and service name for soundtouch.*. See this comment for context: https://github.com/home-assistant/home-assistant/pull/28890#issuecomment-558485691
Related issue (if applicable): Related to #27289
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#11314
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

@raman325 Unfortunately, another conflict. The PR looks good though!

resolved conflicts, this is due to me changing this file in multiple branches


OP-SQLite cannot build with 0.74RC and bridgeless

Description
Hi, I'm trying to test the library against the latest RC and also trying to get bridgeless to work. So far I'm facing two issues with the included example app in my library repo.
iOS:
It seems the codegen script is expecting the src files to be in node_modules, but the builder-bob template links the dependency directly via react-native.config.js:
// react-native.config.js
const path = require('path');
const pak = require('../package.json');

module.exports = {
  dependencies: {
    [pak.name]: {
      root: path.join(__dirname, '..'),
    },
  },
};

And I get the following error when trying to install the pods:

Android
After turning on all the flags (and bumping the minSDK to 23) I get a compilation error but no output to what is wrong:

Steps to reproduce
If you want to take a look yourself here is the branch. Just install deps via yarn and then try to run the example app: cd example && yarn android or the equivalent iOS commands.
React Native Version
0.74.0-rc.0
Affected Platforms
Build - MacOS
Areas
Bridgeless - The New Initialization Flow
Output of npx react-native info
System:
  OS: macOS 14.3.1
  CPU: (11) arm64 Apple M3 Pro
  Memory: 2.49 GB / 36.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 18.19.0
    path: ~/.nvm/versions/node/v18.19.0/bin/node
  Yarn:
    version: 4.0.2
    path: ~/.nvm/versions/node/v18.19.0/bin/yarn
  npm:
    version: 10.2.3
    path: ~/.nvm/versions/node/v18.19.0/bin/npm
  Watchman: Not Found
Managers:
  CocoaPods:
    version: 1.14.0
    path: /Users/osp/.rbenv/shims/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK:
    Android NDK: 26.0.10792818
IDEs:
  Android Studio: 2023.1 AI-231.9392.1.2311.11330709
  Xcode:
    version: 15.2/15C500b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.10
    path: /usr/bin/javac
  Ruby:
    version: 3.3.0
    path: /Users/osp/.rbenv/shims/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.74.0-rc.0
    wanted: ^0.74.0-rc.0
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: true
iOS:
  hermesEnabled: true
  newArchEnabled: true

### Stacktrace or Logs

```text
Pasted above

Reproducer
https://github.com/OP-Engineering/op-sqlite/tree/bridgeless
Screenshots and Videos
No response

Hi @ospfranco, thanks for the issue.
For Android, we are aware that codegen and autolinking are not working with the RC.0 as we needed a commit that landed right after the branch cut. It should be solved in RC1
For iOS, @dmytrorykun, can you provide any insight? I remember that we had a similar report of react-native.config.js not being picked up by Codegen, but I remember we fixed that already. 🤔

For Android, I'm not sure this is the error I'm seeing? seems it doesn't like that I use the CallInvokerHolderImpl reference and it fails the compilation? How am I supposed to get a hold of the CallInvoker?
package com.op.sqlite

import com.facebook.react.bridge.ReactContext
import com.facebook.react.turbomodule.core.CallInvokerHolderImpl

class OPSQLiteBridge {
    private external fun installNativeJsi(
        jsContextNativePointer: Long,
        jsCallInvokerHolder: CallInvokerHolderImpl,
        docPath: String
    )

    private external fun clearStateNativeJsi()
    fun install(context: ReactContext) {
        val jsContextPointer = context.javaScriptContextHolder!!.get()
        val jsCallInvokerHolder =
            context.catalystInstance.jsCallInvokerHolder as CallInvokerHolderImpl
        // Trick to get the base database path
        val dbPath =
            context.getDatabasePath("defaultDatabase").absolutePath.replace("defaultDatabase", "")
        installNativeJsi(
            jsContextPointer,
            jsCallInvokerHolder,
            dbPath
        )
    }

    fun clearState() {
        clearStateNativeJsi()
    }

    companion object {
        val instance = OPSQLiteBridge()
    }
}

For Android, I'm not sure this is the error I'm seeing? seems it doesn't like that I use the CallInvokerHolderImpl reference and it fails the compilation? How am I supposed to get a hold of the CallInvoker?

You're seeing this error because you're using CallInvokerHolderImpl which is annotated as FrameworksAPI. This is a more advanced API that should be used only if you know what you're doing.

Can I ask you why do you need to access CallInvokerHolderImpl.
What's your use case?
What does the installNativeJsi method does?

For the time being, you can overcome this build failure by annotating your class with:
import com.facebook.react.common.annotations.FrameworkAPI

@OptIn(FrameworkAPI::class)
class OPSQLiteBridge {
  ...
}

The methods on op-sqlite run in a different thread, therefore to return the data to JS without crashing the JS context I need to schedule the callback. I have used this extensively on other client modules whenever there needs to be async work that does not block the JS thread, e.g. authentication/fingerprint callbacks.

After adding the annotation I'm getting the following exception when trying to access the callinvokerholderimpl:
2024-02-28 08:24:19.849  2370-2475  OPSQLite                com.op.sqlite.example                E  Install exception: java.lang.UnsupportedOperationException: There is no Catalyst instance in bridgeless mode.

Trying iOS again, and added a console.log on the paths being expanded, indeed it seems it looks in node_modules and then in a prepended path

I guess this is correct on a normal installation but on a library setup where the dependency is not explicitly declared on the package.json but instead linked via react-native.config.js and via tsconfig paths... it will never find the source

@ospfranco sorry, I forgot to post an update here. iOS has been fixed https://github.com/facebook/react-native/pull/43210
The fix will be available in the next RC.

Thanks in the mean time I will patch it

@ospfranco pod install?

Tried a nuke of pods and caches and still getting the same error.
Getting this error because of importing the RCTBridge+Private header apparently? But without it I don't have access to the runtime and callInvoker types.

After adding the annotation I'm getting the following exception when trying to access the callinvokerholderimpl:
2024-02-28 08:24:19.849  2370-2475  OPSQLite                com.op.sqlite.example                E  Install exception: java.lang.UnsupportedOperationException: There is no Catalyst instance in bridgeless mode.

What would be the correct way to access the callinvoker on bridgeless?

hi!!! for android, in new architecture, we will be recommending to use RuntimeExecutor:
https://github.com/facebook/react-native/blob/c3b0a8f1626939cf5c7b3864a5acf9d3dad26fb3/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactContext.java#L219
it should be a simple drop in replacement and it is forward compatible so it will work in bridge as well

Tried a nuke of pods and caches and still getting the same error.
Getting this error because of importing the RCTBridge+Private header apparently? But without it I don't have access to the runtime and callInvoker types.

Edit: ah right bridgeless... so what's the correct way to get access to the jsiRuntime?

for iOS, accessing the raw runtime should be backwards compatible. this build issue seems like something else. dmitry is currently looking at it

@philIip is there a RuntimeExecutor for iOS as well? this is a shared module between iOS and Android, setting up conditional compilation with two different classes will be really painful.

where am I supposed to access the Runtime executor from? It would be massively helpful to get snippets for all this stuff, since I'm not familiar with the internal classes.
I meant that if the callinvoker is only available on ios but not on android then my shared C++ code doesn't work

hi @ospfranco, i have posted https://github.com/reactwg/react-native-new-architecture/discussions/160! please let me know if any of these options do not fit with your use case

ok, that helps a bit, but it seems the runtimeExecutor is platform-dependent? At least from my quick reading through that discussion it seems to take a obj-c closure. My modules are a shared C++ codebase, which means I have shared calls that use the jsCallInvoker.
Here is an example of a JSI Host Function doing work in another C++ thread, then scheduling work via the call invoker to return data to JS.
https://github.com/OP-Engineering/op-sqlite/blob/main/cpp/bindings.cpp#L318
It is also not clear to me at all how to keep compatibility with the old modules as it seems I will either need a lot of conditional compilation to receive the correct objects.

got it!!! thanks for the code pointer! i think in this case, we want to eventually use a C++ TurboModule, but currently we don't have support for cross platform support for an iOS native module to embed a C++ TurboModule for example
as you saw in the post, we build the runtime pointer into the backwards compat layers but not the call invoker. i think that this helps inform that we probably should build the call invoker into those layers as well. we will prioritize that from our end, it's currently in flight!

I already gave my feedback in the original discussion. A C++ turbo module doesn't quite work for some use cases since, I not only need C++ but sometimes I also need access to functions or variables that are only available to native apis (database folder location, get secure bytes via keychain/hardware chip, etc.). So, I think it is still valuable to be able to access all the C++ entities from a native module so pure JSI bindings can be created. Thanks for taking the time to take a look though, and all the help!

I already gave my feedback in the original discussion. A C++ turbo module doesn't quite work for some use cases since, I not only need C++ but sometimes I also need access to functions or variables that are only available to native apis (database folder location, get secure bytes via keychain/hardware chip, etc.). So, I think it is still valuable to be able to access all the C++ entities from a native module so pure JSI bindings can be created. Thanks for taking the time to take a look though, and all the help!

got it! let me move this discussion over there.

What do you mean with DI?
 Mapping and binding functions via funny strings and conversion between java classes and native C++ is cumbersome, but to get to the point, yeah, basically like that.
At least for my use case that's all I need. Another thing was the C++ module initialization. I haven't checked since the original bridgless announcement, but it was cumbersome, requiring user changes. I think callstack manager to migrate some of their libraries and also patch them in a way that requires no user changes to appdelegate but this is not documented anywhere.

What's the status of this issue @ospfranco ? Is there anything we need to pick in an upcoming RC of 0.74?

I still haven't received any info on how to solve the issues. Basically still at square 0.
iOS
I need more exact snippets on how to retrieve the call invoker for iOS (or in its absence the runtimeExecutor, which I didn't find any way to retrieve it from existing exposed classes)
https://github.com/facebook/react-native/issues/43204#issuecomment-1972586329
The problem with the runtime executor is that then my Android and iOS codebases will diverge. So I need to use one or the other.
Android
I still did not manage to get a hold of the call invoker from the Bridgeless Android module.
https://github.com/facebook/react-native/issues/43204#issuecomment-1968380955

For Android https://github.com/facebook/react-native/pull/43400 should fix accessing the JSCallInvoker in Bridgeless.

sorry for the delay! was out sick last week
this should be resolved when https://github.com/facebook/react-native/pull/43400 and https://github.com/facebook/react-native/pull/43314 get picked into 0.74!

For Android #43400 should fix accessing the JSCallInvoker in Bridgeless.

https://github.com/facebook/react-native/pull/43424 as well to fix failing test.

Hey, I have a little bit of free time. What is the status of those patches? Are they merged, can I test a new version of 0.74 this weekend?

Hey, I have a little bit of free time. What is the status of those patches? Are they merged, can I test a new version of 0.74 this weekend?

We'll be doing RC4 with those 4 patches on Monday 👍

I just tried rc.4 and getting the following problems.
on iOS I get the following casting error:

On Android it now compiles but it crashes on runtime when I try call the getInvoker method

Any suggestions?

@ospfranco, i have https://github.com/facebook/react-native/pull/43600 to help unblock iOS. can you try patching it and let me know what happens? i tried pushing a PR against your repo but got blocked by lefthook lol

For Android, I couldn't repro due to gradle issues but from cursory look the crash is happening since the newly added catalyst instance for Bridgeless is returning the interface CallInvokerHolder versus what OPSQLiteBridge expects is an implementing class of this interface i.e. CallInvokerHolderImpl  which is why the cast is failing.
Also, since CallInvokerHolderImpl is a FrameworksAPI: This API is provided only for React Native frameworks and not intended for general users, we recommend directly using the interface. LMK if this works.

@philIip thanks, I will give it a try when I can, this weekend.
@arushikesarwani94 you mean just casting to CallInvokerHolder? I will give it a try thanks

Just tried running it without the cast to Impl and I get this error.

Would it be possible for you guys to run this yourselves? The branch is there and you don't need to do anything except a yarn install and then run:
https://github.com/OP-Engineering/op-sqlite/tree/bridgeless

@philIip I just tried to patch with your changes and I'm getting this error(s):

I think my changes are correct, so don't know why am I getting that missing header error

I changed both node_modules in the example project and parent folder

@ospfranco I was able to repro the insta crash of OPSQLiteExample app locally just not with the same error as yours and rather as a SEG Fault as discussed in chat.

Both the BridgelessCatalystInstance & CallInvokerHolder are getting retrieved correctly upon debugging without needing any cast.
I was able to get past the app crash by updating the cpp-adapter.cpp to exactly the one on main

Since you mentioned the change was not necessary for bridgeless as long as both the runtime and the invoker are there is all good, I think this should unblock you.

Thanks for the help @arushikesarwani94! The app is now building correctly! 🎉
Waiting for instructions regarding iOS

Thanks for the help @arushikesarwani94! The app is now building correctly! 🎉
Waiting for instructions regarding iOS

@ospfranco, can you patch https://github.com/facebook/react-native/pull/43600? that should resolve your build error.

The screenshot I posted with the import error is after patching the changes of #43600. Let me try it again, maybe I did something wrong.

I re-applied the same changes again and I'm getting this:

I tried again, this time I created the patch using patch-package, you can see it here:
https://github.com/OP-Engineering/op-sqlite/pull/60/files#diff-b5757661684d7172a2bb6e4ced36e58fed8300e282b867167bf5ae03158f7191
Now getting this error:

I tried generating a patch-package with the changes but it failed to apply. Maybe some of these files are created on the install step? In any case, manually applying the changes to node_modules/react-native doesn't seem to work for me
Here is the patch file in case you want to check if I did it correctly:
https://github.com/OP-Engineering/op-sqlite/pull/60/files#diff-0c24a4c1e88df14537749bb451dbc3164819872c11700d39fa16f1abe54b1fdc

can you show me the stack trace that's pulling in RCTBridge+Inspector? you should be able to find it in the builds tab here, which should open the build logs:

This is all I can find:

oooo got it, seems like a classic merge conflict lol.... PageTarget got renamed to HostTarget after 0.74 cut. i have to push a change directly to 0.74 that renames HostTarget back to PageTarget in RCTBridge (Inspector) :P

It seems now the error on node modules is gone, but still getting missing property when casting to CxxBridge

do you have the new architecture flag turned on? IIRC in your code you have the import to RCTBridge+Private gated by a compiler flag. you can remove that.

You are absolutely right, sorry, hard to keep track as I'm busy with paid work. Uncommenting it fixes it in the old arch, but I'm getting a undefined module on the new arch. Checking it now.

No type error, but on runtime the cxxbridge is not there

You are absolutely right, sorry, hard to keep track as I'm busy with paid work. Uncommenting it fixes it in the old arch, but I'm getting a undefined module on the new arch. Checking it now.

no need to apologize! just figuring out one thing at a time

No type error, but on runtime the cxxbridge is not there

cool i was just able to repro! :D let me get back to you!

@ospfranco, I can explain why the cxxBridge is nil here!
Here's the goal: When people migrate to our stable apis (for turbomodules and fabric at least), we want them to cut ties with the backwards compatibility layers (i.e: this bridge proxy). That way, people only have to migrate once for the new architecture (i.e: when they adopt the stable apis). And after most people migrate migrate, as the framework authors, we can just sunset the backwards compatibility layers.
The solution here would be this: just undo the turbomodule migration work you did for this module that accesses the bridge. That way, it'll just become a legacy module going through the interop layer, and it'll have access to the bridge. Once we figure out the stable api for give modules access to the runtime (likely c++ turbomodule), you'll just have to re-write this completely. I hope that's fine!

I will give it a try, but this is not a normal turbo module. I just need to access the runtime so I can later bind c++ functions directly

@RSNara i also have to say: i won't migrate this module to c++ turbo module, because I need access to native apis, in this case getting the library folder path and on Android getting the databases path.
Same with other libraries I have, sometimes I call native apis from c++
Would this still work with the approach you suggested?

We've picked the requested change (https://github.com/reactwg/react-native-releases/issues/183) into the 0.74-stable branch this morning. It will be part of 0.74 RC6, due out later today.

@philIip ok, I got it, it is cumbersome and I'm short on time, but I will try to get it soon a report how it goes. Thanks!

Ok, reverting to an old module works in all archs. Will close for now, and I guess we can have a discussion when old modules will be completely retired. Like I said, I don't really care about having to migrate to a C++ Turbo Module, as long as I get to call the native functions that's all I need.


HeNeAr line solutions using line list hash table?

can we do a pattern fit to calibration lamp spectra using line lists?
like a 1-d, kindergarten version of astrometry.net solution ala @davidwhogg & @dstndstn ?

Yes for sure. I think you take sets of three nearby lines and compute the separation ratio (which varies from 0 to 1 and use that as your geometric hash.

This was thought up ~20years ago for IRAF. It was a good idea then, and still is now!

I started cracking at this, but its just a mess...
The beginnings are in the file linehash.py
@migueldvb you might be interested


Travis build fails

After 633fa8ea029f76c0a77a7ebebda2d9d2721f8dfd the travis build fails. The libgdal-dev package is not installed. It seems this https://github.com/travis-ci/travis-ci/issues/3741 wasn't resolved after all. Maybe it is just some hickup at travis and this change wasn't deployed yet. Lets wait a bit and see whether it fixes itself.

Fixed itself.


Copy/paste functionality

On the page I'm testing now, a new URL is generated and I need to select, copy the URL, open a new tab and continue to do more actions and assertions. My question is how can I make this thing work?                  I clicked and selected the new text (URL), but I'm stuck at this step.
I'm aware that TestCafe can't handle multiple tabs. It's fine if the url is opened in the same tab.

Hello Mihai,
TestCafe cannot automate a browser's built-in behavior including the Copy & Paste functionality. We expect that this functionality works correctly as it is tested by browsers' developers. In addition, TestCafe does not support browser tabs: it is possible to run tests in a single window only.
Nevertheless, would you please describe your scenario in greater detail? Why do you wish to copy-paste URL instead of opening this URL directly?

Hello, here is a screen shot of the page I'm automating:
https://www.screencast.com/t/GsmiNpwK0k6

@Mihai0909,
In your scenario, I suggest you implement the following solution:

Access the "Permalink" using Selector;
Take the editor's innerText;
Insert the obtained text into the navigatoTo(innerText) function.

Thank you very much for your response. Works perfect now. We are going to migrate from Selenium Webdriver with Java to TestCafe. Keep up the good work and thanks for the prompt answers.


Log to stdout

I have this change in PR #109...and a bit more (1ab49116655f512aab0695b25eea6ecadf4f823a).  Do you need this change in sooner for something you're working on?


add p3 instance to instanceType

Not sure if this file is manually maintained or auto-generated

Not sure if this file is manually maintained or auto-generated

Currently manually maintained - so thanks for the PR!  But hoping to move to something auto-generated soon so that this and other enums are kept up to date.


email text in multiple languages

Is there any mechanism to have multiple email text files with different languages?
If not do you have any recommendation on how to implement this, I'm happy to volunteer and create a pull request.

I think the best way would be to subcless the view yourself, and override the get_plain_body_template_name, get_html_body_template_name and get_subject_template_name methods to check the current language (with request.LANGUAGE_CODE or something like that). Then you could name your templates something like 'registration_french.html', 'registration_english.html' and return the appropriate one.
I could be wrong though!

Thank you @zaubererty four your question.
As @mcgeeco said, you have to override mentioned methods and create your custom view. Please check https://github.com/sunscrapers/djoser#customization


OLS-155: Remove commented code

Description
There is a bunch of commented code. Seems like left-overs from developing.
Type of change

[X] Refactor

/lgtm


Update MNIST demo to use latest NPM packages.

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Update grandle plugin to 3.0.1

Closes #851
Changelogs:
Gradle plugin: https://developer.android.com/studio/releases/gradle-plugin.html
We have to update these dependencies too:
Support libs: https://developer.android.com/topic/libraries/support-library/revisions.html
Robolectric: https://github.com/robolectric/robolectric/releases
LeakCanary https://github.com/square/leakcanary/blob/master/CHANGELOG.md

Codecov Report

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

@@            Coverage Diff            @@
##             master     #862   +/-   ##
=========================================
  Coverage     97.66%   97.66%
  Complexity      829      829
=========================================
  Files            98       98
  Lines          2744     2744
  Branches        302      302
=========================================
  Hits           2680     2680
  Misses           35       35
  Partials         29       29

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 675a8f6...7592d26. Read the comment docs.


Find In Project: Add tooltips/help links for Include/Exclude file inputs - what is the syntax here?

It is hard to know what the "Include files" and "exclude files" syntax is. It is NOT obvious that it is a comma separated list of.. regex? I haven't seen that before. Please a tooltip or help link or anything here that helps me learn that :)
Thanks!

It's in the docs here: https://code.visualstudio.com/docs/editor/codebasics#_advanced-search-options

I understand it's in the docs - this feature request was to add a tooltip or a help link into the UI.

There is another issue to add a dropdown/button to get some nice pre-filled examples but I don't know when I'll get to that. I think we could add some basic placeholder to the inputs the same as in the Problems pane.


Amplify Push fails with: Parameters: [access, type] must have values

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
18.7.0
Amplify CLI Version
10.8.1
What operating system are you using?
Windows
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Describe the bug
I added predictions to the amplify local backend using amplify cli,
amplify predictions add
√ Please select from one of the categories below · Identify
√ What would you like to identify? · Identify Labels
√ Provide a friendly name for your resource · identifyLabelsbc2c037f
√ Would you like to use the default configuration · Default Configuration
√ Who should have access? · Auth users only

I didn't edit the local backend files and when I tried to push:

🛑 The following resources failed to deploy:
Resource Name: predictionsidentifyLabelsbc2c037f (AWS::CloudFormation::Stack)
Event Type: create
Reason: Parameters: [access, type] must have values

🛑 Resource is not in the state stackUpdateComplete
Name: predictionsidentifyLabelsbc2c037f (AWS::CloudFormation::Stack), Event Type: create, Reason: Parameters: [access, type] must have values

Expected behavior
To push the backend successfully.
Reproduction steps

Add predictions
Push

Project Identifier
e5c65903e25dce83b476572ec4ab9dc3
Log output

# Put your logs below this line

🛑 The following resources failed to deploy:
Resource Name: predictionsidentifyLabelsbc2c037f (AWS::CloudFormation::Stack)
Event Type: create
Reason: Parameters: [access, type] must have values

🛑 Resource is not in the state stackUpdateComplete
Name: predictionsidentifyLabelsbc2c037f (AWS::CloudFormation::Stack), Event Type: create, Reason: Parameters: [access, type] must have values

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

And now I have started getting the problem where it keeps asking me to "Select the authentication method you want to use:" whenever i want to do anything in amplify CLI and when I select "AWS profile" it gives me an error:
amplify push
? Select the authentication method you want to use: AWS profile
? Select the authentication method you want to use: AWS profile
× There was an error pulling the backend environment staging.
🛑 Missing credentials in config, if using AWS_CONFIG_FILE, set AWS_SDK_LOAD_CONFIG=1

I was able to reproduce the issue. marking as bug.
1.
amplify predictions add
√ Please select from one of the categories below · Identify
√ What would you like to identify? · Identify Labels
√ Provide a friendly name for your resource ·
√ Would you like to use the default configuration · Default Configuration
√ Who should have access? · Auth users only

amplify push

Marking as a duplicate of https://github.com/aws-amplify/amplify-cli/issues/12159 but keeping this issue open as it is a slightly different workflow and different parameters

The temporary fix for this is to add the access and type properties to the parameters.json files manually.
"access": "auth",
"type": "ALL"


Add sort boolean benchmark

Which issue does this PR close?
Re #447
Rationale for this change
While reviewing #448  I wanted to measure performance change
What changes are included in this PR?
Benchmark for sorting boolean arrays

was wondering what if you increase it to say 2^16?

Whoops -- not sure why I closed (rather than merging) this branch


Update README

The RepoRater badge in the README.md file is not showing.
Screenshot

Thank you for raising the issue

I would like to work on this

sure, assigned to you @Bashamega 👍  - this was working before, so not sure why it broke, I am super curious though

@eddiejaoude For some reason this issue wasn't closed. Can you close it?

oh strange, thank you 👍


compose operator

done with compose. next will do matrix free, and name change (diffeq* -> sciml) in next PR

tests passing


Need higher resolution

What do I need to do to generate a larger and different size resolution?
from 256x256
To 768x576?

Hello. Sorry for the late reply. I don't actively develop this project anymore.
What you would need is checkpoints of the KP detector and the generator trained on images with said resolutions, along with the accompanying YAML configs. I do not have such checkpoints, and I don't know if they're available openly.


Update network-plugins.md to reflect removed dockershim flags in 1.24.

The network plugins page need to be updated to reflect the removed flags that happened as part of 1.24 dockershim removal.

I am curious about this. We should ideally offer a path forward for folks migrating off of configuring CNI via the Kubelet systemd unit.
As far as I can tell the entire archlinux community's kubernetes systemd units broke when we released 1.24.
I don't think we are responsible for contributin to the arch docs, however I do think having a source of truth here would be important.
Primarily:

Where should we be configuring CNI now that the flags are removed
What should the values be for the suggested systemd files (I have a list I can provide)

/assign @tomkivlin

/triage duplicate
@tomkivlin are you happy to close this as a duplicate?

/triage duplicate
@tomkivlin are you happy to close this as a duplicate?

Yep, thanks @sftim.


change Xpand.EmailTemplateEngine to get the System.Web.Razor assembly…

… via Nuget.  Required for machines that don't have older ASP.NET MVC versions installed.  Also updated to Razor 2, which is the last version to support .NET 4.0

merged with <IP_ADDRESS>

Sorry Tolis, this needs to be backed out of <IP_ADDRESS> - the upgrade to Razor v2 has problems I don't have time to investigate right now.  The "get Razor from Nuget" part is still a nice fix though, so I'll submit another PR with just that, for Razor v1.

what kind of problems?

Errors when it tries to compile a template, exactly like this.  I couldn't find any of the extensibility points people referred to as a fix (any kind of SetTemplateService method).

Or wait - I think we just need to add a WriteAttribute method to EmailTemplate?

Yeah the thread is not so clear why don't you ask back?
On Mon, Oct 12, 2015 at 12:48 AM, Jim Horvath<EMAIL_ADDRESS>wrote:

Or wait - I think we just need to add a WriteAttribute method to
EmailTemplate?
—
Reply to this email directly or view it on GitHub
https://github.com/expand/eXpand/pull/101#issuecomment-147227416.

--
Apostolis Bekiaris
Frameworks Technical Evangelist @ DevExpress
Blog @ http://apobekiaris.blogspot.com
Skype/twitter: tolisss

I'll figure something out with it, but I at least wanted you to know that this was broken as-is in case you were about to release the next version.

I understand that the issue is closed without as you say any pointers to a
fix. However I do not thing is right to downgrade a package release back on
2013 and with such huge user base because of that issue. Maybe other people
may have problems with such a downgrade?
On Sun, Oct 11, 2015 at 9:43 PM, Jim Horvath<EMAIL_ADDRESS>wrote:

I'll figure something out with it, but I at least wanted you to know that
this was broken as-is in case you were about to release the next version.
—
Reply to this email directly or view it on GitHub
https://github.com/expand/eXpand/pull/101#issuecomment-147234268.

--
Apostolis Bekiaris
Frameworks Technical Evangelist @ DevExpress
Blog @ http://apobekiaris.blogspot.com
Skype/twitter: tolisss

Oh, I didn't think it was a downgrade?  I thought eXpand.EmailTemplateEngine has been using Razor v1 all this time.

hmm this case is tricky no the server has v2 installed in the GAC. This
happen because the scripts register the Razorv2 from 3rdpartyassemblies to
the GAC by mistake. So the build server and many users by mistake have
razorv2 in the gac. See also
http://www.expandframework.com/forum/5-installation-configuration-a-updates/4579-nuget-package-for-email-module-issue.html#8981
On Sun, Oct 11, 2015 at 9:59 PM, Jim Horvath<EMAIL_ADDRESS>wrote:

Oh, I didn't think it was a downgrade? I thought
eXpand.EmailTemplateEngine has been using Razor v1 all this time.
—
Reply to this email directly or view it on GitHub
https://github.com/expand/eXpand/pull/101#issuecomment-147235310.

--
Apostolis Bekiaris
Frameworks Technical Evangelist @ DevExpress
Blog @ http://apobekiaris.blogspot.com
Skype/twitter: tolisss

Hmm.  From the RazorEngine thread I linked it sounds like a new part of v2.  If the eXpand template has never had a WriteAttribute method it seems like this may never have worked with v2?

I believe we should take advantage of the
https://github.com/Antaris/RazorEngine and obsolete the current
implementation. What do you think?
On Sun, Oct 11, 2015 at 10:23 PM, Jim Horvath<EMAIL_ADDRESS>wrote:

Hmm. From the RazorEngine thread I linked it sounds like a new part of v2.
If the eXpand template has never had a WriteAttribute method it seems like
this may never have worked with v2?
—
Reply to this email directly or view it on GitHub
https://github.com/expand/eXpand/pull/101#issuecomment-147237571.

--
Apostolis Bekiaris
Frameworks Technical Evangelist @ DevExpress
Blog @ http://apobekiaris.blogspot.com
Skype/twitter: tolisss

I've thought about it.  RazorEngine does support .net 4.0.  And the only dependency is the Razor v2 package I started importing with this PR.  I think it's a good idea.


Client side Request Logger doesn't log GET requests when logBody is set

#1849 fixed this for the server side but the issue still remains on the client side.

I'm using the JdkHttpClient. This is happening because the body stream is not consumed in the convertRequest method.
val publisher = FlowAdapters.toFlowPublisher(
  StreamUnicastPublisher(req.body.chunks.map(_.toByteBuffer))
)
if (req.isChunked)
  BodyPublishers.fromPublisher(publisher)
else
  req.contentLength
    .fold(BodyPublishers.noBody)(BodyPublishers.fromPublisher(publisher, _))

I can see two ways on how to fix this. Either, require all clients to consume the request body or simply add a check inside the request logger:
Client { req =>
  if (logBody && req.contentLength.isDefined)
    Resource.suspend { ... }
  else
    Resource.liftF(Logger
      .logMessage[F, Request[F]](req)(logHeaders, logBody, redactHeadersWhen)(log(_))
    ) *> client.run(req)

The logger has been touched several times since this was reported, so unconfirmed, but cautiously flagging it as still a bug.

Just confirmed this is still an issue with http4s-jdk-http-client version 0.7.0 and 0.23.11 of http4s

@dzanot thanks for confirming. This seems straightforward to fix, would appreciate PR :)

Sure thing: https://github.com/http4s/http4s/pull/6535/files


Tidy up youtube-dl progress

The current state of our youtube-dl progress output can be described as apocalyptic at best. For example -

I propose we either build our own logic to deal with the mess or we use something that already exists out there, like ytdl-core. Example -
const readline = require('readline');
const fs = require('fs');
const ytdl = require('ytdl-core');
// TypeScript: import ytdl from 'ytdl-core'; with --esModuleInterop
// TypeScript: import * as ytdl from 'ytdl-core'; with --allowSyntheticDefaultImports
// TypeScript: import ytdl = require('ytdl-core'); with neither of the above

let video = ytdl('https://www.youtube.com/watch?v=UnSILVWDKL8');

let starttime;
video.pipe(fs.createWriteStream('video.mp4'));

video.once('response', () => {
  starttime = Date.now();
});

video.on('progress', (chunkLength, downloaded, total) => {
  const percent = downloaded / total;
  const downloadedMinutes = (Date.now() - starttime) / 1000 / 60;
  readline.cursorTo(process.stdout, 0);
  process.stdout.write(`${(percent * 100).toFixed(2)}% downloaded `);
  process.stdout.write(`(${(downloaded / 1024 / 1024).toFixed(2)}MB of ${(total / 1024 / 1024).toFixed(2)}MB)\n`);
  process.stdout.write(`running for: ${downloadedMinutes.toFixed(2)}minutes`);
  process.stdout.write(`, estimated time left: ${(downloadedMinutes / percent - downloadedMinutes).toFixed(2)}minutes `);
  readline.moveCursor(process.stdout, 0, -1);
});

Result -

Moreover, terminal-kit (which we already depend on) has a progress bar, would make for a really neat UI -

Passing --verbose would change this neat output to raw youtube-dl output when debugging is necessary.

ytdl-core is a dead end since it's not a youtube-dl wrapper, it's a new implementation that does DOM parsing and is youtube specific. What we need is a youtube-dl output parser.

i tried to make a progress bar a few days ago but i gave up for 2 reasons:
1
are we switching to aria? if yes, i think there s no need to waste time on this
2
youtube-dl has no output, let me explain
my idea was to grab youtube-dl stdout and make a custom progress bar by parsing every stdout line in real time.
the problem here is that i found that youtube-dl does nothing, all the mess you see is ffmpeg.
so you spawn youtube-dl -> youtube-dl prepares ffmpeg -> youtube-dl launches ffmpeg.
afaik, when you spawn a child in node you can get its stdout, but if your child spawns a new child, you will have no control over it.
what we need here is ffmpeg output.
i tried many node libraries but none of them was able to show the ffmpeg output, i don't know if there is an hack or a hidden lib that can do this.
the only solution i see now is that youtube-dl should implement a progress bar. (unless there is an arg that i m not aware of to get some useful output)

I'm trying out a few things of my own, will report back progress. I'll have a closer look ar aria2c as well.

There's a - - show-json in youtube-dl or something like that which i believe generates something sanely parsable. Didn't look too much into it yet.

If you really want to implement aria2c I have some experience working with it, so if you need help I can provide some

@kylon raised a valid point. Why even take a dependency on youtube-dl to begin with.
This seems to work just as fine -
`ffmpeg -headers "Authorization: Bearer ${session.AccessToken}" -i "${video.playbackUrl}" video.mp4`;

..which now opens up a new can of worms:
https://www.npmjs.com/search?q=ffmpeg progress

There are only 3 packages in the page that you linked that were updated in the last year, only 2 in the last 6 months
We should pick one and focus all our collective efforta towards that one

So i replaced our youtube-dl command logic with fluent-ffmpeg.

the error says unknown option
this is the syntax in that issue
command.inputOption('-headers', 'Header1: value\r\nHeader2: value...')
have you tried to split -headers and its value.
i had the same problem with node spawn syntax

Yes, thank you! That was it. It was driving me insane.

Ok, i have some progress -
console.info('Spawning ffmpeg with access token and HLS URL...');
        ffmpeg()
          .input(`"${video.playbackUrl}"`)
          .inputOption([
            '-headers', `"Authorization:Bearer ${session.AccessToken}"`,
          ])
          .format('mp4')
          .saveToFile(`"${outputDirectory}/${video.title}.mp4"`)
          .on('start', cmd => {
            console.log(`Spawned Ffmpeg with command: ${cmd}`);
          })
          .on('codecData', data => {
            console.log(`Input is ${data.video} with ${data.audio} audio.`);
          })
          .on('progress', progress => {
            console.log(`Processing: ${progress.percent} % done`);
          })
          .on('stderr', stderr => {
            console.log(`Stderr output: ${stderr}`);
          })
          .on('error', err => {
            console.log(`An error occurred: ${err.message}`);
          })
          .on('end', info => {
            console.log(`Processing finished: ${info}`);
          });

Now i'm errorring out on
Stderr output: "https://use2-2.api.microsoftstream.com/api/videos/manifest?api-version=1.0\&playbackurl=https://e22-blah-blah-tbna.ism/manifest(format=m3u8-aapl)": No such file or directory

most likely due to this https://github.com/fluent-ffmpeg/node-fluent-ffmpeg/issues/843

I've pushed this to tokencache to ease debugging.

 It's a fine and silly art passing arguments to ffmpeg. I miss youtube-dl :)
        ffmpeg()
          .input(video.playbackUrl)
          .inputOption([
            '-headers', `Authorization:\ Bearer\ ${session.AccessToken}`
          ])
          .format('mp4')
          .saveToFile(`${outputDirectory}${path.sep}${video.title}.mp4`)
...

Lessons learned: Don't try to pass in double quotes, whitespace in headers MUST be escaped, ALWAYS. I figured this out via ffmpeg -loglevel debug.t
Pushed to tokencache.

nice work @snobu, it's working flawlessly 🔝

So, we have
.on('progress', progress

and progress.timemark has the current HLS segment as absolute video timestamp.
In theory we need to find out how to query for video length (in minutes/seconds/whatever) either from API or ffmpeg, then draw our own progress bar from total_video_time minus current timemark.
We can do all that via frame numbers as well if it's easier. progress.frames has total video frames and progress.currentFps has fps. We can apply simple math times hope and dreams (since fps may drop) and estimate a progress bar position.
What can i say more...

This lib looks really nice, a little fat at 50KB unpacked, maybe we can find a simpler implementation since we don't need much.
https://www.npmjs.com/package/cli-progress

@snobu there's a way to implement simulate in ffmpeg (as I'm not finding any in the docs)?

I don't think so. I think we need to rewrite that feature to simply get the access token and return a few stats at the end like video title, output file and so on, without calling ffmpeg at all... or maybe we could call ffmpeg just to get AV codec info.

I'm going to look into implementing the simulation flag as best as I can tomorrow

I may have just found our solution that requires zero effort on our side -

via https://github.com/sidneys/ffmpeg-progressbar-cli.

Just pushed to tokencache, need to test this on Windows as well, looks fantastic on the Mac.

ok when we call the script we have to check what platform we are on because now it doesn't work on windows (there's a PS and a CMD one too)

this thing is not working well on my term, i think a cross platform module is better

i think we can nuke this package and use something simpler
we have the duration and ffmpeg has the current timemark
trying something like this
https://www.npmjs.com/package/progress

Ah i see, yeah ffmpeg-bar looked so promising. Got excited too soon.

working cross-platform? progressbar
tested on linux and win 10 powershell
messy code, but it is not important right now
can you try this on both win and mac? thanks
@snobu @lukaarma
tokencache branch has a broken downloadVideo function, the promise is not working so do not input more than one url
destreamer.ts.zip

Can you please push to a new branch here? Not sure what to pick as base for your .zip since we've merged a few things in since.

Also, you know we just need to make this usable for now it doesn't have to be perfect,

#57
yeah, i don't relly like refactoring, i prefer to write good code :P
anyway, this branch is not ready, it is still not fully working

Well, in our join defense, we changed like 60% of the code from dev so it's expected. Nobody is in a hurry, i agree let's write good code, no pressure.

Closing since our progress bar is now clean and stable. Thanks @kylon!


How to get ThreadList with Messages/Subject contained?

I tried calling the getThreadLists but it doesn't include some messages even the subject. Is there anyway to pull with messages included?

https://github.com/Schmavery/facebook-chat-api/blob/master/DOCS.md#getThreadHistory is probably what you're looking for.


Feature Request: skipOnFirstFailedRule

Like we already have skipOnFirstAppliedRule, It would be nice to have an option  skipOnFirstFailedRule .
For now, if there are three rules A, B and C, and if we need A, B and C to pass to evaluate a condition, then we can stop B and C being evaluated if  A fails.
We can achieve this right now, using skipOnFirstAppliedRule, but it doesn't feel elegant and natural. Having skipOnFirstFailedRule  would be nice.

Great idea! Thanks for the suggestion.
I'll plan that for next release. Keep tuned :-)
Regards

Hi,
I have released version 2.0.0 with support for the "skipOnFirstFailedRule" parameter.
Do not hesitate to try it out :-)
Regards
Mahmoud


python312Packages.frigidaire: 0.18.21 -> 0.18.23

Diff: https://github.com/bm1549/frigidaire/compare/refs/tags/0.18.21...0.18.23
Changelog: https://github.com/bm1549/frigidaire/releases/tag/0.18.23
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 339892 run on x86_64-linux 1

4 packages built:

python311Packages.frigidaire
python311Packages.frigidaire.dist
python312Packages.frigidaire
python312Packages.frigidaire.dist


build-std: Don't treat std like a "local" package.

This changes it so that build-std will not treat the std crates like a "local" package. This has the following changes:

std does not build with incremental. This generally shouldn't be needed, and incremental has various bugs with std crates.
Cargo's dep-info fingerprint tracking will not track the std crate sources, these are tracked via other means.
Cargo's .d dep-info file does not include std crate sources.
Lints are capped to "allow" for std crates, and warnings are not shown by default.

Closes https://github.com/rust-lang/wg-cargo-std-aware/issues/44
Closes https://github.com/rust-lang/wg-cargo-std-aware/issues/55

r? @Eh2406
(rust_highfive has picked a reviewer for you, use r? to override)

I heavily considered the approach suggested in https://github.com/rust-lang/wg-cargo-std-aware/issues/44#issuecomment-530866296 to add a field to Package, but that seemed to be potentially much more difficult to wire up. Packages tend to prefer to be immutable, and this would require teaching Workspace to know it is the std workspace. I'd like to avoid having is_std infect the entire codebase. The downside to not doing that is that the calls to get_profile() need to pass in an explicitly computed "is_local" value. Willing to reconsider, but I think it will end up being a little more complex.
Also, I'm totally up for using a different phrase than "local", open to suggestions!

@bors: r+
Nah this looks good to me. Thanks for this!

:pushpin: Commit b6a4b074ed924878fefb631c7d559ca47331e5eb has been approved by alexcrichton

:hourglass: Testing commit b6a4b074ed924878fefb631c7d559ca47331e5eb with merge ceef92db274acffc2313d1a987cb27552dff450f...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing ceef92db274acffc2313d1a987cb27552dff450f to master...


Rule: SA1200

Rule summary

Key
Value

Title
SA1200UsingDirectivesMustBePlacedCorrectly

CheckId
SA1200

Category
Ordering Rules

Link
https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1200.md

What is the problem
This rule have a conflict This rule have a conflict with root .editorconfig csharp_using_directive_placement = outside_namespace
And there is strong opinions on the internet about this rule - but also a fact about assembly loading issues which should be taken into consideration which should be taken into consideration
Read Scott Hanselman post
And default create new class in Visual Studio have alway used outside_namespace which means much written code over time has been followed the outside_namespace rule.
Suggestion
disable this rule in root .editorconfig with dotnet_diagnostic.SA1200.severity = none

https://github.com/atc-net/atc-coding-rules/blob/main/documentation/CodeAnalyzersRules/StyleCop/SA1200.md


Use more luminance levels and dispose of images

Fixes #19 and #20
This isn't fully tested (I couldn't find an easy way to test the output to see if banding was removed) but should fix both issues.
When you use Histogram equalization the default luminance levels are set to 256 which works for an 8 bit grayscale input. Bumping them up should help.
Let me know if this doesn't help with #20 and I'll have another look once I understand the system better and can view the output.

Thanks so much, James! I wasn't at all expecting you to do my grunt work with #19, but I appreciate it. I'll test out the luminance level change this evening - sounds promising.

No worries, happy to help 😀

Unfortunately this doesn't seem to have made any tangible difference to the banding. I suspect this may just be a artifact of the original image (which is 8 bit greyscale), combined with relatively large areas of slowly gradating colour and just not enough bit depth in the source material.
Which makes me wonder again about adding noise to mask it. I think that's the only real approach if there isn't enough source data, isn't it?

Yeah, try using error diffusion.


Fix multiple bootstrap requests

What does this PR change?

fix to error out multiple bootstrap agent registration requests

Does the PR depend on any other PRs or Issues? If yes, please list them.

No

Checklist
I confirm, that I have...

[x] Read and followed the contributing guide in CONTRIBUTING.md
[ ] Added tests for this PR
[x] Formatted the code using go fmt (if applicable)
[ ] Updated documentation on the Paralus docs site (if applicable)
[x] Updated CHANGELOG.md

closing this as have created a different pr - https://github.com/paralus/paralus/pull/178


Add a main menu

It'd be a great to have a main menu with links to a news page, a FAQ, GitHub, and the Discord server
Like https://mainroad-demo.netlify.app/

There's no need now. Since, the website doesn't have a large amount of content. There's currently only the homepage and the news page which can be found in the center.

First things like the installer menu and the search menu are focused. These items aren't that necessary, for now!

Moreover, they can be added into the sidebar or something. If they have to add these links, I don't think @cpw will want these items to get focused.

I'm talking about making them better :)!
The installer section is not proper. Check this->
The search bar just redirects you to Google.
And more...!
Yes, but these items should still be added, maybe I will make a pull request, later!

Last time I checked the website there was no link to the Discord at all which is where it appears most developer discussions are happening. These links don't necessarily need to be in a main menu but should at least be present on the website. One may expect the installer download to be on the homepage, and there's already a search bar in the sidebar.

The links have been added in https://github.com/neoforged/websites/commit/0e01c96808df4de4ee1f73107ca49b260acc79fe


Move repo

It's OK to move this repo to 6to5 organization like other 6to5 plugins.

What do you think, @sebmck?

@shinnn Sure, sorry I took so long to get back to you. I've added you to the org so you can transfer your repo over.

@sebmck No problem. sorry for my noisy ping. I'll transfer this repo soon.

Moved repo and released v1.0.0.
I'd happily follow the code-style convention of 6to5 team if specified.

@shinnn Feel free to do whatever makes you the most comfortable.

@sebmck I see :+1:


Reproducibility in another text dataset

Hi @hila-chefer,
First of all, congratulations on your paper, the proposal you present is really great!
I am trying to apply your method as an explicability mechanism to the BERT model that I am using to classify the polarization (sentiment) expressed in a set of tweets. My goal is, through your method, to obtain the parts of the text that led to the prediction made. Basically, I have a BERTForSequenceClassification model, which predicts between three target classes, I also have saved the attention arrays returned by the model using output_attentions = True.
I have reviewed the code from bert_pipeline.py, and also BERT-explainability.ipynb, but I still can't quite figure well out how to apply the code.
I was thinking of using the attentions I got from the trained model, but it seems to me that it would not be possible because in generate_LRP, you make use of: .attention.self.get_attn_gradients () and .attention.self.get_attn_cam (). Right?
In any case, what I would have to do would be to configure the pre-trained model, my tweets, outputs, and with that data and my model, pass all that to generate_LRP, then?
Could you please help me with this?

Hi @carlosabcs, thanks for your kind words and for your interest in our work!
I actually have 2 suggestions:

Assuming your code is really based only on BERTForSequenceClassification, you can just use our colab as is! i.e. simply load your own weights instead on the pretrained weights I loaded there, and use an example from your tweets instead of my free form text.
If you do need to implement from scratch, you can use our new method which has no LRP so implementation is easier, all you need is gradients and pure attention maps which is super easy to obtain (one backwards pass and you can use a backward hook for gradients).

Given the information you provided, I think option 1 is effortless to you and should work.
Do let me know if you have anymore questions :)

You're welcome :)
I'm going to try the first option and tell you later how it turned out. Thank you very much for your quick response!

Closing this issue for now. If other problems arise please reopen.

Hi @hila-chefer, it worked!!
Just one more question, how can I run this script locally? Should I only add it to the main folder of the cloned project on my computer?
Oh, and something else, I don't know if you've tried to run your notebook again, but originally it asks me to do model.to("cuda"), otherwise, an error appears.

Great to hear that @carlosabcs!
Feel free to share your results :) also, please consider citing our paper if you use our method :)
To address your questions- the script from our notebook should work if you add it to the main folder of your cloned repository.
About the error you encountered- I actually ran the notebook several times and didn’t get that error, but I’ll be sure to check it again, and fix this issue.

Yeah, of course, I'll cite your paper!
I have another question, is there a way I could use this method with BERTModel?
I mean, I have another dataset for Fake News detection, where I use the BERT representations obtained from BERTModel, not BERTForSequenceClassification because I combine the representations with topological features of the diffusion network of the news, and then use all the combined vector as a feature vector for classification (where I use traditional classification algorithms: RF, LR, SVM, etc). This is our paper (I'm sorry it's not in English 😢). So I would like to use the attention weights obtained from your method to provide a visualization about the parts of each new that got more attention of the BERTModel.

@carlosabcs the answer is probably yes. Our method can provide an attention like matrix that will reflect the impact of each token on each of the other tokens. The structure of your model isn’t really clear to me, but assuming you concatenate another model to BERT, you need to implement LRP for the entire network for the method used in this paper. If you wish to avoid it, you can use our new method with the link I mentioned, in that case you only need to propagate gradients which is way easier.
In any case, a good first step would be to understand if the output of our explainability method suits your needs for this scenario specifically.


boot2docker up really slow

In the Video that shows the Mac OSX installation boot2docker up just taks a few seconds, for me it takes quite a long time and is slower than booting an Ubuntu VM:

I tried installing it via homebrew as well as via this installer and both seem to have equal times. Any reasons why this could be so slow?

I also have the problem with really slow startup. I installed boot2docker on OS X 10.8.5 on a 2012 Macbook Pro Retina and am seemingly waiting forever on first boot.
Waiting for VM and Docker daemon to start...
..........................................................................ooooo

I tried with both version 1.3.2 and 1.4.1 of boot2docker but have been giving up after 20 minutes or so of waiting.

@VolkerH I would suggest using the vurtual box GUI to stop the vm, and then start it using the GUI - so that you can see the bootup output. This may tell you more.

@theRemix  Completely removing the whole VM worked for me as well.

I am having the same issue. Took 15-30 seconds to launch for the first time (on 2014 Mac with loads of hardware). Subsequent launches are crawling. I initially got boot2docker from brew cask. I tried doing the image removal and reinitializing as suggested, but continued to see the same thing. I removed boot2docker from brew and then downloaded the package and tried the installer. Just ran the boot2docker from my Applications and it is still taking a long time (running on 5 minutes now).

Waiting for VM and Docker daemon to start...
..........................................................................oooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo
Started.
Trying to get Docker socket one more time
Error requesting socket: exit status 255
Auto detection of the VM's Docker socket failed.
Please run boot2docker -v up to diagnose.
real	1020m13.457s
user	0m0.290s
sys	0m0.324s

boot2docker -v init
Boot2Docker-cli version: v1.6.2
Git commit: cb2c3bc
2015/05/21 09:34:02 executing: VBoxManage showvminfo boot2docker-vm --machinereadable
2015/05/21 09:34:02 executing: VBoxManage showvminfo boot2docker-vm --machinereadable
2015/05/21 09:34:02 executing: VBoxManage list vms
2015/05/21 09:34:02 executing: VBoxManage createvm --name boot2docker-vm --register
Virtual machine 'boot2docker-vm' is created and registered.

No one really says what the desired startup time should be. I know this might vary depending on hardware but it would still be nice to get and idea if a 30s startup is considered fast or slow. I personally think it's quite slow for a Linux box. My Windows 10 VM boots faster than that.
I have also noticed that startup times are very similar on my 2010 Macbook Pro (SSD) compared to a quad core with raid-0 SSDs.

Sorry, this repository is long-since deprecated in favor of Docker Toolbox (whose usage is now also discouraged in favor of Docker for Windows and Docker for Mac).


chore(client/executor): remove unused error InvalidCode and improve docs

@yjhmelody the CI is not happy.

@bkchr It's happy now ?

@bkchr  Can we merge it ?


Singulo drink - speed up animation

Spin
Media
Old

New

[X] I have added screenshots/videos to this PR showcasing its changes ingame

Do any of the other drinks use delays.

like 30 so

like 30 do

Is some of them are slow also?

If they were ported in the same pr it's very likely.