Error building main - the commit on deps-deply does not exist.

Hi,
I noticed dialog depends on a deps-deploy commit hash that does not exist.
I tried current main.
./bin/build
Error building classpath. Commit not found for slipset/deps-deploy in repo https://github.com/greglook/deps-deploy.git at a138ef59c1239336034c69624458f5b100ac835b

Using commit  "b87c9299761762984bd54a285ca8fa0aac81809f" fixes the issue.
One thing I noticed is that deps-deploy uses slf4j 2.x

I believe dialog can me moved back to slipset deps-deploy since the commit was merged.
https://github.com/slipset/deps-deploy/commits/master
@greglook : should I do a PR ?
;; TODO: change this back to the main repo once https://github.com/slipset/deps-deploy/pull/53 is merged


Android INTERNAL_ERROR code: 8

Hello. Help me please. I've implemented your module: on android 4.4.2 it works perfectly, but if I try on 5.0.1 I got GoogleSigninError INTERNAL_ERROR code: 8.
RN v0.23
RN-google-signin: v0.4.3
I checked everythin from #35 issue, maybe you found correct solution for that, I hope.
Thanks

+1
RN v.20
RN-google-signin: v0.4.0

I had this issue and fixed it by updating the outdated installation of Google play services in my device. You can use this app to check the version installed in your device https://play.google.com/store/apps/details?id=com.weberdo.apps.serviceinfo&hl=en

please try again with version 0.5.1 of the plugin. A lot has changed for android and you should definitely not see this message even on devices without play services.
check the readme / release note for more information. I close this issue but don't hesitate to ping me if something is not right.


Wrong return value for ListView_DeleteColumn

This page incorrectly indicates that ListView_DeleteColumn has no return value.
https://docs.microsoft.com/en-us/windows/desktop/api/commctrl/nf-commctrl-listview_deletecolumn
See older content, which is correct.
https://docs.microsoft.com/en-us/previous-versions/ms909702(v=msdn.10)
Return type is BOOL.  TRUE indicates success. FALSE indicates failure.
Take a survey of all these macros, this is probably not an isolated incident.

This is just one instance of a more general problem, see #1270.

Closing this one, the more general #1270 should cover it.


about the input to Suma

Hello, thanks for your code releasing!
I have some questions about the input to Suma:

Does the visualizer default to using config/default.xml and how can i change config file to another one (for example vlp16.xml)?
If I just only feed Lidar data to Suma (there is no calib.txt and other files) , when I execute Save Poses, is the pose in output file poses.txt referenced to the first frame of the Lidar data?

thanks for your interest in our work and the feedback.

Yes, I should have added it to the Readme; you call ./visualizer [<path/to/config>] [<path/to/data>], where the path to data is a directory containing the .bin files. config and path to data are optional directories.
Currently, the savePoses method doesn't consider the calibration (see [1], not sure when I did this...). So all poses are in the local reference frame of the first LiDAR.

It might be that stuff is quite fitted to KITTI (and their data format), but hopefully it will work our for you.
[1] https://github.com/jbehley/SuMa/blob/2d191f5a5c83e793ef2768611b2d5cad04943a9e/src/visualizer/VisualizerWindow.cpp#L840-L869

Okay, I got it. Thank you very much for your prompt reply!

@jbehley Hello, I have another question to ask for your advice:  whether the ego-motion compensation in the bin format of the KITTI dataset is compensated to the end of the lidar frame time?

Regarding what KITTI did for the motion compensation only the authors know. I guess they used the IMU estimates. But cannot tell for sure.


feature: Set camera IP/Port as variables

I'm wondering if there is any chance of a feature to set the camera IP/port (and, I suppose, camera type, but I don't need that personally) through variables. Currently, if I'm trying to control multiple cameras (up to 7) I need to add a Connection for each camera and separate pages of buttons for each. It would be easier to maintain if I could set up one bunch of buttons and switch cameras by changing variables.
I'd look into implementing this myself, but I don't want to bother if this idea has already been rejected for some reason.

+1
That would be incredibly convenient and save a lot of time if you have multiple identical cameras.


YARN-10500. TestDelegationTokenRenewer fails intermittently.

https://issues.apache.org/jira/browse/YARN-10500

Unclosed DelegationTokenRenewer affects testRMRestartWithExpiredToken. I could easily reproduce the failure by mvn test -Dtest=TestDelegationTokenRenewer but not by mvn test -Dtest=TestDelegationTokenRenewer#testRMRestartWithExpiredToken.
Assertion in testRMRestartWithExpiredToken must wait for token renewal.
The timeout for the waitFor in testTokenThreadTimeout is too strict for the timeout and retry interval.

I got another timeout in testTokenThreadTimeout on my local after submitting this PR. I need additional fix.

I got another timeout in testTokenThreadTimeout on my local after submitting this PR. I need additional fix.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 32s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:

0m  1s
test4tests
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
33m 53s

trunk passed

+1 :green_heart:
compile
1m  3s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
compile
0m 53s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
checkstyle
0m 43s

trunk passed

+1 :green_heart:
mvnsite
1m  0s

trunk passed

+1 :green_heart:
shadedclient
17m 15s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 44s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
0m 37s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+0 :ok:
spotbugs
1m 52s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
1m 50s

trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 51s

the patch passed

+1 :green_heart:
compile
0m 55s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javac
0m 55s

the patch passed

+1 :green_heart:
compile
0m 47s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
javac
0m 47s

the patch passed

-0 :warning:
checkstyle
0m 33s
/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 5 new + 53 unchanged - 2 fixed = 58 total (was 55)

+1 :green_heart:
mvnsite
0m 52s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
shadedclient
15m 13s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 41s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
0m 34s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
findbugs
1m 50s

the patch passed

_ Other Tests _

-1 :x:
unit
93m 45s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 30s

The patch does not generate ASF License warnings.

176m 48s

Reason
Tests

Failed junit tests
hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2619

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 2a5d46cd0350 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 8ec824f2ba4

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/1/testReport/

Max. process+thread count
895 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/1/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 32s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:

0m  1s
test4tests
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
33m 53s

trunk passed

+1 :green_heart:
compile
1m  3s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
compile
0m 53s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
checkstyle
0m 43s

trunk passed

+1 :green_heart:
mvnsite
1m  0s

trunk passed

+1 :green_heart:
shadedclient
17m 15s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 44s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
0m 37s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+0 :ok:
spotbugs
1m 52s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
1m 50s

trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 51s

the patch passed

+1 :green_heart:
compile
0m 55s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javac
0m 55s

the patch passed

+1 :green_heart:
compile
0m 47s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
javac
0m 47s

the patch passed

-0 :warning:
checkstyle
0m 33s
/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 5 new + 53 unchanged - 2 fixed = 58 total (was 55)

+1 :green_heart:
mvnsite
0m 52s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
shadedclient
15m 13s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 41s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
0m 34s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
findbugs
1m 50s

the patch passed

_ Other Tests _

-1 :x:
unit
93m 45s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 30s

The patch does not generate ASF License warnings.

176m 48s

Reason
Tests

Failed junit tests
hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2619

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 2a5d46cd0350 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 8ec824f2ba4

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/1/testReport/

Max. process+thread count
895 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/1/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

The cause of the failure of testTokenThreadTimeout was

the token is expired due to short expiration time (3000).
DelegationTokenRenewerPoolTracker did not re-submitted retrying due to thread synchronization.

I updated thread timeout and retry interval too in order to reduce the test time.

The cause of the failure of testTokenThreadTimeout was

the token is expired due to short expiration time (3000).
DelegationTokenRenewerPoolTracker did not re-submitted retrying due to thread synchronization.

I updated thread timeout and retry interval too in order to reduce the test time.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 30s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:

0m  0s
test4tests
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
33m 53s

trunk passed

+1 :green_heart:
compile
0m 59s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
compile
0m 53s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
checkstyle
0m 41s

trunk passed

+1 :green_heart:
mvnsite
0m 56s

trunk passed

+1 :green_heart:
shadedclient
16m 57s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 45s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
0m 39s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+0 :ok:
spotbugs
1m 47s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
1m 45s

trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 50s

the patch passed

+1 :green_heart:
compile
0m 51s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javac
0m 51s

the patch passed

+1 :green_heart:
compile
0m 45s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
javac
0m 45s

the patch passed

-0 :warning:
checkstyle
0m 33s
/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 6 new + 92 unchanged - 2 fixed = 98 total (was 94)

+1 :green_heart:
mvnsite
0m 49s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
shadedclient
14m 40s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 38s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
0m 34s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
findbugs
1m 49s

the patch passed

_ Other Tests _

+1 :green_heart:
unit
96m 54s

hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 38s

The patch does not generate ASF License warnings.

178m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2619

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux cbf20735d5aa 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 630f8ddd2c7

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/2/testReport/

Max. process+thread count
882 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/2/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 30s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:

0m  0s
test4tests
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
33m 53s

trunk passed

+1 :green_heart:
compile
0m 59s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
compile
0m 53s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
checkstyle
0m 41s

trunk passed

+1 :green_heart:
mvnsite
0m 56s

trunk passed

+1 :green_heart:
shadedclient
16m 57s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 45s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
0m 39s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+0 :ok:
spotbugs
1m 47s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
1m 45s

trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 50s

the patch passed

+1 :green_heart:
compile
0m 51s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javac
0m 51s

the patch passed

+1 :green_heart:
compile
0m 45s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
javac
0m 45s

the patch passed

-0 :warning:
checkstyle
0m 33s
/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 6 new + 92 unchanged - 2 fixed = 98 total (was 94)

+1 :green_heart:
mvnsite
0m 49s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
shadedclient
14m 40s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 38s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
0m 34s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
findbugs
1m 49s

the patch passed

_ Other Tests _

+1 :green_heart:
unit
96m 54s

hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 38s

The patch does not generate ASF License warnings.

178m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2619

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux cbf20735d5aa 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 630f8ddd2c7

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/2/testReport/

Max. process+thread count
882 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/2/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Thanks, @jbrennan333.  I addressed your comment and the theckstyle warnings.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 30s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:

0m  0s
test4tests
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
33m 57s

trunk passed

+1 :green_heart:
compile
1m  1s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
0m 53s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
checkstyle
0m 49s

trunk passed

+1 :green_heart:
mvnsite
0m 58s

trunk passed

+1 :green_heart:
shadedclient
15m 12s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 45s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 38s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+0 :ok:
spotbugs
1m 53s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
1m 51s

trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 50s

the patch passed

+1 :green_heart:
compile
0m 54s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
0m 54s

the patch passed

+1 :green_heart:
compile
0m 46s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
javac
0m 46s

the patch passed

+1 :green_heart:
checkstyle
0m 42s

hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 0 new + 92 unchanged - 2 fixed = 92 total (was 94)

+1 :green_heart:
mvnsite
0m 49s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
shadedclient
12m 52s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 39s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 38s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
findbugs
1m 50s

the patch passed

_ Other Tests _

+1 :green_heart:
unit
94m 19s

hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 28s

The patch does not generate ASF License warnings.

172m 47s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2619

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 4072957b621e 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 98ca6afd177

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/3/testReport/

Max. process+thread count
851 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2619/3/console

versions
git=2.25.1 maven=3.6.3 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


Create directory structure for multiple binaries

Will have to look into the cmd subdirectory structure that I have seen on occasion.

refs atc0005/elbow#233 (and multiple other projects)

From https://github.com/coreos/torus/blob/b53b701fc3af89fdf5f2999ebbc3b778e8c18ccc/Makefile#L11-L16:
WHAT := torusd torusctl torusblk

build: vendor
	for target in $(WHAT); do \
		$(BUILD_ENV_FLAGS) go build $(VERBOSE_$(V)) -o bin/$$target -ldflags "-X $(REPOPATH).Version=$(VERSION)" ./cmd/$$target; \
	done

Essentially each section would contain a similar loop to build/process each binary.


Bluetooth: Mesh: shell & proxy related fixes

The proxy fixes in particular are important since they make it possible to pass more PTS GATT Proxy test cases.

@rymanluk @michal-narajowski FYI


Update crud.php

English fixes

A new inspection was created.

The inspection completed: No new issues

Hello @olipayne
I'v run this sentence against two different grammar checks and they don't seem to complain.
From my understanding both of the forms are correct? https://www.grammar-monster.com/lessons/his_her_their_they_singular_plural.htm
Disclaimer: I am not a native english speaker!
Best,
Pedro

Hi @pxpm,
Thanks for the feedback! As a native English speaker, the use of 'his' assumes that the gender of the user is already known.
Using 'they' is also a much more socially correct gender-neutral singular pronoun, as 'they' has a better chance of being an acceptable pronoun for the reader vs. 'his' or 'her'.
For your reading pleasure: https://www.americandialect.org/2015-word-of-the-year-is-singular-they
If you accept this I'll happily grep through the rest of the documentation to ensure any other references or his/him are addressed in the same manner with a separate PR for consistency.
Cheers!

Thanks for the deep explanation.
I can't argue as I lack the required skills, from the discussion we had it seems you are correct.
If this is the correct wording, we should make sure to apply it whenever we got it wrong (it would probably be in lots of places).
Thanks for raising this issue @olipayne
Best,
Pedro

Totally, we want to be as inclusive and inoffensive as possible. Thanks @olipayne - I imagine it's like that in a lot of places, further PRs about this would be super-helpful.


New UUID generation

addresses #203
This is a breaking change for all configured homekit databases, as it will remove room/area assignments and other things configured in iOS apps
... unless your accessory type was called Object before
Changes:

Without any changes to the configuration file all accessories UUIDs will be generated by  uuid.generate(accessoryType + ":" instead of the constructor name which turned out to be unreliable in that context.  So about all UUIDs will change.
For platforms the platformType will be used, as configured in config.json at eg. "platform":"Nest"
new parameter uuid_base can be used for accessories in config.json to use a different base for UUID generation than the displayName, which might not be unique
platforms can add the same property to their accessories before returned, just as they have a name property right now.

Compiles and runs fine with my installation (node 4.1.1 on raspi wheezy). Tested together with https://github.com/KhaosT/HAP-NodeJS/pull/144
UUIDs are built on platform name ("KNX" in my test), display names or if present, on uuid_base from accessory.
I wouldn't expect any surprises (well that's sounding obvious) but if somebody could test that with node 0.12 the confidence level would be even higher.

Tested and works great. Thanks for patching this up!!

Your welcome, I still have to rename my platform to "Object" to keep my database ...
Just kidding.

Is this still going to break things?

Yes, but only for platforms you're already using that return accessories which use the "new API" style syntax.

@nfarina but we should update platforms to use this new style though right? Hence we'll lose our stuff?

Yes, this was the case before but unfortunately platforms that are upgraded to the new API will end up with different UUIDs. HomeKit won't care but it will remove those accessories from any assigned rooms, scenes, etc.

I wasn't following the other couple of threads very closely. What is this providing? Does this have to do with being able to auto-detect new accessories without restarting? (did I read that correctly somewhere?)

No this is unrelated - this fixes the auto generation of UUIDs from platforms which was previously bugged - it was prepending the string "Object" instead of the accessory constructor name.

:thumbsup: thanks!


4.6.0 distfile: contains pycache files for Python 3.11

It seems there was a bug in creating the 4.6.0  pypi distfile, it contains a pycache file for Python 3.11:
drwxr-xr-x  0 bdbaddog staff       0 Nov. 20 02:22 SCons-4.6.0/SCons/Tool/docbook/__pycache__/
-rw-r--r--  0 bdbaddog staff   37254 Juli 15 02:13 SCons-4.6.0/SCons/Tool/docbook/__pycache__/__init__.cpython-311.pyc

Please make sure this doesn't happen again. Thanks!

Hmmm, thought there was logic to avoid that. I'm only seeing a filter for scons-local though. @bdbaddog ?

@0-wiz-0 - Clearly this was not intentional, we'll do our best.

I'm not even sure how that happened at all to be clear. We'll have to figure it out. We're using the normal wheel building logic..

Hmm. Did a git clean -xfd and then rebuilt and no .pyc. I'll upload a 4.6.0.post1 package shortly.

Thank you, the new release tarball is fine!

@0-wiz-0 - great. We've updated our release procedure to do git clean -xfd going forward, so hopefully that prevents a repeat.


compair for offset when retrieving data in get All root by type method

pull request does not cover the requirement


Add "Checkpoints" like PowerShell Workflows

I have tacked on my own by creating and looking for files - but it would be handy to have it natively built in.
When a boxstarter script restarts, it starts right after the last completed checkpoint.  Helps speed things up on very long workflows.

Hmm. I have thought about this previously. I'd really prefer to keep the scripts "idempotent". I think this is simpler and less error prone in the end. It also sticks to the "pattern" most other automation tools suport.

But thanks for the suggestion and please feel free to send any more our way!


Syntax intro update

ping @pojeda just to make you aware of this. I'll probably finalise this PR tomorrow morning

hopefully this is good for a 60 minute general intro to julia syntax!


Using separate database branches for preview deployments

Build is failing because api.preview.tigrisdata.cloud does not have DB branching support yet.


update README's development instructions

should use boot verify not boot check

"verify your code's correctness and quality with boot verify"


RunOnSave causing an issue when using autosave.

I wrote this RFC on headwinds github page but after seeing the package is not well maintained I figured I would use your fork instead in the hope of getting this issue fixed. My apologies if some of the settings described here are not named the same but I'm sure you will get the idea:
RunOnSave feature is causing an issue when using vscode autosave. The issue is that while I'm typing my tailwind classes, vscode autosaves triggering headwind to reorder. As this reorder applies the cursor does not follow the original classname and I end up writing half of my classname in the center of another, corrupting that classname and leaving the classname I'm trying to declare half written and lost somewhere else in the string.
Could you add another cofiguration setting say headwind.triggerPoint where the user can choose options "runOnSave" or "runOnLineOut". The runOnLineOut would be an option whereby the reordering would happen when the user focuses away from the line or maybe when the cursor focuses away from the class value declaration string (anything within the ""). Or just move the cursor with the reordering of possible, you get the idea.
It's a very handy extension for code readability and therefore productivity. It just becomes unusable when users have vscode autosave enabled and it gets triggered every 5 seconds, corrupting their tailwind classnames.
Thanks,
Donk3yLee.

@donk3ylee That sounds very frustrating. Definitely worth fixing. I'll dig into this later this week. My approaches will generally prefer automatic detection so when people install the extension it "just works". So I'll try and detect active editing and disable reordering. My first approach will be detecting if the selection (cursor is a zero width selection) lies within the detected text range to be reordered and skip just that range. Another approach I might try is attempting to stick the cursor to its position relative to class it currently is within / on the end of as you suggested. If I can't make either of those work, I will investigate the addition of another configuration option.
Aside from those, I think that maintaining cursor position is a generally good idea and so I will make sure that when we change the length of className - such as removing duplicates or extra whitespace - we update the cursor position accordingly.
Thank you for taking the time to submit this issue and bring this to my attention!

No worries Andrew thanks for the quick reply. It's a useful plugin. I think
if you can stick the cursor to the class name your typing as it is
reordered that would be thebest fix like you said.
Cheers,
Donk3ylee
On Tue, 27 Feb 2024, 19:34 Andrew Trefethen, @.***>
wrote:

@donk3ylee https://github.com/donk3ylee That sounds very frustrating.
Definitely worth fixing. I'll dig into this later this week. My approaches
will generally prefer automatic detection so when people install the
extension it "just works". So I'll try and detect active editing and
disable reordering. My first approach will be detecting if the selection
(cursor is a zero width selection) lies within the detected text range to
be reordered and skip just that range. Another approach I might try is
attempting to stick the cursor to its position relative to class it
currently is within / on the end of as you suggested. If I can't make
either of those work, I will investigate the addition of another
configuration option.
Aside from those, I think that maintaining cursor position is a generally
good idea and so I will make sure that when we change the length of
className - such as removing duplicates or extra whitespace - we update the
cursor position accordingly.
Thank you for taking the time to submit this issue and bring this to my
attention!
—
Reply to this email directly, view it on GitHub
https://github.com/Trapfether/tailwind-raw-reorder/issues/14#issuecomment-1967458242,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATWRJY3CWQ3AV7E2QTT3VJLYVYYLPAVCNFSM6AAAAABD4QGBKKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRXGQ2TQMRUGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I have been searching for a solution for so long now, I hope the work is still going on this, and an update soon, thanks


update kube download url

What type of PR is this?
/kind bug
What this PR does / why we need it:
The kube download url has recently changed from storage.googleapis.com/kubernetes-release to dl.k8s.io, which needs to be updated, otherwise the resources cannot be downloaded properly.
Which issue(s) this PR fixes:

Fixes #736
Special notes for your reviewer:

/lgtm


contrib/twitter needs to use oauth

conrtib/twitter needs to be updated to use oauth and not try to use username / password from the obsolete Config table

I've got this working on my build. I'll clean it up and submit a patch.

Pull requested at https://github.com/Kegbot/kegbot/pull/60


🐛 Bug fixed - selection colour

Closes #3

Latest commit: Added colours for setState in React like other colour themes

Before

After


Infer ontolex:partOfSpeech from wn:partOfSpeech

Work on version 1.2 of the GWA schema is ongoing. See: https://github.com/globalwordnet/schemas/issues/58#issuecomment-928959527
It should be possible to infer the Ontolex triple from the GWA relation, but it will require rewriting - or perhaps uncommenting the correct section of - the Apache Jena rule DSL. Once that is done, the bootstrap code should only include the wn:partOfSpeech triple, letting Jena's inference take care of the rest.

WIP is being tracked in the feature/17-pos-inference branch, although it is currently a fair bit behind master. Before continuing work it will have to be rebased.


#19 issue  resolved.

Fixes Issue🛠

Closes #
Description👨💻

Type of change📄

[] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)

How this has been tested✅

Checklist✅

[] My code follows the style guidelines of this project
[] I have performed a self-review of my own code
[] I have commented my code, particularly in hard-to-understand areas
[] I have added demonstration in the form of GIF/video file
[] I am an Open Source Contributor

Screenshots/GIF📷

Pls change the background color and accordingly the text color in dark theme, it should look like yes game is running on the dark theme. Blue is not suitable.

Is it looking good

Yes @kushaljgec2025

Yes

I have commited it.

Nice work!! Congrats on getting your PR merged 🎉 @kushaljgec2025


Modify owasp tekton pipeline to redirect the scan result from taskrun log to github issues

Change as follows:

add-issues  (new) - take owasy scan result and output to github issue by calling  gh create issue cli command
pipeline/owasp - add add-issues and fetch repo, set up workspace
tekton.md - update owasp tekton task run script for owasp scan
task/kustomization - add add-issues.yaml

The one you have is probably GitHub action pipeline not openshift tekton pipeline.  The tekton pipeline from arctiq only dump in log.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Billy Li @.>
Sent: Thursday, April 21, 2022 10:36:34 PM
To: bcgov/pipeline-templates @.>
Cc: Bruce Li @.>; Author @.>
Subject: Re: [bcgov/pipeline-templates] Modify owasp tekton pipeline to redirect the scan result from taskrun log to github issues (PR #28)
@w8896699 approved this pull request.
Looks good to me! One thing I want to mentioned is that the zape scan that registry using right now will also create a github issue. but I didn't find similar code over there. I am not sure how that works, just mentioned here for your reference. https://github.com/bcgov/platform-services-registry/blob/master/.github/workflows/zap_scan.yml
—
Reply to this email directly, view it on GitHubhttps://github.com/bcgov/pipeline-templates/pull/28#pullrequestreview-949459518, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUY3ZEJ4IBEPGGDQ3QSZJKDVGI3GFANCNFSM5UAKLXZQ.
You are receiving this because you authored the thread.Message ID: @.***>

Bruce will create a new PR


CB-2733,  CB-2688 Knox frontend URL to CM + trailing /

@risdenk I updated the jira, can you please review again?


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 840

Questions ?
Please refer to the Backport tool documentation


part_of relationship

Hi,
Can I discover "part_of" relationship between different terms or your program build children field only on "is_a" relationship?

We would like to support that. But unfortunately, it will have to be a bit in the future. Maybe by the end of the year? The immediate priority is in adding support for additional multipletest methods, like Benjimini/Hochberg.


Debian package

This pull request sets some build stages in Travis that use the npm package node-deb to build a Debian package, which is then pushed to packagecloud.io.
There are 2 build stages.

The first builds all branches, with a packagename of "signalk-server-$BRANCH"(including master). So on every commit to every branch you get a package to test with.

The second stage builds the main "signalk-server" package. This happens only on master branch, and only when the commit is tagged with a tag starting in "v[0-9]".

There are 2 deploy stages on each build stage. This is only because of the different distributions (debian/raspbian).
Note, that due to a few dependencies (bcrypt and mdns I think) build-essential is a dependency, and during the package install "npm install" is run to build them. If the package is popular enough it's possible that we could look into pre-compiling them, would need to see how to cross compile for arm though.
Regarding packagecloud, I have an account with the name "SignalK" that I got the opensource plan on. I can add collaborators, or I can look to transfer it over to someone else.

Sorry for letting this lapse, getting back now.
Again, sorry for the conflicts from the docker stuff now in master's .travis.yml. There is now a stage for building x86 and arm docker images in parallel. In theory deb build could run in parallel with them, but not sure if it is a good idea. What do you think?
About the stages: the stages are now copies, except for the version, right? I'd like to avoid duplication, use the same "body" and just create the version conditionally.
I think releases should be based on git tags, not master commits. I think this rhymes well with creating the version conditionally, TRAVIS_TAG looks like it could do the trick.
jq looks superfluous?
I have stumbled a few times when trying to extract data from xml and json with command line hacks. Since the build environment is node I think it would make sense to parse the package.json with a short js script, like echo "const data = require('./package.json'); console.log(data.version)" | node.
How do we specify that this needs node >8?

The stages should be able to run concurrently.
Only difference between the two stages build-deb-branch and build-deb-main are the debian package name. build-deb-branch names it "signalk-server-${TRAVIS_BRANCH}" while build-deb-main names its just "signalk-server".
They could possible combined into one stage. I can look into it.
Regarding releases, currently stage build-deb-main only runs when a tag is present. so the only packages  named "signalk-server" will be the tagged ones. Having the branch related ones permits two things, 1) making sure any changes don't break package building, 2) allows people to easily run pre-release versions (like latest master or a branch) Imagine someone requests support for some new thing, rather then wait for a new "release" they can just apt-get the branch and use it right away. :)
jq is a dependency for node-deb. (a system package, not npm dependency).
Fair point with the scraping json from command-line, your command looks much more "stable" :)
For specifying node8, I'll have a look. I think we can say we depend on "nodejs (>= 8.0)". I'll give it a try :)

@mxtommy did we lose the momentum here? Winter's coming, lots of non-sailing time for tinkering with these....

Don't really know.
AFAIK the npm method for installing and updating has served us well during the summer happening.
The server update does simply npm install for the latest version: https://github.com/SignalK/signalk-server-node/blob/84b007a300b13c651b3be331b0fa92038e153b95/lib/modules.js#L107-L109, but it has been out for just a few weeks and can't really tell how it will do.
On a related note I have been talking with @sbender9 about the setup script creating a separate signalk account and running the server under that, with limited priviledges.
Then there's also OpenPlotter. I just talked to @e-sailing last week about automating the OpenPlotter build as a solution for creating an always up to date RPi sd card image. There lies a practical problem: charts purchased from https://o-charts.org/ can not be (easily?) moved to a new installation, so upgrading via get a new card  is not attractive, even if we discount all your personal settings not being transferable. But it would still be a good starting point for getting into OpenPlotter/SK.

@mxtommy  @tkurki Please feel free to consider Cloudsmith as well for packaging. :-)
It's commercial but has very generous free plans and offerings for open-source projects (with 100GB of storage and 1TB of bandwidth by default). It might work well to consider splitting unstable releases into a separate repository (with support for npm as well if you need it), which will provide stable/unstable channels for your users. For actually building the packages I'd highly recommend fpm (worth a look whether you use Cloudsmith or not).
Full disclosure: I work for Cloudsmith, so might be biased (but only slightly, honest)!


The link for the "complete list of options" points to the outdated list

Describe the bug
The link for the "complete list of options" on the README documentation here points to the analysis_options.3.1.0.yaml file, but I think it should point to the latest analysis options, which is analysis_options.5.0.0.yaml
To Reproduce
Steps to reproduce the behavior:

Go to README documentation.
Click on the "complete list of options" hyperlink.
Verify that the opened link is for a list of options for the 3.1.0 version instead of the up-to-date, 5.0.0, version.

Expected behavior
Analysis options for 5.0.0 should be opened.

https://github.com/VeryGoodOpenSource/very_good_analysis/pull/82


Provide the required Permission for AzureDataBricks Enterprise App

[Enter feedback here]
The page is telling check the permission , but missing 2 main aspect

Permission/Role to which App ?
What Role (least) needed?

Basically we need to provide the AzureDataBricks Enterprise App with the Secret Reader on the KeyVault.

Reference: https://stackoverflow.com/a/63272029/7073340
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6da278b7-2b30-3a2a-8321-f19c7cac96e2
Version Independent ID: c44bb6d9-ddf2-f4ef-2d83-efd67d6a3960
Content: Troubleshoot key vault access issues - Azure Databricks - Workspace
Content Source: databricks/kb/security/troubleshoot-key-vault-access.md
Service: azure-databricks
GitHub Login: @aravishdatabricks
Microsoft Alias: saperla

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

Are you confusing access control with access policies, @jayendranarumugam  ?

Roles are part of Access control (IAM).
There are two models for setting permissions.

Hi @MartinJaffer-MSFT -  Not exactly , either way its Azure RBAC or Vault Access Policies .
What permissions we need to check on the IAM or Vault access and for Which App ?
The troubleshooting page is telling more generic way of just check the permission. But without knowing for which app its difficult to resolve.
To be exact

The point 5 telling Verify the Get and List permissions are applied. But which for app we need to verify that access ??

Hi @jayendranarumugam,
What permissions we need to check on the IAM or Vault access and for Which App ?
By default, every subscription has a AzureDatabricks application registered.

You need to grant access policy to the AzureDatabricks application as shown below:

Hope this helps.

Hi @CHEEKATLAPRADEEP-MSFT exactly ! Can the author can update the original docs with this much clarity ? I didn't create this issue because I don't know the solution. I know the solution before :) I created this issue which is a kind of a feedback to include this much clarity like

Verify the Get and List permissions are applied on AzureDatabricks application

instead of simply mentioning like

Verify the Get and List permissions are applied.

like below screenshot

@jayendranarumugam Thanks for the feedback! I have assigned the issue to the content author to evaluate and add the screenshot.
@leifbro Could you please create a pull request to add an image for clarification?


added ethers instance inside wallection connection object

@snowwhitedev, @BankiLouis - please review and approve when ready.
This change introduces the smart contract interaction library to the wallet connection call back interface. To provide an opinionated way for developers to work with web3 once connected via TKN.
In future we may need to change the libs that we attach here e.g. if they are connected with Solana or Cardano then we can provide the most popular lib for them to work with.
Thanks guys.


用accelerate 多卡训练后请问如何多卡 inference

ValueError: You can't train a model that has been loaded with device_map='auto' in any distributed mode. Please rerun your script specifying --num_processes=1 or by launching with python {{myscript.py}}.
单卡可以跑，多卡报错
多谢！

目前不支持多卡 inference


[keymgr] Abort into Invalid when Creator Root Key is not valid during initialization

Prior to this PR, keymgr could be initialized and would open itself
to operations even if the Creator Root Key shares provided by OTP in the
root_key_i input were invalid (although keymgr did write all-zeros and
all-ones instead of the input from OTP into its key shares if that was
the case).  However, the specification dictates that kemygr must signal
an invalid operation. Since the Creator Root Key is meant to be static after
reset, SW doesn't have to be able to retry the initialization. As a security
defense mechanism, keymgr should abort into the Invalid state in this case.
This commit fixes that behavior, thereby closes https://github.com/lowRISC/opentitan/issues/17776.
 That commit also aligns the DV. All commits before the last one are DV fixes/extensions necessary and sufficient to prepare for the last commit. Please see the commit messages for details.
TODO Items

[ ] Run related TLTs and check their result. --> running
[ ] Debug failing CW310 ROM test.
[x] Check if other steps in the key ladder need similar fix.
--> I think they don't. The inputs to the other key ladder steps don't have valid signals associated with them (which could be improved in a future revision) but they get checked by keymgr_input_checks if they aren't all-zeros or all-ones. The result of that check then feeds into keymgr_ctrl's op_err, which prevents the key ladder from advancing and any keys from being updated, and the error gets reported as invalid input to the err_code CSR.
[ ] Split RTL commit because it fixes two orthogonal problems in keymgr.

Prior to this PR, keymgr could be initialized and would open itself
to operations even if the Creator Root Key shares provided by OTP in the
root_key_i input were invalid (although keymgr did write all-zeros and
all-ones instead of the input from OTP into its key shares if that was
the case). However, the specification dictates that kemygr must signal
an invalid operation. Since the Creator Root Key is meant to be static after
reset, SW doesn't have to be able to retry the initialization. As a security
defense mechanism, keymgr should abort into the Invalid state in this case.

@moidx @ballifatih just wanted to double-check with you that you agree this this reasoning, since I think we haven't specified the next state part of keymgr's behavior when the Creator Root Key is invalid during initialization yet.

Prior to this PR, keymgr could be initialized and would open itself
to operations even if the Creator Root Key shares provided by OTP in the
root_key_i input were invalid (although keymgr did write all-zeros and
all-ones instead of the input from OTP into its key shares if that was
the case). However, the specification dictates that kemygr must signal
an invalid operation. Since the Creator Root Key is meant to be static after
reset, SW doesn't have to be able to retry the initialization. As a security
defense mechanism, keymgr should abort into the Invalid state in this case.

I think moving right away to Invalid makes more sense. It also aligns with our earlier decision in keymgr_dpe:
https://github.com/lowRISC/opentitan/blob/f4962c85dd1fb69a26c6fa2aeb9d59d6b832f330/hw/ip/keymgr_dpe/rtl/keymgr_dpe_ctrl.sv#L480-L481
In general I favor Keymgr behaviors that immediately mark invalid/erroneous operations instead of resuming operation with invalid data (as a form of obfuscation). I think the latter behavior was chosen early in the design specs to conceal the results of fault attempts. However, it only prevents shallow attempts by the attacker, and there is always further ways to deduce that fault attempt worked (I.e. by computing the attestation identities or cryptographic keys that is supposed to be same in the absence of a fault).


Fail to resolve protocol type from static lib that registered from different module.

As titled,
So let's say I have a project that has this modularized workspace setup.
Main App Project which has dependency to StaticLib1 and FeatureA.framework
FeatureA.framework is a dynamic framework that also has dependency to StaticLib1
StaticLib1 is a static lib that is linked statically to FeatureA.framework and MainApp
StaticLib1 has a public protocol SomeProtocol
MainApp act as composition root to register SomeProtocol.self into a global container and provide the detail factory process in the resolve closure.
When calling resolve SomeProtocol.self from inside of the Main app. The factory is found without problem.
When calling resolve SomeProtocol.self from inside FeatureA.framework it fails to resolve since apparently it sees the key of SomeProtocol.self that is registered on the Main App is different with SomeProtocol.self that FeatureA.framework is trying to resolve.
I wonder if this is expected due to the nature of StaticLib1 is linked statically to both the MainApp and FeatureA.framework? If yes is there a workaround to use Swinject with this kind of project setup?
Version Swinject used: 2.8.1
Thank you in advance!

Here a sample workspace that mirror the situation, pardon all the mess (default, iOS 15.4, brokenload xib from FeatureA.framework)  since this is just a quick made project with all default configuration in
https://github.com/agungpratama/ExperimentSwinject
As you can see:
ISomeFeatureAService.self is a protocol declared in FeatureAAPI ( static lib )
AppContainer located in MainApp registers ISomeFeatureAService.self
ViewController located in MainApp successfully resolves ISomeFeatureAService.self
FeatureAViewController located in FeatureA.framework (dynamic framework) failed to resolve ISomeFeatureAService.self
Press the button on the ViewController (first screen after launched) to present FeatureAViewController


Wishlist and compare list are not cached properly

Current behavior
After refreshing pare or restarting browser, wishlist and compare list are empty
Expected behavior
Content of wishlist and compare list should be cached and still available after refresh or browser restart
Steps to reproduce the issue

Open http://test.storefrontcloud.io
Add any product to wishlist and compare list
Refresh page (F5)

Repository
http://test.storefrontcloud.io
Can you handle fixing this bug by yourself?
no
Environment details

Browser: All (tested on Chrome, Firefox, Opera, Edge and Safari - desktop and mobile)
OS: All (Tested on win10, macOS + android and iOS on mobile)

Additional information
Cart seems unaffected.

@filrak how about thi one?


how do I make children

How do I make children with an Oni? im a female oni, and I've given this guy a thorned rose and a ring an now were married. how do we make children

Hello! there is a 5 minute countdown to have kids, so when you log in you have to wait 5 minutes for it to work, along with another 5 minute cooldown after having the kid.   That's all ya need xD

lol :)


MRS UAV System Ubuntu 18.04 ROS Melodic - Build Status failing

Hello,
I need to install the MRS UAV System on Ubuntu 18.04 with ROS Melodic but as you can see on the README, the Build Status is failing. Here is the error I get:

Have you planned to fix it ?

Hey, sorry, there was a brief moment when the gitman installation was broken. Try it now. But pull the mrs_uav_system first.


Internationalization and Localization Support

Multi-language support for Flutree is coming.

[ ] español / Spanish
[ ] Portugese
[ ] Something else

Ingat nk guna gugel translate je hewhehw tpi kene ade org yg tahu language tu supaya tk lari context


Runtime issues with the example

After the migration to rescript-react has been merged, @jsiebern noticed a problem with the example.
I am opening an issue mainly to discuss what is happening, what could be involved and, in case, how this could be fixed.

Using the debugger, it is possible to see that module @material-ui/styles/esm/useTheme/useTheme.js imports ThemeContext from './ThemeContext'. However this is undefined, causing the hook error inside useTheme. This function is called from useStyles, which, if I am not wrong, is called from AppLayout.res:15.
What I don't get is the fact that ThemeContext.js exists inside @material-ui/styles/esm/useTheme. This makes me think that the issue could be related to some strange behavior of webpacking...? Let me know what you think.

Just built version 2.1.0 together with @jsiebern, and we did not encounter any problem with the examples.

Fantastic!!! I just finished checking, and I did not encounter the issue anymore as well. Probably you fixed the issue during the refactory you made since I opened the issue.
Thank you for your awesome work!


[Broken] Facebook download of private (friends-only) video fails with ExtractorError

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dlc version 2020.11.11-3
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.facebook.com/video.php?v=10158180747884436', '--username', 'PRIVATE', '--password', 'PRIVATE', '--verbose']
[debug] Loading archive file None
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dlc version 2020.11.11-3
[debug] Python version 3.8.2 (CPython) - Linux-4.4.0-19041-Microsoft-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4, rtmpdump 2.4
[debug] Proxy map: {}
[facebook] Downloading login page
[facebook] Logging in
[facebook]<PHONE_NUMBER>7884436: Downloading webpage
[facebook]<PHONE_NUMBER>7884436: Downloading webpage
[facebook]<PHONE_NUMBER>7884436: Downloading webpage
ERROR: Cannot parse data; please report this issue on https://github.com/blackjack4494/yt-dlc . Make sure you are using the latest version; see  https://github.com/blackjack4494/yt-dlc  on how to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/home/sofia-wsl/.local/lib/python3.8/site-packages/youtube_dlc/YoutubeDL.py", line 830, in extract_info
    ie_result = ie.extract(url)
  File "/home/sofia-wsl/.local/lib/python3.8/site-packages/youtube_dlc/extractor/common.py", line 532, in extract
    ie_result = self._real_extract(url)
  File "/home/sofia-wsl/.local/lib/python3.8/site-packages/youtube_dlc/extractor/facebook.py", line 482, in _real_extract
    _, info_dict = self._extract_from_url(
  File "/home/sofia-wsl/.local/lib/python3.8/site-packages/youtube_dlc/extractor/facebook.py", line 380, in _extract_from_url
    raise ExtractorError('Cannot parse data')
youtube_dlc.utils.ExtractorError: Cannot parse data; please report this issue on https://github.com/blackjack4494/yt-dlc . Make sure you are using the latest version; see  https://github.com/blackjack4494/yt-dlc  on how to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.

Description

Attempting to download a private youtube video from a friend's post results in the above error and the video fails to download.
I have tried multiple videos set to "friends only" with both youtube video link formats, any one you have access to should suffice
https://www.facebook.com/[username]/videos/[video-id]/
and
https://www.facebook.com/video.php?v=[video-id]

Checklist

[x]  I'm reporting a broken site support
[x]  I've verified that I'm running youtube-dlc version 2020.11.11-3
[x]  I've checked that all provided URLs are alive and playable in a browser
[x]  I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x]  I've searched the bugtracker for similar issues including closed ones

Verbose log
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.facebook.com/video.php?v=10158180747884436', '--username', 'PRIVATE', '--password', 'PRIVATE', '--verbose']
[debug] Loading archive file None
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dlc version 2020.11.11-3
[debug] Python version 3.8.2 (CPython) - Linux-4.4.0-19041-Microsoft-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4, rtmpdump 2.4
[debug] Proxy map: {}
[facebook] Downloading login page
[facebook] Logging in
[facebook]<PHONE_NUMBER>7884436: Downloading webpage
[facebook]<PHONE_NUMBER>7884436: Downloading webpage
[facebook]<PHONE_NUMBER>7884436: Downloading webpage
ERROR: Cannot parse data; please report this issue on https://github.com/blackjack4494/yt-dlc . Make sure you are using the latest version; see  https://github.com/blackjack4494/yt-dlc  on how to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/home/sofia-wsl/.local/lib/python3.8/site-packages/youtube_dlc/YoutubeDL.py", line 830, in extract_info
    ie_result = ie.extract(url)
  File "/home/sofia-wsl/.local/lib/python3.8/site-packages/youtube_dlc/extractor/common.py", line 532, in extract
    ie_result = self._real_extract(url)
  File "/home/sofia-wsl/.local/lib/python3.8/site-packages/youtube_dlc/extractor/facebook.py", line 482, in _real_extract
    _, info_dict = self._extract_from_url(
  File "/home/sofia-wsl/.local/lib/python3.8/site-packages/youtube_dlc/extractor/facebook.py", line 380, in _extract_from_url
    raise ExtractorError('Cannot parse data')
youtube_dlc.utils.ExtractorError: Cannot parse data; please report this issue on https://github.com/blackjack4494/yt-dlc . Make sure you are using the latest version; see  https://github.com/blackjack4494/yt-dlc  on how to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.

Description
Attempting to download a private youtube video from a friend's post results in the above error and the video fails to download.
I have tried multiple videos set to "friends only" with both youtube video link formats, any one you have access to should suffice
https://www.facebook.com/[username]/videos/[video-id]/
and
https://www.facebook.com/video.php?v=[video-id]

both of your sample links come up as page missing/broken

They are simply templates, showing the format of youtube video links.
As the videos affected are ones set to friends only and I do not know your facebook friends, I cannot provide sample links that you can access. You can test this yourself by finding a video on your feed set to "friends only" and attempting to download it.

yd-dlc Cannot Fetch (In General Private Videos), it requires authentications, although in this case yd-dlc possibly cannot authenticate with fb even with specifying (username,password) probably because it's doesn't support fb auth .
You'll have to make the video publicly accessible in order to download then switch back to private .

yd-dlc Cannot Fetch (In General Private Videos), it requires authentications, although in this case yd-dlc possibly cannot authenticate with fb even with specifying (username,password) probably because it's doesn't support fb auth .
You'll have to make the video publicly accessible in order to download then switch back to private .

It successfully authenticates in the verbose log shared above, and entering an incorrect password causes it to fail to login. This is not an account with 2FA installed, which is the only situation where facebook auth is not supported to my knowledge unless there has been a regression in yt-dlc

#52
This appears related to this issue which i missed on my initial search, hopefully that fix gets merged soon!


Wallets nav2 header

Factor SendButton and DropdownButton from the original header into wallets common components

r? @keybase/react-hackers

Thanks @chrisnojima, made changes PTAL


Add goaled.js

Hello, this is my Sprig game. I'm pretty new to programming, so please tell me if there is anything I can do to make it better.
It's a game where you need to push the ball into a teleporting goal. I hope you like it!

Cool game! Really tricky and is fun to play! Thanks for the PR.
One minor issue. One line 194, it will error when there is no goal on the screen, like the game over screen for example.
To prevent that, I suggest you add a check to see if there is a goal on screen. Let us know if you run into any issues!

This is a nice straightforward game. Could you add a background to make it more visually interesting?
I also get the error Lucas mentioned.
@leomcelroy

Okay I added a background, made it look better, and fixed that error.

I had a lot of fun with your game!


Approach to Branded Types & RPC-related Validation

Do U.HexString and comparable branded types buy us much in terms of safety / DX?

Yes, branded types give us much better DX, not just because of safety, but because they describe what things are. string is impenetrable and near-useless, but MultiAddressString is much less so.
However, I think we should rework our approach to branded types.
type Branded<T, Brand extends PropertyKey, V = undefined> = T & Record<Brand, V>;

declare const _hex: unique symbol;
export type Hex<T extends Uint8Array = Uint8Array> = Branded<string, typeof _hex, T>;

declare const _encoded: unique symbol;
export type Encoded<T> = Branded<Uint8Array, typeof _encoded, T>;

declare const _hash: unique symbol;
export type Hash<T = unknown, K extends HasherKind = HasherKind> = Branded<Uint8Array, typeof _hash, [T, K]>

export type HexEncoded<T> = Hex<Encoded<T>;
export type HexHash<T = unknown, K extends HasherKind = HasherKind> = Hex<Hash<T, K>>;

With this, instead of manually creating a branded MultiAddressString, we can simply use HexEncoded<MultiAddress>. We can also have things like
export function encodeHex<T>($t: Codec<T>, value: T): HexEncoded<T> {
  ...
}
export function decodeHex<T>($t: Codec<T>, hex: HexEncoded<T>): T {
  ...
}

And then type-safely call encodeHex($multiAddress, multiAddress) or decodeHex($multiAddress, multiAddressStr).

I like this approach of integrating branded types into our conversion utils. Nice. That signature––HexEncoded<MultiAddress>––is beautiful.
Why the T extends Uint8Array in the Hex type params?

Also, I'm not sure if the branded types belong in utils/branded.ts. How would you feel about moving branded types into a top-level branded.ts file? Or better yet, into a new_types/mod.ts?

Why the T extends Uint8Array in the Hex type params?

Because, ultimately, the data contained in a hex string is a Uint8Array. If that Uint8Array itself represents data, then we can narrow it further, like we do with Hex<Encoded<T>> and Hex<Hash<T>>.
It also opens up the possibility of signatures like this in our utils/hex.ts:
export function encode<T extends Uint8Array>(array: T): Hex<T> {
  ...
}
export function decode<T extends Uint8Array>(hex: Hex<T>): T {
  ...
}

Also, I'm not sure if the branded types belong in utils/branded.ts. How would you feel about moving branded types into a top-level branded.ts file? Or better yet, into a new_types/mod.ts? We could include validations/conversions in this file as well. Perhaps the hex utils belong in here too.

Perhaps they should be moved in with primitives (or whatever it is renamed to)?

relevant branded types modeled in #233
TODO RPC validation: #325


Remove deprecated classes #52

This PR fixes issue #52 and removes deprecated reporter classes.

why close this?

I will set up a new branch/PR without so many commits.


Implemented popoverClassName (with default props and types)

Codecov Report

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

@@           Coverage Diff           @@
##           master     #141   +/-   ##
=======================================
  Coverage   38.38%   38.38%
=======================================
  Files           8        8
  Lines         198      198
=======================================
  Hits           76       76
  Misses        122      122

Impacted Files
Coverage Δ

src/defaultProps/selectFieldDefaultProps.js
100% <ø> (ø)
:arrow_up:

src/types/selectFieldTypes.js
4.16% <ø> (ø)
:arrow_up:

src/SuperSelectField.js
39.16% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Almost perfect PR, thanks
Can you a description add in Readme, after noMatchFoundStyle please ?

All done :+1:

Sorry maybe my modified comment crossed your committing, but can you amend your last commit message with feat: add popoverClassName please ?

Ok cool I think I've done it :1st_place_medal:

That second commit is weird o.O ?
Merge branch 'master' into master

Yeah, it keeps giving me merge conflicts and then automatically does that to fix it. This is acting strangely...

Making a new PR since this one just isn't working...


Implement command getKey

Please go above 100 tests in integration also!

https://docs.microsoft.com/en-us/azure/key-vault/keys/about-keys
Key Vault doesn't support EXPORT operations. Once a key is provisioned in the system, it cannot be extracted or its key material modified.
KeyVault gives you only public part. Possibly we can explore "backup" solution, but sounds hacky.
Private part never leaves KeyVault in my opinion (which I think is good) and you "just" somehow refer to a propriate key in a code should you need to encrypt/decrypt ...
No reason to implement "download public part", therefore closing the issue.


Ranboes New URL (Cannot see LN's in Ranboes because of their new url)

Steps to reproduce

Open LN Reader
Navigate to explore menu (compass button)
Search for Ranobes and enter
Tap the globe button at top right.

Expected behavior
Open Ranobes and/or be able to search for LN's under Ranobes
Actual behavior
Following message:
The site is temporarily disabled
Go to the new site - https://ranobes.top
LNReader version
1.1.15
Android version
Android 13
Device
Samsung Galaxy A23
Other details
No response
Acknowledgements

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

If this is an issue with a source, I should be opening an issue in the sources repository.

This has been already fixed in develop, you can either try a dev build or wait for next release.


Revert "Pokestops / Evolves / Transfers Logged to DB"

Reverts PokemonGoF/PokemonGo-Bot#4232
Caused crash reported in #dev channel, need revert.
Traceback (most recent call last):   File "pokecli.py", line 596, in <module>     main()   File "pokecli.py", line 106, in main     bot.tick()   File "/Users/bstpierre/PokemonGo-Bot/pokemongo_bot/__init__.py", line 532, in tick     if worker.work() == WorkerResult.RUNNING:   File "/Users/bstpierre/PokemonGo-Bot/pokemongo_bot/cell_workers/spin_fort.py", line 83, in work     conn.execute('''INSERT INTO pokestop_log (pokestop, exp, items) VALUES (?, ?, ?)''', (fort_name, str(experience_awarded), str(items_awarded)))                     sqlite3.OperationalError: no such table: pokestop_log

I just took a look and I am not able to reproduce this, does anyone know what user had this issue? I'd love to track it down


test(uri): fix URI parse error tests

Add also test parse authority without port

[x] The commit messages match the guidelines in https://github.com/hyperium/hyper/blob/master/CONTRIBUTING.md#git-commit-guidelines

Thanks!


add NATS support

Issue Ref: #669
Description:
Add NATS support into Kubeless.
[PR Description]
This PR adds a first-cut integration of NATS into Kubeless. NATS provides wide variety of messaging options, this PR adds suport for https://nats.io/documentation/concepts/nats-queueing/
Please see the documentation included in the PR on how to use the functionality.
To the reviewer:
Commits are logically organised to make easier to follow.
8a38f8c6cb546cdbac8b3899f96341adc3e02fb9 (HEAD -> nats, origin/nats) NATS documentation
0fdb77e3f517c45fe5c74c978cbaa9fbb3bec9b7 fix tests
085e38f501126216c941ce3fb18194d0487444d7 build, CI and other misc stuff
0868bc84656ec1ff58955e5f3514609561232303 CLI support for NATS
90427b67f07810b9cc6df43c510bf11e3648c25d core nats trigger functionality
2254359450820910968bdd54646ffae72fa4f563 auto-generated code from code generator
b904a1d33d1fe325becfb8db380fee85ac6f607b API spec for nats trigger
2345b2a614c304adffd927aede296ad5caaf8e7a vendor nats-io/go-nats and nats-io/nuid

TODOs:

[x] Ready to review
[x] Automated Tests
[x] Docs

Planning to add some more IT tests, but PR is good to get started with review.

We should also add an additional flag to the kubeless function deploy  to complement the --trigger-topic flag and knows if the target is Kafka or NATS. Or just remove the --trigger-topic flag to clarify that PubSub triggers have a different life cycle and should be created/deleted independently. I would do the second since right know if you create and delete a function with --trigger-topic an orphaned KafkaTrigger is left.

@andresmgot I have removed --trigger-topic flag in the kubeless function deploy call. Updated quick start guide for Kafka as well to reflect this.

thanks for the review @andresmgot @wallyqs
@wallyqs you should be able to use the functionality by building image from the master. Ping me on slack if you run into any issue. Would be nice to get your feedback and possible future enhacements.


Emit willfocus from immediatelyResetRouteStack

The Navigator component inconsistently emits the willfocus event.
While it is emitted in Navigator#resetTo, Navigator#push, Navigator#pop and event Navigator#replaceAtIndex it is not emitted from Navigator#immediatelyResetRouteStack. This leads to surprising inconsistencies when working with these events.
With the PR I suggest to emit the willfocus event before resetting the route stack.

@dzautner updated the pull request - view changes

@facebook-github-bot shipit

Thanks for importing. If you are a Facebook employee, you can view this diff on Phabricator.


classifier problem

Model filename: /home/swf/work/0509/facenet/models/facenetclean/20170512-110547/20170512-110547.pb
Traceback (most recent call last):
File "src/classifier.py", line 170, in
main(parse_arguments(sys.argv[1:]))
File "src/classifier.py", line 69, in main
facenet.load_model(args.model)
File "/home/swf/work/0512/facenet/src/facenet.py", line 375, in load_model
tf.import_graph_def(graph_def, name='')
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/importer.py", line 258, in import_graph_def
op_def = op_dict[node.op]
KeyError: u'FIFOQueueV2'

Hi,
This sounds like a tensorflow version problem. This repo is currently tested against TF 1.0. Which version are you using?

I update my tensorflow to TF.1.0, the problem disappeared.


Question: why --start/end option are in timestamp and not in datetime ?

It could be easier to launch sim with start/end as datetime format instead of timestamp format.
I can do the PR...
@@ -17,8 +17,8 @@ module.exports = function container (get, set, clear) {
-      .option('--start <timestamp>', 'start at timestamp')
-      .option('--end <timestamp>', 'end at timestamp')
+      .option('--start <datetime>', 'start ("YYYYMMDDhhmm")')
+      .option('--end <datetime>', 'end ("YYYYMMDDhhmm")')

@@ -49,13 +49,13 @@ module.exports = function container (get, set, clear) {
-          so.start = moment(so.start).valueOf()
+          so.start = moment(so.start, "YYYYMMDDhhmm").valueOf()

-          so.end = moment(so.end).valueOf()
+          so.end = moment(so.end, "YYYYMMDDhhmm").valueOf()

Dear issue reporter,
We have slightly changed our github issue policy and would now kindly request folks that have questions that they ask them in our zenbot subreddit.
You can find it here: https://reddit.com/r/zenbot
This issue will be closed, but if you disagree with your ticket being marked as a question feel free to leave a comment defending your case.
Thanks for contributing time and effort!
Greetings,
🤖


[Add]: set up flow to add lms to course

General Info

Pivotal Tracker Story
[ ] Breaking?

Changes
Implement the POST api/v1/courses/{course_id}/lms, which creates one record in the course_to_lms table.
Testing

Test if it raises exception when course_id and lms_id are invalid;
Test if it raises exception when external_course_id is missing.
Test if it raises exception if this bind between such this course and lms exits before.
Test if such a target record could be successfully created in the course_to_lms table.

Documentation
The related tables in this implementation part in our E-R diagram are shown below.

Checklist

[x] Name of branch corresponds to story

#39 also has these changes. It would be great if you could remove this pr's code from the other one and resolve the comments I left on that one :)

Hi Connor, thanks for your comments! I've already made changes based on your comments on #39. Sorry for the inconvenience caused by the huge pr. And for the test coverage, I can assure you that every line of the course_to_lms controller has been touched by tests. But I still have no idea why the test coverage decreased a little.


Create CODE_OF_CONDUCT.md

@TomTranter Don't forget to delete branches associated with merged pull requests.


add other_names for CA

CA looks good, MN looks like it changes some ids & deletes some extras information that should be left in the file for a handfull, but most look fine just swapping where other_names is located


v0.18.2-beta: forge download from https://maven.minecraftforge.net/ returns 404

when trying to install a modpack:
Server file not present, downloading...
(Point --server-dir and --server-file to existing targets to skip this step)
Downloading file from: https://maven.minecraftforge.net/net/minecraftforge/forge/47.2.17/forge-47.2.17-installer.jar
http status 404
is this because they changed the file host url or because this particular forge file isn't hosted there anymore?

apparently it's a server outage and nothing permanent


fix: update node and semantic release versions

it uses now the lastest semantic release, node support from 6 to 10 and commitizen updated too

:tada: This PR is included in version 5.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feature logits problem when num_classes is small

Hello,
Thanks for your great work! Your idea is brilliant!
The meta network use feature logits as input to compute the fitness between sub-networks. But when the num classes is small such as 2, the information loss in feature logits could be very severe, which will makes the meta network hard to learn.
So have you use the feature before last fully connection to learn the meta network? If so, can you share the result.
Best,

Hi,
Thanks for your interest in our work!
For now, our search experiments are performed on ImageNet(Number of classes is 1000). We haven't tried datasets with small number of classes. This could be an interesting problem in our future work - How to select the teacher model with limited class information.
We have tried the feture map before the last fully connection. However, using feature map brings no significant improvement and increase the training time. So, we just use the last fully connection as the input of the meta matching network.
Best,
Hao.

thanks for your reply ~


Update repositories.txt with karol-brejna-i/RemoteDebug

@ArduinoBot would you mind rechecking the PR?


${None} converted to string even if argument default value is None

If we have a keyword like
def example(arg: str = None):
    print(arg, type(arg))

and call it like
Example    ${None}

the explicitly given Python None will be converted to a string. This is a regression in RF 4.0, with earlier versions None is passed as-is. Luckily this isn't a very common situation as it's generally better to not pass an argument at all in which the argument will have the original default value.

There were many changes in RF 4.0 that contributed to this regression:

Nowadays non-string arguments are also converted (#3735).
Conversion to string is supported (#3736).
Union conversion (#3738) was implemented so that arguments are always converted, even if they would have one of the accepted types, and conversion is attempted in the order types are listed in the Union.

The last point may sound surprising because the example didn't use Union, but in fact Python's typing.get_type_hints that we use adds it automatically. What happens in practice is that arg: str = None is converted to arg: Optional[str] = None which in turn is just a different way to write str: Union[str, None] = None.
The underlying bug is thus that when we have type hint like Union[str, None] and call it with a real None, we do conversion even if the type is one of the accepted types. This is exactly the same problem as the one reported by #3897. Because the manifestation of the bug is so different, I decided to submit a separate issue.

This problem was fixed by PR #3895. Still need to add an explicit test for exactly this case before merging.


[BUG] secondary_structure_percentage doctest raises pandas-related AttributeError

I just launch the doctest code written in the rstoolbox.analysis.secondary_structure_percentage function docstring:
from rstoolbox.io import parse_rosetta_file
from rstoolbox.analysis import secondary_structure_percentage
import pandas as pd
pd.set_option('display.width', 1000)
pd.set_option('display.max_columns', 500)
df = parse_rosetta_file("../rstoolbox/tests/data/input_ssebig.minisilent.gz",
                        {'scores': ['score'], 'structure': 'C'})
df = secondary_structure_percentage(df, 'C')
df.head()

It raises the following error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/florent/anaconda3/envs/rstoolbox/lib/python3.8/site-packages/rstoolbox/analysis/structure.py", line 240, in secondary_structure_percentage
    df2 = df.apply(lambda row: secondary_structure_percentage(row,
  File "/home/florent/anaconda3/envs/rstoolbox/lib/python3.8/site-packages/pandas/core/frame.py", line 7541, in apply
    return op.get_result()
  File "/home/florent/anaconda3/envs/rstoolbox/lib/python3.8/site-packages/pandas/core/apply.py", line 180, in get_result
    return self.apply_standard()
  File "/home/florent/anaconda3/envs/rstoolbox/lib/python3.8/site-packages/pandas/core/apply.py", line 255, in apply_standard
    results, res_index = self.apply_series_generator()
  File "/home/florent/anaconda3/envs/rstoolbox/lib/python3.8/site-packages/pandas/core/apply.py", line 284, in apply_series_generator
    results[i] = self.f(v)
  File "/home/florent/anaconda3/envs/rstoolbox/lib/python3.8/site-packages/rstoolbox/analysis/structure.py", line 240, in <lambda>
    df2 = df.apply(lambda row: secondary_structure_percentage(row,
  File "/home/florent/anaconda3/envs/rstoolbox/lib/python3.8/site-packages/rstoolbox/analysis/structure.py", line 253, in secondary_structure_percentage
    dfp = df.append(pd.Series([float(csse['H']) / len(sse), float(csse['E']) / len(sse),
  File "/home/florent/anaconda3/envs/rstoolbox/lib/python3.8/site-packages/pandas/core/series.py", line 2688, in append
    return concat(
  File "/home/florent/anaconda3/envs/rstoolbox/lib/python3.8/site-packages/pandas/core/reshape/concat.py", line 274, in concat
    op = _Concatenator(
  File "/home/florent/anaconda3/envs/rstoolbox/lib/python3.8/site-packages/pandas/core/reshape/concat.py", line 395, in __init__
    axis = sample._constructor_expanddim._get_axis_number(axis)
AttributeError: 'function' object has no attribute '_get_axis_number'

I did a quick test with pandas 0.24.1 and it worked fine, so my best guess is that there was a change in the _Concatenator class for the pandas v1 major update.
I kind of solved that issue with a quick and dirty hack in rstoolbox.analysis.structure.py (just tested in my pandas 1.1.0 python environment). So if your interested, just let me know and I'll submit a PR.
>>> rstoolbox.show_versions():
yaml: 5.3.1
pandas: 1.1.0
seaborn: 0.10.1
libconfig: 0.9.2
six: 1.15.0
networkx: 2.4
>>> rstoolbox.__version__
'1.0.1'

Hi @FloLangenfeld !
Thanks for your interest!
This comes, of course, from the bigger issue, which is the general integration of pandas 1 into the code 😕
I also need to fix the CI with travis, as it has changed a bit since last time I checked this code, but if you open the PR I'll be happy to merge it as soon as I can.

@jaumebonet
I've opened the PR with the small trick here : #124
And yeah, pandas 1 was a game-changer in so many ways that one may ask why not just add pandas<1.0.0 in the requirements :laughing:


[copyright infringement] please rename your packages and organisation on github

Hello,
Naming your organization and your packages symfonyboot is going to create confusion for the community.
Also Symfony is a trademark that must be used within the policy described at https://symfony.com/trademark
Please rename your organization and all related packages.
Thanks for your understanding.

Hello Nicolas
I understand your position and I will rename this project. I apologize for any trouble it has caused.

Hi Nicolas, I sent this project to my personal namespace for fix this copyright issue. Thanks for your patience.


Add option to set whether multiple SelectMenu should close on selecting an item

Description
It is now not possible to easily close the dropdown from the SelectMenu when an item is selected (or maybe when max is reached?). This is by design and derived from HeadlessUI.
Proposal
It would be nice if you could provide the option closeOnSelect which defaults to false.
<USelectMenu @update:modelValue="closeSelectMenu" ref="selectMenu" close-on-selection></USelectMenu>

{
  default: {
    clearSearchOnClose: false,
    showCreateOptionWhen: 'empty',
    closeOnSelection: false
  }
}

Current solution
Current no so beautiful work-around example could be the following:
<script>
  const selectMenu = ref();

  const closeSelectMenu = () => {
    nextTick(() => {
      if (selectMenu.value) {
        const button = $selectMenu.value?.trigger.$el.querySelector('button')
        if(button) button.parentNode.dispatchEvent(new Event('click'))
      }
    });
  }
</script>
<template>
  <USelectMenu @update:modelValue="closeSelectMenu" ref="selectMenu"></USelectMenu>
</template>

Additional context
No response

Unfortunately this is not possible to achieve with Headless UI. This should be possible to achieve in v3 though since you can control with v-model:open: https://ui3.nuxt.dev/components/select-menu#control-open-state


gsd: show untracked files in git stashes

It's easy to lose untracked files in some stash since they are never listed anywhere.
It would be nice if the gsd alias (or e.g. gsD) also included untracked files.

gdu shows untracked files.

I mean stashed but untracked files. gdu only shows currently untracked files.
The dangerous scenario is this:

need to stash everything -> gss
look at the stashed work with gsd some time later, see nothing worth keeping and drop it
realize that the important part was in the untracked files (which are not shown by gsd)

What's the switch to show what you want? It's an alias, not a function.

Maybe something like this:
alias gsD="gsd; echo 'Untracked files:'; git diff --stat --patch 4b825dc642cb6eb9a060e54bf8d69288fbee4904 'stash^3'"


Event Publisher C_1290 SOAP Web Service Request Mgt.

I would like to have this event in place so that other HTTPWebRequest values such as ContentType can be changed:

can't you use instead the newly added HttpClient and co. AL types?

Good point, I should be able to in going forward, I am just working in an older environment at the moment.


Use keep-alive for http-client

keep-alive is not required by Surreal and Rust SDK do not use it. So it is not very important, but can be faster and ligther to use instead of independent requests


MultipleFileUploadEditor(AllowNonImage = true)

Dear @volkanceylan,
I have some trouble with the MultipleFileUploadEditor: it's working fine with images, text file, excel file etc. but if I try with a pdf or an executable file (about 3MB) then I get this:

with the related exception in the BeforeSave() method.
I'm doing something wrong?
Also I've seen that the plugin (jQuery File Upload Plugin 5.40.0) it's a bit outdated...it can be safely upgraded with the last version from here https://blueimp.github.io/jQuery-File-Upload/ or it requires also a server side patch?
Thank you very much
Bye

Sorry some connection problem: duplicated!


Incorporate logit processors into generation API

@rlouf FYI

By that do you mean for all models that can be used in Outlines?

Closing since this is goind to be fixed in #556


Istio sidecar-proxy is not using workload certs to connect to remote Istiod

Bug description
Istio sidecar proxy running on VM, is not using workload certs after initial connection with token. When token expires after 3600s, I will get following errors.
sidecar-proxy-on-vm-logs: warn    xdsproxy        upstream terminated with unexpected error rpc error: code = Unauthenticated desc = authentication failure
remote-isitod-logs:
error	ads	Failed to authenticate client from ip:49908: Authenticator ClientCertAuthenticator: no verified chain is found;

Authenticator KubeJWTAuthenticator: failed to validate the JWT from cluster "Kubernetes": the service account

authentication returns an error: [invalid bearer token, Token has expired., unknown]

After I regenerate and provide new token to sidecar-proxy connection resumes.
/etc/istio/proxy/envoy-rev0.json file always has PROV_CERT as /etc/certs/root-cert.pem.
even in my sidecar.env is set as /etc/certs folder.
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Expected behavior
Since token is temp authentication method, I'd expect sidecar-proxy to use workload certs generated on /etc/certs folder.
Steps to reproduce the bug
followed this docs
Version (include the output of istioctl version --remote and kubectl version --short and helm version --short if you used Helm)
Istio 1.10.0
How was Istio installed?
istioctl install -f config.yaml - IstioOperator options.
sidecar-proxy on vm is installed by following istio.io docs.
Environment where the bug was observed (cloud vendor, OS, etc)
AWS, Amazon Linux2.
Additionally, please consider running istioctl bug-report and attach the generated cluster-state tarball to this issue.
Refer cluster state archive for more details.

failed to fetch token from file: open ./var/run/secrets/tokens/istio-token: no such file or directory
I got this error when I restarted the istio agent and removed the token.

@howardjohn Can you advise here please?
It's not using the certs. It keeps using the temp token

Code to configure tls: https://github.com/istio/istio/blob/b53449263b1b23d68a04ddef64a9b6fe64946c6d/pkg/istio-agent/xds_proxy.go#L644
So its basically just looking at the PROV_CERTS

PROV_CERT as /etc/certs/root-cert.pem

PROV_CERT is supposed to be a folder like /etc/certs

@howardjohn  If I set PROV_CERT, not PROV_CERTS as /etc/certs, should this work?
I had it set as PROV_CERT in /var/lib/envoy/sidecar.env. I was still seeing PROV_CERT as /etc/certs/root-ca.pem.
If you can tell me the right place to set that variable ? Here /var/lib/istio/config/mesh or /var/lib/envoy/cluster.env?

@howardjohn  I think PROV_CERTS is wrong variable right ? Should be PROV_CERT?  but I'm not sure where to set it
I would really appreciate your help

@hzxuzhonghu Can please advise here?
After setting JWT_POLICY="" in sidecar.env, I see a log  Using existing certs, I expect Istio to start using certs right after when certs are generated.
but PROV_CERT is still set to /etc/cert/root-ca.pem, not sure where to set the var PROV_CERT:  /etc/cert.
On Istiod I get
Authentication failed for <IP_ADDRESS>:40840: Authenticator ClientCertAuthenticator at index 0 got error: no verified chain is found. Authenticator KubeJWTAuthenticator at index 1 got error: target JWT extraction error: no HTTP authorization header exists
so it's expecting the token.
@howardjohn

client cert is always the first authenticator - you can also get this info in the log as it tried client cert first and failed.
It fails because below two env var is missing -- I will work on a fix for this soon.
@Arsen-Uulu
One workaround for this is to add below two env var into your sidecar.env
$ echo "PROV_CERT=/var/run/secrets/istio" >> sidecar.env
$ echo "OUTPUT_CERTS=/var/run/secrets/istio" >>  sidecar.env

@irisdingbj is it ok to put certs to /var/run? If VM gets restarted, secrets folder will be gone.
I will try this workaround and let you know if it works

/var/run is tmpfs and after reboot secrets folder will be gone. Is there a way to specify diff location?

/var/run is tmpfs and after reboot secrets folder will be gone. Is there a way to specify diff location?

You can specify any folder according to your need actually, Above is just an exmaple. Just make sure istio-proxy has related authority for this folder
sudo chown -R istio-proxy the-folder-you-defined

@irisdingbj my remote Istiod is behind IngressGateway that exposed as type LoadBalancer handling TLS with my signed cert. If I don't provide my cert to istio-sidecar it's complaining certificate is signed my unknown authority, but if I do provide in /etc/certs/root-cert.pem, connection is established using the temp istio-token, but once token expires, I get error from Istiod that no no valid certificate chain found. Can you suggest some work around? Please

One possible work around is to let your sidecar use system certs like what external control plane[https://istio.io/latest/docs/setup/install/external-controlplane/]  is currently doing:
      proxyMetadata:
        XDS_ROOT_CA: /etc/ssl/certs/ca-certificates.crt
        CA_ROOT_CA: /etc/ssl/certs/ca-certificates.crt

@irisdingbj the ca-certificates.crt should contain only my cert? or Istio CA root cert as well?

warn	serverca	Authentication failed for <IP_ADDRESS>:33914: Authenticator ClientCertAuthenticator at index 0 got error: no verified chain is found.
I'm getting this error. I'm not sure why it can't validate the client certs

@irisdingbj my issue is that Gateway has the cert that I created for my DNS_NAME($ISTIOD_INGRESS),
How I can provide that cert to istio-sidecar trust store running on VM so I don't get this error?
  warn    sds     failed to warm certificate: failed to generate workload certificate: create certificate: rpc error: code = Unavailable desc = connection error: desc = "transport: authentication handshake failed: x509: certificate signed by unknown authority"
If I provide it in root-cert.pem I no longer get that error , but I get no valid certificate chain found.
If I provide as chain with Istio root CA cert I still get that error no valid certificate chain found`.

https://istio.io/latest/docs/setup/install/external-controlplane/  This will help.


[Quantum]Add Azure quantum api key support

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Quantum

/azp run

Can someone manually trigger the integration tests to rerun? The service is deployed.