Hn dev

Hello @HorlavaNastassya! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file clinicaaddl/clinicaaddl/classify/inference.py:

Line 8:11: E401 multiple imports on one line
Line 10:1: E402 module level import not at top of file
Line 11:1: E402 module level import not at top of file
Line 12:1: E402 module level import not at top of file
Line 13:1: E402 module level import not at top of file
Line 205:36: E225 missing whitespace around operator
Line 207:25: E225 missing whitespace around operator
Line 210:25: E225 missing whitespace around operator
Line 216:24: E225 missing whitespace around operator
Line 216:61: E225 missing whitespace around operator

In the file clinicaaddl/clinicaaddl/cli.py:

Line 119:1: W293 blank line contains whitespace
Line 123:38: E226 missing whitespace around arithmetic operator
Line 715:5: E303 too many blank lines (2)

In the file clinicaaddl/clinicaaddl/quality_check/__init__.py:

Line 2:22: W292 no newline at end of file

In the file clinicaaddl/clinicaaddl/tools/deep_learning/augmentations.py:

Line 57:1: E402 module level import not at top of file
Line 58:1: E402 module level import not at top of file

In the file clinicaaddl/clinicaaddl/tools/deep_learning/cnn_utils.py:

Line 49:1: W293 blank line contains whitespace
Line 50:5: E265 block comment should start with '# '
Line 157:1: W293 blank line contains whitespace
Line 245:5: E265 block comment should start with '# '
Line 270:17: E225 missing whitespace around operator
Line 270:19: E226 missing whitespace around arithmetic operator
Line 270:24: E226 missing whitespace around arithmetic operator
Line 270:37: E226 missing whitespace around arithmetic operator
Line 270:42: E226 missing whitespace around arithmetic operator
Line 272:17: E225 missing whitespace around operator
Line 280:26: E231 missing whitespace after ':'
Line 638:1: E302 expected 2 blank lines, found 1

In the file clinicaaddl/clinicaaddl/tools/deep_learning/data.py:

Line 584:39: E225 missing whitespace around operator
Line 601:1: W293 blank line contains whitespace
Line 602:15: E401 multiple imports on one line

In the file clinicaaddl/clinicaaddl/tools/deep_learning/iotools.py:

Line 106:18: E225 missing whitespace around operator
Line 107:13: E225 missing whitespace around operator
Line 108:56: E226 missing whitespace around arithmetic operator
Line 111:18: E225 missing whitespace around operator
Line 112:13: E225 missing whitespace around operator
Line 113:38: E231 missing whitespace after ','
Line 114:108: E226 missing whitespace around arithmetic operator
Line 120:1: E302 expected 2 blank lines, found 1

In the file clinicaaddl/clinicaaddl/tools/deep_learning/models/__init__.py:

Line 3:83: E231 missing whitespace after ','
Line 3:142: E231 missing whitespace after ','
Line 23:58: E231 missing whitespace after ','
Line 23:125: E231 missing whitespace after ','
Line 24:23: E225 missing whitespace around operator
Line 24:36: E231 missing whitespace after ':'

In the file clinicaaddl/clinicaaddl/tools/deep_learning/models/image_level.py:

Line 194:1: E302 expected 2 blank lines, found 1
Line 203:1: E302 expected 2 blank lines, found 1
Line 210:1: E303 too many blank lines (3)
Line 214:1: E302 expected 2 blank lines, found 1
Line 223:1: E302 expected 2 blank lines, found 1
Line 229:1: E303 too many blank lines (3)

In the file clinicaaddl/clinicaaddl/tools/deep_learning/models/modules.py:

Line 140:1: E265 block comment should start with '# '
Line 141:1: E305 expected 2 blank lines after class or function definition, found 1
Line 141:1: E402 module level import not at top of file
Line 142:1: E302 expected 2 blank lines, found 0
Line 152:1: E302 expected 2 blank lines, found 1
Line 237:1: E265 block comment should start with '# '
Line 238:1: E302 expected 2 blank lines, found 1
Line 257:1: E303 too many blank lines (3)
Line 350:30: E261 at least two spaces before inline comment
Line 350:31: E262 inline comment should start with '# '
Line 372:47: E226 missing whitespace around arithmetic operator
Line 374:36: E226 missing whitespace around arithmetic operator
Line 374:50: E226 missing whitespace around arithmetic operator
Line 379:36: E226 missing whitespace around arithmetic operator

In the file clinicaaddl/clinicaaddl/tools/tsv/kfold_split.py:

Line 84:20: E225 missing whitespace around operator
Line 85:19: E225 missing whitespace around operator

In the file clinicaaddl/clinicaaddl/train/__init__.py:

Line 4:42: W292 no newline at end of file

In the file clinicaaddl/clinicaaddl/train/train_singleCNN.py:

Line 8:11: E401 multiple imports on one line
Line 11:1: E402 module level import not at top of file
Line 12:1: E402 module level import not at top of file
Line 16:1: E402 module level import not at top of file
Line 17:1: E402 module level import not at top of file
Line 18:1: E402 module level import not at top of file
Line 101:22: E225 missing whitespace around operator
Line 114:9: E265 block comment should start with '# '


[RHOAIENG-15710] fix the version of codeflare-sdk shown for 2024.1

Fix the Codeflare-SDK version for the 2024.1 images.

[1] https://issues.redhat.com/browse/RHOAIENG-15710

I checked the actual Codeflare-SDK version via quay package metadata feature for images in the metadata - all that have Codeflare-SDK installed have it in version 0.21.
Description

How Has This Been Tested?

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/lgtm

Thank you guys for a quick reviews.
We don't need to build the images for this change:
/override ci/prow/images
Let's move this on then:
/approve


Scroll-spy doesn't work correctly on Firefox

Hello!
Apparently Firefox is treating differently scrollIntoView thus it causes a bug when selecting one of the bottom elements in the nav view page gets scrolled even further down from that element as if there is a recursive loop.
Thank you!

Hey @kirudenk I was able to reproduce and it's a pretty simple fix (https://github.com/anvilco/spectaql/pull/299). We'll have a new release out soon with the fix.

@kirudenk should be fixed in 1.0.14...


Fix #221 - Deepcopy widgets

Thank you for the help, it's merged and I'll push out a new release later today :)


HBASE-24249 Move code in FSHDFSUtils to FSUtils and mark related clas…

…ses as final

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 12s
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.

_ master Compile Tests _

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

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

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

+1 :green_heart:
spotbugs
4m 50s
master passed

_ Patch Compile Tests _

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

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

+1 :green_heart:
checkstyle
0m 23s
The patch passed checkstyle in hbase-common

-0 :warning:
checkstyle
1m 31s
hbase-server: The patch generated 11 new + 1204 unchanged - 80 fixed = 1215 total (was 1284)

-0 :warning:
checkstyle
0m 20s
hbase-mapreduce: The patch generated 1 new + 103 unchanged - 7 fixed = 104 total (was 110)

-0 :warning:
checkstyle
0m 13s
hbase-backup: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

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

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

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

+1 :green_heart:
hadoopcheck
12m 10s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
5m 37s
the patch passed

_ Other Tests _

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

48m 20s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

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

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / a9c1ca9577

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

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

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

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

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

modules
C: hbase-common hbase-server hbase-mapreduce hbase-backup hbase-it hbase-examples U: .

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

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

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

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

_ Prechecks _

_ master Compile Tests _

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

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

+1 :green_heart:
compile
2m 54s
master passed

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

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

_ Patch Compile Tests _

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

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

+1 :green_heart:
compile
2m 54s
the patch passed

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

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

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

_ Other Tests _

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

-1 :x:
unit
128m 18s
hbase-server in the patch failed.

+1 :green_heart:
unit
12m  8s
hbase-mapreduce in the patch passed.

+1 :green_heart:
unit
9m 54s
hbase-backup in the patch passed.

+1 :green_heart:
unit
1m 11s
hbase-it in the patch passed.

+1 :green_heart:
unit
2m 33s
hbase-examples in the patch passed.

187m 56s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 752ae72377c4 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
dev-support/hbase-personality.sh

git revision
master / a9c1ca9577

Default Java
1.8.0_232

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1586/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

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

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

modules
C: hbase-common hbase-server hbase-mapreduce hbase-backup hbase-it hbase-examples U: .

_ Patch Compile Tests _

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

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

+1 :green_heart:
compile
3m 17s
the patch passed

+1 :green_heart:
javac
3m 17s
the patch passed

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

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

-0 :warning:
javadoc
0m 39s
hbase-server in the patch failed.

-0 :warning:
javadoc
0m 20s
hbase-mapreduce in the patch failed.

-0 :warning:
javadoc
0m 17s
hbase-backup in the patch failed.

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

_ Other Tests _

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

-1 :x:
unit
172m  4s
hbase-server in the patch failed.

+1 :green_heart:
unit
15m 23s
hbase-mapreduce in the patch passed.

+1 :green_heart:
unit
11m 34s
hbase-backup in the patch passed.

+1 :green_heart:
unit
0m 59s
hbase-it in the patch passed.

+1 :green_heart:
unit
2m 20s
hbase-examples in the patch passed.

239m 23s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

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

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / a9c1ca9577

Default Java
2020-01-14

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

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

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

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

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

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

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

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

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

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

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1586/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

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

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

modules
C: hbase-common hbase-server hbase-mapreduce hbase-backup hbase-it hbase-examples U: .

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

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 15s
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.

_ master Compile Tests _

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

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

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

+1 :green_heart:
spotbugs
4m 51s
master passed

_ Patch Compile Tests _

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

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

+1 :green_heart:
checkstyle
0m 25s
The patch passed checkstyle in hbase-common

-0 :warning:
checkstyle
1m 30s
hbase-server: The patch generated 1 new + 1203 unchanged - 81 fixed = 1204 total (was 1284)

+1 :green_heart:
checkstyle
0m 19s
hbase-mapreduce: The patch generated 0 new + 103 unchanged - 7 fixed = 103 total (was 110)

+1 :green_heart:
checkstyle
0m 13s
The patch passed checkstyle in hbase-backup

+1 :green_heart:
checkstyle
0m 15s
The patch passed checkstyle in hbase-it

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

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

+1 :green_heart:
hadoopcheck
12m 14s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
5m 33s
the patch passed

_ Other Tests _

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

48m 12s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

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

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 84f2e95dd5

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

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

modules
C: hbase-common hbase-server hbase-mapreduce hbase-backup hbase-it hbase-examples U: .

_ Patch Compile Tests _

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

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

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

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

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

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

-0 :warning:
javadoc
0m 41s
hbase-server in the patch failed.

-0 :warning:
javadoc
0m 22s
hbase-mapreduce in the patch failed.

-0 :warning:
javadoc
0m 19s
hbase-backup in the patch failed.

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

_ Other Tests _

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

-1 :x:
unit
129m 18s
hbase-server in the patch failed.

+1 :green_heart:
unit
12m 29s
hbase-mapreduce in the patch passed.

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

+1 :green_heart:
unit
1m  6s
hbase-it in the patch passed.

+1 :green_heart:
unit
2m 35s
hbase-examples in the patch passed.

192m  1s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

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

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 84f2e95dd5

Default Java
2020-01-14

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

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

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

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

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

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

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

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

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

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

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1586/2/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

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

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

modules
C: hbase-common hbase-server hbase-mapreduce hbase-backup hbase-it hbase-examples U: .

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

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

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

_ Prechecks _

_ master Compile Tests _

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

+1 :green_heart:
mvninstall
4m 21s
master passed

+1 :green_heart:
compile
3m 14s
master passed

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

+1 :green_heart:
javadoc
2m 38s
master passed

_ Patch Compile Tests _

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

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

+1 :green_heart:
compile
2m 59s
the patch passed

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

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

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

_ Other Tests _

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

-1 :x:
unit
197m  8s
hbase-server in the patch failed.

+1 :green_heart:
unit
15m 49s
hbase-mapreduce in the patch passed.

+1 :green_heart:
unit
12m 17s
hbase-backup in the patch passed.

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

+1 :green_heart:
unit
2m 20s
hbase-examples in the patch passed.

265m  9s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

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

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 84f2e95dd5

Default Java
1.8.0_232

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1586/2/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

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

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

modules
C: hbase-common hbase-server hbase-mapreduce hbase-backup hbase-it hbase-examples U: .

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

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

The failure of TestAccessController is caused by HBASE-24211.
PTAL @saintstack
Thanks.


GCDS- POC header

Summary | Résumé
POC using GCDS Header
https://github.com/cds-snc/platform-forms-client/issues/4273

Won't compile
 GET /en/sla?_rsc=bcp9r 200 in 1743ms
TypeError: Cannot read properties of undefined (reading 'split')
    at Object.read (webpack-internal:///(ssr)/./node_modules/i18next-browser-languagedetector/dist/esm/i18nextBrowserLanguageDetector.js:92:30)
    at Object.lookup (webpack-internal:///(ssr)/./node_modules/i18next-browser-languagedetector/dist/esm/i18nextBrowserLanguageDetector.js:109:22)

 ○ Compiling /api/auth/[...nextauth] ...
 ⚠ ./node_modules/linkedom/commonjs/canvas.cjs
Module not found: Can't resolve 'canvas' in '/Users/timarney/projects/platform-forms-client/node_modules/linkedom/commonjs'

Import trace for requested module:
./node_modules/linkedom/commonjs/canvas.cjs
./node_modules/linkedom/esm/html/canvas-element.js
./node_modules/linkedom/esm/shared/html-classes.js
./node_modules/linkedom/esm/index.js
<EMAIL_ADDRESS>./app/layout.tsx
 GET /api/auth/session 200 in 2707ms
[08:26:06.824] WARN: NextAuth warning - Code: debug-enabled

Server.js
var import_linkedom = require("linkedom");
(() => {
  if (typeof window !== "undefined") {
    return;
  }
  const { document, customElements, Event, CustomEvent, DocumentFragment, DOMParser, HTMLElement: LinkedomHTMLElement, HTMLTemplateElement, MutationObserver } = (0, import_linkedom.parseHTML)("...");
  class HTMLElement extends LinkedomHTMLElement {
    static get observedAttributes() {
      return HTMLElement._observedAttributes;
    }
    static set observedAttributes(value) {
      HTMLElement._observedAttributes = value;
    }
  }
  HTMLElement._observedAttributes = [];
  class Storage {
    key() {
    }
    getItem() {
    }
    setItem() {
    }
    removeItem() {
    }
    clear() {
    }
  }
  const localStorage = new Storage();
  class ResizeObserver {
    observe() {
    }
    unobserve() {
    }
    disconnect() {
    }
  }
  class CSSStyleDeclaration {
    getPropertyPriority() {
    }
    getPropertyValue() {
    }
    item() {
    }
    removeProperty() {
    }
    setProperty() {
    }
  }
  Object.assign(globalThis, {
    document,
    customElements,
    Event,
    CustomEvent,
    DocumentFragment,
    DOMParser,
    HTMLElement,
    HTMLTemplateElement,
    MutationObserver,
    localStorage,
    ResizeObserver,
    CSSStyleDeclaration,
    getComputedStyle: function getComputedStyle() {
      return new CSSStyleDeclaration();
    },
    cancelAnimationFrame: function cancelAnimationFrame() {
    },
    addEventListener: function addEventListener() {
    },
    removeEventListener: function removeEventListener() {
    },
    requestAnimationFrame: function requestAnimationFrame(callback) {
      callback();
    }
  });
})();

@ethanWallace @daine
Any thoughts on how to get around the linkedom issue above?
I can see the same errors throwing on this example but in this case it does render the header.
https://codesandbox.io/p/devbox/hyzfcx

@timarney You should be able to add this bit to your next.config.
  webpack: (config) => {
    // Fixes dependency issues with `canvas` and `perf_hooks` of linkedom
    config.resolve.fallback = { canvas: false, perf_hooks: false };

    return config;
  },

Thanks @ethanWallace  ---
Hitting a further issue
 ⨯ uncaughtException: TypeError: Cannot read properties of undefined (reading 'split')
    at Object.read (webpack-internal:///(ssr)/./node_modules/i18next-browser-languagedetector/dist/esm/i18nextBrowserLanguageDetector.js:92:30)

Pushed a commit comment out the lang detect for now.

With workarounds in place we're able to get a component rendering.

Need to dig into a few things:

CSP
i18N - browser detect script issue.

@ethanWallace have you folks looked at nonces yet?
https://stenciljs.com/docs/csp-nonce
Our CSP is very strict

Having done some initial investigation into feasibility we have 2 initial blockers

Content security policy https://github.com/cds-snc/platform-forms-client/pull/4281#issuecomment-2358313268
i18N - browser detect script issue https://github.com/cds-snc/platform-forms-client/pull/4281#issuecomment-2355728595

@timarney We haven't explored how a team could use that nonce helper function yet.

Closing this PR as it was only for feasibility check per https://github.com/cds-snc/platform-forms-client/issues/4273
Thanks for the help @ethanWallace

@timarney could you send a few info my way around your setup on nonces? We have CSP as well but haven't implemented the nonce method but it's something we can look at. I'd be curious to know the implementation details that you have around it.

@daine for the nonce
It might be possible if the setNonce function was exposed for use to use we might be able to set it as we do have the ability to pass the generated nonce to components i.e. on the Form Ownership we pass the nonce to React Select / Emotion.   There's further issues with the nonce on our end but noting that's how we've done it for other libs.
  const { session, ability } = await authCheckAndThrow().catch(() => ({
    session: null,
    ability: null,
  }));
...
  const nonce = await getNonce();

    return (
      <ManageForm
        nonce={nonce}

export const FormOwnership = ({
  nonce,

const cache: EmotionCache = createCache({
    key: "gc-forms",
    nonce: nonce || "xyz123",
  });


Random public port assignment to a service (if not specified)

https://github.com/rancher/rancher/issues/2097

Port range is defined on per environment (project) basis. New API field is introduced for the project:

servicesPortRange (string, optional, default value "49153-65535", CRU permissions)

If service has a port defined as "80" or "80/tcp" (with no public port specified), public port will get automatically assigned to the port from environment.servicesPortRange

Port gets assigned to the service on its creation, and the assignment gets removed after service is removed. Once disassociated, the port can be used by other services.

Random port can't have duplicates among other ports generated in random manner.
We don't validate random port against public ports specified by the user explicitly. The assumption is that the servicesPortRange is reserved for random allocation, and user shouldn't specify ports from this range when creates service/standalone instance.
@ibuildthecloud @vincent99 please review
Also split test_svc_discovery test into smaller files

We would need a UI support for modifying default port range for the environment @vincent99

To expand on this a little more, it would also be nice if there was the ability to increment port numbers for a service (assuming this isn't what's happening here). That way you get more predictable port numbering patterns.
For example, start range number + container ID in service.

@msh100 The way this works is that the service gets a random port assigned and that port is used with every single container.  If the user puts ports: 80 it effectively becomes ports: 45242:80.  This way every container has the same port but just on different hosts.

@ibuildthecloud ah thanks for the clarification - is there currently a way of exposing a port and having multiple containers of that service on a single host? As I am assuming this PR doesn't help out there then.

@msh100 Run the service without publishing ports with whatever scale you want, then a load balancer pointing to it and listening on the actual public port with scale <= number of hosts.

@alena1108 Can you remove: code/packaging/app/keys.properties.  You might as well put that in the .gitignore


KAFKA-7080: replace numSegments with segmentInterval

See also KIP-319.
Replace number-of-segments parameters with segment-interval-ms parameters in various places. The latter was always the parameter that several components needed, and we accidentally supplied the former because it was the one available.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

LGTM! cc @bbejeck for another view.

@guozhangwang , I think this PR is ready for a final review.

Merged to trunk. @vvcephei thanks!
Please feel free to update KIP-319's adoption version on the wiki.


Raw string with backslash should not highlight letters following \

Issue Type: Bug
A raw string like below, is highlighting h and w and p and r.
file_name = r'C:\hello\world\python_scripts\results{0}_auto_test_results.log'
It should not.
Thank you.
VS Code version: Code 1.36.1 (2213894ea0415ee8c85c5eea0d0ff81ecc191529, 2019-07-08T22:59:35.033Z)
OS version: Windows_NT x64 10.0.17134

Extension
Author (truncated)
Version

pgformatter
bra
1.6.5

python-extension-pack
don
1.6.0

gc-excelviewer
Gra
2.1.32

MagicPython
mag
1.1.0

python
ms-
2019.6.24221

vscodeintellicode
Vis
1.1.8

@parsh78 what language is this?

This is Python.

From: Alex Ross<EMAIL_ADDRESS>Sent: Tuesday, July 30, 2019 2:43:33 AM
To: microsoft/vscode<EMAIL_ADDRESS>Cc: Parag Shah<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/vscode] Raw string with backslash should not highlight letters following \ (#78140)
@parsh78https://github.com/parsh78 what language is this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode/issues/78140?email_source=notifications&email_token=AA5DEYV5NMSFXHS4ID6FSPTQCAEMLA5CNFSM4IHX5YHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3DNESY#issuecomment-516346443, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA5DEYQKWSK3DYZ753WSYALQCAEMLANCNFSM4IHX5YHA.

Duplicate of https://github.com/MagicStack/MagicPython/issues/155. Can you do as suggested here?


implement transport api for PPL inter-plugin communication

Signed-off-by: Zhongnan Su<EMAIL_ADDRESS>Update
After discussion in the thread here https://github.com/opensearch-project/common-utils/pull/155, we decided for now we'll  keep the transport interfaces in sql repo, instead of common-utils for the following reasons.

there's no consumer plugin for this feature yet. Because the response format is not encapsulated. Currently on trasnport layer there's only a json string in the response. Ideally we want a ResultSet or DataFrame like response for clients to access data. An issued is created w.r.t this.
This change is to the request model, which is the input of the SQL/PPL engine. Having it separated into 2 repos is not development friendly. E.g, the CI will fail for sure, because CI only pulls against released common-utils

Description
Add a transport layer for handle PPL request. This will expose transport interface for inter-plugin communication. The existing Rest layer handler is also being refactored to make use of the transport layer

Create TransportPPLQueryAction

refactored RestPPLQueryAction to service as a route layer, moved the query request handling logic to TransportPPLQueryAction.

Register action and handler in SQLPlugin

Refactored PPLQueryRequest to be writable, for the transport layer commuication

Create transport response model that currently holds a string response.

Add common-utils as dependency, since the request/response models will be exposed in common-utils for other plugin to consume as well.

TODO:

Request and response format are strings for now.(may need modification in future version, especially response format)
Implement transport interface for SQL, in a similar manner

Issues Resolved
#521
CI/CD

The CI is failing because of the pending PR in common-utils
[Fixed]local build failed because of jarhell caused by ml-commons. Raised another PR in their repo to fix. https://github.com/opensearch-project/ml-commons/pull/328

| java.home: /Library/Java/JavaVirtualMachines/jdk-14.0.2.jdk/Contents/Home
| examining directory: /Users/szhongna/Desktop/Projects/OpenSearch/sql/integ-test/build/classes/java/test
| examining directory: /Users/szhongna/Desktop/Projects/OpenSearch/sql/integ-test/build/resources/test
| examining jar: /Users/szhongna/Desktop/Projects/OpenSearch/sql/sql-jdbc/build/libs/opensearch-sql-jdbc-<IP_ADDRESS>.jar
| examining jar: /Users/szhongna/Desktop/Projects/OpenSearch/sql/sql-jdbc/build/libs/opensearch-sql-jdbc-<IP_ADDRESS>.jar
| Exception in thread "main" java.lang.IllegalStateException: jar hell!
| class: org.opensearch.jdbc.ResultSetImpl
| jar1: /Users/szhongna/Desktop/Projects/OpenSearch/sql/sql-jdbc/build/libs/opensearch-sql-jdbc-<IP_ADDRESS>.jar
| jar2: /Users/szhongna/Desktop/Projects/OpenSearch/sql/sql-jdbc/build/libs/opensearch-sql-jdbc-<IP_ADDRESS>.jar
|       at org.opensearch.bootstrap.JarHell.checkClass(JarHell.java:317)
|       at org.opensearch.bootstrap.JarHell.checkJarHell(JarHell.java:212)
|       at org.opensearch.bootstrap.JarHell.checkJarHell(JarHell.java:99)
|       at org.opensearch.bootstrap.JarHell.main(JarHell.java:83)

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':integ-test:jarHell'.
> Process 'command '/Library/Java/JavaVirtualMachines/jdk-14.0.2.jdk/Contents/Home/bin/java'' finished with non-zero exit value 1

Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #533 (7949e17) into main (d3a76c9) will decrease coverage by 32.47%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #533       +/-   ##
=============================================
- Coverage     95.23%   62.76%   -32.48%
=============================================
  Files           276       10      -266
  Lines          7410      658     -6752
  Branches        537      119      -418
=============================================
- Hits           7057      413     -6644
+ Misses          299      192      -107
+ Partials         54       53        -1

Flag
Coverage Δ

query-workbench
62.76% <ø> (ø)

sql-engine
?

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

Impacted Files
Coverage Δ

...c/main/java/org/opensearch/sql/ppl/PPLService.java

...arch/storage/system/OpenSearchSystemIndexScan.java

...g/opensearch/sql/data/utils/ExprValueOrdering.java

...arch/storage/script/filter/FilterQueryBuilder.java

...pensearch/sql/planner/physical/WindowOperator.java

...ript/aggregation/dsl/AggregationBuilderHelper.java

...rch/planner/logical/OpenSearchLogicalIndexAgg.java

...search/executor/protector/ResourceMonitorPlan.java

.../sql/planner/physical/PhysicalPlanNodeVisitor.java

...nsearch/sql/expression/window/WindowFunctions.java

... and 256 more

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 d3a76c9...7949e17. Read the comment docs.

seems missing files, SQLActions, TransportQueryResponse

@penghuo Those are interfaces that should be defined in common-utils and used by any caller plugins. Currently I have a PR pending for common-utils. Once that's merged and released to maven central, this PR will pass CI

Addressed all comments. Local build keeps failing during ./gradlew build, > Task :integ-test:jarHell FAILED
| sun.boot.class.path: null
| java.home: /Library/Java/JavaVirtualMachines/jdk-14.0.2.jdk/Contents/Home
| examining directory: /Users/szhongna/Desktop/Projects/OpenSearch/sql/integ-test/build/classes/java/test
| examining directory: /Users/szhongna/Desktop/Projects/OpenSearch/sql/integ-test/build/resources/test
| examining jar: /Users/szhongna/Desktop/Projects/OpenSearch/sql/sql-jdbc/build/libs/opensearch-sql-jdbc-<IP_ADDRESS>.jar
| examining jar: /Users/szhongna/Desktop/Projects/OpenSearch/sql/sql-jdbc/build/libs/opensearch-sql-jdbc-<IP_ADDRESS>.jar
| Exception in thread "main" java.lang.IllegalStateException: jar hell!
| class: org.opensearch.jdbc.ResultSetImpl
| jar1: /Users/szhongna/Desktop/Projects/OpenSearch/sql/sql-jdbc/build/libs/opensearch-sql-jdbc-<IP_ADDRESS>.jar
| jar2: /Users/szhongna/Desktop/Projects/OpenSearch/sql/sql-jdbc/build/libs/opensearch-sql-jdbc-<IP_ADDRESS>.jar
|       at org.opensearch.bootstrap.JarHell.checkClass(JarHell.java:317)
|       at org.opensearch.bootstrap.JarHell.checkJarHell(JarHell.java:212)
|       at org.opensearch.bootstrap.JarHell.checkJarHell(JarHell.java:99)
|       at org.opensearch.bootstrap.JarHell.main(JarHell.java:83)

It doesn't related to my change. Any idea how can I resolve this? @penghuo @joshuali925

doctest and gradle test gradle integTest can pass

build/libs/opensearch-sql-jdbc-<IP_ADDRESS>.jar

clean you local build folder will help.

@penghuo Hi Peng, I have moved the req/response model from common-utils to sql repo, and added unit test. CI now can pass. Could you please take a look? Thanks


If my dataset is not a time series with uniform intervals, will the models still be effective?

For instance, if my data looks like :
ds                      y
2024-10-01       y1
2024-10-02       y2
2024-10-05       y3
2024-10-14       y4
2024-10-16       y5
2024-10-19       y6
....

Hello! The main problem will be in setting the correct frequency to create the dates, as we only support valid pandas or polars offset alias. In this case, I suggest you use integers instead of dates and specify freq=1. The model will still work, since internally, it tries to map an input sequence to an output sequence.
Let me know if that answers your question!

Hello! The main problem will be in setting the correct frequency to create the dates, as we only support valid pandas or polars offset alias. In this case, I suggest you use integers instead of dates and specify freq=1. The model will still work, since internally, it tries to map an input sequence to an output sequence.
Let me know if that answers your question!

It's great, this method solve my problem, thank you!


Do not use full urls for including images

Use relative URLs instead.

@yozw I believe we had to make this change to support readthedocs (which we are trying to move to).
cc: @nicholas-leonard

Ah, in that case the last two links should be fixed (as they do have relative urls). Let's see what @nicholas-leonard says.

Friendly ping to @nicholas-leonard.


Support manual state verification

@cathay4t this is ready for review, integration tests are missing. I will add them if you like the overall approach and code.

Basically, I like the idea of manual checkpoint and it works well mostly.
Things to fix:

During handling of the above exception, another exception occurred: issue on:

No privileged.
Create 2+ checkpoint.
Commit or rollback after timeout.

These errors are fixed now.

Add integration test for nmstatectl to simulate the use of above cases.

WIP.

In stead of showing detail implementation, please consider this patch:
diff --git a/libnmstate/netapplier.py b/libnmstate/netapplier.py
index bb45441..59c3ace 100644
--- a/libnmstate/netapplier.py
+++ b/libnmstate/netapplier.py
@@ -124,8 +124,10 @@ def _apply_ifaces_state(desired_state, verify_change, commit,
                 _verify_change(desired_state)
         if not commit:
             return checkpoint
-    except nm.checkpoint.NMCheckPointCreationError:
-        raise NmstateConflictError('Error creating a check point')
+    except nm.checkpoint.NMCheckPointCreationError as e:
+        six.raise_from(
+            NmstateConflictError(
+                'Error creating a check point: {}'.format(str(e))), None)

 def _verify_change(desired_state):
diff --git a/libnmstate/nm/checkpoint.py b/libnmstate/nm/checkpoint.py
index 4b8211e..ddf0358 100644
--- a/libnmstate/nm/checkpoint.py
+++ b/libnmstate/nm/checkpoint.py
@@ -18,6 +18,7 @@
 import logging

 import dbus
+import six

 import libnmstate.nm.nmclient

@@ -118,7 +119,7 @@ class CheckPoint(object):
                           dbuspath, timeout)
             self._dbuspath = dbuspath
         except dbus.exceptions.DBusException as e:
-            raise NMCheckPointCreationError(str(e))
+            six.raise_from(NMCheckPointCreationError(str(e)), None)

     def destroy(self):
         try:

In stead of showing detail implementation, please consider this patch:

Yes, I have considered it but I do not see why this would be an advantage. The implementation details are stored in the __context__ attribute, therefore it is IMHO not part of an API. The default exception handler is just using it for a more useful traceback and the correct way is to handle the exception instead of making them less useful.


Electrolux centurio iq3.0

Hello. Can I connect the Electrolux Centurio IQ 3.0 water heater via homebridge?

Hi @silvious22
Can you see your water heater in the Electrolux app and control it? If so, would you mind changing the password to your account and sending me the credentials to my mail: tomekkleszcz[at]icloud.com, so I could reverse-engineer the communication between the app and Electrolux servers and add support for this device in this plugin?
Thank you!


Ignore RFC-incompliant date ranges

Original issue: https://github.com/domainaware/parsedmarc/issues/282
We potentially want a slightly different approach - specifically I think it might make more sense to make this configurable in the parser options.
We should also make sure to obtain/create a sample that has this issue.

Hello @nhairs,
What is the reason you want RFC-incompliant reports to be parsed? Shouldn't rule-violations be ignored by default?
Thank you!
Kind Regards,
Sebastiaan

Hi @RealSebFox,
That's my plan: ignore by default but allow accepting in case some decides they need them.


Keyboard issue on search

On search if the item is clicked on the menu when the keyboard is showing on iOS it only dismisses the keyboard but does not hide the open menu.
Also when applied in Keyboard Scrollview the field scrolls up when keyboard is open and menu opens from under the field, but when the keyboard is dismissed, the menu stays in the opened position and the field scrolls down to the actual position which looks odd.

hi @abhisekmallik ,
I'm working on it.

On search if the item is clicked on the menu when the keyboard is showing on iOS it only dismisses the keyboard but does not hide the open menu.
Also when applied in Keyboard Scrollview the field scrolls up when keyboard is open and menu opens from under the field, but when the keyboard is dismissed, the menu stays in the opened position and the field scrolls down to the actual position which looks odd.

keyboard issues on android by running the example

input container with placeholder scrolls up but the dropDown container stays at the previous possition

I am having this same issue as @AkshayPatel7000. The search bar avoids the keyboard just fine, but the dropdown is hidden by the keyboard, so when we search for items in the dropdown, we can't see any results since the keyboard is hiding it.

hi All,
Please install new version 2.3.1 and re-check this issue.

Hi @hoaphantn7604, sorry for the long delay in checking this. I downloaded the code and ran the example in expo and the dropdown is still being hidden by the keyboard like in the photo by @AkshayPatel7000. I messed around with the code and have a fix for the logic as to when it renders the dropdown on top, but the keyboard keeps getting dismissed upon mounting the dropdown on top. I assume the keyboard keeps getting dismissed due to how the dropdown is being rendered. I was hoping you could take a look at this when you get a chance please and thank you!
Here is my work in progress changes: #139
I have also attached a video that shows how the dropdown rendering is messed up currently
https://user-images.githubusercontent.com/16094260/216484305-63e16078-0359-4498-a9cb-f125e0d52045.mp4

Hi @hoaphantn7604, I was able to tinker around with this some more and realized the problem was with conditionally rendering the search input. Since the keyboard references the search input, then un-rendering it and rendering a different search input will cause the keyboard to be dismissed.
To fix this, I conditionally render the list instead as that will not affect the keyboard when it disappears and is rendered before/after the search input.
Please see the same PR #139 for the new changes and feel free to modify any of my changes as you see fit. This change does make the dropdown behave how I expect it should when the keyboard is showing.

Hi @hoaphantn7604, I was able to tinker around with this some more and realized the problem was with conditionally rendering the search input. Since the keyboard references the search input, then un-rendering it and rendering a different search input will cause the keyboard to be dismissed.
To fix this, I conditionally render the list instead as that will not affect the keyboard when it disappears and is rendered before/after the search input.
Please see the same PR #139 for the new changes and feel free to modify any of my changes as you see fit. This change does make the dropdown behave how I expect it should when the keyboard is showing.

hi @myou11 ,
Thank you create PR. I merged your PR and it raised a new issue.
https://user-images.githubusercontent.com/50919571/216881926-f8897b9e-09d5-4363-9991-ed6a34954aea.mp4

hi All,
This issue has been resolved and is being tested.

hi All,
This issue is resolved. Please install later version 2.6.0


Feature/reorganizing code

This file structure makes the code more manageable in my head (hbu?). Any opinions on the file names? I dasherized the camel-case file names because I found them slightly easier to parse that way. Let me know if you have any input.

Looks good. Remember we still need a base application layout for later (without the menu and other junk in it).


Rendering Mapbox GL Style as base layer

How to reproduce

Leaflet version I'm using:1.0.1
Browser (with version) I'm using: Firefox 49.0.2
OS/Platform (with version) I'm using: Windows 7 Enterprise
step 1: Created Mapbox Style in Mapbox Studio
step 2: Added the tile normally using L.tilelayer()
step 3: Map appears in the browser but few of the layers are not rendered completely. Only partial render and not at all most of the times.

When I load the same Mapbox style using Mapbox GL API (mapbox-gl-js), the layers load fine.
What behaviour I'm expecting and which behaviour I'm seeing
All layers of the map to be rendered properly with L.tilelayer()
Minimal example reproducing the issue
Using Leaflet API: http://playground-leaflet.rhcloud.com/vaxe
Same base map with using Mapbox GL API.: http://playground-leaflet.rhcloud.com/naxil

As far as I can see, this is not Leaflet issue, Leaflet correctly displays the images returned by Mapbox's API. Any differenes you see is a difference in how Mapbox render the styling depending on if its done in the browser (Mapbox GL) or on Mapbox's servers (Leaflet). You should report this to Mapbox.

Thanks @perliedman . Will report it to Mapbox.

I have now posted it here for cross-reference.
https://github.com/mapbox/mapbox-gl-js/issues/3630

For the sake of completeness, the issue was fixed by adding @2x for retina display.
https://www.mapbox.com/api-documentation/#retina


TypeError in Opera Mobile 32

Catched in Sentry:
TypeError
object is not a function
/scripts/libs/dom4.js at line 2

/*! (C) Andrea Giammarchi - @WebReflection - Mit Style License */
(function(window){"use strict";function createDocumentFragment(){return document.createDocumentFragment()}function createElement(nodeName){r {snip}

User:
Opera Mobile 32.0.2254
Android 4.2.1

Note about r {snip}: it's Sentry's artifact, in real-life there is:
/*! (C) Andrea Giammarchi - @WebReflection - Mit Style License */
(function(window){"use strict";function createDocumentFragment(){return document.createDocumentFragment()}function createElement(nodeName){return document.createElement(nodeName)}

it'd be awesome if you could use the dom4.max.js file instead so I understand which object is not actually a function.
I'm already happy it wasn't just undefined is not a function error 😄

it'd be awesome if you could use the dom4.max.js file instead so I understand which object is not actually a function.

I was hoping you have a test environment.
I could not run a similar version of Opera neither in the emulator, nor at my smartphone.
OK, I'll switch to dom4.max.js. :disappointed:

I was hoping you have a test environment.

I've tested manually through this page every device/browser in the list but maybe I've missed Opera Mobile 32.0.2254 for Android 4.2.1 there 🤷♂

I could not run a similar version of Opera neither in the emulator, nor at my smartphone.

I ran it in the ARM (not x86) emulator. But I can't connect via chrome://inspect/#devices. But the site on which the error was occurred looks good.

I've tested manually through this page every device/browser in the list but maybe I've missed Opera Mobile 32.0.2254 for Android 4.2.1 there :man_shrugging:

This page is green.
But user, triggered this error, is real, it's not a bot.
Let's wait for another event, but with max version. :man_shrugging:

Let's wait for another event, but with max version. 🤷♂

I guess that never happened. Closing.


[Feature Request] SSL Cert reload API in C++

Reference - https://groups.google.com/forum/#!searchin/grpc-io/ssl$20arpit|sort:date/grpc-io/TUsC_MTIv2U/oRhuDxWaBgAJ
What version of gRPC and what language are you using?
1.10.0/C++
I have discussed this issue on the above mailing list thread and brought up in the last community sync up. I am filing this issue here so it is officially in consideration.
Appreciate your time.
Thanks.

Hi,
Is this on your radar to provide support for?
Thanks.

@abaldeva Do you want to write c++ and contribute to the community?

I don't have any plans. But if it is not on your radar, let me know and I'll see if I can find out someone at my company to do this (I am leaving for a long vacation soon).
Thanks.

@jiangtaoli2016 is there any update? Has grpc support reload SSL certificate in C++?

I do not think gRPC security team has any imminent plan to support the credential reload in C++. However, you can use TLSCredentials and TLSServerCredentials that currently has support for credential reload.


Running WW3 with sea-state dependent drag coefficient

Hi, What should I do to run the WW3 version in the CoastalApp (standalone) with a sea state dependent drag coefficient, that follows the GFDL distribution? Should I compile the WW3 code with the FLD1 option in the switch file and then add the 2nd line in the following picture to ww3_grid.inp file? Is this enough?

@aliabdolali
Angelos

@sarriegi Please ask this in a ww3 forum. @aliabdolali what do you suggest!?


No longer working with Ubuntu 20.04

With python 2 removed from Ubuntu 20.04, and most other recent distros, youtube-dl-gui can no longer be run.

Use this fork instead.

@axcore thanks for the suggestion. I tried it but it didn't work, meaning it couldnt download youtube videos. Also, it is more than what i need in a GUI for youtube-dl and I designed the UI for youtube-dl-gui. :D

If it doesn't work, submit an issue at the git
If you want an interface exactly like youtube-dl-gui, submit an issue for that as well
youtube-dl-gui is not being maintained, so submitting issues here is not going to help you.

Issue filed: https://github.com/axcore/tartube/issues/77
Thanks for the notice that it isn't maintained anymore. Are you interested in forking it again to just do a gkt3 upgrade to the code?
For others who stumble upon this, I was able to get it running on Ubuntu 20.04 with these steps.

install these packages - python-is-python2 python2 python-wxgtk3.0
install this package, as its not longer in the repo - https://packages.ubuntu.com/disco/python-twodict
install the bionic version - http://ppa.launchpad.net/nilarimogard/webupd8/ubuntu/pool/main/y/youtube-dlg/youtube-dlg_0.4-1~webupd8~bionic9_all.deb

@philipzae I looked at updating youtube-dl-gui once before, but the author has provided no clues about how it is packaged. I think it will be much simpler to implement axcore/tartube#77

I noticed it stopped working most of the time again here on w7 aswell, i get the dreaded "Error" message when trying to download both new and old clips off of YT. I have already updated manually via settings but unfortunately the issue still persists

tartube can't change the playlist and reload download while the download is still in progress


Override with custom styles in external component

Describe the bug
I overrided the styles and it worked properly in localhost (vue-cli-service serve) but when I compile with (vue-cli-service build) my custom are added to chunk-vendors.css but at the end of the  tag in index.html file the default styles are adeed.
Reproduction

Create a project called "A" that uses vue-auto-dropzone
Override styles using a css file imported as "import './assets/styles.css'"
Bundle and publish this project "A"
Use project "A" into another project "B"
Build project "B"

// Your code here

Solution: mark styles as !important;


Dock Icon should bounce and probably show #of unread messages

It would be great if the dock icon would bounce as long as there are unread messages in any chat-window...
Even better if it would also show a tiny number indication the number of messages unread (or at least the number of conversations with unread messages), best if configurable (eg. bounce for 30secs, and show #of messages or bounce for 10secs and show number of conversations with unread messages) or similar.
(Issue migrated from https://trac.jitsi.org/ticket/910)

The dock icon should now show the number of missed calls or whatever number the history button would show.


[BUG] Remove the SHA256 from the key in AES function

AES currently does a SHA on the key by default, this should not be the case! It should be done with a different block if needed.

Done https://github.com/openbullet/openbullet/commit/70e667a141ff082b26f8c2861b9c0447133c7e02


Helia v5

It would be great to have this merged (and a new version released) since we are also in the process of upgrading helia.

@haadcode I have moved the pinning test into the orbitdb replication. It didn't seem to make sense having a separate test suite for one test.


Incorrect register disassembly

For some reason, EBP sometimes gets confused with EBX. First noticed in a left shift instruction.

Switched to an up-to-date disassembly engine. Problem should be fixed.


Notification text editable

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
Other
🔖 Feature description
It will be nice and helpful if we can edit the message/structure of the notification text.
For example, currently discord and telegram use a lot of information, which makes the text big.
When many servers go offline, it's pure confusion on discord.
One line per notification will be cool and much more organized when use a lot of servers.
✔ Solution
make it editable.
❓ Alternatives
No response
📝 Additional Context
No response

+1


Incorrect function name

This function was not defined, but I think it should be calling New-InvalidArgumentError?

This change is

@mgreengit - although the change  I'm scratching my head as to why the error came up.
The function New-InvalidOperationError is defined in here: https://github.com/PlagueHO/xCertificate/blob/dev/DSCResources/CertificateCommon/CertificateCommon.psm1#L185 - just above New-InvalidArgumentError.
What I probably need to do though is add some unit tests to xPDT to cover this. I'll raise an issue to get this done.

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

Comments from Reviewable

Sorry - wrong URL (pasted the one from my fork). Correct one: https://github.com/PowerShell/xCertificate/blob/dev/DSCResources/CertificateCommon/CertificateCommon.psm1#L185

Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable

@mgreengit @PlagueHO Weird. No idea what happened there.


I can't get the request body param from the ServerHttpRequest in spring cloud version  Greenwich. I spent the whole day but faild. It's terrible!

Saying I can't do something isn't helpful. Please describe in detail what your want to accomplish and what you've tried (code is helpful). We do this regularly in gateway, so it must be something you're doing.

I can't get the request param from the request body

is any progress about this feature?

Closing this issue waiting for additional details

Saying I can't do something isn't helpful. Please describe in detail what your want to accomplish and what you've tried (code is helpful). We do this regularly in gateway, so it must be something you're doing.


Fix incorrect Cargo.toml syntax

Replaces commit with rev in all Cargo.toml files

Closing this PR in favor of #6 (fixed the same issue in obce and pallet-asset-chain-extension)
@h4nsu Thank you for noticing!


Photo View App return Roku home screen after two hours

Insignia Roku Build: 4200-20 v9.4.0 updated December 2020
TCL Roku Build: 4207-93 v9.4.0 Updated February 2021
Photo View Build: 5.616.20210216 v3.7.1
Hey Chris,
I work in a museum where we use Roku TVs to display maps and information around our building. Currently, we have 5 Insignia TVs and 1 TCL TV. Over the last 2 weeks we've noticed an ongoing issue where the Photo View app will close on all displays after 2 hours of use.  Each TV has a Google image folder with 10 or less dedicated images at any given time.
I have been troubleshooting all TVs in an attempt to isolate the issue:

Turn the TVs off and back on
Completely restart the TVs and remove power for a few minutes
Check for new updates
Check and change TV settings

 Sleep settings are off

 Screen Saver is off

 Power Saving is off

 Bandwidth saver is off

Check for Photo View updates
Used other apps both on and off our internal network

Right now, we have 4 of 6 TVs disconnected from the network running the images through Roku Media Player.
The TCL TV is still using Photo View on the network and exiting after two hours.
One Insignia TV is running Roku Media Player via USB while connected to the network (the idea behind this is if Media Player closes too, it would be a Roku issue).
The only thing I haven't tried was hard resetting the TV and entering our credentials again.  We definitely prefer using Photo View and have had minimal/no issues at all.  It's made our life easier and we really do like it.  If this is a bug, hopefully this helps bring attention to it.  If there is anything else I should try, I'm more than happy to give it a shot.  Thanks for all your time with this app.

We've started experiencing a similar issue in the past two weeks using the Photo View App on 8 Hisense TVs.
Our slideshows are short 2-15 images.
In the past we have slideshows ran the full 11 hours that we are open. Within the past two weeks the slideshows timeout to the Google Photo Albums screen at almost exactly the 2 hour mark. This is consistent with all 8 TVs.
I have copied nathanb065 troubleshooting on our units but the issues persists.
Hisense Roku TV
Roku TV C219X
Software Version 9.4.0 - build 4200-94
System Update: Last Updated Dec 18, Last Checked Feb 24

We are using 10 TCL Tvs. We are also seeing the app close down after 2 hours.  We have tried all settings recommended by Roku.

With the recent Roku updated OS, they implemented multiple Bandwidth/power saving features. I should have access to a TCL TV soon, so i'll see if I can duplicate. I'm not able to duplicate using standard Roku players but it wouldn't be the first time a new Roku OS implemented a bug in the TCL TV.
If you are using the PhotoView built in screensaver, does it continue playing after 2 hours? Try to use the slideshows on the screensaver side and see if you can reproduce. That'll help in determining a potential cause.
Have you seen this same issue while using any other apps that are sitting idle?

I only use the photo view app on these tvs.
I wish there would even be a way to roll back the TCL software.
On Wed, Feb 24, 2021, 9:36 PM Chris Taylor<EMAIL_ADDRESS>wrote:

With the recent Roku updated OS, they implemented multiple Bandwidth/power
saving features. I should have access to a TCL TV soon, so i'll see if I
can duplicate. I'm not able to duplicate using standard Roku players but it
wouldn't be the first time a new Roku OS implemented a bug in the TCL TV.
If you are using the PhotoView built in screensaver, does it continue
playing after 2 hours? Try to use the slideshows on the screensaver side
and see if you can reproduce. That'll help in determining a potential cause.
Have you seen this same issue while using any other apps that are sitting
idle?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/chtaylo2/Roku-GooglePhotos/issues/375#issuecomment-785552110,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS7K6ZWM2WBJ2JD7M6AAYJTTAXAS5ANCNFSM4YBILHPA
.

Chris, we also only use the Photo View App on our Hisense TVs but I will test the screen saver.
If there’s another App you’d recommend testing with please let me know and I’ll give it a try.

I have exactly to same probleme, photoview leave after 2 hours

I've been away from work for a few days.  I show up a few hours before we open on the weekends and will test out the screen saver on a TCL and Insignia TV. Hopefully I'll have an update tomorrow morning on this.

I have the same problem.  I was using Photoview as an all-day presentation medium.  2 hour cutoff makes it pretty useless to me.  If anyone can suggest another app, it would be great.

Good news.  I was able to replicate the issue on a TCL tv this weekend and have a work around. I’ll publish a fix for the issue tomorrow and should be approved by Wednesday.
It’s due to a bug in the Roku TVs and the work around is able to trick it into not triggering the 2 hour time out.  I’ve had this this running for 8 hours and no issues.

Was just about to post I have the same issue till I noticed your reply.  This would probably work for any Roku device?  We have 1 TCL Roku TV and 4 Roku Expresses on non smart TV's (all having the same issue)

@fieroloki - Yes, will apply to any Roku player/devices.

Here's the bug fix release if you want to test before it goes live:  https://my.roku.com/account/add/QGGTVR
Please post your findings.  Thx

@fieroloki - Yes, will apply to any Roku player/devices.
Sounds good. Thank you! I have them all testing now.

These all kicked back to the roku home screen right at the 2 hour mark.  When should the v3.7.2 build<PHONE_NUMBER>1 be available for update? So far the app isn't showing an update. If it is live, then I need to dig into our firewall logs and see what's being blocked.

We have currently updated our profile with the Photo View Beta app and are testing as well.  Will comment in a few hours with our findings.  Thanks for the work you put in on this

We have loaded the fix to our 7 Hisense Roku TVs and will update in two hours.

@fieroloki yes v3.7.2

beta channel? Just trying to make sure our firewall isn't blocking an update

@fieroloki yes, beta channel.
They ran without issue for 9 hours yesterday.

@fieroloki yes, beta channel.
They ran without issue for 9 hours yesterday.

OK, thank you! I'll wait on a general release patch

Hello!  Was curious when the fix will go live for everyone?

Roku approved and released it 5 minutes ago.
For business users of this free channel, please take time to donate to continue supporting the development: https://www.photoviewapp.com/donations/
Thanks

Roku approved and released it 5 minutes ago.
For business users of this free channel, please take time to donate to continue supporting the development: https://www.photoviewapp.com/donations/
Thanks

Thank you!  I will see if i can get them to update, and yes, i plan on donating

Roku approved and released it 5 minutes ago.
For business users of this free channel, please take time to donate to continue supporting the development: https://www.photoviewapp.com/donations/
Thanks

I think we are golden. Quick question though, Looks like there is a 12 hour limit. Is this on the app side or a Roku issue?

Thanks!  Works just like I'd hoped.  Louise Casey
-----Original Message-----
From: Chris Taylor<EMAIL_ADDRESS>To: chtaylo2/Roku-GooglePhotos<EMAIL_ADDRESS>Cc: louisecasey<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Sent: Wed, Mar 3, 2021 10:08 am
Subject: Re: [chtaylo2/Roku-GooglePhotos] Photo View App return Roku home screen after two hours (#375)
Roku approved and released it 5 minutes ago.For business users of this free channel, please take time to donate to continue supporting the development: https://www.photoviewapp.com/donations/Thanks—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


Allow to disable SignUp page when using SAML Authentication

Is your feature request related to a problem? Please describe.
When I set AUTHENTICATION_PROVIDER to "saml" and login successfully, OM will display the JOIN UI for signup.

Describe the solution you'd like
I expect that OM will offer the option to enable or disable this feature (JOIN UI) because there are occasions when companies prefer to restrict members from editing their information.

@nqvuong1998 you have a enableSelfSignup : ${AUTHENTICATION_ENABLE_SELF_SIGNUP:-true} in the authentication config section.


Error: NHS tries to sign with DS, Prod

PROD ЕЦП помилка
More there https://drive.google.com/drive/folders/1FmD4Fxc7LfA3NtZJoT-7U_wRefb2WU38

after communication with assigner it was found out that there is no issue and the task can be closed


Output warning for different environment env_var keys

Hello folks.
Would it be possible for me to add a feature to nmesos? It would be really handy to have nmesos check if all the environments have the same env_var keys set and output a warning if the environments have different env_var keys. We ran into some issue and this feature could prevent this.
I am thinking about something like:
object ConfigReader {
  type MissingEnvVarKey = String

  def findEnvVarDifferences(config: Config): Map[EnvironmentName, List[MissingEnvVarKey]]
}

This function would create List of all existing env_var keys from config.environments and then compare every environment's env_var keys (environment.container.env_vars) to that source of truth list.

if there was only one environment, then of course there should be no differences
if there are 2 environments (e.g. dev and prod) and the second one (prod) doesn't have a key, that is specified in the first one (dev) then it should print the missing key for prod. (or vice versa)
if there are 3 or more environments, then it should print out the missing keys for each environment.

This could be called here
  def parseEnvironment(file: File, environmentName: String, logger: Logger): ConfigResult = {
    parse(file, logger) match {
      case InvalidYaml(msg) =>
        ConfigError(msg, file)

      case ValidYaml(config, hash) =>
        val envVarDifferences = findEnvVarDifferences(config)
        if (envVarDifferences.nonEmpty) {
          logger.logBlock("Environment env_var keys not equal") {
            for ( diff <- envVarDifferences ) {
              logger.error(s"Environment ${diff._1}: Missing env_var keys: ${diff._2.mkString(",")}")
            }
          }
        }
        environmentFromConfig(config, environmentName, hash, file)
    }
  }

Best,
Tom
(cc @relistan)

Hi @patoms (cc @relistan),
Of course it's possible to add such feature via a pull request! It seems a great idea, I'm pretty sure @felixgborrego would be happy to help with the implementation bits and review the PR. Thanks for your proposal! 🤝

Awesome. I‘ll wrap it up and submit it Monday for a first review. :)
Thanks!


Mekanism + opencomputers, energy cables stop transferring energy after reboot

Please use the search functionality before reporting an issue. Also take a look at the closed issues!
Issue description:
mekanism universal energy cables stop transferring power to opencomputers energy converter after reboot.
Steps to reproduce:

Install opencomputers <IP_ADDRESS> & mekanism <IP_ADDRESS>0
Set up any type computer with open computers
power opencomputer power converter with universal cable
4.reboot
(cable will not transmit power until the cable touching the energy converter is broken then re-created)

Version (make sure you are on the latest version before reporting):
Forge:<IP_ADDRESS>25
Mekanism: <IP_ADDRESS>0
Other relevant version:
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
[gist/pastebin/etc link here]

#4676


Color variable name printed wrong in 0.3.5

This bug came with latest update. Didn't have it earlier.
So I have defined variable $color-white: #fffff; in sass. And then I'll print it out with KSS like:
// Colors
//
// Markup:
{$modifiers}

//
// $color-white - white
//
// Styleguide 1.0

Now styleguide prints out following string "3px solid darken(#ffffff, 5%)" on top of color. There are several other colors with random text instead of name and some colors work fine.
It might be that if color is used somewhere and it is used in special way then name of it is getting some extra crap.
Here is several examples of output I get instead of color name:

lighten(2px solid lighten(#b0b0b1, 10%),20%)
2px solid 2px solid #e0e0e1
2px solid 2px solid 2px solid 2px solid 1px solid 1px solid 2px solid 2px solid 2px solid 2px solid 2px solid #0090f1

Fixed in 0.3.6 release (reverted gonzales parser to a known working version)


draft: add error codes

PoC for proposed error code changes to the spec.
Holding off on opening PR to lnd until a few more decisions have been made.

Closing for now, settled on reusing existing err msg. May reopen later to reimpl.


Remove defer from useLoadingState hook

[X] Implement

[ ] Test

[ ] Merge

[ ] Release (as major)

Coverage remained the same at 100.0% when pulling ae68044bc311a1519faceb7f0a9fb2a228a67430 on remove-defer into b8852b077abeb90bbd4fd7bfe91683ac48769614 on master.


feat(logging): Adding logrus for the logging in pod-delete experiment

Adding logrus for the logging in pod-delete experiment

fixes: #3

Logs when TTL is disabled
time="2020-05-19T09:30:01Z" level=info msg="[PreReq]: Getting the ENV for the  experiment"
time="2020-05-19T09:30:01Z" level=info msg="[PreReq]: Updating the chaos result of pod-delete experiment (SOT)"
time="2020-05-19T09:30:01Z" level=info msg="The application informations are as follows:" Label="run=nginx" Namespace=test Ramp Time=10
time="2020-05-19T09:30:01Z" level=info msg="[Status]: Verify that the AUT (Application Under Test) is running (pre-chaos)"
time="2020-05-19T09:30:01Z" level=info msg="[Status]: Checking whether application pods are in running state"
time="2020-05-19T09:30:01Z" level=info msg="nginx-7989b5f7f4-jttw2 Pod is in Running State"
time="2020-05-19T09:30:01Z" level=info msg="[Status]: Checking whether application containers are in running state"
time="2020-05-19T09:30:01Z" level=info msg=" nginx container of pod nginx-7989b5f7f4-jttw2 is in Running State"
time="2020-05-19T09:30:06Z" level=info msg="[Ramp]: Waiting for the 10s ramp time before injecting chaos"
time="2020-05-19T09:30:16Z" level=info msg="[Info]: Killing the following pods" PodList="[nginx-7989b5f7f4-jttw2]"
time="2020-05-19T09:30:21Z" level=info msg="[Wait]: Wait for the chaos interval 10s"
time="2020-05-19T09:30:31Z" level=info msg="[Status]: Verification for the recreation of application pod"
time="2020-05-19T09:30:31Z" level=info msg="[Status]: Checking whether application pods are in running state"
time="2020-05-19T09:30:31Z" level=info msg="nginx-7989b5f7f4-mg2pl Pod is in Running State"
time="2020-05-19T09:30:31Z" level=info msg="[Status]: Checking whether application containers are in running state"
time="2020-05-19T09:30:31Z" level=info msg=" nginx container of pod nginx-7989b5f7f4-mg2pl is in Running State"
time="2020-05-19T09:30:32Z" level=info msg="[Info]: Killing the following pods" PodList="[nginx-7989b5f7f4-mg2pl]"
time="2020-05-19T09:30:37Z" level=info msg="[Wait]: Wait for the chaos interval 10s"
time="2020-05-19T09:30:47Z" level=info msg="[Status]: Verification for the recreation of application pod"
time="2020-05-19T09:30:47Z" level=info msg="[Status]: Checking whether application pods are in running state"
time="2020-05-19T09:30:47Z" level=info msg="nginx-7989b5f7f4-lsslc Pod is in Running State"
time="2020-05-19T09:30:47Z" level=info msg="[Status]: Checking whether application containers are in running state"
time="2020-05-19T09:30:47Z" level=info msg=" nginx container of pod nginx-7989b5f7f4-lsslc is in Running State"
time="2020-05-19T09:30:47Z" level=info msg="[Info]: Killing the following pods" PodList="[nginx-7989b5f7f4-lsslc]"
time="2020-05-19T09:30:52Z" level=info msg="[Completion]: pod-delete chaos is done"
time="2020-05-19T09:30:52Z" level=info msg="[Ramp]: Waiting for the 10s ramp time after injecting chaos"
time="2020-05-19T09:31:02Z" level=info msg="[Confirmation]: The application pod has been deleted successfully"
time="2020-05-19T09:31:02Z" level=info msg="[Status]: Verify that the AUT (Application Under Test) is running (post-chaos)"
time="2020-05-19T09:31:02Z" level=info msg="[Status]: Checking whether application pods are in running state"
time="2020-05-19T09:31:02Z" level=info msg="nginx-7989b5f7f4-nvps2 Pod is in Running State"
time="2020-05-19T09:31:02Z" level=info msg="[Status]: Checking whether application containers are in running state"
time="2020-05-19T09:31:02Z" level=info msg=" nginx container of pod nginx-7989b5f7f4-nvps2 is in Running State"
time="2020-05-19T09:31:07Z" level=info msg="[The End]: Updating the chaos result of pod-delete experiment (EOT)"


State not updating when sending event from UITextField

Describe the bug
I have a simple UITableView with a custom UITableViewCell which contains a UITextField when the text changes i'm sending an event from the view store  in order to get the new text and apply some transformations.The UI freezes and the UITextField loses focus.
To Reproduce
CounterListApp.zip
Expected behavior
State must receive the new text without freezing the UI.
Environment

Xcode 11.6
Swift 5.2
OS (if applicable): iOS 13

@sger I think the issue you are seeing is due to the fact that reloadData() is called on the table view for every key stroke. The data source should probably only be driven by insertions/removals/moves. You can get halfway there with the following change:
 self.viewStore?.publisher.counters
+  .removeDuplicates(by: { $0.count == $1.count })
   .sink(receiveValue: { [weak self] in self?.dataSource = $0 })

This won't account for moves, but if you add some kind of identifier to each row you could detect that, as well.

@sger Did the above solution work for you and is this issue OK to close out?

@stephencelis Thanks this was the solution.


Missing window focused event

Window Unfocused event was implemented, but respective focus event is still not fired

done in master branch

Thanks a lot!


Support oracular caching of variable-size slabs.

Closes #12.

I didn't add the LRU equivalent because I couldn't figure out how a caller might efficiently defragment the shared buffer. The oracular case only requires a defragmentation at every cycle (and only if slabs are reused); but for LRU, we would potentially have to do a defragmentation for every slab load. This is most obvious in the simple case of consecutive access, where the next access might require all existing slabs to shift along, effectively increasing runtime with cache size - not good!
You might say that we don't need a shared buffer at all, but if each slab gets its own allocation, we just pass the problem onto malloc. Then, each slab either grows its capacity to the largest slab, if the memory is re-used without freeing; or we pay the cost of constant (de)allocations when we force malloc to use the exact cache size (and even then, this would be a subset of the real block of memory that malloc requested from the OS). If memory usage is going to increase anyway, applications might prefer to just use LruSlabCache with an upper bound on the slab size.


Only show overflow widget if there is an overflow

Currently the overflow widget is displayed no matter whether there is an overflow or now
Would it be possible to only show if something overflows
also would it be possible to know how many children widgets were cut off ?

I need these features for this package to fit my use case as well

You can wrap the last child in VisibilityDetector link and check if it was visible. if it's not visible that means the children has overflowed.

You can wrap the last child in VisibilityDetector and check if it was visible. if it's not visible that means the children has overflowed.

and then show the overflow widget based on this condition.

@Mufaddal1125 Can you give me an example? I tried to implement it but it doesn't work

For people who wants to avoid displaying for few milliseconds the "See more" overflow widget before to hide it if needed, you can do it this way (without VisibilityDetector) :
  final GlobalKey<State<StatefulWidget>> _lastChildKey = GlobalKey();
  int? _lastChildVisibility;

@override
  void initState() {
    super.initState();
    // To detect if there is an overflow based on the visibility of the last item
    WidgetsBinding.instance.addPostFrameCallback((_) {
      if (_lastChildKey.currentContext != null) {
        final RenderBox renderBox =
            _lastChildKey.currentContext!.findRenderObject()! as RenderBox;
        final Size size = renderBox.size;
        if (size.width == 0 && size.height == 0) {
          setState(() {
            _lastChildVisibility = 0;
          });
        }
      }
    });
  }

ExtendedWrap(
        overflowWidget:
            _lastChildVisibility != null && _lastChildVisibility == 0 ? _overflowWidget(context) : null,
        maxLines: 2,
        children: _getChildren(),
      ),

List<Widget> _getChildren() {
   final List<Widget> children = <Widget>[];
   for (int i = 0; i < widget.children.length;) {
     final Widget chip = Padding(
       key: (i == widget.children - 1) ? _lastChildKey : null,
       padding: const EdgeInsets.only(right: 4, bottom: 4),
       child: _chip(widget.children[i].name),
     );
     children.add(chip);
     i++;
   }
   return children;
 }

Last solution works great


ggf.customer_lifetime_value produces NaN clv

Hi there, I successfully fitted a modified beta geo model as well as a gamma gamma model. However, when I use the customer_lifetime_value function from the gg model, all CLV are NaN.
DISCOUNT_a = 0.06                # annual discount rate
LIFE = 12                        # lifetime expected for the customers in months

discount_m = (1 + DISCOUNT_a)**(1/12) - 1     # monthly discount rate

clv = ggf.customer_lifetime_value(
        transaction_prediction_model = bgf,
        frequency = df_rftv["frequency"],
        recency = df_rftv["recency"],
        T = df_rftv["T"],
        monetary_value = df_rftv["monetary_value"],
        time = LIFE,
        freq = "D",
        discount_rate = discount_m)

Hey @littlesweet1129,
Passing in Panda series as arguments can lead to unstable behavior because many of the internal methods in the legacy fitters module expect numpy arrays. Try making the following changes to your code:
clv = ggf.customer_lifetime_value(
        transaction_prediction_model = bgf,
        frequency = df_rftv["frequency"].values,
        recency = df_rftv["recency"].values,
        T = df_rftv["T"].values,
        monetary_value = df_rftv["monetary_value"].values,
        time = LIFE,
        freq = "D",
        discount_rate = discount_m)

It's a good practice to always use the df['column'].values convention with all of the legacy Fitter models. The Model module I'm currently building as a replacement will not have this issue because the entire dataframe is provided as an argument instead of the separate array components.


[FEATURE]: Integration to diffusers framework

Describe the feature
Diffusers has become a common framework for stable diffusion. It seems that you already have a plan to integrate to diffusers. What's the progress? I'm looking forward to using it.

Thanks for your attention to our project. Integration to diffuser is in our schedule. We are planning to add ColossalAI to dreambooth, then we will consider working on the other projects. We are also looking for warmhearted developers to help us on finish that roadmap.

We have updated a lot. This issue was closed due to inactivity. Thanks.
https://github.com/hpcaitech/ColossalAI/tree/main/examples/images


Fix remaining instances of the JAORMX namespace

These are required in order to properly build this package.

/test e2e


Bump actions/checkout and actions/github-script to their latest major versions

This addresses deprecation warnings related to the version of Node used in the older actions.

📚 Documentation preview 📚: https://readthedocs-preview--40.org.readthedocs.build/en/40/

Gentle ping @humitos :)

@humitos


script didn't do anything?

am i missing something obvious?
i ran iwr -useb https://git.io/debloat|iex via powershell (admin rights), chose "remove all bloat", it did it thing and the powershell window appeared to do what it should do, however once i rebooted and then logged in a new user nothing was removed.
all start tiles were still there.
all the "bloat" was still in place.
i ran it on Win0 21H2 19044.1620

Did It remove the content from the Admin account?

Same here, the script executed but nothing was deleted.

I can confirm this, script does nothing.  In my case I had already unpinned the start menu tiles and this script not only didn't remove any bloatware, it even re-pinned all the tiles.

For me, it did remove the pinned tiles, but didn't remove any of the bloatware using the "remove all bloatware" button.


Fixed Major level code smells from backend (CS16)

Replaced variables that shadow protected words, type identifiers, merged nested "if" statements and redundant conditionals, made a regex less complicated.

close as they were included in #707


Initial Dotnet SDK proposal for Dapr workflows

Dotnet SDK proposal for Dapr workflows.
This is an initial proposal for Dapr workflows regarding the structure and the methods to be considered.
The SDK implementation will be a wrapper to DurableTask dotnet client.
The nuget dependencies we will be using are:

Microsoft.DurableTask.Client, Link
Dapr.AspNetCore , Link

There are a two flavors of programming model SDK implementation of dapr workflows.

For console apps.
Asp.net, object oriented programming.

We will be creating two different nuget packages:

Dapr.Workflows
Dapr.Workflows.AspNetCore

Path where the projects will be added. https://github.com/dapr/dotnet-sdk/tree/master/src
The methods that will be supported by dotnet-sdk for dapr workflows are as follows:
1. Workflow Service Collection Extensions
- AddWorkflow, add to the services
Eg:
builder.Services.AddWorkflow(options =>
{
..
}

2. Workflow Runtime Options
- RegisterWorkflow
Eg:
builder.Services.AddWorkflow(options =>
{
    options.Workflows.RegisterWorkflow<Input, Output>("workflowName", implementation: async (context, input) =>
    {
    ...
    });
}

- AddWorkflowsToRegistry
The method will be called inside AddWorkflow method in Extension class
Eg:
workerBuilder.AddTasks(registry =>
            {
                options.AddWorkflowsToRegistry(registry);
                ...
            });

- RegisterActivity
options.RegisterActivity<Input, Output>("activityName", implementation: async (context, input) =>
{
...
});

- AddActivityToRegistry
The method will be called inside AddWorkflow method in Extension class
workerBuilder.AddTasks(registry =>
{
    ...
    options.AddWorkflowsToRegistry(registry);
    ...
    options.AddActivityToRegistry<MethodActivity>();

    options.AddActivityToRegistry<string, string>("method", (context, input) => $"{input}");
});

3. Workflow Context
The workflow context object is used to call different methods in that workflow's context. Here are a few methods:
The workflow context methods are called in the workflow definition.
- PublishEvent
context.PublishEvent( pubSubName: "pubsubName", topic: "topicName", payload: $"data");

- InvokeMethodAsync
await context.InvokeMethodAsync(httpMethod: HttpMethod.Post, appId: "appId", methodName: "methodName", data: data);

- OutputBindingAsync
await context.InvokeBindingAsync(bindingName: "bindingName", operation: "exec", data: data);

- Expose workflowId
string workflowId = context.InstanceId;

- CreateTimer
TimeSpan delay = TimeSpan.FromSeconds(3);
context.CreateTimer(delay, CancellationToken.None);

- SetCustomStatus
context.SetCustomStatus("waiting");

- WaitForExternalEventAsync
TimeSpan approvalTimeout = TimeSpan.FromHours(12);
await context.WaitForExternalEventAsync("Approval", approvalTimeout);

Note Add AddWorkflow method to Dapr.Workflows.AspNetCore and others to Dapr.Workflow

An example showing the usage of few of the above methods:
using System.Text.Json.Serialization;
using Dapr.Workflow;
using Microsoft.AspNetCore.Mvc;
using JsonOptions = Microsoft.AspNetCore.Http.Json.JsonOptions;

// The workflow host is a background service that connects to the sidecar over gRPC
WebApplicationBuilder builder = WebApplication.CreateBuilder(args);

// Add workflows. NOTE: This could alternatively be placed in another project.
builder.Services.AddWorkflow(options =>
{
    // Inline order processing workflow logic
    options.RegisterWorkflow<OrderPayload, OrderResult>("ProcessOrder", implementation: async (context, input) =>
    {
        ArgumentNullException.ThrowIfNull(input, nameof(input));

        // Notify order received
        context.PublishEvent(
            pubSubName: "notifications-pubsub",
            topic: "notifications",
            payload: $"Received order {context.InstanceId} for {input.Name} at {input.TotalCost:c}");

        // Invoke the inventory service to reserve the specified items
        InventoryResult? result = await context.InvokeMethodAsync<InventoryResult>(
            httpMethod: HttpMethod.Post,
            appId: "inventory",
            methodName: "reserve-inventory",
            data: new { item = input.Name, quantity = input.Quantity });

        if (result?.success != true)
        {
            context.SetCustomStatus($"Insufficient inventory for {input.Name}");
            return new OrderResult(Processed: false);
        }

        // Orders >= $1,000 require an approval to be received within 24 hours
        if (input.TotalCost >= 1000.00)
        {
            // Notify waiting for approval
            TimeSpan approvalTimeout = TimeSpan.FromHours(24);
            string message = $"Waiting for approval. Deadline = {context.CurrentUtcDateTime.Add(approvalTimeout):s}";
            context.SetCustomStatus(message);
            context.PublishEvent(pubSubName: "notifications-pubsub", topic: "notifications", payload: message);

            try
            {
                // Wait up to 24-hours for an "Approval" event to be delivered to this workflow instance
                await context.WaitForExternalEventAsync("Approval", approvalTimeout);
            }
            catch (TaskCanceledException)
            {
                // Notify approval deadline expired
                context.PublishEvent(
                    pubSubName: "notifications-pubsub",
                    topic: "notifications",
                    payload: $"Approval deadline for order {context.InstanceId} expired!");

                return new OrderResult(Processed: false);
            }

            // Notify approval received
            message = $"Received approval at {context.CurrentUtcDateTime:s}";
            context.SetCustomStatus(message);
            context.PublishEvent(pubSubName: "notifications-pubsub", topic: "notifications", payload: message);
        }

        // Invoke the payment service
        await context.InvokeMethodAsync(
            httpMethod: HttpMethod.Post,
            appId: "payments",
            methodName: "process-payment",
            data: new { amount = input.TotalCost, currency = "USD" });

        // Notify order processed successfully
        context.PublishEvent(
            pubSubName: "notifications-pubsub",
            topic: "notifications",
            payload: $"Order {context.InstanceId} was processed successfully!");

        return new OrderResult(Processed: true);
    });
});

WebApplication app = builder.Build();

app.Run("http://<IP_ADDRESS>:8080");

Classes to provide class-based syntax:
We have a class called WorkflowBase to extend from to provide class based implementation. The OnRunAysnc method to implement the functionality.

WorkflowBase

Example for using WorkflowBase class is shown below.
public class SampleWorkflow : WorkflowBase<string, string>
{
    protected async override Task<string?> OnRunAsync(WorkflowContext context, string? input)
    {
        // Notify order received
        context.PublishEvent(
            pubSubName: "notifications-pubsub",
            topic: "notifications",
            payload: $"Received order {context.InstanceId} for {input.Name} at {input.TotalCost:c}");
    }
}

Can you add examples of code showing this being used?

Hi! Are you guys considering the Serverless Workflow specification? We have a dotnet SDK that might help. :)

@ricardozanini we are considering this, yes, though that would be part of a separate proposal. Most likely we'd be looking to implement a runtime for the SLWF spec, in which case it would be useful to use your SDK for spec validation. Happy to discuss this further on https://github.com/dapr/dapr/issues/4576 if you'd like.

Dotnet-sdk issue is linked here. https://github.com/dapr/dotnet-sdk/issues/979


Charge Spreading and PSF Notebooks

Adding two notebooks describing psf and charge spreading calculations for MOXSI

I did not mean to close this PR...I think this happened when I moved the repo from private to public. Either way, I'll sort out a way to get the notebooks in.


DOC: Improved documentation for diag_indices and diag_indices_from

Part of https://github.com/google/jax/issues/21461

Thanks @jakevdp! it's done.


allow security manager provide error message

Cherry pick the security manager error message PR.

Codecov Report

Merging #68 into airbnb-master will decrease coverage by 0.08%.
The diff coverage is 30.76%.

@@               Coverage Diff                @@
##           airbnb-master     #68      +/-   ##
================================================
- Coverage          65.68%   65.6%   -0.09%
================================================
  Files                346     346
  Lines              21801   21805       +4
  Branches            2415    2415
================================================
- Hits               14320   14305      -15
- Misses              7470    7489      +19
  Partials              11      11

Impacted Files
Coverage Δ

superset/views/core.py
73.67% <16.66%> (-1.16%)
:arrow_down:

superset/security.py
73.09% <42.85%> (-1.12%)
:arrow_down:

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 6f957bb...6595032. Read the comment docs.


Add solarhosting, Remove groom

Added/Removed:

This pull request adds solar hosting as an alternative to replit that does not need a creditcard
Removes groom.io as its no longer running

Proof: Its been down since Jan 13 according to the internet archive.  https://web.archive.org/web/20240113062409/https://goorm.io/

Fix the spelling and I'll merge

Fix the spelling and I'll merge

Alright should be fixed now!