HBASE-24640 [branch-1] Purge use of VisibleForTesting

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

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

_ branch-1 Compile Tests _

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

+1 :green_heart:
mvninstall
8m  0s
branch-1 passed

+1 :green_heart:
compile
2m 23s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
compile
2m 46s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
checkstyle
5m  0s
branch-1 passed

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

+1 :green_heart:
javadoc
2m 11s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
2m 34s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+0 :ok:
spotbugs
2m 44s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
7m 47s
branch-1 passed

_ Patch Compile Tests _

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

+1 :green_heart:
mvninstall
1m 59s
the patch passed

+1 :green_heart:
compile
2m 19s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javac
2m 19s
the patch passed

+1 :green_heart:
compile
2m 40s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

-1 :x:
javac
0m 44s
hbase-server-jdkAzulSystems,Inc.-1.7.0_272-b10 with JDK Azul Systems, Inc.-1.7.0_272-b10 generated 2 new + 4 unchanged - 2 fixed = 6 total (was 6)

-1 :x:
checkstyle
0m 35s
hbase-common: The patch generated 3 new + 335 unchanged - 10 fixed = 338 total (was 345)

-1 :x:
checkstyle
0m 14s
hbase-procedure: The patch generated 1 new + 20 unchanged - 1 fixed = 21 total (was 21)

-1 :x:
checkstyle
0m 45s
hbase-client: The patch generated 5 new + 491 unchanged - 16 fixed = 496 total (was 507)

+1 :green_heart:
checkstyle
0m 13s
hbase-metrics-api: The patch generated 0 new + 1 unchanged - 1 fixed = 1 total (was 2)

-1 :x:
checkstyle
0m 15s
hbase-hadoop2-compat: The patch generated 1 new + 7 unchanged - 4 fixed = 8 total (was 11)

-1 :x:
checkstyle
2m 24s
hbase-server: The patch generated 28 new + 2413 unchanged - 83 fixed = 2441 total (was 2496)

+1 :green_heart:
checkstyle
0m 17s
The patch passed checkstyle in hbase-rsgroup

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

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

+1 :green_heart:
hadoopcheck
4m 34s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
2m  5s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
2m 34s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
findbugs
8m 47s
the patch passed

_ Other Tests _

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

+1 :green_heart:
unit
3m 57s
hbase-procedure in the patch passed.

+1 :green_heart:
unit
2m 42s
hbase-client in the patch passed.

+1 :green_heart:
unit
0m 16s
hbase-metrics-api in the patch passed.

+1 :green_heart:
unit
0m 37s
hbase-hadoop2-compat in the patch passed.

+1 :green_heart:
unit
98m 28s
hbase-server in the patch passed.

+1 :green_heart:
unit
12m 21s
hbase-rsgroup in the patch passed.

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

199m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/1/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux c6d2c53bf241 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-2697/out/precommit/personality/provided.sh

git revision
branch-1 / edccfe4

Default Java
Azul Systems, Inc.-1.7.0_272-b10

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:Azul Systems, Inc.-1.8.0_262-b19 /usr/lib/jvm/zulu-7-amd64:Azul Systems, Inc.-1.7.0_272-b10

javac
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/1/artifact/out/diff-compile-javac-hbase-server-jdkAzulSystems,Inc.-1.7.0_272-b10.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/1/artifact/out/diff-checkstyle-hbase-common.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/1/artifact/out/diff-checkstyle-hbase-procedure.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/1/artifact/out/diff-checkstyle-hbase-client.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/1/artifact/out/diff-checkstyle-hbase-hadoop2-compat.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/1/artifact/out/diff-checkstyle-hbase-server.txt

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

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

modules
C: hbase-common hbase-procedure hbase-client hbase-metrics-api hbase-hadoop2-compat hbase-server hbase-rsgroup U: .

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

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

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

This message was automatically generated.

I just merged https://github.com/apache/hbase/pull/2658, it probably introduced one conflict with this PR.

@Reidddddd Will rebase.
I can also make a pass over these checkstyle warns. Was expecting some legacy issues to turn up.

Rebased.
Pushed checkstyle fixes.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

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

_ branch-1 Compile Tests _

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

+1 :green_heart:
mvninstall
7m 55s
branch-1 passed

+1 :green_heart:
compile
2m 24s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
compile
2m 51s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
checkstyle
5m 21s
branch-1 passed

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

+1 :green_heart:
javadoc
2m 18s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
2m 43s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+0 :ok:
spotbugs
2m 42s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
7m 46s
branch-1 passed

_ Patch Compile Tests _

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

-1 :x:
mvninstall
1m  8s
root in the patch failed.

-1 :x:
compile
0m 26s
hbase-server in the patch failed with JDK Azul Systems, Inc.-1.8.0_262-b19.

-1 :x:
javac
0m 26s
hbase-server in the patch failed with JDK Azul Systems, Inc.-1.8.0_262-b19.

-1 :x:
compile
0m 30s
hbase-server in the patch failed with JDK Azul Systems, Inc.-1.7.0_272-b10.

-1 :x:
javac
0m 30s
hbase-server in the patch failed with JDK Azul Systems, Inc.-1.7.0_272-b10.

+1 :green_heart:
checkstyle
0m 33s
hbase-common: The patch generated 0 new + 335 unchanged - 10 fixed = 335 total (was 345)

-1 :x:
checkstyle
0m 15s
hbase-procedure: The patch generated 1 new + 20 unchanged - 1 fixed = 21 total (was 21)

+1 :green_heart:
checkstyle
0m 44s
hbase-client: The patch generated 0 new + 491 unchanged - 16 fixed = 491 total (was 507)

+1 :green_heart:
checkstyle
0m 12s
hbase-metrics-api: The patch generated 0 new + 1 unchanged - 1 fixed = 1 total (was 2)

+1 :green_heart:
checkstyle
0m 14s
hbase-hadoop2-compat: The patch generated 0 new + 7 unchanged - 4 fixed = 7 total (was 11)

-1 :x:
checkstyle
2m 22s
hbase-server: The patch generated 7 new + 2413 unchanged - 96 fixed = 2420 total (was 2509)

+1 :green_heart:
checkstyle
0m 16s
The patch passed checkstyle in hbase-rsgroup

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

-1 :x:
shadedjars
1m 45s
patch has 16 errors when building our shaded downstream artifacts.

-1 :x:
hadoopcheck
1m  0s
The patch causes 16 errors with Hadoop v2.8.5.

-1 :x:
hadoopcheck
2m 43s
The patch causes 16 errors with Hadoop v2.9.2.

-1 :x:
javadoc
0m 29s
hbase-server-jdkAzulSystems,Inc.-1.8.0_262-b19 with JDK Azul Systems, Inc.-1.8.0_262-b19 generated 1 new + 8 unchanged - 0 fixed = 9 total (was 8)

-1 :x:
javadoc
0m 40s
hbase-server-jdkAzulSystems,Inc.-1.7.0_272-b10 with JDK Azul Systems, Inc.-1.7.0_272-b10 generated 1 new + 8 unchanged - 0 fixed = 9 total (was 8)

-1 :x:
findbugs
0m 29s
hbase-server in the patch failed.

_ Other Tests _

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

+1 :green_heart:
unit
4m 21s
hbase-procedure in the patch passed.

+1 :green_heart:
unit
3m  9s
hbase-client in the patch passed.

+1 :green_heart:
unit
0m 21s
hbase-metrics-api in the patch passed.

+1 :green_heart:
unit
0m 46s
hbase-hadoop2-compat in the patch passed.

-1 :x:
unit
0m 41s
hbase-server in the patch failed.

+1 :green_heart:
unit
15m 47s
hbase-rsgroup in the patch passed.

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

97m 28s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux b405acddfc8c 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
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-2697/out/precommit/personality/provided.sh

git revision
branch-1 / 4640b22

Default Java
Azul Systems, Inc.-1.7.0_272-b10

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:Azul Systems, Inc.-1.8.0_262-b19 /usr/lib/jvm/zulu-7-amd64:Azul Systems, Inc.-1.7.0_272-b10

mvninstall
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/patch-mvninstall-root.txt

compile
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/patch-compile-hbase-server-jdkAzulSystems,Inc.-1.8.0_262-b19.txt

javac
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/patch-compile-hbase-server-jdkAzulSystems,Inc.-1.8.0_262-b19.txt

compile
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/patch-compile-hbase-server-jdkAzulSystems,Inc.-1.7.0_272-b10.txt

javac
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/patch-compile-hbase-server-jdkAzulSystems,Inc.-1.7.0_272-b10.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/diff-checkstyle-hbase-procedure.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/diff-checkstyle-hbase-server.txt

shadedjars
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/patch-shadedjars.txt

hadoopcheck
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/patch-javac-2.8.5.txt

hadoopcheck
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/patch-javac-2.9.2.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/diff-javadoc-javadoc-hbase-server-jdkAzulSystems,Inc.-1.8.0_262-b19.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/diff-javadoc-javadoc-hbase-server-jdkAzulSystems,Inc.-1.7.0_272-b10.txt

findbugs
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/patch-findbugs-hbase-server.txt

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/artifact/out/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/2/testReport/

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

modules
C: hbase-common hbase-procedure hbase-client hbase-metrics-api hbase-hadoop2-compat hbase-server hbase-rsgroup U: .

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

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

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

This message was automatically generated.

Let me make another pass to fix up checkstyle and insure VisibleForTesting is really gone. In one precommit it seems to have come back somewhere.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

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

_ branch-1 Compile Tests _

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

+1 :green_heart:
mvninstall
7m 56s
branch-1 passed

+1 :green_heart:
compile
2m 27s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
compile
2m 50s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
checkstyle
5m 25s
branch-1 passed

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

+1 :green_heart:
javadoc
2m 21s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
2m 42s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+0 :ok:
spotbugs
2m 47s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
7m 54s
branch-1 passed

_ Patch Compile Tests _

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

+1 :green_heart:
mvninstall
1m 54s
the patch passed

+1 :green_heart:
compile
2m 29s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

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

+1 :green_heart:
compile
2m 50s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

-1 :x:
javac
0m 45s
hbase-server-jdkAzulSystems,Inc.-1.7.0_272-b10 with JDK Azul Systems, Inc.-1.7.0_272-b10 generated 2 new + 4 unchanged - 2 fixed = 6 total (was 6)

+1 :green_heart:
checkstyle
0m 39s
hbase-common: The patch generated 0 new + 335 unchanged - 10 fixed = 335 total (was 345)

+1 :green_heart:
checkstyle
0m 18s
hbase-procedure: The patch generated 0 new + 20 unchanged - 1 fixed = 20 total (was 21)

+1 :green_heart:
checkstyle
0m 47s
hbase-client: The patch generated 0 new + 491 unchanged - 16 fixed = 491 total (was 507)

+1 :green_heart:
checkstyle
0m 14s
hbase-metrics-api: The patch generated 0 new + 1 unchanged - 1 fixed = 1 total (was 2)

+1 :green_heart:
checkstyle
0m 17s
hbase-hadoop2-compat: The patch generated 0 new + 7 unchanged - 4 fixed = 7 total (was 11)

-1 :x:
checkstyle
2m 37s
hbase-server: The patch generated 2 new + 2417 unchanged - 92 fixed = 2419 total (was 2509)

+1 :green_heart:
checkstyle
0m 19s
The patch passed checkstyle in hbase-rsgroup

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

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

+1 :green_heart:
hadoopcheck
4m 43s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
2m  8s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
2m 40s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
findbugs
9m  2s
the patch passed

_ Other Tests _

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

+1 :green_heart:
unit
4m  4s
hbase-procedure in the patch passed.

+1 :green_heart:
unit
2m 43s
hbase-client in the patch passed.

+1 :green_heart:
unit
0m 18s
hbase-metrics-api in the patch passed.

+1 :green_heart:
unit
0m 39s
hbase-hadoop2-compat in the patch passed.

+1 :green_heart:
unit
90m 35s
hbase-server in the patch passed.

+1 :green_heart:
unit
13m 55s
hbase-rsgroup in the patch passed.

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

195m 39s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/3/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 753df3773eb7 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-2697/out/precommit/personality/provided.sh

git revision
branch-1 / 4640b22

Default Java
Azul Systems, Inc.-1.7.0_272-b10

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:Azul Systems, Inc.-1.8.0_262-b19 /usr/lib/jvm/zulu-7-amd64:Azul Systems, Inc.-1.7.0_272-b10

javac
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/3/artifact/out/diff-compile-javac-hbase-server-jdkAzulSystems,Inc.-1.7.0_272-b10.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2697/3/artifact/out/diff-checkstyle-hbase-server.txt

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

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

modules
C: hbase-common hbase-procedure hbase-client hbase-metrics-api hbase-hadoop2-compat hbase-server hbase-rsgroup U: .

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

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

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

This message was automatically generated.


CFE does not report error when a setter's parameter type is not assignable to the corresponding getter's return type.

Example:
class C {
  String get foo => "42";
  void set foo(int x) {}
}
main() {
  new C().foo;
  new C().foo = 42;
}

This should cause an error.
The spec says:

It is a compile-time error if a class has a setter named v= with argument type T and a getter named v with return type S, and S may not be assigned to T.

The analyzer shows a warning.

cc @dgrove @leafpetersen
If we plan on fixing this post Dart 2.0 it will be breaking.

Will the analyzer switch warnings to errors before Dart 2?  @devoncarew
If not, we need to treat doing so later as a non-breaking change.  As long as the analyzer gives an error about this though, the missing CFE error isn't blocking.

Will the analyzer switch warnings to errors before Dart 2?

Right now we're showing the severity levels from strong mode:

It is a compile-time error

I don't know if this is an isolated bug or we need to promote a group of warnings to errors. Either way, it's an easy fix.

If not, we need to treat doing so later as a non-breaking change.

A warning is already a failing severity from the POV of dartanalyzer, so promoting it to an error won't be hugely breaking for users.

For posterity, this is the mismatched_getter_and_setter_types error code.

A warning is already a failing severity from the POV of dartanalyzer, so promoting it to an error won't be hugely breaking for users.

I can't repro. When I run dartanalyzer and it prints a warning I get a zero exit code. Internally we probably default to --fatal-warnings but that is not the default externally.

No, you're right, I see that --fatal-warnings defaults to false.

Bringing back into Dart2Stable

It looks like the warning ==> error spec changed happened recently: https://github.com/dart-lang/sdk/commit/a07b2a10c6da58e0f3c461d778e51b128e2772a5. I'll open an issue for this.

https://github.com/dart-lang/sdk/issues/33837

But I'm not sure the CFE portion of this needs to be in dart2 stable.

Since the analyzer issue #33837 is in Dart2Stable, this doesn't need to be.

I'm guessing this is unplanned since the decision was made to keep this a warning.


Browser refresh is broken in CASMgmt Dashboard. Generates a 404

Brief description of changes applied:

Refreshing the browser generates a 404. Routing issue with Angular SPA. This config change fixes the browser refresh issue, although other robust recommended solutions may exist.

Any documentation on how to configure, test:

Open CasMgmt Dashboard. Search and navigate to a server. Refresh the browser.

Any possible limitations, side effects, etc

Nothing so far.

Reference any other pull requests that might be related:

N/A

@tsschmidt  I'll be closing this PR. I don't see this issue happening with the latest 6.3.3-SNAPSHOT build.


Getting started

It would be really useful here to have a Getting started path that just used the Developer Token to query Bing Ads API. I'm trying to use the Management Model for Direct Advertisers so an Oauth walkthrough is of limited use.

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

ID: e5819ecf-4666-9f16-b735-64bd65610bdb
Version Independent ID: 5b66a562-2c4b-a5fc-b8ed-ffaf6ad43c73
Content: Walkthrough: Bing Ads Web Application in PHP - Bing Ads
Content Source: BingAds/bingads-12/guides/walkthrough-web-application-php.md
Service: bing-ads
Product: unspecified
GitHub Login: @eric-urban
Microsoft Alias: eur

@tobyberesford it isn't possible to use the Bing Ads API with only a DeveloperToken. You must authenticate with credentials of a user who has access to a Bing Ads account. Starting in version 12 OAuth is required. I'm reviewing your other feedback now, and will reply to each. Thanks!


Transitive dependencies are not resolved

I have the same problem described in issue #12 which is closed.
An example repository can be found here: https://github.com/realityfilter/bug-transitive-deps
the structure is
app: uses liba
liba: uses libb
libb
Just run nx serve app
Resulting in the error:
  The following dependencies are imported but could not be resolved:
  @trans-dep/libb (imported by /Users/vb/Desktop/nx-vite-dep/trans-dep/packages/liba/src/lib/liba.ts)

I tried the workaround described in issue #12:
tsconfigPaths({
  root: '../..',
})

with no success.

what do the debug logs say?
DEBUG="vite-tsconfig-paths" vite

> nx run app:serve

2022-02-22T18:08:05.762Z vite-tsconfig-paths crawling "/path/Desktop/trans-dep"
2022-02-22T18:08:05.764Z vite-tsconfig-paths options: {
  projects: [
    '/path/Desktop/trans-dep/packages/app/tsconfig.json',
    '/path/Desktop/trans-dep/packages/liba/tsconfig.json',
    '/path/Desktop/trans-dep/packages/libb/tsconfig.json'
  ],
  extensions: [ '.ts', '.tsx', '.js', '.jsx', '.mjs' ]
}
2022-02-22T18:08:05.768Z vite-tsconfig-paths config loaded: {
  configPath: '/path/Desktop/trans-dep/packages/app/tsconfig.json',
  include: [],
  exclude: [ 'node_modules', 'tmp' ],
  allowJs: undefined,
  baseUrl: '/path/Desktop/trans-dep',
  paths: {
    '@trans-dep/liba': [ 'packages/liba/src/index.ts' ],
    '@trans-dep/libb': [ 'packages/libb/src/index.ts' ]
  },
  outDir: undefined
}
2022-02-22T18:08:05.768Z vite-tsconfig-paths compiled globs: {
  included: [],
  excluded: [
    /^\.\/node_modules\/((?:[^/]*(?:\/|$))*)$/,
    /^\.\/tmp\/((?:[^/]*(?:\/|$))*)$/
  ]
}
2022-02-22T18:08:05.769Z vite-tsconfig-paths config loaded: {
  configPath: '/path/Desktop/trans-dep/packages/liba/tsconfig.json',
  include: [],
  exclude: [ 'node_modules', 'tmp' ],
  allowJs: undefined,
  baseUrl: '/path/Desktop/trans-dep',
  paths: {
    '@trans-dep/liba': [ 'packages/liba/src/index.ts' ],
    '@trans-dep/libb': [ 'packages/libb/src/index.ts' ]
  },
  outDir: undefined
}
2022-02-22T18:08:05.769Z vite-tsconfig-paths compiled globs: {
  included: [],
  excluded: [
    /^\.\/node_modules\/((?:[^/]*(?:\/|$))*)$/,
    /^\.\/tmp\/((?:[^/]*(?:\/|$))*)$/
  ]
}
2022-02-22T18:08:05.770Z vite-tsconfig-paths config loaded: {
  configPath: '/path/Desktop/trans-dep/packages/libb/tsconfig.json',
  include: [],
  exclude: [ 'node_modules', 'tmp' ],
  allowJs: undefined,
  baseUrl: '/path/Desktop/trans-dep',
  paths: {
    '@trans-dep/liba': [ 'packages/liba/src/index.ts' ],
    '@trans-dep/libb': [ 'packages/libb/src/index.ts' ]
  },
  outDir: undefined
}
2022-02-22T18:08:05.771Z vite-tsconfig-paths compiled globs: {
  included: [],
  excluded: [
    /^\.\/node_modules\/((?:[^/]*(?:\/|$))*)$/,
    /^\.\/tmp\/((?:[^/]*(?:\/|$))*)$/
  ]
}
2022-02-22T18:08:05.771Z vite-tsconfig-paths options: {
  projects: [
    '/path/Desktop/trans-dep/packages/app/tsconfig.app.json'
  ],
  extensions: [ '.ts', '.tsx', '.js', '.jsx', '.mjs' ]
}
2022-02-22T18:08:05.772Z vite-tsconfig-paths config loaded: {
  configPath: '/path/Desktop/trans-dep/packages/app/tsconfig.app.json',
  include: [ '**/*.ts' ],
  exclude: [ '**/*.spec.ts', '**/*.test.ts' ],
  allowJs: undefined,
  baseUrl: '/path/Desktop/trans-dep',
  paths: {
    '@trans-dep/liba': [ 'packages/liba/src/index.ts' ],
    '@trans-dep/libb': [ 'packages/libb/src/index.ts' ]
  },
  outDir: '../../dist/out-tsc'
}
2022-02-22T18:08:05.772Z vite-tsconfig-paths compiled globs: {
  included: [ /^\.\/((?:[^/]*(?:\/|$))*)([^/]*)\.ts$/ ],
  excluded: [
    /^\.\/((?:[^/]*(?:\/|$))*)([^/]*)\.spec\.ts$/,
    /^\.\/((?:[^/]*(?:\/|$))*)([^/]*)\.test\.ts$/,
    /^\.\.\/\.\.\/dist\/out-tsc\/((?:[^/]*(?:\/|$))*)$/
  ]
}
2022-02-22T18:08:05.805Z vite-tsconfig-paths resolved: {
  id: '@trans-dep/liba',
  importer: '/path/Desktop/trans-dep/packages/app/src/main.ts',
  resolvedId: '/path/Desktop/trans-dep/packages/liba/src/index.ts',
  configPath: '/path/Desktop/trans-dep/packages/app/tsconfig.app.json'
}
The following dependencies are imported but could not be resolved:

  @trans-dep/libb (imported by /path/Desktop/trans-dep/packages/liba/src/lib/liba.ts)

Are they installed?

Could you please reproduce the issue in a Stackblitz or the ./demo folder of this repo (then open a PR)?
So I can debug easier :)

There were two problems in our nx workspace:

no root tsconfig.json file. Nx only has a tsconfig.base.json
no allowJs, checkJs is configured in tsconfig.json and no loose flag was set, so importing vue files was not successfull

So no bug, closing this.

I ran into this as well and have posted how I managed resolved it based on the notes in this thread and elsewhere in the following issue: https://github.com/aleclarson/vite-tsconfig-paths/issues/12#issuecomment-1081160667


slow

What is the issue?
The program is very slow
3.10 ollama
OS
Windows
GPU
Intel
CPU
Intel
Ollama version
3.10

your hardware ?

And what model are you using ? I’ve pushed my Mac M2 (64 GB RAM) to its limits just trying to use LLaMA 3.1:70B." - ollama went hard to sigkill

You can use ollama ps to figure out if the model is loaded into CPU or GPU.

That fact they list intel GPU is very telling on why it is very slow.

lama3.1 8q
no viga
Unusually slow, I deleted the first one and reinstalled it, and the same problem.

i7
64ram

unless you have a modern nVidia or AMD GPU, it will continue to be slow

and what do you think is slow, can you think faster ?, my llama3.1:70B model is slow, like 12 minutes for a response, but i find it rather fast enough and workable, i can give it tasks in the future to solve

Hi @ayttop , sorry this is going slow for you. It's because you're using CPU inference (which you can tell with ollama ps). I'm not certain what your GPU is, but the Intel GPUs unfortunately aren't supported (yet). There are a number of issues already around supporting that, so I'm going to go ahead and close this issue (like #1590) so I'll go ahead and close this issue.


Possible to use with custom API?

Hi,
I've API that also has auth by JWT can I plug this next-auth to my API?
Thank you

I don't know what your API looks like but you can try creating a custom provider and configure it with your own API. You can find more about this in the provider docs.

I mean like I've API write with Nest.js that serving JWT Authentication. So can I use it with next-auth?

So JWT support is definitely something I want to add, and MIGHT make it in for 2.0 (undecided).
However, it's a big tricker to accommodate folks with an existing API, but is very useful to think about as it might shape how the implementation works and what callbacks are exposed.

A screenshot of what cookie it is stored in looks like in a browser inspector (e.g. name, path, prefix, expiry, etc) would also helpful.

For my API I use Set-Cookie header with httpOnly and secure (on production)

@wiput1999 Thanks! I'll get back to this ticket in a bit!

I'm going to track this in  #167 if that's alright.
I have linked back to this issue for reference, but am setting out thoughts there; feel free to chime in if you have any specific requirements or see needs that are not addressed by proposal!


Refs 184: add ouia and tooltips

Hello
Further to
https://github.com/content-services/content-sources-frontend/pull/47
Adding some more OUIA IDs for the edit form and fixing surrounding tool tips.
https://issues.redhat.com/browse/HMSCONTENT-184

https://issues.redhat.com/browse/HMSCONTENT-184

⚠ This task currently requires qe-approval, but this PR does not fully resolve the task.  Please contact QE to determine appropriate testing required.


Fixed: The add button is not disabled

Closes #149
Changes proposed
Now the save button is disabled for empty as well as for invalid repo URLs, it only enables If the repo URL is valid.
Screenshots
Screencast from 2024-08-16 16-50-05.webm

Please fix the failing tests

Sure will look into it


Heatmap: colors

The heamap colors do not work well with the basemap.  The grey used for the lowest scale of the heatmap blend in both to the terrain and the buildings.  For example, see the heatmap dot in the center of the attached image.  We need to use more striking colors.

Changed colors, the color representing the lowest value probably needs further adjustment.

new colors available for testing at http://dev.jdarchive.org/heatmap/web/en/search#view_type=event&


Dataframe.concat generating large numbers of tasks with overlapping time indexed dataframes

When I concat DateTimeIndexed dataframes together, that have overlapping partitions,  dask generates a large number of repartition-merge , repartition-split tasks. With 400 dataframes I have seen it generate 600,000 tasks even when each dataframe only has one partition, I can't reproduce quite the same number using data generated inline, but did get 300k.
I want to establish if this is just a catch of the way concat works and if it is, find a way to mitigate it for my dataset.
Here is the sample of code that generates 320800 tasks from 400 dataframes.
from datetime import datetime, timedelta

import dask.dataframe as dd
import numpy as np
import pandas as pd

one_day = timedelta(days=1)
start = datetime(2017, 1, 1)
end = datetime(2017, 1, 2)
dataframes = []
for i in range(0, 400):
    one_second =<PHONE_NUMBER>
    d_range = pd.date_range(start=start, end=start + one_day, freq='{0}ns'.format(one_second + i))
    df = pd.DataFrame(np.random.randn(len(d_range), 1), index=d_range, columns=list('A'))
    df = dd.from_pandas(df, npartitions=1)
    dataframes.append(df)
print(dd.concat(dataframes, interleave_partitions=True))

Thank you for the reproducible example.  Taking a look now.

OK, so we have several dataframes with slightly overlapping divisions
|     | |
|     |  |
|     |   |
|     |    |

When we align these divisions we come up with the following output divisions
|     | ||||

Which we then tell everyone to match
|     | |
|     | ||
|     | |||
|     | ||||

Each repartitioning has about as many divisions as we have dataframes, so this is roughly O(n^2).
One solution here would be to change the align_partitions function to be less conservative, and willing to merge partitions a bit more.  This is a bit dangerous (merging partitions here may overwhelm memory) but might be ok to do a bit.
In this particular case we probably wanted to convert these divisions
|     | |
|     |  |
|     |   |
|     |    |

Into this output
|     |    |

Or, if we wanted to be safe, maybe something like the following:
|     |  |  |

The relevant code is align_partitions in dataframe/multi.py
I don't immediately see an obvious clean way to do this that is decent in all situations.  I imagine this might take some thought.  @shughes-uk if you have any interest in diving in here that would be welcome.  The align_partitions code is not trivial, but is well separated from the rest of dask.dataframe.
The line to change is this one:
    divisions = list(unique(merge_sorted(*[df.divisions for df in dfs1])))

We would need to be a bit more clever there

I feel a little in over my head but i'll have a look

@eriknw you might enjoy this problem.  It's interesting, not clear how best to handle all situations, and well isolated.

ping received, thanks!


[BEAM-10543] Upgrade Kafka cross-language python tests

Added additional parameters to Kafka's Read cross-language transform and upgrade the integration test to be run in postcommit suite instead of separate task.
The params max_num_records and max_read_time are very useful for testing.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run Python 3.8 PostCommit

Run Python 3.8 PostCommit

Run Python 3.8 PostCommit

pull_request

Run Python 2 PostCommit

@chamikaramj Could I ask you for review?

R: @ihji can you review ?

@ihji @chamikaramj ping

Thanks, looks good to me overall.
Is there any way to share the pipeline codes in CrossLanguageKafkaIOTest and CrossLanguageKafkaIO? Could we share the pipeline codes and use max_num_records only in CrossLanguageKafkaIOTest? The internal test only checks sum_counter so you could keep any value in the PCollection afterward and assert it.

Run Python 3.7 PostCommit

@ihji @chamikaramj ping

Thanks, looks good to me overall.
Is there any way to share the pipeline codes in CrossLanguageKafkaIOTest and CrossLanguageKafkaIO? Could we share the pipeline codes and use max_num_records only in CrossLanguageKafkaIOTest? The internal test only checks sum_counter so you could keep any value in the PCollection afterward and assert it.

It required a small modification (triggers don't work with bounded source - max_num_records changes unbounded to bounded) and sum_counter returns [None], so I use just ReadFromKafka and skip the rest of the read pipeline.

 Thanks!

Thanks! I've cleared up the commit history so the change is ready to be merged :)

@chamikaramj Could you merge if you don't have any further suggestions?


Drill down from package statistics into package version statistics

When viewing package downloads for a package, grouped by Versions, click the Download number to drill into the package downloads for a package version.
If you're grouped by Version AND other fields, we should navigate into the version-specific statistics, grouped by the other fields you were already grouped by.

"package downloads for a package versions" is unreachable.
We should definetly link to it.
@anangaur ?


fix leg 6 log rorate output

Seems like there is a config change on the log rorate, the proper fix should be adding something in/vertica-kubernetes/tests/e2e-leg-6/verify-server-logrotate/sql-input/logrotate.sql, to setup the log level or something so we don't get the new generated lines. But I can't find anything on the public doc: https://docs.vertica.com/24.4.x/en/admin/monitoring/rotating-log-files/


Provide a changelog

Provide a changelog to alow user keep up with changes made on each version.
The changelog can be provided or on a changelog file or using the Github releases page.

https://github.com/pakrym/jab/blob/main/CHANGELOG.md


screencopy: handle compositor not setting damage

Damage the full output if the compositor didn't submit damage but did
submit a buffer.

Added your suggestions in separate commits and retested to make sure my use-case still works (which it does). If the commits look good to you I'd say this is good to go.


Spectrum Calculation Crashes

ChroMo crashes if spectrum calculation is attempted with no prior segmentation. Should provide a notification, instead.

Fixed in https://github.com/danilexn/chromo/commit/1ff60b45ff9d3bcd9c1d9378ab59713e70a81fc3


Migrate Delete Detector to SDK using SDKRestClient

Is your feature request related to a problem?
Part of https://github.com/opensearch-project/OpenSearch/issues/5224
What solution would you like?
Migrate the Delete Detector Rest API to Extensions SDK using SDKRestClient.
Use the implementation of #353 as a reference.  Add new issues for any steps you skip.
Implement the following API endpoints:

[ ] DELETE _plugins/_anomaly_detection/detectors/<detectorId>

Previously, the draft PR was blocked because of the actions coming as null on SDK which has been resolved.
From the below error, AD task is not being searched because .opendistro-anomaly-detection-state index is not created. The index gets created while starting the detector here. Keeping this issue on hold unless https://github.com/opensearch-project/opensearch-sdk-java/issues/383 is done.
18:05:32.063 [httpclient-dispatch-2] ERROR org.opensearch.ad.task.ADTaskManager - Failed to search AD task for detector ejOl3YYBSYtm-lGhtfp-
org.opensearch.OpenSearchStatusException: OpenSearch exception [type=index_not_found_exception, reason=no such index [.opendistro-anomaly-detection-state]]
        at org.opensearch.rest.BytesRestResponse.errorFromXContent(BytesRestResponse.java:207) ~[opensearch-3.0.0-SNAPSHOT.jar:3.0.0-SNAPSHOT]
        at org.opensearch.client.RestHighLevelClient.parseEntity(RestHighLevelClient.java:2228) ~[opensearch-rest-high-level-client-3.0.0-SNAPSHOT.jar:?]
        at org.opensearch.client.RestHighLevelClient.parseResponseException(RestHighLevelClient.java:2205) ~[opensearch-rest-high-level-client-3.0.0-SNAPSHOT.jar:?]
        at org.opensearch.client.RestHighLevelClient$1.onFailure(RestHighLevelClient.java:2116) [opensearch-rest-high-level-client-3.0.0-SNAPSHOT.jar:?]
        at org.opensearch.client.RestClient$FailureTrackingResponseListener.onDefinitiveFailure(RestClient.java:729) [opensearch-rest-client-3.0.0-SNAPSHOT.jar:?]
        at org.opensearch.client.RestClient$1.completed(RestClient.java:450) [opensearch-rest-client-3.0.0-SNAPSHOT.jar:?]
        at org.opensearch.client.RestClient$1.completed(RestClient.java:434) [opensearch-rest-client-3.0.0-SNAPSHOT.jar:?]
        at org.apache.hc.core5.concurrent.BasicFuture.completed(BasicFuture.java:123) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.concurrent.ComplexFuture.completed(ComplexFuture.java:79) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.client5.http.impl.async.InternalAbstractHttpAsyncClient$1$3$1.completed(InternalAbstractHttpAsyncClient.java:299) [httpclient5-5.1.4.jar:5.1.4]
        at org.apache.hc.core5.http.nio.support.AbstractAsyncResponseConsumer$2.completed(AbstractAsyncResponseConsumer.java:108) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.http.nio.entity.AbstractBinAsyncEntityConsumer.completed(AbstractBinAsyncEntityConsumer.java:84) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.http.nio.entity.AbstractBinDataConsumer.streamEnd(AbstractBinDataConsumer.java:81) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.http.nio.support.AbstractAsyncResponseConsumer.streamEnd(AbstractAsyncResponseConsumer.java:149) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.client5.http.impl.async.HttpAsyncMainClientExec$1.streamEnd(HttpAsyncMainClientExec.java:235) [httpclient5-5.1.4.jar:5.1.4]
        at org.apache.hc.core5.http.impl.nio.ClientHttp1StreamHandler.dataEnd(ClientHttp1StreamHandler.java:280) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.http.impl.nio.ClientHttp1StreamDuplexer.dataEnd(ClientHttp1StreamDuplexer.java:366) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.http.impl.nio.AbstractHttp1StreamDuplexer.onInput(AbstractHttp1StreamDuplexer.java:335) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.http.impl.nio.AbstractHttp1IOEventHandler.inputReady(AbstractHttp1IOEventHandler.java:64) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.http.impl.nio.ClientHttp1IOEventHandler.inputReady(ClientHttp1IOEventHandler.java:41) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.reactor.InternalDataChannel.onIOEvent(InternalDataChannel.java:133) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.reactor.InternalChannel.handleIOEvent(InternalChannel.java:51) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.reactor.SingleCoreIOReactor.processEvents(SingleCoreIOReactor.java:178) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.reactor.SingleCoreIOReactor.doExecute(SingleCoreIOReactor.java:127) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.reactor.AbstractSingleCoreIOReactor.execute(AbstractSingleCoreIOReactor.java:85) [httpcore5-5.1.5.jar:5.1.5]
        at org.apache.hc.core5.reactor.IOReactorWorker.run(IOReactorWorker.java:44) [httpcore5-5.1.5.jar:5.1.5]
        at java.lang.Thread.run(Thread.java:832) [?:?]
        Suppressed: org.opensearch.client.ResponseException: method [POST], host [http://<IP_ADDRESS>:9200], URI [/.opendistro-anomaly-detection-state/_search?typed_keys=true&max_concurrent_shard_requests=5&ignore_unavailable=false&expand_wildcards=open&allow_no_indices=true&ignore_throttled=true&search_type=query_then_fetch&batched_reduce_size=512&ccs_minimize_roundtrips=true], status line [HTTP/1.1 404 Not Found]
{"error":{"root_cause":[{"type":"index_not_found_exception","reason":"no such index [.opendistro-anomaly-detection-state]","index":".opendistro-anomaly-detection-state","resource.id":".opendistro-anomaly-detection-state","resource.type":"index_or_alias","index_uuid":"_na_"}],"type":"index_not_found_exception","reason":"no such index [.opendistro-anomaly-detection-state]","index":".opendistro-anomaly-detection-state","resource.id":".opendistro-anomaly-detection-state","resource.type":"index_or_alias","index_uuid":"_na_"},"status":404}

Delete Detector worked with the correct response.
Tested with DELETE method
API: <IP_ADDRESS>:9200/_extensions/_ad-extension/detectors/exLuTYcBzPWgaZWl_VLb
Response
{
    "_index": ".opendistro-anomaly-detectors",
    "_id": "LSXrTYcB70sQaKGvIW8c",
    "_version": 2,
    "result": "deleted",
    "forced_refresh": true,
    "_shards": {
        "total": 2,
        "successful": 1,
        "failed": 0
    },
    "_seq_no": 2,
    "_primary_term": 1
}


"Pleșoiu " , "Popești " , "Predești " , "Schitu " , "Șerbăneasa " , "Tufanii " , "Valea Bălcească " . "Valea Viei" are listed in the drop-down menu

The villages are listed in the drop-down menu alongside its corresponding county and city, according to the law
Preconditions : the  website is up and runing. After every step delete the text you typed from the search bar.

Step 1
Write in the search bar "Pleșoiu "

Expected results
The location appears in the drop-down menu as "Sat Pleșoiu , VÂLCEA (Nicolae Bălcescu)"

Step 2
Write in the search bar "Popești "

Expected results
The location appears in the drop-down menu as "Sat Popești , VÂLCEA (Nicolae Bălcescu)"

Step 3
Write in the search bar "Predești "

Expected results
The location appears in the drop-down menu as "Sat Predești , VÂLCEA (Nicolae Bălcescu)"

Step 4
Write in the search bar "Schitu "

Expected results
The location appears in the drop-down menu as "Sat Schitu , VÂLCEA (Nicolae Bălcescu)"

Step 5
Write in the search bar "Șerbăneasa "

Expected results
The location appears in the drop-down menu as "Sat Șerbăneasa , VÂLCEA (Nicolae Bălcescu)"

Step 6
Write in the search bar "Tufanii "

Expected results
The location appears in the drop-down menu as "Sat Tufanii , VÂLCEA (Nicolae Bălcescu)"

Step 7
Write in the search bar "Valea Bălcească"

Expected results
The location appears in the drop-down menu as "Sat Valea Bălcească, VÂLCEA (Nicolae Bălcescu)"

Step 8
Write in the search bar "Valea Viei"

Expected results
The location appears in the drop-down menu as "Sat Valea Viei, VÂLCEA (Nicolae Bălcescu)"


SrvTake.java:44-47: Continue to integrate with Servlet...

The puzzle 682-74f1da61 from #682 has to be resolved:
https://github.com/yegor256/takes/blob/87f8ee3a80cb27136838f640677b1890f5adda79/src/main/java/org/takes/servlet/SrvTake.java#L44-L47
The puzzle was created by Kirill on 24-Oct-18.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@paulodamaso/z please, pay attention to this issue

Job #865 is now in scope, role is DEV

The job #865 assigned to @ypshenychka/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@0crat refuse

@0crat refuse (here)

@ypshenychka The user @ypshenychka/z resigned from #865, please stop working. Reason for job resignation: Order was cancelled

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @ypshenychka/z

The job #865 assigned to @olenagerasimova/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@olenagerasimova/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat refuse

@0crat refuse

@0crat refuse (here)

@olenagerasimova The user @olenagerasimova/z resigned from #865, please stop working. Reason for job resignation: Order was cancelled

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @olenagerasimova/z

The job #865 assigned to @robsonataide/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

The user @robsonataide/z resigned from #865, please stop working. Reason for job resignation: It is older than 10 days, see §8

Resigned on delay, see §8: -30 point(s) just awarded to @robsonataide/z

@0crat assign fabriciofx

@0crat assign fabriciofx (here)

@paulodamaso The job #865 assigned to @fabriciofx/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @paulodamaso/z

The user @fabriciofx/z resigned from #865, please stop working. Reason for job resignation: It is older than 10 days, see §8

Resigned on delay, see §8: -30 point(s) just awarded to @fabriciofx/z

@0crat assign @fabriciofx

@0crat assign @fabriciofx (here)

@paulodamaso The job #865 assigned to @fabriciofx/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @paulodamaso/z

@g4s8 Besides unit tests for RqFrom, ResponseOf and SrvTake which others requirements do you think important to finish the servlet integration?

@fabriciofx I think we can add servlet session to takes identity mapping


How to deploy the server?

I want to test the server.
I'm using Windows-10.
I downloaded the files.
What do I need to install additionally on Windows-10 to test the server?
Where to place the project files?
Where to place and how to run the server file?
Where to place and how to run the client file?

Hi @it19862 and welcome to ReactPHP! :-)
It's a bit unclear what you're trying to achieve exactly from the details you've posted and it sounds like you may be new to PHP. As a starting point, I would recommend checking out our website on https://reactphp.org/ and see if you can follow some of our quickstart examples.
If you have PHP installed and used Composer to install your package of choice, you should be able to run each example on the command line like php examples/demo.php. If you don't know where to start, you may also like to take a look at this: https://framework-x.org/docs/getting-started/quickstart/
I hope this helps! :+1:


Cannot train model on google colab

I switched to using google colab but i run into the following error
`IndexError                                Traceback (most recent call last)
 in ()
----> 1 learn.fit_one_cycle(4, lr_max=1e-3)
20 frames
/usr/local/lib/python3.7/dist-packages/torch/nn/functional.py in nll_loss(input, target, weight, size_average, ignore_index, reduce, reduction)
2262                          .format(input.size(0), target.size(0)))
2263     if dim == 2:
-> 2264         ret = torch._C._nn.nll_loss(input, target, weight, _Reduction.get_enum(reduction), ignore_index)
2265     elif dim == 4:
2266         ret = torch._C._nn.nll_loss2d(input, target, weight, _Reduction.get_enum(reduction), ignore_index)
IndexError: Target 2 is out of bounds.
`

Can you share your code via a gist so I can take a look?
Just a guess, but if your doing classification you'll want to make sure you tell your transformer model how many classes (or else for multilabel how many labels) you are predicting.
See here and in particular this code:
task = HF_TASKS_ALL.SequenceClassification

pretrained_model_name = "distilroberta-base"
config = AutoConfig.from_pretrained(pretrained_model_name)
config.num_labels = len(lbl_cols)

hf_arch, hf_config, hf_tokenizer, hf_model = BLURR_MODEL_HELPER.get_hf_objects(pretrained_model_name,
                                                                               task=task,
                                                                               config=config)


add japanese localization

added the localization over here aswell, please use whichever is more convenient.

This will be it.
Thanks a lot. :)


WIP: Reimbursement

Coverage decreased (-0.8%) to 83.135% when pulling 5744cf72abab7a7a770fa5e909f803e98793880a on reimbursement into 283193cab1e01b2fb0db3dcb8554b9bea1ef9826 on master.

Coverage decreased (-0.5%) to 83.344% when pulling 0097e7699020fdcfe53f8fbf1ec13d8f2e490d8c on reimbursement into 283193cab1e01b2fb0db3dcb8554b9bea1ef9826 on master.

Coverage decreased (-0.01%) to 83.881% when pulling 7afd9e1b86ac29d26edfa38efee40aafd35628ac on reimbursement into 283193cab1e01b2fb0db3dcb8554b9bea1ef9826 on master.

Coverage decreased (-0.2%) to 81.436% when pulling 5592a5bee124a90cb4ba36ce54721ef6fd26e66a on reimbursement into 8b8adc21b6d9ed5c96924751920f40e0d091c780 on master.

Coverage increased (+2.4%) to 83.973% when pulling cb20522cfd9de45b3db8625ccbbdbe37b0e11d25 on reimbursement into 8b8adc21b6d9ed5c96924751920f40e0d091c780 on master.

Changes Unknown when pulling d129395a388cb5a4a86b2ce9a85e8eb32d2c3423 on reimbursement into ** on master**.

Changes Unknown when pulling cd59163c49892fdcb9ccfababbc80aa61db78d5a on reimbursement into ** on master**.

Changes Unknown when pulling f746022e420551eef01220a09beb9ac4a69cf979 on reimbursement into ** on master**.

Changes Unknown when pulling fb8f3832b93210341d61a3fe942e6e5c0f162a31 on reimbursement into ** on master**.

Changes Unknown when pulling 41a6062873c413d8659588ec18fcf05cc8454e99 on reimbursement into ** on master**.

Changes Unknown when pulling 15fe3ddba3fe9d4e81e22d05697456fec8374e46 on reimbursement into ** on master**.

Changes Unknown when pulling 184ff2a0bd8ba2b610c909956daf8a08c01d6d07 on reimbursement into ** on master**.

Changes Unknown when pulling b76178f74e612e8610e9d8848759b34f58358ee0 on reimbursement into ** on master**.

Changes Unknown when pulling b76178f74e612e8610e9d8848759b34f58358ee0 on reimbursement into ** on master**.


Filter invalid transactions

Add logic to somehow filter out/check for malformed transactions. Actually we are only worried about malformed, should check to see how this is handled in Arbitrum.

I will close this ticket once https://github.com/EspressoSystems/op-geth/pull/2 is merged, since invalid transactions will be handled. We have a separate ticket for transactions that don't even deserialize, https://github.com/EspressoSystems/op-geth/issues/4, which we probably won't get to this sprint.


i18n with multiple_files misidentify "md" within the slug name as extension

Describe the bug
When using i18n with multiple_files structure with slug name containing "md", Netlify CMS incorrectly identify the md as the file extension, and insert lx.md there instead (where lx is the language).
To Reproduce

Create a yml file with the following i18n options:

i18n:
  structure: multiple_files
  locales: [th, en]
  default_locale: th

Create a collection, for example, with

folder: content/members
path: "{{title}}/index"
media_folder: ''
public_folder: ''
i18n: true
fields:
  - {name: title, label: Title, i18n: duplicate}

Create a new "member" object whose name contains "md", for example, "Samdy"

Expected behavior
The following files get created:
/content/members
  - /samdy
    - /index.en.md
    - /index.th.md
    - /profile.png

Actual behavior
/content/members
  - /saen.mdy/index.md
  - /samdy/profile.png
  - /sath.mdy/index.md

Applicable Versions:

netlify-cms-app 2.14.26
netlify-cms-core 2.38.5
netlify-cms 2.10.96
Git provider: GitHub
OS: Mac OS 11
Browser version: chrome 88

CMS configuration
backend:
  name: github
  repo: ...
  branch: main
  accept_roles:
    - admin
    - editor

publish_mode: editorial_workflow

site_url: ...

media_folder: media_folder

i18n:
  structure: multiple_files
  locales: [th, en]
  default_locale: th

collections:
- name: members
  label: Members
  label_singular: member
  identifier_field: memberid
  create: true
  folder: content/members
  path: "{{slug}}/index"
  preview_path: '/members/{{memberid}}/'
  media_folder: ''
  public_folder: ''
  summary: '{{name}} - {{org}}'
  i18n: true
  fields:
    - {name: memberid, label: Member ID, i18n: duplicate}
    - {name: name, label: Name, i18n: true}
    - {name: image, label: 'Image', widget: image, i18n: duplicate}
    - {name: email, label: Email, i18n: duplicate, required: false}
    - {name: body, label: Body, widget: markdown, i18n: true}

Hey @artt, shared this on Slack but post it here again for visibility. I think the responsible line of code could be this one:
https://github.com/netlify/netlify-cms/blob/53061710a9e3b338781e34f11491eeb65d23b8d5/packages/netlify-cms-core/src/lib/i18n.ts#L82


[dependencies] tslib >= 1.7.0 < 2

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[x] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
The version dependency of tslib for akita is < 2.
Expected behavior
Is it possible to use it with the version 2.0.0?
I am wondering if this is still on purpose due to an incompatibility or if you could simply update it to allow to be compatible with the version 2.0.0.
What is the motivation / use case for changing the behavior?
Upgrading to Angular 10 on my project with the CLI throw an error.
Using the flag force allow nevertheless to update the dependency.
Note that even if Akita dependency seems to be incompatible with tslib I already have the 2.0.0 version and everything seems fine.
Example
This command:
ng update @datorama/akita @datorama/akita-cli @datorama/akita-ngdevtools
Produce this error:
Package "@datorama/akita-ngdevtools" has an incompatible peer dependency to "tslib" (requires "^1.10.0", would install "2.0.0")
Package "@datorama/akita" has an incompatible peer dependency to "tslib" (requires ">= 1.7.0 < 2", would install "2.0.0")

Environment

Angular version: 10.0.2

@xbaun what do u think?

It seems that typescript must be upgraded to >3.8, to use tslib 2.0.0: https://github.com/microsoft/tslib/releases/tag/2.0.0

By requiring tslib 2.0.0 or higher we will probably break current projects based on older ts versions. Best option would be to compile Akita without --importHelpers flag, thus directly generating the helper functions and removing the tslib peer dependency.

@xbaun thank you for the insight.
If you do not plan to change the compilation method is it still required to disallow tslib on 2.0.0 version?
I am not sure, but I think that if you change the restriction to higher version it will not force to download the 2.0.0 since it will meet the >= 1.7.0 range for older versions and also avoid this error.
Also, if this was mentioned in the documentation for older versions that tslib should be set below the 2.0.0 version it would avoid potential mistakes when upgrading.
What do you think? Thanks.

I am not sure, but I think that if you change the restriction to higher version it will not force to download the 2.0.0 since it will meet the >= 1.7.0 range for older versions and also avoid this error.

The problem would be if Akita is compiled with a ts version lower than 3.9 (what is currently the case), it would require a tslib version < 2.0.0 to be installed which isn't compatible to 2.0.0 because of breaking changes in the helper functions. So if a project using Akita is relaying on tslib >= 2.0.0 (thus a newer ts version), then Akita would probably break by importing the wrong tslib version expecting the old helper functions. And the other way around.
So I would enable inlining the helper functions so that the dependency to any specific tslib version is removed. Projects using Akita can then choose any ts/tslib version they need.

Any update on this?

Hello,
As state by @C0ZEN, if you force the update angular 10 and akita are working nice together.
But according to @xbaun, it seems it should not work.
How could we explain this ?

@oliveti not work with older versions only.

@oliveti this issue (which is not a bug but a feature-request) is only possible when you have tslib on 2.X.X on your project and you use Akita which requires > 2.

@xbaun can we do sth about this? Akita seems to work fine with tslib v2, TS v4 and Angular v10.1, but the peer dependency warning is pretty annoying. Maybe you could release a new majore version which depends on tslib v2 and TS > 3.9 ?

I recently started a new project in Angular 10. This issue is preventing me - and possibly many others - from using Akita.

@Wintermute79 how is it preventing you from using Akita? It's just a harmless warning.

I'm going to upgrade everything to the latest versions and release Akita v6. It'll take some time, but you can ignore the warning for now.

@Wintermute79 how is it preventing you from using Akita? It's just a harmless warning.

When trying to install Akita via npm install @datorama/akita from within the Angular 10 project directory, it wont complete, giving the following errors:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/tslib
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer tslib@">= 1.7.0 < 2" from<EMAIL_ADDRESS>npm ERR! node_modules/@datorama/akita
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

When I realised the cause and tried again from another path, it succeeded. So my apologies for being not clear enough in my earlier post.

@Wintermute79 seems like npm bug. Try to update your npm cli.

@hakimio thanks for the hint; I guess I will switch to Yarn then.

The upgrade is released with v6.

The upgrade is released with v6.


Expose schema object attribute inheritance in introspection

Every inheriting schema object introspection now has an
inherited_attrs property containing an array of names of attributes
that were inherited from a parent as opposed to defined locally.

How about calling it 'inherited'? I don't like 'Attrs', but making it 'attributes' would be too long.

inherited is confusing IMO since it may be attributed to the object itself and not it's attributes.  We may spell attrs as fields if that helps (and we also need to settle on that in general),


Validate ZNode ACLs prior to upgrade, log unexpected values and fail

Closes #2890

It may also be desirable to check all paths, keeping a list of any nodes that are incorrect and then failing if the list size is > 0 and providing the complete list. That way they may be able to correct things once and the re-try the upgrade instead of needing to "find" each node individually.

That's what should happen. It should log all of them, then throw the exception at the end after the visit method.


useTheme

Explain the feature
Allows you to toggle between light and dark themes and remembers it from local storage.

now will work on this feature 😄  ... it might take some time cause my university exams are near...

Ok, I'll try to advertise the repo and get more contributors 😄

is rhymojs website not active now ?

Yup, while writing a lot of boilerplate for testing and stuff, I barely started writing the code for the website. It's gonna be a MEMN (MERN but the R is NextJS) stack website with blogs, etc. The repo for it is @rhymojs/rhymojs.com

Ha

Yup, while writing a lot of boilerplate for testing and stuff, I barely started writing the code for the website. It's gonna be a MEMN (MERN but the R is NextJS) stack website with blogs, etc. The repo for it is https://github.com/rhymojs/rhymojs.com

Have you planned any design system (structure) for the website ?
Cause I'm really interested to build it with you ?

I already created the sitemap, now I am planning to document features.

https://dev.to/xeptao/12-steps-to-creating-a-serious-startup-1lm3 - I am following my own blog and we are on the 5th step

This is the sitemmap:

The paths with the [brackets] are dynamic routes

Maybe i could create a markdown file for the sitemap, would be a lot more professional :D

I thought of it something like this...
const useTheme = (): SwitchTheme => {
   const [theme, setTheme] = useLocalStorage('theme');
   const [toggle, reverseToggle] = useToggle(theme !== "" ? theme === 'light' ? false : true : false);

   const themeSwitcher = () => reverseToggle()

   if(theme)
   {
       document.body.parentElement?.setAttribute('data-theme',theme)
   }

   useEffect(() =>{
       setTheme(toggle ? 'dark' : 'light')
   },[toggle]);

   return [theme, () => themeSwitcher()];

}

Seems pretty good, what code editor do you use by the way?

Seems pretty good, what code editor do you use by the way?

VS code

I do, do you use ESLint, because I setup prettier with it and it formats ts, and tsx extensions

I do, do you use ESLint, because I setup prettier with it and it formats ts, and tsx extensions

yeah i do

Ok, please change your default formatter to EsLint so it can format files

https://prod.liveshare.vsengsaas.visualstudio.com/join?247F762AE52827AAE8B52C566E7128105A6E - live share link

Ok, please change your default formatter to EsLint so it can format files

i have set it to default previously and i also i have joined the live share session

ok, come to the file i'm editing

Hey, do you see the file?

ok, i'm starting another session

https://prod.liveshare.vsengsaas.visualstudio.com/join?B6E9A47D7CEFAE96B325889579A7DA2D00C7

ok, i'm starting another session

okkk


Check box editor supports commitOnChange and InstantEdit

This resolves #2552.
With clicksToEdit set to 1, it makes for a super easy to edit checkbox column.
Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[x] Caught up with develop branch as of last change.
[ ] Added CHANGELOG entry, or determined not required.
[x] Reviewed for breaking changes, added: no breaking changes
[x] Updated doc comments / prop-types, or determined not required.
[x] Reviewed and tested on Mobile: n/a
[x] Created Toolbox branch: not required

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

Closing -- main idea has been incorporated in
https://github.com/xh/hoist-react/pull/2679


Change Biomole manual data to petal data pipeline

Is there a commit associated with this issue?


sc2api : Update Convert CloakState

There is an issue with receiving of the observation object (#301).
Particularly, received data is converted to unit pool in the sc2_proto_to_pods.cc file. And the conversion function checks that received data is correct and all units are in known state.
For some reason the game sets flag has_cloak and send CloakState as CloakedUnknown.
Conversion routine did not take it into account and returned status failure. So that, unit pool stays not initialized.
This request adds new cloak state CloakedUnknown and updates the conversion routine, so it returns success in such case,

Aren't these files supposed to be auto-generated b gRPC

@minchopaskal you should be right
I was unsure if I have a correct tools and able to find a proper way to regenerate it from scratch. I have decided just to fix obvious error to play with api. And as we see at #316 there still are many issues which I miss.


style(types): update interface imports to use type, some descriptions

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Not able to push files if path is not present

If the push command is executed for specific files/folders and the path is not jet present in Google Drive, these folders are not transferred.
There is no error message.

Hello @philipp923 welcome to drive.
Would you mind giving me a bit more information as well as diagnostics? What commands did you execute? What version are you?
drive pushes upstream content that is not or may have been modified so this issue pretty much defeats the purpose of it. Hopefully we can get this solved asap.

Hi odeke-em,
thanks for the fast reply. Very nice to notice that the project is active.
I tested a little bit and came up with a minimal example.
Folder structure:
gdrive <- root folder GoogleDrive

folderNotOnline

fileNotOnline.txt

philipp@x220:/xxx/gdrive$ drive push folderNotOnline/fileNotOnline.txt
Resolving...

/folderNotOnline/fileNotOnline.txt
Addition count 1
Proceed with the changes? [Y/n]: Y
1 / 1 [=====================================================================================================] 100.00 % 0

Everything looks fine, but the folderNotOnline as well as the fileNotOnline.txt are not transmitted to GoogleDrive.
philipp@x220:/xxx/gdrive$ drive stat folderNotOnline/fileNotOnline.txt
/folderNotOnline/fileNotOnline.txt: remote path doesn't exist
philipp@x220:/xxx/gdrive$ drive stat folderNotOnline/
/folderNotOnline: remote path doesn't exist
drive is working if I push the entire folderNotOnline:
philipp@x220:/xxx/gdrive$ drive push folderNotOnline/
Resolving...

/folderNotOnline
/folderNotOnline/fileNotOnline.txt
Addition count 2
Proceed with the changes? [Y/n]: Y
2 / 2 [====================================================================================================] 100.00 % 2s

For my use case it would be very helpful to have the ability to push individual sub-folders.
Here are the versions I am using:
drive version 0.1.5
go version go1.2.1 linux/amd64
Linux x220 3.16.0-33-generic #44-Ubuntu SMP Thu Mar 12 12:19:35 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
Thanks in advance and have a nice day.
Philipp

I can reproduce the minimal case, too

Hello @philipp923 sorry for the late reply, Friday nights offset Saturdays by a bit ;)
Oh yes in deed, this issue should be present because now it just dawned onto me that during remoteAdditions I was doing lookups of parents instead of remoteMkdirAll of parents which should be the case. This behaviour was changed a couple of weeks ago when the Google API client was buggy and was uploading files title 'Untitled Title' for every folder for which I used remoteMkdirAll, hence I backed off. However the API client got fixed and I also patched that in drive however not in that spot. Thank you very much for bringing this.
I am pushing up the fix in a few minutes.

For more details the Google Go api client offending change ages ago was https://github.com/google/google-api-go-client/commit/901b69e534b7fd805ed4d27388b165db81eea51a#commitcomment-9471898
The latest fix in drive is at https://github.com/odeke-em/drive/commit/53ce76e1ff1aa1749a51cb9d1e9438dc3bde4682

Please re-open if it persists, am making a release right now.

Hi odeke-em,
the new version of drive is working very well.
Thank you very much for the fast support!
Best regards,
Philipp


Make it more obvious that custom tasks / notes exist

https://github.com/KennethLuczko/myroutebuddy/pull/20


Python script to scrape links

Description
Added a script to scrape all links from a given url.
Fixed #87
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added screenshots(if any) that prove my fix is effective or that my feature works

@all-contributors please add namanshah01 for code, doc


Fix cub::DeviceReduce::Arg{Max,Min} for inf values

This PR addresses the following issue related to properly handling inf values by cub::DeviceReduce::Arg{Max,Min}.

cc: @asi1024 for vis

@senior-zero Thanks for quick fix! Question: Would the same bug affect the block-level (or any level blow) reduction? In CuPy they also have CUB block reduction that could kick in in certain code paths.

Hello @leofang!
Our block and warp scope reductions do not provide Arg{Min,Max} functionality, so they shouldn't be affected by the bug.


Bubble clustering

Hey Matt, spent a couple hours trying different ways to cluster to force nodes. Started by trying understand the force clustering Observable code - but it's a bit too complex. Then tried using the force-cluster npm package, which I think would get us there but:

my force("cluster") is going to be competing with .force(x) and I'm not sure how to get these to work together
the function I am using to cluster by position is likely incorrect.

`const clusterCenters = positions.map((position, index) => {
                const angle = (index / positions.length) * Math.PI * 2;
                const radius = Math.min(innerWidth, height) / 3;
                const x = Math.cos(angle) * radius + innerWidth / 2;
                const y = Math.sin(angle) * radius + height / 2;
                return { x, y };
            });`

have you posted your code?
also what's the link to the observable code?

I have! It's the D3 instructions page you linked in the feedback sorry, here.

I'm just not seeing this in your code. what component are you in?

Gotcha! It looks like you're on an older version, can you git pull? I also just made some new pushes to main over the last hour (made some adjustments for feedback), you should see the clustering function commented out above the rerunSimulation function.

oh darn i was on my own branch from our pairing session. my bad!
On Fri, Jun 7, 2024 at 9:36 AM Tom @.***> wrote:

Gotcha! It looks like you're on an older version, can you git pull? I also
just made some new pushes to main over the last hour (made some adjustments
for feedback), you should see the clustering function commented out above
the rerunSimulation function.
Bubbles.svelte is the right component.
—
Reply to this email directly, view it on GitHub
https://github.com/tomvaillant/sleep-training/issues/10#issuecomment-2154855826,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAV7FLVTBQDANM4WM5S4H7TZGGZN3AVCNFSM6AAAAABI2VHER2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJUHA2TKOBSGY
.
You are receiving this because you commented.Message ID:
@.***>

ok i got the observable example working in svelte. will replace it with your bubbles component in a branch!

i just pushed a branch, refactoring your code a bit—hope it helps!

WOW. That's a huge refactor. I'll dig into this and try to understand it, maybe on our next call you can walk me through a bit :) Thanks man.


BUG: Russian translation of the month

Pay Attention, Please:
https://github.com/jenssegers/date/blob/master/src/Lang/ru.php#L30
https://github.com/jenssegers/date/blob/master/src/Lang/ru.php#L43
We really see identical array keys with different values.
This only applies to May month, for example:
Date::setLocale('ru');
echo \Jenssegers\Date\Date::instance(new \DateTime('2019-05-10'))->format('M j, D H:i');

Result:
{0}май|{1}мая 10, пт 00:00
However, the correct result will be:
май 10, пт 00:00

Thank you, in the future I will migrate to carbon 2.

str_replace('{0}Май|{1}Мая', 'Май', mb_convert_case($dd->format('M Y'), MB_CASE_TITLE, "UTF-8"));


Update to API will cause issue with Group User Call Statistics

Odin Patch Release 7.4.17 is now available
Greetings Odin users! This batch of release notes covers all the features, updates, and bug fixes published throughout July, 2024.
Normally, we segment releases into smaller chunks, but in this case the Odin team feels grouping the last few releases together makes the most sense, as many front and backend elements distrubuted across these releases work in concert.
Accordingly, we strongly advise any client wishing to upgrade beyond 7.4.13 to request an upgrade directly to 7.4.17.
As a reminder, all of these features are live and available for review in your instance of Odin via the Lab feature.
Please email<EMAIL_ADDRESS>if you'd like to get on the schedule for an upgrade.
New Features
ODIN WEB
For User Bulk Features, the Voice Messaging User Screen CSV report download now includes 3 additional columns: sendVoiceMessageNotifyEmail, voiceMessageDeliveryEmailAddress, and voiceMessageCarbonCopyEmailAddress. [ADO-10600]
Enterprise and group admins can now enable and disable admins at their level of clearance or lower. [9863]
Updates
ODIN WEB
Improved the performance of the Service Utilization Report [ADO-8413]
Added the ability to filter the Group User Call Report to remove users who do not have any calls during the designated period [ADO-10633]
Odin group Devices and group Device Configuration as well as user Devices were in some cases requiring tag values. Odin now allows null Tag Values in all cases as well as correctly passes along "true" or "false" values. [ADO-10785]
While on the user landing page, the links to settings for calling features are now in focus order. [ADO-9665]
The downloadable CSV report User Bulk Features - Voice Messaging User Screen now includes an additional column: voiceMessageNotifyEmailAddress. [ADO-11272]
When using the "Enable Authentication" option of the user create callback, the system should update the user authentication and raise a "user.authentication.update" event regardless of whether or not the authentication service is previously enabled. [ADO-11585]
ODIN API
Performing a GET call on the call records user stats returns an empty array instead of an object with null properties when no matching records are found. [ADO-10847]
Bugfixes
ODIN WEB
Corrected a bug that was preventing hunt group agent reordering. [ADO-11439]
Fixed various bugs and implemented several optimizations for date picker. [ADO-10644]
ODIN API
Corrected issues with the Phonism Webhook fix for the user.busyLampField.update event. [ADO-10727]

Odin has changed what is returned from the api null/0 which is going to break the method (group user call statitstics) we built in reporter.py. Need to adjust for this


If no version provided what about use GitSummary as VERSION?

@codeskyblue that's not a bad idea at all. However a developer of a project would know which one to pick, and if they find GitSummary suitable, they can just use it in their program?
The problem with GitSummary is, it picks up all tags. So if you're making docs releases for your project using separate tags, we have no generic way of saying --match v[0-9\.]+$, so you might end up with a weird tag name like docs-v2.1.0 in GitSummary.

Use regexp match is a good way(seems). Before see your repo, I build my go project use a shell script
!/bin/bash -

set -eu

VERSION=$(git describe --abbrev=0 --tags)
REVCNT=$(git rev-list --count HEAD)
DEVCNT=$(git rev-list --count $VERSION)
if test $REVCNT != $DEVCNT
then
        VERSION="$VERSION.dev$(expr $REVCNT - $DEVCNT)"
fi
echo "VER: $VERSION"

GITCOMMIT=$(git rev-parse HEAD)
BUILDTIME=$(date -u +%Y/%m/%d-%H:%M:%S)

LDFLAGS="-X main.VERSION=$VERSION -X main.BUILDTIME=$BUILDTIME -X main.GITCOMMIT=$GITCOMMIT"
if [[ -n "${EX_LDFLAGS:-""}" ]]
then
        LDFLAGS="$LDFLAGS $EX_LDFLAGS"
fi

build() {
        echo "$1 $2 ..."
        GOOS=$1 GOARCH=$2 go build \
                -tags bindata \
                -ldflags "$LDFLAGS" \
                -o dist/gohttpserver-${3:-""}
}

build linux arm linux-arm
build darwin amd64 mac-amd64
build linux amd64 linux-amd64
build linux 386 linux-386
build windows amd64 win-amd64.exe

The version is come from
VERSION=$(git describe --abbrev=0 --tags)
REVCNT=$(git rev-list --count HEAD)
DEVCNT=$(git rev-list --count $VERSION)
if test $REVCNT != $DEVCNT
then
        VERSION="$VERSION.dev$(expr $REVCNT - $DEVCNT)"
fi

@codeskyblue so I believe the GitSummary variable today  is pretty usable as is if you're using tags only to release versions of your repository. (if you don't agree, let me know) and if your project does not have a VERSION file (which is something you already know as the developer of the project), then you'd just use GitSummary variable and not Version, right?

No, I don't agree, usally version can be found in tags, so why have to write a VERSION file. However you are the owner of this repo, if you don't agree I can fork it and make my own.

@codeskyblue VERSION file is optional. If GitSummary field is enough for you, why not just use it? I just did not get your point. :-)

GitSummary is summary, it got the different meaning of version. Not just name, because next month when you review your code, you will be wired, and say what is the GitSummary and where is version?

Oh I see your point. You're right we could offer GitSummary as Version if the file does not exist. I tend to agree that ideally tags should be used correctly however they rarely are used correctly.
I think since the project is not so much adopted just yet, we can consider implementing it so that GitSummary is assigned to Version if the file does not exist. Would you consider contributing it?

Yes, ofcourse, anyway, I need to use such tool


[Task]: Harmonize and reorganize dependencies

What needs to happen?
Dependencies are not correctly managed in Hop.
We have a lot of duplicate dependencies and it is unclear in many situations which dependency is used/required in each module.
Clean up and reorganize the dependencies.

Create lib modules with the shared dependencies which then can be used via dependency management.
Make version management simpler
Remove as much version information and dependencies from the main pom

Issue Priority
Priority: 2
Issue Component
Component: Other

.take-issue


Feature request numpy.unique axis=0

Feature request
I want to go from
a = np.array([[1, 1, 1, 0, 0, 0],
              [0, 1, 1, 1, 0, 0],
              [0, 1, 1, 1, 0, 0],
              [1, 1, 1, 0, 0, 0],
              [1, 1, 1, 1, 1, 0]])

to
array([[1, 1, 1, 0, 0, 0],
       [0, 1, 1, 1, 0, 0],
       [1, 1, 1, 1, 1, 0]])

Using numpy I would use
unique_rows = np.unique(a, axis=0)
however, the axis parameter is not supported yet. I tried some workarounds based on this SO question, but I havn't figured out a sollution yet.

Many thanks for the request! A runnable MWR to test for the feature in future is:
import numpy as np
from numba import njit

a = np.array([[1, 1, 1, 0, 0, 0],
              [0, 1, 1, 1, 0, 0],
              [0, 1, 1, 1, 0, 0],
              [1, 1, 1, 0, 0, 0],
              [1, 1, 1, 1, 1, 0]])

@njit
def f(x):
    return np.unique(a, axis=0)

print(f(a))

which presently gives:
numba.core.errors.TypingError: Failed in nopython mode pipeline (step: nopython frontend)
No implementation of function Function(<function unique at 0x7fb93b057dc0>) found for signature:

 >>> unique(readonly array(int64, 2d, C), axis=Literal[int](0))

There are 2 candidate implementations:
  - Of which 2 did not match due to:
  Overload in function 'np_unique': File: numba/np/arrayobj.py: Line 2088.
    With argument(s): '(readonly array(int64, 2d, C), axis=int64)':
   Rejected as the implementation raised a specific error:
     TypingError: got an unexpected keyword argument 'axis'
  raised from /home/gmarkall/numbadev/numba/numba/core/typing/templates.py:791

During: resolving callee type: Function(<function unique at 0x7fb93b057dc0>)
During: typing of call at /home/gmarkall/numbadev/issues/7663/repro.py (13)

File "repro.py", line 13:
def f(x):
    return np.unique(a, axis=0)
    ^

I wrote this a while back, which gets the job done.

@nb.jit(nopython=True, cache=True)
def nb_unique(input_data, axis=0):
    """2D np.unique(a, return_index=True, return_counts=True)

    Parameters
    ----------
    input_data : 2D numeric array
    axis : int, optional
        axis along which to identify unique slices, by default 0
    Returns
    -------
    2D array
        unique rows (or columns) from the input array
    1D array of ints
        indices of unique rows (or columns) in input array
    1D array of ints
        number of instances of each unique row
    """

    # don't want to sort original data
    if axis == 1:
        data = input_data.T.copy()

    else:
        data = input_data.copy()

    # so we can remember the original indexes of each row
    orig_idx = np.array([i for i in range(data.shape[0])])

    # sort our data AND the original indexes
    for i in range(data.shape[1] - 1, -1, -1):
        sorter = data[:, i].argsort(kind="mergesort")

        # mergesort to keep associations
        data = data[sorter]
        orig_idx = orig_idx[sorter]
    # get original indexes
    idx = [0]

    if data.shape[1] > 1:
        bool_idx = ~np.all((data[:-1] == data[1:]), axis=1)
        additional_uniques = np.nonzero(bool_idx)[0] + 1

    else:
        additional_uniques = np.nonzero(~(data[:-1] == data[1:]))[0] + 1

    idx = np.append(idx, additional_uniques)
    # get counts for each unique row
    counts = np.append(idx[1:], data.shape[0])
    counts = counts - idx
    return data[idx], orig_idx[idx], counts

Someday, when I figure out how to avoid writing 8 different implementations for various combinations of np.uniques returns, I'll contribute this back into numba.

@rishi-kulkarni Many thanks for the pointer! I'll add the "good second issue" label in case someone else also would like to look at how to contribute it back to Numba - your implementation does provide a good starting point.
@lukasalexanderweber Does the implementation above help you get by for your use case?

@gmarkall yeah thank you very much!

I wrote this a while back, which gets the job done.
@nb.jit(nopython=True, cache=True)
def nb_unique(input_data, axis=0):
    """2D np.unique(a, return_index=True, return_counts=True)

    Parameters
    ----------
    input_data : 2D numeric array
    axis : int, optional
        axis along which to identify unique slices, by default 0
    Returns
    -------
    2D array
        unique rows (or columns) from the input array
    1D array of ints
        indices of unique rows (or columns) in input array
    1D array of ints
        number of instances of each unique row
    """

    # don't want to sort original data
    if axis == 1:
        data = input_data.T.copy()

    else:
        data = input_data.copy()

    # so we can remember the original indexes of each row
    orig_idx = np.array([i for i in range(data.shape[0])])

    # sort our data AND the original indexes
    for i in range(data.shape[1] - 1, -1, -1):
        sorter = data[:, i].argsort(kind="mergesort")

        # mergesort to keep associations
        data = data[sorter]
        orig_idx = orig_idx[sorter]
    # get original indexes
    idx = [0]

    if data.shape[1] > 1:
        bool_idx = ~np.all((data[:-1] == data[1:]), axis=1)
        additional_uniques = np.nonzero(bool_idx)[0] + 1

    else:
        additional_uniques = np.nonzero(~(data[:-1] == data[1:]))[0] + 1

    idx = np.append(idx, additional_uniques)
    # get counts for each unique row
    counts = np.append(idx[1:], data.shape[0])
    counts = counts - idx
    return data[idx], orig_idx[idx], counts

Someday, when I figure out how to avoid writing 8 different implementations for various combinations of np.uniques returns, I'll contribute this back into numba.

The Code does not work because of the usage of np.all(axis=1), due to numpy.all only being supported in its basic form without any optional arguments. Source: https://numba.pydata.org/numba-doc/dev/reference/numpysupported.html#calculation

Oops, here's an overload for that. It only handles 2D arrays.
@overload(np.all)
def np_all(x, axis=None):

    # ndarray.all with axis arguments for 2D arrays.

    @register_jitable
    def _np_all_axis0(arr):
        out = np.logical_and(arr[0], arr[1])
        for v in iter(arr[2:]):
            for idx, v_2 in enumerate(v):
                out[idx] = np.logical_and(v_2, out[idx])
        return out

    @register_jitable
    def _np_all_flat(x):
        out = x.all()
        return out

    @register_jitable
    def _np_all_axis1(arr):
        out = np.logical_and(arr[:, 0], arr[:, 1])
        for idx, v in enumerate(arr[:, 2:]):
            for v_2 in iter(v):
                out[idx] = np.logical_and(v_2, out[idx])
        return out

    if isinstance(axis, types.Optional):
        axis = axis.type

    if not isinstance(axis, (types.Integer, types.NoneType)):
        raise TypingError("'axis' must be 0, 1, or None")

    if not isinstance(x, types.Array):
        raise TypingError("Only accepts NumPy ndarray")

    if not (1 <= x.ndim <= 2):
        raise TypingError("Only supports 1D or 2D NumPy ndarrays")

    if isinstance(axis, types.NoneType):

        def _np_all_impl(x, axis=None):
            return _np_all_flat(x)

        return _np_all_impl

    elif x.ndim == 1:

        def _np_all_impl(x, axis=None):
            return _np_all_flat(x)

        return _np_all_impl

    elif x.ndim == 2:

        def _np_all_impl(x, axis=None):
            if axis == 0:
                return _np_all_axis0(x)
            else:
                return _np_all_axis1(x)

        return _np_all_impl

    else:

        def _np_all_impl(x, axis=None):
            return _np_all_flat(x)

        return _np_all_impl

When using your np_all function, it raises "NameError: name 'overload' is not defined", what am I doing wrong?

I updated my comment to include the necessary imports.

Hi rishi-kulkarni and thanks for the reply. Does this mean that I essentially create an array with 15k elements and then truncate it once it's filled up? I would have to create a total of 15k*15k elements? I know that for my systems this can easily go up to 1M meaning that there would be 10**12 elements, perhaps too much for RAM, right?

Hm, yeah, that's probably too much - I was assuming you were looking for 2D slices that are unique in a 3D array, for example. Another approach- as far as I'm aware, you only need to use numba's typed lists if you're passing in a Python list - however, you can just do a regular list comprehension inside of a decorated function no problem. Am I mistaken?