Backport "HBASE-24418 Consolidate Normalizer implementations" to branch-2.3

Simplify our Normalizer story to have just a single, configurable
implementation.

fold the features of MergeNormalizer into
SimpleRegionNormalizer, removing the intermediate abstract class.
configuration keys for merge-only features now share a common
structure.
add configuration to selectively disable normalizer split/merge
operations.
RegionNormalizer now extends Configurable instead of creating a
new instance of HBaseConfiguration or snooping one off of other
fields.
avoid the extra RPCs by using MasterServices instead of
MasterRpcServices.
boost test coverage of all the various flags and feature
combinations.

Signed-off-by: Michael Stack<EMAIL_ADDRESS>Signed-off-by: Viraj Jasani<EMAIL_ADDRESS>Signed-off-by: huaxiangsun<EMAIL_ADDRESS>

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

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

_ Prechecks _

_ branch-2.3 Compile Tests _

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

-1 :x:
mvninstall
0m 12s
root in branch-2.3 failed.

-1 :x:
compile
0m  9s
root in branch-2.3 failed.

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

-0 :warning:
javadoc
0m  9s
root in branch-2.3 failed.

-0 :warning:
javadoc
0m 14s
hbase-common in branch-2.3 failed.

-0 :warning:
javadoc
0m 23s
hbase-server in branch-2.3 failed.

_ Patch Compile Tests _

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

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

-1 :x:
compile
0m  9s
root in the patch failed.

-0 :warning:
javac
0m  9s
root in the patch failed.

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

-0 :warning:
javadoc
0m  9s
root in the patch failed.

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

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

_ Other Tests _

-1 :x:
unit
0m  9s
root in the patch failed.

4m 46s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 4bf6bfb79e22 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
branch-2.3 / 39e03ab5ac

Default Java
2020-01-14

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

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

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

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

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

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

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

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

javac
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/1/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-root.txt

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

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

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

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

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

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

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

modules
C: . hbase-common hbase-server U: .

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

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

_ branch-2.3 Compile Tests _

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

+1 :green_heart:
mvninstall
3m 45s
branch-2.3 passed

+1 :green_heart:
checkstyle
2m 36s
branch-2.3 passed

+0 :ok:
refguide
12m 55s
branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
spotbugs
12m 50s
branch-2.3 passed

_ Patch Compile Tests _

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

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

+1 :green_heart:
checkstyle
2m 38s
root: The patch generated 0 new + 103 unchanged - 1 fixed = 103 total (was 104)

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

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

+0 :ok:
refguide
7m 33s
patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
hadoopcheck
21m 49s
Patch does not cause any errors with Hadoop 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
13m 34s
the patch passed

_ Other Tests _

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

92m 57s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense refguide xml spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux 7ab3339651d2 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
branch-2.3 / 39e03ab5ac

refguide
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/1/artifact/yetus-general-check/output/branch-site/book.html

refguide
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/1/artifact/yetus-general-check/output/patch-site/book.html

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

modules
C: hbase-common hbase-server . U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/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.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 14s
Docker mode activated.

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

_ Prechecks _

_ branch-2.3 Compile Tests _

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

+1 :green_heart:
mvninstall
4m 50s
branch-2.3 passed

+1 :green_heart:
compile
2m 58s
branch-2.3 passed

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

+1 :green_heart:
javadoc
3m 59s
branch-2.3 passed

_ Patch Compile Tests _

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

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

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

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

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

-0 :warning:
javadoc
0m 50s
hbase-server generated 25 new + 1 unchanged - 0 fixed = 26 total (was 1)

-0 :warning:
javadoc
2m 57s
root generated 25 new + 16 unchanged - 0 fixed = 41 total (was 16)

_ Other Tests _

+1 :green_heart:
unit
376m 52s
root in the patch passed.

419m  9s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 0b7fe01bbb1d 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / 39e03ab5ac

Default Java
1.8.0_232

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/1/artifact/yetus-jdk8-hadoop2-check/output/diff-javadoc-javadoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/1/artifact/yetus-jdk8-hadoop2-check/output/diff-javadoc-javadoc-root.txt

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

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

modules
C: hbase-common hbase-server . U: .

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

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

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

_ Prechecks _

_ branch-2.3 Compile Tests _

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

-1 :x:
mvninstall
0m 11s
root in branch-2.3 failed.

-1 :x:
compile
0m  9s
root in branch-2.3 failed.

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

-0 :warning:
javadoc
0m  9s
root in branch-2.3 failed.

-0 :warning:
javadoc
0m 15s
hbase-common in branch-2.3 failed.

-0 :warning:
javadoc
0m 23s
hbase-server in branch-2.3 failed.

_ Patch Compile Tests _

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

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

-1 :x:
compile
0m  9s
root in the patch failed.

-0 :warning:
javac
0m  9s
root in the patch failed.

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

-0 :warning:
javadoc
0m  9s
root in the patch failed.

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

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

_ Other Tests _

-1 :x:
unit
0m  9s
root in the patch failed.

4m 30s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

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

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / 6ffa36b061

Default Java
2020-01-14

mvninstall
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/2/artifact/yetus-jdk11-hadoop3-check/output/branch-mvninstall-root.txt

compile
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/2/artifact/yetus-jdk11-hadoop3-check/output/branch-compile-root.txt

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

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

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

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

mvninstall
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/2/artifact/yetus-jdk11-hadoop3-check/output/patch-mvninstall-root.txt

compile
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/2/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-root.txt

javac
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/2/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-root.txt

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

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

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

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

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

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

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

modules
C: . hbase-common hbase-server U: .

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

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

_ branch-2.3 Compile Tests _

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

+1 :green_heart:
mvninstall
3m 47s
branch-2.3 passed

+1 :green_heart:
checkstyle
2m 28s
branch-2.3 passed

+0 :ok:
refguide
11m 42s
branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
spotbugs
11m 40s
branch-2.3 passed

_ Patch Compile Tests _

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

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

+1 :green_heart:
checkstyle
2m 27s
root: The patch generated 0 new + 103 unchanged - 1 fixed = 103 total (was 104)

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

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

+0 :ok:
refguide
7m  7s
patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
hadoopcheck
18m 16s
Patch does not cause any errors with Hadoop 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
12m 12s
the patch passed

_ Other Tests _

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

82m 39s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense refguide xml spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux b02ec0d95c0d 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / 6ffa36b061

refguide
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/2/artifact/yetus-general-check/output/branch-site/book.html

refguide
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/2/artifact/yetus-general-check/output/patch-site/book.html

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

modules
C: hbase-common hbase-server . U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/2/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.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

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

_ Prechecks _

_ branch-2.3 Compile Tests _

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

+1 :green_heart:
mvninstall
3m 53s
branch-2.3 passed

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

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

+1 :green_heart:
javadoc
3m 11s
branch-2.3 passed

_ Patch Compile Tests _

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

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

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

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

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

-0 :warning:
javadoc
0m 35s
hbase-server generated 25 new + 1 unchanged - 0 fixed = 26 total (was 1)

-0 :warning:
javadoc
2m 24s
root generated 25 new + 16 unchanged - 0 fixed = 41 total (was 16)

_ Other Tests _

+1 :green_heart:
unit
386m 11s
root in the patch passed.

419m 27s

Subsystem
Report/Notes

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

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

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 7ccefd3fcb07 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / 6ffa36b061

Default Java
1.8.0_232

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/2/artifact/yetus-jdk8-hadoop2-check/output/diff-javadoc-javadoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/2/artifact/yetus-jdk8-hadoop2-check/output/diff-javadoc-javadoc-root.txt

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

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

modules
C: hbase-common hbase-server . U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1834/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.


make credentials faster

The google/auth package provides a general-purpose caching
with FetchAuthTokenCache. The implementation is aimed to be compatible
with PSR-6.
This commit introduces a faster cache, usable in conjuction with
FetchAuthTokenCache.
It queries for the last received token and reuses it if the token has
not expired.
In this way, we avoid sanitizing cache keys and performing table
lookups.
In my testing, this increases our throughput by ~10%.

@bshaffer A problem with the current implementation is that this level of cache does not honor the FetchAuthTokenCache's lifetime config. AFAICT, items in FATC expires lifetime seconds into the future, while this reuse-last-cache expires whenever the token reports itself to be expired.
I don't think this will be a big problem in practice, but it sounds a little scary. Maybe we use gate this behind a different config? WDYT?

Codecov Report

Merging #111 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
+ Coverage   90.03%   90.05%   +0.02%
==========================================
  Files          32       32
  Lines        2238     2243       +5
==========================================
+ Hits         2015     2020       +5
  Misses        223      223

Impacted Files
Coverage Δ

src/GrpcCredentialsHelper.php
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

These changes seem fine, and a 10% increase in throughput is pretty huge. But what is the justification for making these changes here, instead of improving the caching in the auth library? It is very surprising to me that we can get that much of a speedup just by avoiding hitting the existing caching solution, and suggests that that might be a better place for the changes. Does that make sense?

@michaelbausor Your thoughts absolutely makes sense. At a high level, I think it's the same reason that your CPU has multi-layered cache.
More specifically, the existing caching solution is more general, and the interface allows for an implementation where multiple PHP processes share the cache.
This is immensely useful. PHP processes are short-lived, serving only one request, so it'd be inefficient for every process to fetch credentials over network.
There is a cost to this generality though. To conform to standard and avoid surprises, the implementation needs to deal with long keys, multiple key-value pairs, etc. To continue with the CPU analogy, this implementation is like an L2 cache.
The new layer implemented in this PR doesn't conform to the standard and only caches within-process. Instead, it uses this flexibility to go faster: a bit like an L1 cache.
The existing implementation isn't "bad". However, if you need to look things up in a map, it's just going to be slower than accessing a variable.

@pongad that does make sense - that was a good explanation :-)
I think that this change is something we should do, but that we should also file an issue, either here or in the auth library, to do a larger review/rework. I had a quick look at the way this is done in gax, and I think that we can make it a lot better. For example, the enableCaching flag is used to wrap a FetchAuthTokenInterface in a cache, but provides no way to use any PSR caching implementation other than the default MemoryCacheItemPool. So someone who wanted to use caching across requests would need to set enableCaching to false, and pass in a FetchAuthTokenCache object they had instantiated themselves.
It also seems to me that we want the behaviour you have implemented to always apply, regardless of whether or not the enableCaching flag is set. Is there ever anyone who wants to make network requests to retrieve a token when they already have a valid token? I don't think so, and I don't think we need/want to provide any way for users to do that.
@bshaffer has a big PR open at the moment that definitely affects this class, and many others in Gax. @bshaffer -does it make sense to merge this now? Or to wait, and refactor after your PR goes in?

Closing this out, as it is implemented in #145


Update the name of the config file to the name that is actually being…

… used. Add a Running section.

Was this for this repo?

Yes.

Then do you want to explain the commits, since they don't currently make any sense to be merged (config.local.json is the default config jsbin looks for).

Please see line 11 and line 20 of Gruntfile.js. The default is indeed config.node.json.

Oh god, yeah, that gruntfile isn't used at all except for building the production (min) js files.
 Sorry you've been going down the wrong path.


Extracting out tokenization alignment functions

Changes:
Extracting out tokenization alignment functions from probing/ to src/utils/retokenize.py. This will allow non-probing tasks to also use these tokenization alignment methods.

Not sure if this is ready to review yet, but can you also refactor retokenize_edge_data.py to import these functions so they aren't defined twice?
Also, can you test with some sample data for each tokenizer (SPR2 should be small and run quick) and compare to the old version? Thanks!

Tested on spr2!


assign groups to projects while publishing workbooks

Is there a way to assign projects to groups based on the requirement

Support this request
Assume I have to create project (default), workbook default and data source default permissions as separate calls.
Created a dictionary item for the capabilities:
capability_wrkbk_viewer = {TSC.Permission.Capability.Read: TSC.Permission.Mode.Allow,
TSC.Permission.Capability.Filter: TSC.Permission.Mode.Allow,
TSC.Permission.Capability.ExportData: TSC.Permission.Mode.Allow,
TSC.Permission.Capability.ExportImage: TSC.Permission.Mode.Allow}
Attempt to create permissions
workbook_rule = [TSC.PermissionsRule(grantee=group_item,capabilities=capability_wrkbk_viewer)]
workbook_permission = server.projects.update_workbook_default_permissions(project_item,workbook_rule)
Returns error status 501 unsupported request default-permissions workbooks not supported

Is there a way we can directly add groups to projects, instead of specifying permission to workbook

Is there a way we can directly add groups to projects, instead of specifying permission to workbook

You can do this using server.projects.populate_permissions
Check out the project endpoint: https://github.com/tableau/server-client-python/blob/master/tableauserverclient/server/endpoint/projects_endpoint.py


Use DynamicJson in CLU samples

For simpler samples and to provide feedback about DynamicJson to @annelo-msft, rewrite at least the CLU samples using DynamicJson. As a stretch, might be good to rewrite all tests using DynamicJson (start with samples, though, in case there are any blockers) so we can stop generating models using the old LLC generator. That would decrease the overall amount of work to update and should make it easier for the code gen folks to iterate on changes to the generator stack (one less potential roadblock).

Thanks for filing this to track the work, Heath!
I'm adding the Azure.Core label as well, because your feedback on the behavior of DynamicJson will help raise our confidence that we can move it out of Azure.Core.Experimental.  Please share any issues you find with me and I'll do my best to fix them ASAP!


Android: scrollReponderScrollWithoutAnimationTo on ListView doesn't do anything.

The documentation has me believing that calling scrollResponderScrollWithoutAnimationTo on a ListView ref should allow me to set the scroll position on Android. It looks like it's not doing anything. Is this known behavior?

That method is deprecated (although it should still work?). Try using scrollResponderScrollTo() instead.

@jkhustler Are you using a RefreshControl? There is an issue that scrollTo is not working when using that (#5725).

@nicklockwood I've tried that also, still no dice :/
@janicduplessis Yes I am! This explains it. So I am not going crazy. Thanks so much for your response :)

Closing since this is a dupe

@jkhustler Great, I just submitted a PR so it should get fixed soon :)


Can't pass in launch debug argument raw to the shell

I'm trying to pass in an argument for my Jest config via create-react-app (react-scripts-ts). Since moduleNameMapper isn't a Jest option CRA lets me override by default, I have to pass it via CLI arguments.
This works fine in our npm scripts, like so:
react-scripts-ts test --env=jsdom --moduleNameMapper='{\"^~/(.*)\":\"<rootDir>/src/$1\"}'

But now I'm trying to configure the same command to debug our tests via VS Code. Here's my launch.json as it is right now:
{
  // Use IntelliSense to learn about possible attributes.
  // Hover to view descriptions of existing attributes.
  // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Debug CRA Tests",
      "type": "node",
      "request": "launch",
      "runtimeExecutable": "${workspaceRoot}/node_modules/.bin/react-scripts-ts",
      "args": [
        "test",
        "--runInBand",
        "--no-cache",
        "--env=jsdom",
        "--moduleNameMapper='{\"^~/(.*)\":\"<rootDir>/src/$1\"}'",
        "--showConfig"
      ],
      "cwd": "${workspaceRoot}",
      "protocol": "inspector",
      "console": "integratedTerminal",
      "internalConsoleOptions": "neverOpen"
    }
  ]
}

The issue is that VS Code is forcing the moduleNameMapper arg to be quoted if it contains any quotes. This makes it so Jest can't parse the argument and then it fails to use my additional mapping override (you can tell with the --showConfig arg).
node_modules/.bin/react-scripts-ts --inspect-brk=16624 test --runInBand --no-cache --env=jsdom "--moduleNameMapper='{\"^~/(.*)\":\"<rootDir>/src/$1\"}'" --showConfig

Since the moduleNameMapper expects a JSON object as the argument value, is there any way to get VS Code to just let me pass it in raw?

VSCode Version: 1.22.2
OS Version: Mac OS 10.12.6

Steps to Reproduce:

Create a launch.json file
In the args array, add a argument that would contain a quote like "--moduleNameMapper='{\"^~/(.*)\":\"<rootDir>/src/$1\"}'"
Inspect debug output to see VS Code adds quotes around the argument string

Does this issue occur when all extensions are disabled?: Yes

You're right that it's interpreted as JSON before it's passed to the process, so you'll have to escape any special characters same as you would in a JS string.
If you want --moduleNameMapper='{\"^~/(.*)\":\"<rootDir>/src/$1\"}' passed to the process, you'll have to escape the \ like
--moduleNameMapper='{\\\"^~/(.*)\\\":\\\"<rootDir>/src/$1\\\"}'
You can look at what it writes to the Debug Console to check it.

I'm positive I tried that but I'll double check, I don't think it's the
escape characters, I think it's the surrounding quotes.
On Wed, May 2, 2018, 19:55 Rob Lourens<EMAIL_ADDRESS>wrote:

Closed #48967 https://github.com/Microsoft/vscode/issues/48967.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/48967#event-1606737391, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAiaa8eArCmWTGNVgx1p31E-EDCfH0nTks5tulWCgaJpZM4TtEYp
.

Yeah, I've tried this--the issue isn't really escaping, it's the fact VS Code adds surrounding quotes which the program doesn't expect and fails to parse as an argument option.
My proposal is to allow VS Code to optionally allow unquoted (i.e. raw) arguments to the shell for this kind of use case.
Something like:
args: [
        "test",
        "--runInBand",
        "--no-cache",
        "--env=jsdom",
        { raw: true, arg: "--moduleNameMapper='{\"^~/(.*)\":\"<rootDir>/src/$1\"}'" },
        "--showConfig"
]

Does it work for you if you don't use "console": "integratedTerminal"?

@roblourens It still showcases the same issue, the output shows that it is not using surrounding quotes but symptoms are the same (i.e. the parameter isn't picked up) which makes me think it's still being quoted behind the scenes. Pasting the same command into a separate terminal works fine, like before.

How can I try this or verify that it's working? I tried the exact command that you pasted originally:

I tried it and I think the problem is actually the single quotes around the object. They are necessary in a shell but not when passed to the process directly. Try
"--moduleNameMapper={\"^~/(.*)\":\"<rootDir>/src/$1\"}",
that seems to work for me.

Yes it works! All I had to do was also escape $1 with \\$1. For reference, this is the config that works:
{
      "name": "Debug Tests",
      "type": "node",
      "request": "launch",
      "runtimeExecutable": "${workspaceRoot}/node_modules/.bin/react-scripts-ts",
      "args": [
        "test",
        "--runInBand",
        "--no-cache",
        "--env=jsdom",
        "--moduleNameMapper={\"^~/(.*)\":\"<rootDir>/src/\\$1\"}"
      ],
      "cwd": "${workspaceRoot}",
      "protocol": "inspector",
      "console": "integratedTerminal"
    }


Radix sort

Implementing Radix Sorting using the approach described in https://eprint.iacr.org/2019/695.pdf
Before calling mpc_sort function, caller is expected to transform the bits into modulus conversion and send secret shared inputs to the function.
The entire flow of the implementation is represented in this diagram.

This diff implements finding order of each input in the sorted list. Following will be handled in later diffs

Asynchronous calls and also helper communication
Optimization e.g. remove quadratic and linear randoms from king
Oblivious Shuffle
Stubs to calculate number of bytes exchanged between helpers

Closing this request, since have created a separate one on top of Ben's implementation of ReplicatedSecretSharing


Mangio-RVC installed on Windows11 PC with NVIDEA GPU 4060 and Go-web.bat also displays the RVC webpage but nothing is working

Mangio-RVC installed on Windows11 PC with NVIDEA GPU 4060 and Go-web.bat also displays the RVC webpage but nothing is working. Issue 1: When I use Vocals removal tab and drop a wav file, slect a model and click convert button, nothing happens.
Issue 2: When I place a download model .pth file under wwights folder and place .index file under /logs folder and click refresh button in 'model inference' tab, nothing happens.
Why nothing is working while 7B setup successfully installed in windows PC with high-end GPU and voice processing webpage also displays as localhost:7897 with all 5 main tabs..but nothing happens when using and placing downloaded model files like .pth or even trying to seperate vocals from a wav file.
Someone please help..appreciate very much.
Thanks

I meant 7GB and not 7B.


ModuleNotFoundError: No module named 'meta_analysis'

Hi!
I installed everything as specified in the instructions.
When starting the file bron_arango.py returns the error: Module Not Found Error: No module named 'meta_analysis'.
What to do?

It could be that your PYTHONPATH is not set correctly. I usually run it like below from the root of the repository:
PYTHONPATH graph_db/bron_arango.py

@hembergerik thank you very much!


More env overrides

The CI apparently had a network glitch.

Interesting idea. I'll have to think about it.

This is implemented in an opt-in way by https://github.com/rust-lang/cc-rs/pull/818. So I don't think we want this anymore.

I looked at the change, you do not want to have this feature opt-in since it would sort of defeat the purpose of having a standard way to pass optional flags to any crate that's using cc-rs.


Allow sorting for last-added/Newest in template / appearance selection window.

This is a nice to have feature, but would help. In most cases, the template you want to use, is a template which was last added. So it would be easier to find than search alphabetically.

Forgot about this issue and created a new one (#49), sorry, I am going to close this because it says the same but adds more detail.


⚠ Bofrost.be has degraded performance

In bfad767, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1339 ms

Resolved: Bofrost.be performance has improved in f36b678.


usage of correct repository context

How to categorize this PR?

/area usability
/kind bug
/priority 3
What this PR does / why we need it:
Currently the correct repository context (set via installation, Landscaper Context or Component Overwrite) is not used while resolving component descriptors and resources.
This PR enables the usage of the correct repository context in various places and adds test to find regressions.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

/test

The coding looks good to me.
I think the possibility to specify a repository context at so many different location is very confusing for the end user (and for me :-) ):
* [Default repo context in landscaper](https://github.com/gardener/landscaper/blob/4f17eb467cc6cb400fe1effa22c4fb1db64d677d/apis/config/v1alpha1/types_landscaper_config.go#L24)

* [Installation alternative 1](https://github.com/gardener/landscaper/blob/4f17eb467cc6cb400fe1effa22c4fb1db64d677d/apis/core/types_installation.go#L49)

* [Installation alternative 2](https://github.com/gardener/landscaper/blob/4f17eb467cc6cb400fe1effa22c4fb1db64d677d/apis/core/types_installation.go#L45)

I'm not sure if there are other possibilities. Currently this information is somehow spread in the documentation (e.g. link1, link2). Might it be possible and a good idea to combine and relate the different possibilities?

I agree with you. There is actually another possibility with component overwrites, which is currently being worked on:
https://github.com/gardener/landscaper/pull/430
We should definately document this better. However, I would like to wait until the changes with the component overwrite are done and the document the complete picture. What do you think?

That's a good idea.


[docs][base-ui] Structure and style revisions for Component docs

Part of #38320 - continuation of #38529
This PR updates the structure and formatting of the Base UI docs to eliminate some unneeded copy and code snippets. I've also done another pass on style and grammar, notably capitalizing component names across the board.
This is the last round of editing that I intend to do on the existing content of the Base docs before the stable release.

removed boilerplate copy and code snippets from ## Component sections along with ### Usage and ### Basics headers
capitalized component names, occasionally added <ComponentCase /> for clarity
heavy revisions to (Dropdown) Menu page in light of the addition of the <Dropdown /> component
more detailed revisions to Select page as it's changed significantly since the first draft
detailed editing and reorganizing of Number Input page

Pages with significant changes:

Menu - https://deploy-preview-38826--material-ui.netlify.app/base-ui/react-menu/
Select - https://deploy-preview-38826--material-ui.netlify.app/base-ui/react-select/
Number Input - https://deploy-preview-38826--material-ui.netlify.app/base-ui/react-number-input/

One more tiny change: I dropped the CSS classes section from Menu doc because they are all documented in the API tab now. I believe it's the only page that has this section.


[High Priority] fix: fetch measurement version info from Resolve Service with Information.vi

[x] This contribution adheres to CONTRIBUTING.md. (Required)
[ ]  Automatically post PR comments with images for G code changes? (Recommended for small changes)

What does this Pull Request accomplish?

Created a wrapper for Resolve Service with Information.vi, client VI of Discovery service and added the client wrapper VI to the NI Discovery V1 Client.lvlib.
Created an automated testcase for the client wrapper and added the testcase to Discovery Tests.lvlib.
Updated the Get Measurement Metadata.vi in Client generator to fetch the measurement version information from Resolve Service with Information.vi instead of Get Metadata.vi of the measurement client.

Why should this Pull Request be merged?

Currently, Client generator fetches the measurement version info from the Get Metadata.vi method of the measurement client. The version feature handling is enabled only in the 2024 Q4 version of Discovery Service and the latest Measurement Plug-In SDK Service packages of LabVIEW and Python.
For Python measurements (which use Measurement Plug-In SDK v2.0.0 package) which do not have the version parameter, Discovery Service internally sets the version parameter as 0.0.1 during measurement registration.
So, fetching the version information using Get Metadata.vi from the measurement causes version mismatch between the version specified in the measurement .service config file and the version Discovery Service has assigned to the measurement during registration. This mismatch causes the generated measurement client to fail in resolving the service.
This update is essential to provide backward compatibility for the client with the previous Measurement Plug-In SDK Service package.

What testing has been done?

Tested by creating and running client for Python measurements which use Measurement Plug-In SDK v2.0.0 package.

Get Measurement Metadata.vi
Diff image:
Before:

After:


Add jbcs_offline_install into the README

The variable is missing in the README's table.

Great! Thanks :)
@guidograzioli I think the job failing is unrelated, are you fine with merge this change despite that?

Sure, jbcs PRs fail because of credentials not available in forks

Thanks for the PR!


Fix few issues with EventListener and EventListenerOrEventObject

https://github.com/Microsoft/TSJS-lib-generator/pull/352 introduced a few issues, this PR addresses them.
Fix Microsoft/TypeScript#21948
Fix Microsoft/TypeScript#21922
Supersedes https://github.com/Microsoft/TSJS-lib-generator/pull/374

There are too many dependencies on EventListener being an object type and  not a union in our Real World code bases that i do not think it make sense to remove it. it will be a massive breaking change for very little value.
I have added special handling to EventListener as a call back interface to avoid the break.


Fix gateway names

These should match what we have in master and be consistent with other
components.

You missed a 'make build' or update for the demo. Good we have a test.

Which was not required apparently. We'll need a separate PR to fix demo


Add column policy_usage to aws_iam_policy table

Integration test logs

Logs
> make
go build -o<EMAIL_ADDRESS>*.go
> steampipe -v
steampipe version 0.10.0
> steampipe plugin list
+--------------------------------------------------+---------+-------------+
| Name                                             | Version | Connections |
+--------------------------------------------------+---------+-------------+
| hub.steampipe.io/plugins/turbot/aws@latest       | local   | aws         |
| hub.steampipe.io/plugins/turbot/steampipe@latest | 0.1.3   | steampipe   |
+--------------------------------------------------+---------+-------------+

Example query results

Results
> select name, policy_usage from aws_iam_policy where is_aws_managed is false
+---------------------------------------------------------------------------------------------------------------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
| name                                                                                | policy_usage                                                                                |
+---------------------------------------------------------------------------------------------------------------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
| AWSControlTowerCloudTrailRolePolicy                                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"AWSControlTowerCloudTrailRole"}],"PolicyUsers":null}                              |
| AWSLambdaBasicExecutionRole-912ec387-a0d6-4caa-bfc3-163d3567be4b                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"testtest-role-83s8u3w0"}],"PolicyUsers":null}                                     |
| CloudTrailPolicyForCloudWatchLogs_82e4c54d-ee0a-40ff-b7db-1b06d895b80c                                                          | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"cloudtrailrole"}],"PolicyUsers":null}                                             |
| AWSLambdaBasicExecutionRole-8eb2ae33-24a0-47d4-bed4-61beb0e1529f                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"eventbridgetest-role-0hmq5sxq"}],"PolicyUsers":null}                              |
| AWSControlTowerAdminPolicy                                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"AWSControlTowerAdmin"}],"PolicyUsers":null}                                       |
| AWSLambdaBasicExecutionRole-6ba5a28f-1dbe-49bf-b9bf-c2ca442eb788                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"test-role-rvxs2cxf"}],"PolicyUsers":null}                                         |
| AWSLambdaBasicExecutionRole-75c44454-4fb6-456f-bbcc-ddf7bad05f3a                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"jira-test-role-flrbia7j"}],"PolicyUsers":null}                                    |
| AWSLambdaBasicExecutionRole-d670d758-62c0-4b90-b2a0-1207f053437a                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"ec2-sg-eventbridge-test-role-kpw56vsr"}],"PolicyUsers":null}                      |
| AWSControlTowerStackSetRolePolicy                                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"AWSControlTowerStackSetRole"}],"PolicyUsers":null}                                |
| aws-security-hub-automated-response-and-remediation-PlaybookServiceCatalog-14R9J8OT-SHARRCatalogUserPolicy58114AE9-3HQV00N9JLQ9 | {"IsTruncated":false,"Marker":null,"PolicyGroups":[{"GroupId":"SECRET","GroupName":"SO0111-SHARR_Catalog_User_ap-northeast-2"}],"PolicyRoles":null,"PolicyUsers":null}                 |
| AWSLambdaBasicExecutionRole-5d6522bb-d636-4050-b33e-7bd2f2bbe55c                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":null,"PolicyUsers":null}                                                                                |
| aws-security-hub-automated-response-and-remediation-PlaybookServiceCatalog-14R9J8O-SHARRCatalogAdminPolicyE502F9A1-3WXBFGS8FNBC | {"IsTruncated":false,"Marker":null,"PolicyGroups":[{"GroupId":"SECRET","GroupName":"SO0111-SHARR_Catalog_Admin_ap-northeast-2"}],"PolicyRoles":null,"PolicyUsers":null}                |
| AWSLambdaBasicExecutionRole-11e3627b-35ab-4db4-8bdd-936d0bf890c6                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"testtest-role-l7mgva91"}],"PolicyUsers":null}                                     |
| AWSLambdaBasicExecutionRole-7716607f-c0e1-4d00-9f0f-faaa814be624                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"test1-role-r4car3dq"}],"PolicyUsers":null}                                        |
| AWSLambdaBasicExecutionRole-1160f530-5214-4f93-a2da-a1bae6d04e75                                                                | {"IsTruncated":false,"Marker":null,"PolicyGroups":null,"PolicyRoles":[{"RoleId":"SECRET","RoleName":"cloudtrail-test-role-w25a9j0l"}],"PolicyUsers":null}                              |
+---------------------------------------------------------------------------------------------------------------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+

@cjsrkd3321 Thanks for using Steampipe!! very glad to see this PR. We will review and take appropriate steps.

I love the idea of making this data available in Steampipe, but am wondering if it would be better as a separate table?
The aws_iam_policy table is already a heavy / slow table to query, and this per-row Hydrate function with it's own pagination will definitely create a lot of API calls. select * from aws_iam_policy will now be dependent on these sub-listing / paginated calls for performance.
What if we moved this to be a separate aws_iam_policy_usage table?

I love the idea of making this data available in Steampipe, but am wondering if it would be better as a separate table?
The aws_iam_policy table is already a heavy / slow table to query, and this per-row Hydrate function with it's own pagination will definitely create a lot of API calls. select * from aws_iam_policy will now be dependent on these sub-listing / paginated calls for performance.
What if we moved this to be a separate aws_iam_policy_usage table?

I agree your comment. But, It's hard to work right away. basically, It would be nice to make it possible to search based on policy_arn. If you have other optinions, please comment.


HDC 4.4.25 stops mapping partway through drive

Expect:

HDC performs consistently during a drive

Actual:

HDC stops mapping, after idling for 10+ minutes

Steps to Reproduce:

Plug HDC into portable battery (RoyPow)
Drive for 45 minutes and map with HDC (3:00 PDT)
Allow images to download & upload
Park for 30 minutes with HDC still on (3:45-4:15 PDT)
Drive and wait for images

Was able to reproduce error on Jumpy-Red-Player.

Plug HDC into portable battery (RoyPow)
Drive for 45 minutes and map with HDC (3:00 PDT)
Allow images to download & upload
Park for 30 minutes with HDC still on (3:45-4:15 PDT)
Drive and wait for images

Logs Sent

Mishmear / glistening-magenta-sculptor 3/27/2024

11:30 uk time mapping stopped after I stopped at a traffic light for a little bit 😭
Logs and db sent

Dragon
Logs & DB sent for around 9:00 MDT (15:00 UTC)

Was parked between 8:50 and 9:05 ish. Something happened in between there. Started moving again no mapping

Can't reproduce


Cannot delete last remaining job/school from user profile

Steps to Reproduce

As a logged-in user, visit the user settings page (/settings/)
On the "Education" tab, type any value in the "Institution" field
Save your changes
Reload the page

Expected behavior
A "remove" link should be visible in the top right of the section for that institution
Actual behavior
No "remove" link is available. Setting the "name" field to an empty string and attempting to save raises an error.
As a result, once a user enters a job or educational institution, it is not possible for them to remove all entries from their profile.

@Johnetordoff --- You want to take a look at this?

It turns out, there is a conflict in a couple of PRs that address issues on that page, and this issue should be resolved by @billyhunt in #2531.
Closing :)


Gift Link preview

The link preview is not correct
https://gift.bitcointribe.app/FhH37FtNtadx8gL97

Ref: https://github.com/firebase/firebase-ios-sdk/issues/5333

Not as per plan


_setDefaultFormat incorrectly wrap

...

Try to paste an copied HTML like

111

222

the _setDefaultFormat will wrap the
s into

111

222

The 2.33.0 version has been updated.
If this issue has not been resolved, please reopen this issue.
Thank you.


[FEATURE] Add plugin variables for Fabric API Key / Secret and logic to pass it down to the SDK

Feature request

Add plugin variables for Fabric API Key / Secret and logic to pass it down to the SDK.

We did migration from Fabric to Firebase and linked firebase project to the Fabric integration but the crash data is not arriving to Firebase Crashlytics Web Interface (only crash-free metric gets updated)
This is from https://docs.fabric.io/android/examples/firebase/faq.html
A: If you’ve completed the click-through Fabric migration flow to migrate to Firebase, you can keep using the Fabric Crashlytics SDK. In that case, you should leave your old Fabric API key in your app.
After you link your app, you can choose to upgrade to the Firebase Crashlytics SDK in beta for iOS or Android, which requires making updates to your code.
This plugin both uses Fabric & Crashlytics SDK atm but does not provide ability to set / implement  API Keys usage.

It is not necessary to include the Fabric API key with this plugin to receive Crashlytics data in the Firebase console - once you have linked your project with Firebase, the Fabric key is no longer needed to upload crash data.
I have myself moved production apps from Fabric to Firebase which use this plugin and they receive Crashlytics data in Firebase console without need for the Fabric API keys.
However, you need to unlink your Fabric app from Firebase before Crashlytics data will flow through to the Crashlytics console in Firebase as outlined here

I already did that and thats np for me. But I don't this can be called complete migration - unlinking does delete older fabric data and firebase assumes it's new app.

@dpa99c when we try to install version 9.0.0 or 9.0.1-cli we get an error at run time of the mobile app.  The error reads [Fabric] failed to download settings Error Domain=FABNetworkError Code=-5 "(null)" UserInfo={status_code=403, type=2, request id=, content_type=application/json; charset=utf-8}
We don't have a Fabric account linked to our Firebase project as we're using a new Firebase project.  Do we need to link a Fabric account to our fresh Firebase project to resolve this issue?
Any help would be appreciated.

How did you manage to solve this @phillipmoon ?


How to train the model on kitti dataset？

Thank the author for the great work!
I want to run through this model on the kitti format data set. After modifying the data part in the configuration file and the relevant part in the train_pipeline, I find it is always difficult to run through, and the following bugs appear. Is there a configuration file description for kitti dataset in this code?
Traceback (most recent call last):   File "tools/train.py", line 249, in <module>     main()   File "tools/train.py", line 238, in main     train_model(   File "/public/chenrunze/wl/mm2/mmdetection3d-0.17.1/mmdet3d/apis/train.py", line 28, in train_model     train_detector(   File "/public/chenrunze/miniconda3/envs/di_help_zq/lib/python3.8/site-packages/mmdet/apis/train.py", line 170, in train_detector     runner.run(data_loaders, cfg.workflow)   File "/public/chenrunze/miniconda3/envs/di_help_zq/lib/python3.8/site-packages/mmcv/runner/epoch_based_runner.py", line 127, in run     epoch_runner(data_loaders[i], **kwargs)   File "/public/chenrunze/miniconda3/envs/di_help_zq/lib/python3.8/site-packages/mmcv/runner/epoch_based_runner.py", line 50, in train     self.run_iter(data_batch, train_mode=True, **kwargs)   File "/public/chenrunze/miniconda3/envs/di_help_zq/lib/python3.8/site-packages/mmcv/runner/epoch_based_runner.py", line 29, in run_iter     outputs = self.model.train_step(data_batch, self.optimizer,   File "/public/chenrunze/miniconda3/envs/di_help_zq/lib/python3.8/site-packages/mmcv/parallel/distributed.py", line 52, in train_step     output = self.module.train_step(*inputs[0], **kwargs[0])   File "/public/chenrunze/miniconda3/envs/di_help_zq/lib/python3.8/site-packages/mmdet/models/detectors/base.py", line 237, in train_step     losses = self(**data)   File "/public/chenrunze/miniconda3/envs/di_help_zq/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl     return forward_call(*input, **kwargs)   File "/public/chenrunze/miniconda3/envs/di_help_zq/lib/python3.8/site-packages/mmcv/runner/fp16_utils.py", line 98, in new_func     return old_func(*args, **kwargs)   File "/public/chenrunze/wl/mm2/mmdetection3d-0.17.1/mmdet3d/models/detectors/base.py", line 59, in forward     return self.forward_train(**kwargs)   File "/public/chenrunze/zq/DeepInteraction-main/projects/mmdet3d_plugin/models/detectors/deepinteraction.py", line 200, in forward_train     img_feats, pts_feats = self.extract_feat(   File "/public/chenrunze/zq/DeepInteraction-main/projects/mmdet3d_plugin/models/detectors/deepinteraction.py", line 140, in extract_feat     new_img_feat, new_pts_feat = self.imgpts_neck(img_feats[0], pts_feats[0], img_metas, pts_metas) TypeError: 'NoneType' object is not subscriptable

There are some modules in the code that depend on the nuscenes dataset. You need to read the code carefully and modify them.

@406143996 hi, have u trained deepinteraction on kitti? i modified it to suit kitti dataset but the training result is sooo bad, i wonder if i made some mistakes.

@406143996 hi, have u trained deepinteraction on kitti? i modified it to suit kitti dataset but the training result is sooo bad, i wonder if i made some mistakes.

can you please share your config?
I'd be very grateful. Thank you

@406143996 hi, have u trained deepinteraction on kitti? i modified it to suit kitti dataset but the training result is sooo bad, i wonder if i made some mistakes.

can you please share your config? I'd be very grateful. Thank you

same bad result, cant train deepinteraction on kitti


"Grindu" is listed in the drop-down menu

Description:
When user writes "Grindu" the location appears in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Grindu"
Expected results
The location appears in the drop-down menu as "Comuna Grindu, ** IALOMIȚA**"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Grindu"

Expected results
The location appears in the drop-down menu as "Sat Grindu, IALOMIȚA (Grindu)".


Radar map labels in multiple lines

I want my radar chart labels to be in multiple lines because of length issues. However when I put a "\n" in my labels it doesn't work and still shows them as a single line. These are my labels:
data: {         labels: ["PMO / WPM SKILL ", "COMMUNICATION SKILL", "PRODUCT AND PROCESS KNOWLEDGE ", "LEADERSHIP SKILL ",             "SITE DEPLOYMENT SPECIFIC SKILL ", "GENERAL ENGINEERING SKILL ", "SITE QUALITY SKILL "] . . .
For example if I put "\n" in "Communication skill" like this "COMMUNICATION \n SKILL" it doesn't make it into to lines, it still remains a single line!
So I was wondering if there is a way to have labels in multiple lines?

See the answer here: https://github.com/chartjs/Chart.js/issues/5985#issuecomment-454961888


chore(common): Remove node 16 build

What?
Remove testing/support for node 16 (post-EOL)
Why?
Node 16 is EOL.
@bigcommerce/api-client-developers

Build will be fixed by https://github.com/bigcommerce/sample-app-nodejs/pull/215


[UI] Add ability to render query plans

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
I can currently download query plans in dot format from the scheduler REST API and then use graphviz to transform them into images. I would like to be able to view them directly in the UI as well.
Describe the solution you'd like
Perhaps https://github.com/magjac/d3-graphviz will help
Describe alternatives you've considered
None
Additional context
None

I'd love to have a go at this. It seems rather straightforward (though I am not too familiar with Typescript). Would the image open in a new page, in a modal box, or it would be like hide/show below the query status?


DRAFT: to do following recent AMY release

I'm keeping a list here of things to come back to. No action needed until I update this issue.
To do

[ ] Any <XX> Homework training progress should be changed to Lesson Contribution. Original date and URL should remain the same.
[ ] Any <XX> Demo training progress should be changed to Demo. Original date should remain the same.
[ ]

@pbanaszkiewicz I think this is complete - can you verify?

@maneesha
Re 1: I'm seeing 3871 training progress entries with <XX> Homework instead of Lesson Contribution.
Re 2: I'm seeing 3838 training progress entries with <XX> Demo instead of Demo.
Should we leave them be in the database, or migrate them? Maybe they are benign, I think I developed the logic in a way that accounts for Carpentry-specific homework or Demo.

We should run a migration on this data.

Fixed by SQL update.


Production Build Styling/CSS is "Bigger" Than Dev Build

First off, GREAT work.  This boilerplate has been immensely helpful.
Bug
Production Build Styling is Bigger Than Dev Build
Description
Developing the app is fine, the UI looks correct with no clipping or overflowing.
When I go to build for production, the UI is some % bigger across the board.
How can I reproduce this problem?
Build for local, then build for Darwin.
If visual, provide a screenshot.
Dev Version:

Production Version:

Tell me about your development environment.

Node version: 8.11.1
NPM version: 6.1.0
vue-cli version: 3.0.0-rc.3
Operating System: MacOS High Sierra 10.13.5

@joemccann  hi there, I was wondering if you managed to solve this problem as I am experiencing the same issue.Any pointers would be greatly appreciated. Thanks

I did not
On Mon, Dec 31, 2018 at 6:49 AM Rob Clarke<EMAIL_ADDRESS>wrote:

@joemccann https://github.com/joemccann hi there, I was wondering if
you managed to solve this problem as I am experiencing the same issue.Any
pointers would be greatly appreciated. Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SimulatedGREG/electron-vue/issues/660#issuecomment-450653944,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC8arlSNNjD93Q8W9tad5dRsXu9vzoCks5u-iPhgaJpZM4VRpnK
.

@joemccann After some searching around I cam across something called devicePixelratio described on msdn as;

The Window property devicePixelRatio returns the ratio of the resolution in physical pixels to the resolution in CSS pixels for the current display device.

I did a check of this on both build and dev instances of electron-vue and low and behold they are different dev being 0.9128 and the build environment being 1. Not sure if this is the reason but this discrepancy might account for the different size renderings.

Yes exactly
On Mon, Dec 31, 2018 at 19:41 Rob Clarke<EMAIL_ADDRESS>wrote:

@joemccann https://github.com/joemccann After some searching around I
cam across something called devicePixelratio described on msdn as;
The Window property devicePixelRatio returns the ratio of the resolution
in physical pixels to the resolution in CSS pixels for the current display
device.
I did a check of this on both build and dev instances of electron-vue and
low and behold they are different dev being 0.9128 and the build
environment being 1. Not sure if this is the reason but this discrepancy
might account for the different size renderings.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SimulatedGREG/electron-vue/issues/660#issuecomment-450707926,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC8ajsYtBDV4zHTbDKn6NDxgaTJi7emks5u-tjkgaJpZM4VRpnK
.

--
Sent from Joe in Real Life

@joemccann Maybe its enough to just multiply all your css size values by 0.9 during dev in order to get the desired result when built. I give it a try and get back to you

👌🏼
On Mon, Dec 31, 2018 at 19:57 Rob Clarke<EMAIL_ADDRESS>wrote:

@joemccann https://github.com/joemccann Maybe its enough to just
multiply all your css size values by 0.9 during dev in order to get the
desired result when built. I give it a try and get back to you
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SimulatedGREG/electron-vue/issues/660#issuecomment-450708355,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC8audvCB7EFVpIT7AGLyuBl-0Iadbbks5u-tyegaJpZM4VRpnK
.

--
Sent from Joe in Real Life

This same problem affects quasar apps (quasar 0.17.20, which uses electron 4.0.5).  Running MacOS High Sierra 10.14.3 on a 15" MBP with two external 4K monitors.  I'm not exactly sure how one would go about multiplying all the CSS values since the CSS is injected by the framework...

@jpriebe I have found that by using exclusively rem values for elements, em for font sizes and by scoping each component more predictable results can be seen, however there is still a difference between dev and prod. From what I have read, it's got something to with how the css/scss is compiled for production.
I hope that helps


1 record in a RecordSet

Hi.
I wonder if you can add more extra features to your project. In this case, it's something that I am using when a recordset only has one record.
I want to have a JSON object with the data of that record, and not having a collection of data.
The code I'm using is:
function ajustaJSON(str)
ajusta = replace(str, "[", "")
ajustaJSON = replace(ajusta, "]", "")
end function
function Recordset1reg2JSON(AdoRS)
Dim oJSONarr
set oJSONarr = new JSONarray
oJSONarr.LoadRecordset AdoRS
Recordset1reg2JSON = ajustaJSON(oJSONarr.Serialize())
end Function
This two functions are used like this:
Dim oJSON
set oJSON = new JSONobject
jsonString = Recordset1reg2JSON(AdoRS)
oJSON.Parse(jsonString)
Now, oJSON has several properties that match the fields of the record, with their values!
It could be great if you could add a new feature similar to LoadRecordSet that can do all this step in only one function. You don't need to ask for a property name, becuase the property names are the fields names of the recordset!
Thanks in advance!

Hi. You can simply get the element from the array:
Dim oJSONarr
set oJSONarr = new JSONarray
oJSONarr.LoadRecordset AdoRS

Dim items, oJSON
items = oJSONarr.items
set oJSON = items(0)

I should implement an easier way to get items by index soon. It will be something like:
Dim items, oJSON
set oJSON = oJSONarr(0)

Is this enough to help?

See issue #6, as it will be closed when the new funcionality become available.

Thanks for the tip #6!
As I told you, I sometimes know already that a recordset has only ONE record, and that's the way I thought I can do it.
I wonder why you cann't use oJSONarr inside oJSON object, so you can to it in one step, and not using two objects. I know that it suppose to mix classes!

I could implement a property or a method to load only the first record and return the object instead of an array.
The concern about the 2 classes is that they represent different things. One represent an object and the other a collection. Each one has ist's own responsabilities. Although not perfect, it's somewhat conformant with the SRP in SOLID.

I've implemented the LoadFirstRecord(rs) in jsonObject class. With this, you get exactly what you asked for:
jsonObj.loadFirstRecord myRecordSet

response.write jsonObj("id") ' outputs: 1

I hope you like it.


Take a picture not working

On iPad or iPhone, when I choose "Take a photo" instead of "Use existing" to choose an image to crop,
the crop area will be blank and the image is not loaded.
Could you have a look at this?
Thanks.

+1, this is a huge problem for me with my mobile app ;)
I've found two other directives (aside from compatibility I prefer ngImgCrop) that work properly on iOS:
http://jsbin.com/fovovu/1/edit?js,output (http://andyshora.com/angular-image-cropper.html)
http://think2011.github.io/localResizeIMG/ (https://github.com/think2011/localResizeIMG/blob/gh-pages/LocalResizeIMG.js)

+1, this is a huge problem for me with my mobile app ;)
I've found two other directives (aside from compatibility I prefer ngImgCrop) that work properly on iOS:
http://jsbin.com/fovovu/1/edit?js,output (http://andyshora.com/angular-image-cropper.html)
http://think2011.github.io/localResizeIMG/ (https://github.com/think2011/localResizeIMG/blob/gh-pages/LocalResizeIMG.js)

+1, huge for me also

+1, huge for me also

From my above referenced alternate libraries, I wanted to call attention to
      if( navigator.userAgent.match(/iphone/i) ) {
        var mpImg = new MegaPixImage(item);
     }

It seems that there is a special method MegaPixImage for reading the data that mobile safari pushed to the DOM's <input>.  Perhaps we could add that?

From my above referenced alternate libraries, I wanted to call attention to
      if( navigator.userAgent.match(/iphone/i) ) {
        var mpImg = new MegaPixImage(item);
     }

It seems that there is a special method MegaPixImage for reading the data that mobile safari pushed to the DOM's <input>.  Perhaps we could add that?

Thoughts @alexk111?

Thoughts @alexk111?

Well I've firgured this out. Method "setNewImageSource" which checks for image rotation caused this. Add an onload handler fixed this problem.

@anhdd-savvycom can you post code?

@brianfeister I replaced method from line 1549 of compile/unminified/ng-img-crop.js with below:
this.setNewImageSource=function(imageSource) {
    image=null;
    resetCropHost();
    events.trigger('image-updated');
    if(!!imageSource) {
        var newImage = new Image();
        if(imageSource.substring(0,4).toLowerCase()==='http') {
            newImage.crossOrigin = 'anonymous';
        }
        newImage.onload = function(){
            events.trigger('load-done');

            cropEXIF.getData(newImage,function(){
                var orientation=cropEXIF.getTag(newImage,'Orientation');

                if([3,6,8].indexOf(orientation)>-1) {
                    var canvas = document.createElement("canvas"),
                    ctx=canvas.getContext("2d"),
                    cw = newImage.width, ch = newImage.height, cx = 0, cy = 0, deg=0;
                    switch(orientation) {
                        case 3:
                            cx=-newImage.width;
                            cy=-newImage.height;
                            deg=180;
                            break;
                        case 6:
                            cw = newImage.height;
                            ch = newImage.width;
                            cy=-newImage.height;
                            deg=90;
                            break;
                        case 8:
                            cw = newImage.height;
                            ch = newImage.width;
                            cx=-newImage.width;
                            deg=270;
                        break;
                    }

                    canvas.width = cw;
                    canvas.height = ch;
                    ctx.rotate(deg*Math.PI/180);
                    ctx.drawImage(newImage, cx, cy);

                    image=new Image();
                    image.onload = function() {
                        resetCropHost();
                        events.trigger('image-updated');
                    };

                    image.src = canvas.toDataURL("image/png");
                } else {
                    image=newImage;
                    resetCropHost();
                    events.trigger('image-updated');
                }
            });
        };
        newImage.onerror=function() {
            events.trigger('load-error');
        };
        events.trigger('load-start');
        newImage.src=imageSource;
    }
};

Hope this help.

You can't rely on onload alone to trigger as it never fires for cached images. see something like this -> http://mikefowler.me/2014/04/22/cached-images-load-event/

@mike-suggitt to put it in context though, we're talking about images captured via iOS "take a picture / video" for an input field.  Since those images are generated at the time of capture, there is no way they would ever be cached.  I'll be the first one to admit that I don't understand all of the moving parts here. My goal is just to get iOS file input fields to accept camera captures (since I'm working on a mobile app ;)

@brianfeister fair enough but probably best not to change a global method. If the cause of the problem is the image rotation work then maybe pick that out? I currently use ngImgCrop without that feature set and it works fine on a mobile (bar the image rotation obviously!)

@mike-suggitt I agree. It's not a solution if it creates a new problem in its place.

@mike-suggitt I'm not seeing an option to disable rotation, where is that?

I'm still having issues even with the fix from @anhdd-savvycom above... tried a couple of things... Does anyone have a js file that's working for them?

sorry @brianfeister I wasn't clear. I simply didn't include the commit related to image rotation in my code base. You should be able to pick out the code easily enough though and comment it out/delete it. Just look at the diff to see where the offending code sits.

anhdd-savvycom's fix worked for me.  But that means I'm locally modifying the source file in the meantime.  Any word from alexk111 on whether or not he will include this fix officially?

For me, 0.3.2, on an iPhone 6 would pull up a picture from the camera roll properly, but failed on taking a picture (blank picture, unresponsive afterwards). 0.3.1 would take a picture or pull one from camera roll, but did not succeed in calculating EXIF.  I tried putting the EXIF code in 0.3.1, but it did not work... which makes me think that EXIF code works fine on desktop, but not in iPhone app.   BTW, it was a hybrid app, built with Phonegap, so not Chrome

Looks like I spoke too soon.  anhdd-savvycom's fix isn't working for me.  "Take a Picture" is working on Chrome on the iPad for me, but Safari does not work.

+1 - I'm experiencing this on an iPhone 5S 8.1 in both Safari and Chrome. Also I'm experiencing that some images from the photo gallery load, while others don't. My theory is that it has something to do with filesize - but I can't say for sure since I haven't tested it yet.

I believe the issue is to do with the way the images are manipulated when EXIF data exists. There is a call to .toDataURL("image/png") from the HTML5 canvas that doesn't work if the image taken by the camera is too big. I have just fixed the issue within iblank's fork of this project, which also has a bunch of lovely stuff in it, including fixed aspect ratios.
The pull request is here. https://github.com/iblank/ngImgCrop/pull/1
I have created a GIST (untested) for those of you using the unforked version of the lib.
https://gist.github.com/hbbh/d5f21c4120665f9429e3
this is a replacement for the setNewImageSource function (as per @anhdd-savvycom comment above), which is originally found in crop-host.js file and should just replace the same function in the compilled version. I haven't created a pull request as I've not had time to test in the original repo.

+1 Awesome library, but not so useful without the mobile support. It would be by far the best image cropping library if it is mobile compliant.

any official word on this from alexk111?

This issue could be solved by adding a rotate button to the UI. If you guys agree this is the proper approach, we'll create a PR to add the enhancement.

Is there a definitive solution for this issue? I am experiencing it right now on my iPhone 5C.

Any update on this issue ? I am facing the same and changes in file ng-img-crop.js didn't work for me.

Changes from @anhdd-savvycom did work for me.

I have captured an image by using webcam. But did not get the right output, following was my code
//  $scope.capturedImageUrl has dataUrl of the image generaed from canvas
    $scope.myImage='';
    $scope.myCroppedImage='';
    var file = $scope.capturedImageUrl;
    var reader = new FileReader();
    $scope.$apply(function($scope){
        $scope.myImage = $scope.capturedImageUrl;
    });
    reader.readAsDataURL(file);

Returns error:  "$apply already in progress"
If I remove $scope.$apply
Returns error: TypeError: Failed to execute 'readAsDataURL' on 'FileReader': parameter 1 is not of type 'Blob'.
Am I missing something? Or doing totally wrong.

   <img-crop image="myImage" result-image="myCroppedImage" result-image-quality="1.0" change-on-fly="true" result-image-size="50"  @*on-load-begin="startspinner()" on-load-done="closetimer()"*@></img-crop>
Just remove the CallBacks : Onloadstart and onloadstop and things might be fine...


fix: stepresult intepolations does not accept multiple matches

Changes
fixes: #7764

This pr enables users to use multiple step results for interpolation in step command and environment variables, for examples, users can do something like "$(steps.foo.results.res.first)-$(steps.foo.results.res.second)" set env variables for step.

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[x] Has Tests included if any functionality added or changed
[x] pre-commit Passed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
 fix: cannot use multiple step results at the same time for interpolation.

/kind fix

/hold

/kind bug

/kind bug

/hold cancel

This is ready for review now ! thanks

/cherry-pick release-v0.56.x

@vdemeester, @JeromeJu  PTAL 🙏


SQL: Fix precision of TIMESTAMP types.

Druid stores timestamps down to the millisecond, so we should use
precision = 3. Setting this wrong sometimes caused milliseconds
to be ignored in timestamp literals.
The patch moves type creation to a utility method and fixes the
precision there.
Fixes #5337.

👍


TerminalOptions.shellArgs should accept string on Windows

In order to align it with how tasks and the terminal.integrated.shellArgs.windows setting work.

@dbaeumer Is this something that would also apply to ShellExecution#shellArgs. Is this a different thing despite the same name?

@jrieken I would say we are OK here. The shellArgs are on ShellExecutionOptions to define args for the shell itself. The actual ShellExecution has a command: string or args: string[] to support a single command line. In the terminal this is one setting and therefore supports a string.


Update default catalog categories

Similar to https://github.com/kalisio/kano/issues/265.

Waiting for https://github.com/kalisio/kdk/issues/568 to be closed first.


Disconnecting a consumer takes seconds

Describe the bug
I observe both on production with AWS MSK but also locally with just a single consumer on a local Kafka broker that disconnecting a consumer (consumer.disconnect()) often takes 2-4 seconds. Is that a bug in kafkajs or is that expected behaviour?

I am able to connect to kafka consumer and consume messages in AWS lambda but when I'm trying to disconnect the it gets stuck. it will never gets disconnected and the lambda get timedout after 15 minute. next time when it connects I get kafka rebalancing error "The group is rebalancing, so a rejoin is needed"


Improve ODBC interaction: Accept a connection string

Is your feature request related to a problem? Please describe.
Yes. I've been playing with this nice tool and discovered that msodbcsql18 is barely usable with SQL Server instances running in Docker locally. The thing is that version 18 brings breaking changes by switching Encrypt option to true by default. This leads to a problem with self-signed certificates not being trusted by the driver.
More information on that can be found here:
https://techcommunity.microsoft.com/t5/sql-server-blog/odbc-driver-18-0-for-sql-server-released/ba-p/3169228
Describe the solution you'd like
Currently, it's possible to provide db-host, db-user, and db-password via both environment variables and CLI arguments. However, it's not possible to further customize the connection string. The proposal for pynonymizer is to accept a connection string from its arguments. This will allow clients to customize the connection information and will improve forward compatibility of the tool with future releases of ODBC drivers.
Describe alternatives you've considered
It looks like it's possible to inject additional connection string arguments via db-host argument by escaping the hostname with ;. Though it feels like a hack rather than a proper solution and I haven't tried that. Chances are it will not work at all.
https://github.com/rwnx/pynonymizer/blob/master/pynonymizer/database/mssql/__init__.py#L127
Additional context
I put together a Dockerfile that can be used to easily reproduce the issue - https://github.com/LDan1117/pynonymizer.docker/blob/main/Dockerfile#L10
Just replace msodbcsql17 with msodbcsql18.

I totally agree, this needs to change for mssql/odbc, the abstraction doesnt fit!
What do you think about a connection string argument that would be mutually exclusive with the db host/db user/password arguments?

Hi, this change is in the v2 milestone and will be released this week.
You can now pass "(local)\SQLEXPRESS" to the --db-host option and it should override other options. you can customize the connection string yourself using --mssql-connection-string


Using angular2-google-maps with angular version beta.11

Do you plan to upgrade angular2-google-maps with latest version of angular2?
(2.0.0-beta.11)
Thx

@ldtung release coming today with beta.11 support


Push to talk support.

autoplay in m.audio and m.recording event (required for vector-im/vector-web#1358)
Generalizing the m.typing event instead may be a better idea.

sorry for the delay on vetting this :(
I like the autoplay flag - I'm less certain about a whole new typing notification event in the form of m.recording.  Typing notifs are already heavily special-cased both on servers & clients, and telling everyone they have to special case another event type there too is going to cause needless pain.  I'm wondering if we can mux the info into m.typing instead... @erikjohnston any thoughts on this?

Right, I felt the ame.

Generalizing the m.typing event instead may be a better idea.

In general, in contet of thi and alo m.audio/m.file, I think a better olution i needed for derived event.
Broken keyboard :cry:

@ara4n do you have the conn on this or do you want me to take a look?

@ara4n Any updates on this?

can we just slap it as an optional field on m.typing? i don't think it needs fullblown derived events...

@ara4n looks good to go?

Do typing notifications even support arbitrary keys?

Is there any update on this?

The lack of push to talk support is one of the biggest things holding matrix back right now compared to proprietary competitors like Discord.

There is now a work in progress implementation at matrix-org/matrix-react-sdk#690, so we might want to make a decision here.

@aviraldg i think the last question was from our side (@kegsay's review discussion from https://github.com/matrix-org/matrix-doc/pull/310#r68506675) if you can address that :)
Otherwise, modulo the comment I just added, this LGTM

@ara4n I believe I updated the language to address @Kegsay's concerns?

@aviraldg : You don't handle the missing case (if there is no autoplay, should it start playing automatically), and you don't use RFC-style SHOULD or SHOULD NOT keywords to indicate playability, you just use the more casual "should".

Ah, thanks for explaining. I'll update it accordingly.
On Tue 14 Feb, 2017, 4:19 PM Kegsay<EMAIL_ADDRESS>wrote:

@aviraldg https://github.com/aviraldg : You don't handle the missing
case (if there is no autoplay, should it start playing automatically),
and you don't use RFC-style SHOULD or SHOULD NOT keywords to indicate
playability, you just use the more casual "should".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/matrix-org/matrix-doc/pull/310#issuecomment-279674195,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOlRD5q3qNvS2I6SN39xw8vjKgWrY6oks5rcYajgaJpZM4IGApw
.

--
Regards,
Aviral Dasgupta

any news on this?

This conflicts pretty badly with the spec as it stands today. Like other PRs, this one looks to have been before its time - an MSC for this change would be appreciated. Thanks for the PR in any case!


Release 5.13.6

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

Please upload report for BASE (master@e40d0a1). Learn more about missing BASE report.
Report is 7 commits behind head on master.

Files with missing lines
Patch %
Lines

Viewer/ecflowUI/src/VRepeatAttr.cpp
0.00%
29 Missing :warning:

Viewer/ecflowUI/src/RepeatEditor.cpp
0.00%
25 Missing :warning:

...ibs/attribute/src/ecflow/attribute/RepeatRange.hpp
74.35%
20 Missing :warning:

...service/src/ecflow/service/mirror/MirrorClient.cpp
27.77%
13 Missing :warning:

libs/node/src/ecflow/node/NodeChange.cpp
0.00%
9 Missing :warning:

libs/node/src/ecflow/node/MirrorAttr.cpp
22.22%
7 Missing :warning:

libs/core/src/ecflow/core/Chrono.cpp
0.00%
6 Missing :warning:

libs/node/src/ecflow/node/AvisoAttr.cpp
0.00%
6 Missing :warning:

libs/attribute/src/ecflow/attribute/RepeatAttr.hpp
37.50%
5 Missing :warning:

Viewer/ecflowUI/src/NodeViewDelegate.cpp
0.00%
3 Missing :warning:

... and 3 more

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #139   +/-   ##
=========================================
  Coverage          ?   48.84%
=========================================
  Files             ?     1184
  Lines             ?    98267
  Branches          ?    14778
=========================================
  Hits              ?    47996
  Misses            ?    50271
  Partials          ?        0

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


Version change to 0.11.0

This change is

r? @vinipsmaker
(maidsafe_highfive has picked a reviewer for you, use r? to override)


#81: Option to enable floating table of contents

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev-1.0.0@f38a8ac). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             dev-1.0.0     #100   +/-   ##
============================================
  Coverage             ?   87.10%
============================================
  Files                ?       10
  Lines                ?      318
  Branches             ?        0
============================================
  Hits                 ?      277
  Misses               ?       30
  Partials             ?       11

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 f38a8ac...735f636. Read the comment docs.


Expected endmethod (structure)

This error is displayed when you do a loop with using assigning field-symbol.

what language version is configured? inline loops works from v740sp02
can you help pasting the text/raw code here?

I missed very important detail: It's a screenshot from a transpiler app 😄

the transpiler only works with 702 syntax, use https://rules.abaplint.org/downport/ to downport first

Downport doesn't seem to make sense here. Issue started with Abaplint reporting the same error Expected endmethod (structure). Transpiler helped to aim the line with the real issue. The initial class that had this error had 1500 lines of code XD
So if you say that transpiler only works for 702 and I set abaplint to check syntax for 702 then how newer syntax keywords are caught and reported?

downport it before passing it to the transpiler, keep developing your code in the high syntax, 702 downport is just an intermediate step to transpiling,

I rather would like to develop in 702 and have abaplint check if I didn't do any mistakes. I don't like the auto downport.

btw. what do you use to make such recordings?

set the syntax version in abaplint.json to what your target version, eg. 702
I typically use https://www.screentogif.com to do gifs

set the syntax version in abaplint.json to what your target version, eg. 702
Yes. I did that. That's why I have this issue :D I get "Expected endmethod (structure)" instead of something like "Inline declaration of field-symbols is not supported in 702"

yea, need some better error messages -> there is https://github.com/abaplint/abaplint/issues/2267 already open for similar problem

closing -> #2267


[BUG] Faulty cast to boolean

Hey again!
https://github.com/sms77io/php-client/blob/ec0339d72bcdfa930b80ab1c8cd2e2838b944d46/src/Response/Sms.php#L19
(bool)"false" does not evaluate to the boolean value FALSE but to the boolean value TRUE.
https://www.php.net/manual/en/language.types.boolean.php
;)
Cheers!

thanks for reporting @estebanthebad will fix asap in the next release but im quite busy with other stuff atm - feel free to open a pull request on your own though :)

Fixed with latest release.


The following PR adds List() functionality so that users can see what's in the box.

The walk function is nice, but sometimes you just want a []string of files inside the box.

Thanks!


[Bug]: F4SR14-LED first channel does not send CMD - RO response works correctly

Contact Details
No response
What happened?
My system is connected via FGW14-USB and it works fine - except for ID 00000001. I cannot send any signals but receiving works fine. The device is F4SR14 LED. Is it a normal behavior, that 00000001 is not working? Have not found a workaround. All of my other devices work fine. Thanks
Adapter Version
0.9.1
JS-Controller Version
5.0.16
Node JS Version
18.18.2
NPM Version
9.8.1
Operating System
Mac OS
Relevant log output
No response
Anything else?
No response

Okay. now I removed it from device list, added it again - and it works. sorry for bothering ;-)


how to leave a room / disconnect / stop listening?

the streaming API will allow me to connect, but it seems to remember connections between different instances of my app. I'm getting multiple instances of a message sent to me.
Is there a way to disconnect?
Do i have to revoke my apps API key?
https://developer.gitter.im/docs/streaming-api
https://github.com/gitterHQ/gitter-node-stream/blob/master/gitter-stream.js
edit: this seems to have cleared up, maybe there was a delay for unsubscribes? either way would be nice to have an api for. I know it's possible to listen twice to the same room, as I've seen that behavior.
ps I'm using
        var chats = room.streaming().chatMessages();
        chats.on('chatMessages', function(message) {
...

this was a test mode server issue

reopening. seems like an intermittent problem, and not sure the root of it.
rebooting my own app has no effect, it seems the subscriptions to events are applied on the server, so working on an app that frequently restarts (eg running tests) seems to over-subscribe me to events, such that any message gets multiple on events fired to me.

@dcsan are you able to show us any code?

@mydigitalself thanks for the quick response. I added you to the private repo
https://github.com/dcsan/gitterbot/tree/master/nap
joining rooms is here:
https://github.com/dcsan/gitterbot/blob/4-multiple-joins/nap/lib/bot/GBot.js#L267-L282
using GitterHelper
on running my tests, which does connect to the API, I'm sometimes seeing this:
Not possible to join the room:  [Error: 429 {"error":"API rate limit exceeded"}] bothelp
Not possible to join the room:  [Error: 429 {"error":"API rate limit exceeded"}] bothelp/GeneralChat
Not possible to join the room:  [Error: 429 {"error":"API rate limit exceeded"}] bothelp/JS-Basics
Not possible to join the room:  [Error: 429 {"error":"API rate limit exceeded"}] bothelp/Frontend

and then seeing lots of dupe echoes.
then running tests a couple mins later i see:
Not possible to join the room:  [Error: 403 {"error":"Permission denied"}] bothelp

and things are back to normal...
is there a way with the API to see if i'm already listening to events for that room?

closing again! I think this is due to my testrunner running tests in parallel or something... not 100% sure but I cant repro in the app atm and don't want to waste your time til I can identify the repro steps.


Initial value for known_devices

It would be nice to be able to set an initial state for devices when HA starts up.
Perhaps just an extra attribute when reading known_devices?

This is only a problem for location based device trackers as OwnTracks and only for developers that often restart their systems.
Instead of fixing this on a per platform/component basis, I have this story that could help

Much better idea (and perhaps even a good general solution. Now we can re-start - makes sense to restore state across a re-start).


Doesn't seem to work anymore

All the modules recently are resulting in:
INVALID PARAMS: -2010
Looks like Nissan changed their API expectations again.  I was working on a Ruby Gem to control carwings (Nissan connect) but this has proved to be a blocker to making any more progress :-\
I need to analyze what the android app does.

edit: Nevermind, looks like my blowfish isn't working... after using the blowfish crypto in python, and passing that in manually it works... strange.


Backports for 3.3.3

This is the proposed PR for the 3.3.3 release.

Copyright notice ought to be 2006-2022 (it's 2006-2020 now).
920610 and 920620 are still in the PR. We changed that meanwhile on v4.0/dev

That's correct:

Updating the year is part of the release procedure and will happen at release.
The backport PRs do not yet contain the 2 new PRs #2780 #2782 merged into 4.0 a few hours back. I will try backporting the 2 new changesets asap which is 13:00-15:00 CEST for me.

Copyright update: Thanks. I just noticed, we update the notice with a 2nd line actually.
# Copyright (c) 2021-2022 Core Rule Set project. All rights reserved.

That ought to be added by hand, is not it?

Thanks for fixing the tests @RedXanadu 🥳


New version: SomePythonThings.WingetUIStore.Pre-release version 3.1.1-beta0

Pull request has been created with komac v2.3.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

/azp run

Usual issue, can be merged

/azp run

Automatic Validation ended with:

Installation failed with exit code 1

(Automated response - build 886.)

https://github.com/marticliment/UniGetUI/releases/download/3.1.1-beta0/UniGetUI.Installer.exe

Normal behaviour, can be merged

Newer version so closing


Skip copy of html5 plugin default css

I would like to not copy commonltr.css and commonrtl.css when using a plugin that depends on org.dita.html5 but my Ant-fu is not strong enough.
I have identified the target html5.css in the html5 build as the location of this copy instruction:
  <target name="html5.css" unless="user.csspath.url" description="Copy CSS files">
    <condition property="user.copycss.yes">
      <and>
        <equals arg1="${args.copycss}" arg2="yes"/>
        <isset property="args.css.present"/>
      </and>
    </condition>
    <property name="user.csspath.real" location="${output.dir}/${user.csspath}"/>
    <mkdir dir="${user.csspath.real}"/>
    <!-- Always copy system default css files -->
    <copy todir="${user.csspath.real}">
      <fileset dir="${dita.plugin.org.dita.html5.dir}/css" includes="*.css"/>
    </copy>
    <!-- Copy user specify css file when required -->
    <antcall target="html5.copy-css-user"/>
  </target>

Is there a way to override the first <copy> instruction?

There is no easy way to skip the copy. A guard should be added to html5.css to make it easier to skip it.

Yep, that would be swell.
The only way I found was to add a step named html5.css to my plugin build file. It seems to override the one in the core plugin. If I understand well it has something to do with the import task in Ant.