Generate JSON response sample

Generate JSON response sample when schema changes.
The sample files are located in sui/open_rpc/samples, they will be regenerated together with openrpc.json when we run cargo -q run --example generate-json-rpc-spec -- record, the files can be used for downstream testing.

Test Plan for TS-side changes
Manual Testing
Transactions
Landing

MoveCall

Publish

TransferCoin

AddressView
Aggregate Fungible Tokens section

Expanded Fungible Tokens section

Object View
Coin

NFT with image

NFT without image

Hi team,
Thanks, Patrick! Great work, as always. Does this test plan represent a common workflow?
Let me know how we can make best use of this work. Should we link to the samples from our JSON RPC page? Perhaps reuse the screenshots for Sui Explorer docs? Would love to expand the flow presented here with details for each step.


PHOENIX-6309 : Use maven enforcer plugin to ban imports from illegal packages (as per project guidelines)

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
15m 42s
master passed

+1 :green_heart:
compile
1m 32s
master passed

+1 :green_heart:
javadoc
1m 17s
master passed

_ Patch Compile Tests _

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

+1 :green_heart:
compile
1m 32s
the patch passed

+1 :green_heart:
javac
1m 32s
the patch passed

+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.

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

_ Other Tests _

-1 :x:
unit
177m 54s
root in the patch failed.

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

212m 35s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1082/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1082

Optional Tests
dupname asflicense javac javadoc unit xml compile

uname
Linux 381bf3490809 4.15.0-126-generic #129-Ubuntu SMP Mon Nov 23 18:53:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / fe0b3ca

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1082/2/artifact/yetus-general-check/output/patch-unit-root.txt

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

Max. process+thread count
5755 (vs. ulimit of 30000)

modules
C: . U: .

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

versions
git=2.7.4 maven=3.3.9

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
15m 42s
master passed

+1 :green_heart:
compile
1m 32s
master passed

+1 :green_heart:
javadoc
1m 17s
master passed

_ Patch Compile Tests _

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

+1 :green_heart:
compile
1m 32s
the patch passed

+1 :green_heart:
javac
1m 32s
the patch passed

+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.

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

_ Other Tests _

-1 :x:
unit
177m 54s
root in the patch failed.

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

212m 35s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1082/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1082

Optional Tests
dupname asflicense javac javadoc unit xml compile

uname
Linux 381bf3490809 4.15.0-126-generic #129-Ubuntu SMP Mon Nov 23 18:53:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / fe0b3ca

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1082/2/artifact/yetus-general-check/output/patch-unit-root.txt

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

Max. process+thread count
5755 (vs. ulimit of 30000)

modules
C: . U: .

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

versions
git=2.7.4 maven=3.3.9

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

This message was automatically generated.

LGTM  +1 for the master branch. However, do we have a plan to do the same thing for the 4.x? I remember that we don't have enforcement for certain important. For example, we have import com.google.common.collect.ImmutableMap; and import org.apache.phoenix.thirdparty.com.google.common.collect.ImmutableMap;. Do we have a plan to clean the code and do the same thing at the 4.x branch?

LGTM  +1 for the master branch. However, do we have a plan to do the same thing for the 4.x? I remember that we don't have enforcement for certain important. For example, we have import com.google.common.collect.ImmutableMap; and import org.apache.phoenix.thirdparty.com.google.common.collect.ImmutableMap;. Do we have a plan to clean the code and do the same thing at the 4.x branch?

Thanks @yanxinyi for the review. @stoty has already created PHOENIX-6234 for replacing guava with thirdparty in 4.x.
Until then, the only enforcement we can see is to restrict old logging framework: #1083

Thanks @yanxinyi for the review. @stoty has already created PHOENIX-6234 for replacing guava with thirdparty in 4.x.
Until then, the only enforcement we can see is to restrict old logging framework: #1083

On master, this is somewhat redundant, as dependency-check will fail the build in these cases.
That was never backported to 4.x  (but eventually should be)
However, as dependency-check runs AFTER the tests, and the tests still fail more oftent than not, this may be useful if this runs before the tests, (and on 4.x.)
The commons-logging and commons-lang stuff definitely applies to 4.x

On master, this is somewhat redundant, as dependency-check will fail the build in these cases.
That was never backported to 4.x  (but eventually should be)
However, as dependency-check runs AFTER the tests, and the tests still fail more oftent than not, this may be useful if this runs before the tests, (and on 4.x.)
The commons-logging and commons-lang stuff definitely applies to 4.x

this may be useful if this runs before the tests, (and on 4.x.)

Yes, it does catch this before running tests.

this may be useful if this runs before the tests, (and on 4.x.)

Yes, it does catch this before running tests.

Does 4.x still have commons-lang ? That is probably a regression, and you may want to fix it in #1083 and add the rule.

Does 4.x still have commons-lang ? That is probably a regression, and you may want to fix it in #1083 and add the rule.

4.x has many references to commons-lang. Let me fix them and update #1083 quickly.

4.x has many references to commons-lang. Let me fix them and update #1083 quickly.

In that it probably not a regression, I just never got around to backporting that cleanup to 4.x
I'll leave it to your discretion wheter you want to that for ticket.

In that it probably not a regression, I just never got around to backporting that cleanup to 4.x
I'll leave it to your discretion wheter you want to that for ticket.

Alright, Thanks @stoty @yanxinyi . Let me create that backport Jira then. Better not change it for now in 4.x. Sounds good?

Alright, Thanks @stoty @yanxinyi . Let me create that backport Jira then. Better not change it for now in 4.x. Sounds good?

On master, commons-lang was upgraded as part of PHOENIX-4880

On master, commons-lang was upgraded as part of PHOENIX-4880

Sounds good. (good to know it wasn't me who skipped the 4.x backport)

Sounds good. (good to know it wasn't me who skipped the 4.x backport)

My bad, in fact we can use 1.1.0 instead.

My bad, in fact we can use 1.1.0 instead.

While copying from other project, I somehow missed looking at latest versions. Let me update PR.

While copying from other project, I somehow missed looking at latest versions. Let me update PR.

Attaching sample build failure:
[WARNING] Rule 2: de.skuzzle.enforcer.restrictimports.rule.RestrictImports failed with message:

Banned imports detected in TEST code:

Reason: Use commons lang 3
	in file: org/apache/phoenix/end2end/AlterTableWithViewsIT.java
		org.apache.commons.lang.StringUtils (Line: 41, Matched by: org.apache.commons.lang.**)

Analysis took 0 seconds

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Apache Phoenix 5.1.0-SNAPSHOT ...................... SUCCESS [  1.701 s]
[INFO] Phoenix Hbase 2.4.0 compatibility .................. SUCCESS [  3.718 s]
[INFO] Phoenix Hbase 2.3.0 compatibility .................. SUCCESS [  0.946 s]
[INFO] Phoenix Hbase 2.2.1 compatibility .................. SUCCESS [  0.893 s]
[INFO] Phoenix Hbase 2.1.6 compatibility .................. SUCCESS [  0.620 s]
[INFO] Phoenix Core ....................................... FAILURE [  3.595 s]
[INFO] Phoenix - Pherf .................................... SKIPPED
[INFO] Phoenix Client Parent .............................. SKIPPED
[INFO] Phoenix Client ..................................... SKIPPED
[INFO] Phoenix Server ..................................... SKIPPED
[INFO] Phoenix - Tracing Web Application .................. SKIPPED
[INFO] Phoenix Assembly ................................... SKIPPED
[INFO] phoenix-tools ...................................... SKIPPED
[INFO] Phoenix Client Embedded 5.1.0-SNAPSHOT ............. SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 12.127 s
[INFO] Finished at: 2021-01-11T12:57:06+05:30
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M3:enforce (banned-illegal-imports) on project phoenix-core: Some Enforcer rules have failed. Look above for specific messages explaining why the rule failed. -> [Help 1]
[ERROR]

Attaching sample build failure:
[WARNING] Rule 2: de.skuzzle.enforcer.restrictimports.rule.RestrictImports failed with message:

Banned imports detected in TEST code:

Reason: Use commons lang 3
	in file: org/apache/phoenix/end2end/AlterTableWithViewsIT.java
		org.apache.commons.lang.StringUtils (Line: 41, Matched by: org.apache.commons.lang.**)

Analysis took 0 seconds

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Apache Phoenix 5.1.0-SNAPSHOT ...................... SUCCESS [  1.701 s]
[INFO] Phoenix Hbase 2.4.0 compatibility .................. SUCCESS [  3.718 s]
[INFO] Phoenix Hbase 2.3.0 compatibility .................. SUCCESS [  0.946 s]
[INFO] Phoenix Hbase 2.2.1 compatibility .................. SUCCESS [  0.893 s]
[INFO] Phoenix Hbase 2.1.6 compatibility .................. SUCCESS [  0.620 s]
[INFO] Phoenix Core ....................................... FAILURE [  3.595 s]
[INFO] Phoenix - Pherf .................................... SKIPPED
[INFO] Phoenix Client Parent .............................. SKIPPED
[INFO] Phoenix Client ..................................... SKIPPED
[INFO] Phoenix Server ..................................... SKIPPED
[INFO] Phoenix - Tracing Web Application .................. SKIPPED
[INFO] Phoenix Assembly ................................... SKIPPED
[INFO] phoenix-tools ...................................... SKIPPED
[INFO] Phoenix Client Embedded 5.1.0-SNAPSHOT ............. SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 12.127 s
[INFO] Finished at: 2021-01-11T12:57:06+05:30
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M3:enforce (banned-illegal-imports) on project phoenix-core: Some Enforcer rules have failed. Look above for specific messages explaining why the rule failed. -> [Help 1]
[ERROR]

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
16m  7s
master passed

+1 :green_heart:
compile
1m 31s
master passed

+1 :green_heart:
javadoc
1m 17s
master passed

_ Patch Compile Tests _

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

+1 :green_heart:
compile
1m 36s
the patch passed

+1 :green_heart:
javac
1m 36s
the patch passed

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

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

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

_ Other Tests _

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

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

304m  9s

Reason
Tests

Failed junit tests
phoenix.end2end.index.GlobalMutableNonTxIndexIT

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1082/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1082

Optional Tests
dupname asflicense javac javadoc unit xml compile

uname
Linux fd5ecc1bcde0 4.15.0-126-generic #129-Ubuntu SMP Mon Nov 23 18:53:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / fe0b3ca

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1082/3/artifact/yetus-general-check/output/patch-unit-root.txt

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

Max. process+thread count
5740 (vs. ulimit of 30000)

modules
C: . U: .

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

versions
git=2.7.4 maven=3.3.9

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
16m  7s
master passed

+1 :green_heart:
compile
1m 31s
master passed

+1 :green_heart:
javadoc
1m 17s
master passed

_ Patch Compile Tests _

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

+1 :green_heart:
compile
1m 36s
the patch passed

+1 :green_heart:
javac
1m 36s
the patch passed

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

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

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

_ Other Tests _

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

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

304m  9s

Reason
Tests

Failed junit tests
phoenix.end2end.index.GlobalMutableNonTxIndexIT

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1082/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1082

Optional Tests
dupname asflicense javac javadoc unit xml compile

uname
Linux fd5ecc1bcde0 4.15.0-126-generic #129-Ubuntu SMP Mon Nov 23 18:53:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / fe0b3ca

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1082/3/artifact/yetus-general-check/output/patch-unit-root.txt

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

Max. process+thread count
5740 (vs. ulimit of 30000)

modules
C: . U: .

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

versions
git=2.7.4 maven=3.3.9

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

This message was automatically generated.


(BKR-504) Update Solaris package name for release version

The planned Solaris package names for the puppet-agent package were
changed to incorporate a release version, as already handled for other
platforms. Update the Solaris package name to include the hard-coded
release 1 naming.

Verified with {facter,hiera,puppet}#stable.

Tested with old and new packages on hiera#stable pre-suite.

ok to test

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/1464/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/1465/
Test FAILed.


Export Function worker internal stats via Prometheus

Motivation
Collect some metrics on the performance of internal modules of the function worker.  Export the metrics via prometheus

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


source available?

Is there source code I could find somewhere? Trying to make the app run on mac.

nah i used an app

which app?

which app?

Nativefier


feat: support exlude windows

add exlude_windows,   some windows do not need manage the window state for example splashscreen

For future contributions, please sign you commits, thanks again.

thanks, i will sign my commits next time:)


Task to consume messages from ActiveMQ

Request has arrived to create a task for consuming messages from ActiveMQ.

Initial implementation of the Task has been done.


Replicate CI job dependency + try to address flakiness of tests

Issue

Alex introduced a dependency for tests (to save time and cycles running heavy integration tests if unit tests or lints fail). We do not yet have this in this repository
We are not raising on blocked in the scale_application helper function
Somehow, scaling the database more than one unit at a time is causing the following error trace (https://github.com/canonical/mysql-k8s-operator/actions/runs/3204633434/jobs/5236555089#step:4:541). Not suggesting that this is the root cause

Solution

Replicate the dependencies for tests to this repo
Raise on blocked
Scale up one application at a time to see if that reduces the frequency of the error

Release Notes
Replicate CI job dependency + try to address flakiness of tests

Thanks for the amazingly quick reviews!


Error handling from C++

A formal notion of operation failure should exist at the C ABI level.  This likely entail returning nullptr or error codes from every method.

@bwasti reached out in the FFI channel on Discord RE this and per Jarred, there are 5 bits that we might use to tag the ptr value returned with an error code value.
Digging through Bun's codebase, this seems the most relevant to how Bun FFI tags pointers at present, which provides some insight into how such an approach to error handling from C++ might take shape: https://github.com/oven-sh/bun/blob/0c2a207d82f42caa1d1f360c5ed56c92443c4795/src/bun.js/api/FFI.h

After thinking about trying to find a simpler solution for comprehensive error handling, I think the simplest approach (that would make it easy to implement custom backends), would be to just print the error from native code and return nullptr; this would mean that anytime JS receives a nullptr, we can immediately throw an error with stack trace (except for those cases where there's an implemented fallback we'll want to try first).
But this would seem best of both worlds in that it removes the added complexity/limitations of tagging pointers with error codes and should provide all the same benefits in that we'll get comprehensive error info from native code + stack trace from JS with insight into where the native Fn throwing the error was called.

I think we might be able to close this. 💯


Filterable list page: Content enhancement - update wireframes to show filter

Which page is this about?
https://cfpb.github.io/design-system/pages/filterable-list-pages
What kind of issue is this?
Add one or more of the issue labels below to the right-hand sidebar AND issue title.
(We strongly encourage you to make content changes yourself where possible!)

Content enhancement (e.g. confusing or redundant content, small content items need to be added, etc.)

Describe your issue
Update the wireframes to show a filter? Otherwise, we're just showing the (unmodified) browse & sublanding wireframes. Or possibly remove the wireframes altogether and refer to the browse and sublanding pages.
Size this request (1=tiny, 5=enormous)
1
Add any additional background info as needed
Make sure NOT to include internal links.
Thanks. Don't forget to add labels indicating issue type and size before submitting!
Once you've submitted your issue, please add it to the content backlog project board: https://github.com/cfpb/design-system/projects/1

This could be a good contender for design system day, but we'd need to track down the wireframe source files.

This could be a good contender for design system day, but we'd need to track down the wireframe source files.

Alright, I updated the wireframes and merged it in! https://github.com/cfpb/design-system/pull/1149
Annnnnd then realized I had an error in one of the images I created, so I'm in the process of trying to re-upload it.

Alright, I updated the wireframes and merged it in! https://github.com/cfpb/design-system/pull/1149
Annnnnd then realized I had an error in one of the images I created, so I'm in the process of trying to re-upload it.

Okay, this should be fixed and done thanks to https://github.com/cfpb/design-system/pull/1151

Okay, this should be fixed and done thanks to https://github.com/cfpb/design-system/pull/1151


How can I visualize those Gaussians?

Great work, by the way. How can I visualize those Gaussians? I encountered an issue with the original SIBR_viewer. How should I solve it? Thank you.
[SIBR] --  INFOS  --:   Initialization of GLFW
[SIBR] --  INFOS  --:   OpenGL Version: 4.6.0 NVIDIA 535.171.04[major: 4, minor: 6]
Number of input Images to read: 40
Number of Cameras set up: 40
[SIBR] --  INFOS  --:   Error: can't load mesh '/mnt/SG6T/xyz/DATASET/deblurnerf/real_camera_motion_blur/blurgirl/sparse/0/points3d.txt.
[SIBR] --  INFOS  --:   Mesh contains: colors: 1, normals: 1, texcoords: 0
[SIBR] --  INFOS  --:   Mesh '/mnt/SG6T/xyz/DATASET/deblurnerf/real_camera_motion_blur/blurgirl/sparse/0/points3d.ply successfully loaded. 1 meshes were loaded with a total of  (0) faces and  (3947) vertices detected. Init GL ...
[SIBR] --  INFOS  --:   Init GL mesh complete
[SIBR] --  INFOS  --:   Loading 83539 Gaussian splats
[SIBR] --  INFOS  --:   Initializing Raycaster
[SIBR] --  INFOS  --:   Interactive camera using (0.009,1100) near/far planes.
Switched to trackball mode.
[SIBR] ##  ERROR  ##:   FILE /mnt/SG6T/xyz/BAGS/SIBR_viewers/src/projects/gaussianviewer/renderer/GaussianView.cpp
LINE 545, FUNC onRenderIBR
A CUDA error occurred during rendering:an illegal memory access was encountered. Please rerun in Debug to find the exact line!terminate called after throwing an instance of 'std::runtime_error'
what():  See log for message errors
Aborted (core dumped)

SAME ERROR FOR ME
Thank you in advance.


PC API login/auth errors starting 11/17/2021

Starting ~2d ago (17-NOV), started getting these Personal Capital API login/auth errors...
% ./wx.ad.py ro
Traceback (most recent call last):
  File "./wx.ad.py", line 60, in <module>
    pc.login( pc_cred_user , pc_cred_pass ) ;
  File "/usr/local/lib/python3.4/site-packages/personalcapital/personalcapital.py", line 44, in login
    raise Exception()
Exception
...on code from this library that has run flawlessly for well over a year...
 34     def login(self, username, password):
 35         initial_csrf = self.__get_csrf_from_home_page(base_url)
 36         csrf, auth_level = self.__identify_user(username, initial_csrf)
 37
 38         if csrf and auth_level:
 39             self.__csrf = csrf
 40             if auth_level != AuthLevelEnum.USER_REMEMBERED:
 41                 raise RequireTwoFactorException()
 42             self.__authenticate_password(password)
 43         else:
 44             raise Exception()
Is anyone else experiencing this and/or have a fix/workaround? Tried disabling 2FA, no dice. Have logged a support case with PC but not holding my breath.
Thanks!
Ian

+1 with the exact same errors starting on 11/17/2021

Thx for verifying @khite1983 ... good in a way to know it's not specific/limited to me. Maybe chuck a Support case in as well.
(Unfortunately they didn't give me a case # for you to reference ... just an email thread, where I commented just now about your confirmation that this is broken.)
I'm dusting off my Plaid credentials & exploring that as an alternative if PC Support doesn't engage.

Same issue here as well.

Looks like update in cloudflare, a small homepage structure change, and API payloads.  My solution here (new dependency added, cloudscraper) #21

same issue.

Dumb question: how do I incorporate your changes before your PR is approved?
On Sat, Nov 20, 2021 at 1:36 PM UnusualPi @.***> wrote:

Looks like update in cloudflare, a small homepage structure change, and
API payloads. My solution here (new dependency added, cloudscraper
https://github.com/VeNoMouS/cloudscraper) #21
https://github.com/haochi/personalcapital/pull/21
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/haochi/personalcapital/issues/20#issuecomment-974714765,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEWS5DRVP5DMAWEPA5SYXVTUNAIE7ANCNFSM5IMRH7SQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@ianvanhoven it really depends on your implementation, but you'll definitely have to install cloudscraper.  For the code changes in the personalcapital.py file, you could either edit directly in your python site-packages directory or you could just download the file, include in your script directory, and make the modifications on that file.  I'd recommend the latter, when you use import in python it looks for files in the same directory before looking in site-packages.  Hope that helps!

Thanks, @UnusualPi. I applied your fix and it resolved the login error issue on my end. I had to upgrade a couple of required package versions to make it fully work though. Thanks again for the solution!

Was about to come here and make a similar pull request! Thanks for making this. For those coming along trying to fix this for themselves, add this to your requirements.txt instead:
git+git://github.com/UnusualPi/personalcapital.git@cd0c88f78912c928ad8fafcce05864a43d914d54

Thanks @UnusualPi ... still at a loss on this :(  Any additional
suggestion/guidance appreciated.
I made your changes directly in...
/usr/local/lib/python3.4/site-packages/personalcapital/personalcapital.py
...re-confirmed I have cloudscraper...
% sudo pip install cloudscraper
Requirement already satisfied: cloudscraper in
/usr/local/lib/python3.4/site-packages (1.2.58)
Requirement already satisfied: pyparsing>=2.4.7 in
/usr/local/lib/python3.4/site-packages (from cloudscraper) (2.4.7)
Requirement already satisfied: requests>=2.9.2 in
/usr/local/lib/python3.4/site-packages (from cloudscraper) (2.20.0)
Requirement already satisfied: requests-toolbelt>=0.9.1 in
/usr/local/lib/python3.4/site-packages (from cloudscraper) (0.9.1)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in
/usr/local/lib/python3.4/site-packages (from requests>=2.9.2->cloudscraper)
(3.0.4)
Requirement already satisfied: idna<2.8,>=2.5 in
/usr/local/lib/python3.4/site-packages (from requests>=2.9.2->cloudscraper)
(2.7)
Requirement already satisfied: urllib3<1.25,>=1.21.1 in
/usr/local/lib/python3.4/site-packages (from requests>=2.9.2->cloudscraper)
(1.24.3)
Requirement already satisfied: certifi>=2017.4.17 in
/usr/local/lib/python3.4/site-packages (from requests>=2.9.2->cloudscraper)
(2019.9.11)
...but getting the following error when executing my script...
% tail -0f log/2021/wx.log.202112.ad.err
[Wed Dec  1 09:29:50 2021] ERROR: Package import failure; exiting...
[Wed Dec  1 09:29:50 2021] ERROR: invalid syntax (init.py, line 202)
^C
...with init.py containing the following...
% cat init.py
from .personalcapital import PersonalCapital, RequireTwoFactorException,
TwoFactorVerificationModeEnum
...yet seeing the following (seemingly good) in STDOUT when running script
w/ verbose logging...
% python -v wx.ad.py
[ ... ]
code object from
/usr/local/lib/python3.4/site-packages/personalcapital/init.py
created
'/usr/local/lib/python3.4/site-packages/personalcapital/pycache/init.cpython-34.pyc'
wrote
'/usr/local/lib/python3.4/site-packages/personalcapital/pycache/init.cpython-34.pyc'
code object from
/usr/local/lib/python3.4/site-packages/personalcapital/personalcapital.py
created
'/usr/local/lib/python3.4/site-packages/personalcapital/pycache/personalcapital.cpython-34.pyc'
wrote
'/usr/local/lib/python3.4/site-packages/personalcapital/pycache/personalcapital.cpython-34.pyc'
destroy cloudscraper
destroy personalcapital.personalcapital
destroy personalcapital
[ ... ]
On Sun, Nov 21, 2021 at 7:14 AM UnusualPi @.***> wrote:

@ianvanhoven https://github.com/ianvanhoven it really depends on your
implementation, but you'll definitely have to install cloudscraper. For the
code changes in the personalcapital.py file, you could either edit directly
in your python site-packages directory
https://stackoverflow.com/questions/122327/how-do-i-find-the-location-of-my-python-site-packages-directory
or you could just download the file, include in your script directory,
and make the modifications on that file
https://csatlas.com/python-import-file-module/. I'd recommend the
latter, when you use import in python it looks for files in the same
directory before looking in site-packages. Hope that helps!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/haochi/personalcapital/issues/20#issuecomment-974834963,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEWS5DXBW63BU7SSFVSLC7LUNEED7ANCNFSM5IMRH7SQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

LOL ... so took a wild guess that my old OpenBSD box where I host this
script is just in some janky state -- and so I ported the script to a very
recently-created AWS EC2 instance.
*Good news: *no more pkg import errors :)
Bad news:  cloudscraper.exceptions.CloudflareChallengeError: Detected a
Cloudflare version 2 Captcha challenge, This feature is not available in
the opensource (free) version.
Is anyone else seeing this?
On Wed, Dec 1, 2021 at 10:05 AM Ian Van Hoven @.***> wrote:

Thanks @UnusualPi ... still at a loss on this :(  Any additional
suggestion/guidance appreciated.
I made your changes directly in...
/usr/local/lib/python3.4/site-packages/personalcapital/personalcapital.py
...re-confirmed I have cloudscraper...
% sudo pip install cloudscraper
Requirement already satisfied: cloudscraper in
/usr/local/lib/python3.4/site-packages (1.2.58)
Requirement already satisfied: pyparsing>=2.4.7 in
/usr/local/lib/python3.4/site-packages (from cloudscraper) (2.4.7)
Requirement already satisfied: requests>=2.9.2 in
/usr/local/lib/python3.4/site-packages (from cloudscraper) (2.20.0)
Requirement already satisfied: requests-toolbelt>=0.9.1 in
/usr/local/lib/python3.4/site-packages (from cloudscraper) (0.9.1)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in
/usr/local/lib/python3.4/site-packages (from requests>=2.9.2->cloudscraper)
(3.0.4)
Requirement already satisfied: idna<2.8,>=2.5 in
/usr/local/lib/python3.4/site-packages (from requests>=2.9.2->cloudscraper)
(2.7)
Requirement already satisfied: urllib3<1.25,>=1.21.1 in
/usr/local/lib/python3.4/site-packages (from requests>=2.9.2->cloudscraper)
(1.24.3)
Requirement already satisfied: certifi>=2017.4.17 in
/usr/local/lib/python3.4/site-packages (from requests>=2.9.2->cloudscraper)
(2019.9.11)
...but getting the following error when executing my script...
% tail -0f log/2021/wx.log.202112.ad.err
[Wed Dec  1 09:29:50 2021] ERROR: Package import failure; exiting...
[Wed Dec  1 09:29:50 2021] ERROR: invalid syntax (init.py, line 202)
^C
...with init.py containing the following...
% cat init.py
from .personalcapital import PersonalCapital, RequireTwoFactorException,
TwoFactorVerificationModeEnum
...yet seeing the following (seemingly good) in STDOUT when running script
w/ verbose logging...
% python -v wx.ad.py
[ ... ]
code object from
/usr/local/lib/python3.4/site-packages/personalcapital/init.py
created
'/usr/local/lib/python3.4/site-packages/personalcapital/pycache/init.cpython-34.pyc'
wrote
'/usr/local/lib/python3.4/site-packages/personalcapital/pycache/init.cpython-34.pyc'
code object from
/usr/local/lib/python3.4/site-packages/personalcapital/personalcapital.py
created
'/usr/local/lib/python3.4/site-packages/personalcapital/pycache/personalcapital.cpython-34.pyc'
wrote
'/usr/local/lib/python3.4/site-packages/personalcapital/pycache/personalcapital.cpython-34.pyc'
destroy cloudscraper
destroy personalcapital.personalcapital
destroy personalcapital
[ ... ]
On Sun, Nov 21, 2021 at 7:14 AM UnusualPi @.***>
wrote:

@ianvanhoven https://github.com/ianvanhoven it really depends on your
implementation, but you'll definitely have to install cloudscraper. For the
code changes in the personalcapital.py file, you could either edit directly
in your python site-packages directory
https://stackoverflow.com/questions/122327/how-do-i-find-the-location-of-my-python-site-packages-directory
or you could just download the file, include in your script directory,
and make the modifications on that file
https://csatlas.com/python-import-file-module/. I'd recommend the
latter, when you use import in python it looks for files in the same
directory before looking in site-packages. Hope that helps!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/haochi/personalcapital/issues/20#issuecomment-974834963,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEWS5DXBW63BU7SSFVSLC7LUNEED7ANCNFSM5IMRH7SQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I wonder if @haochi would be willing to assign ownership of this repo to another dev so that we could add this fix?

A little help, please. I've applied the fix as detailed by @UnusualPi & installed cloudscaper, but am getting
 File "/usr/local/lib/python3.9/site-packages/personalcapital/personalcapital.py", line 1, in <module>
    import cloudscraper
ModuleNotFoundError: No module named 'cloudscraper'.

I've installed cloudscaper on the host and in the docker container - is there some path/link I need to create to get this resolved?
thanks

Ok, I'm super close to getting this working...Getting the prompt for the verification code from Personal Capital, but when I click the 'Verify' button am getting the response:
Failed to call service configurator/configure.authenticate_password() missing 1 required positional argument: 'password'
Any thoughts?
thanks

@elstevega You'll have to update the file you're running the main scripts in to include the additional parameter. UnusualPi's fix added an additional parameter that gets passed in and which is expected by the PersonalCapital parent class in the personalcapital module.
In your case above, you're passing in the password that you get with (config.get(CONF_PASSWORD) but because it's the only thing being passed in and not being passed in explicitly the parent class is treating is as the first parameter (username) and saying you're missing the password.
To fix it you can modify your authenticate_password method to pass in the email as well. I'm not sure what your actual script file looks like but you should be able to do something like result = pc.authenticate_password(config.get(CONF_EMAIL),config.get(CONF_PASSWORD)) (assuming you're able to reference both your email and password with config.get).

@Aharic - thanks for the info - that did the trick. (I was close...)
:)

Argh, I spoke too soon. While the initial SMS validation piece now works (thanks @Aharic ), when I reboot the HA server, it loses the connection info and kicks back the below in the log file:
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform
    await asyncio.shield(task)
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/personalcapital/sensor.py", line 130, in setup_platform
    pc.login(config.get(CONF_EMAIL), config.get(CONF_PASSWORD))
  File "/usr/local/lib/python3.9/site-packages/personalcapital/personalcapital.py", line 44, in login
    raise Exception()
Exception

That bit of code is the update from UnusualPi...code block below - any thoughts? Thanks so much in advance!!
    def login(self, username, password):
        initial_csrf = self.__get_csrf_from_home_page(base_url)
        csrf, auth_level = self.__identify_user(username, initial_csrf)

        if csrf and auth_level:
            self.__csrf = csrf
            if auth_level != AuthLevelEnum.USER_REMEMBERED:
                raise RequireTwoFactorException()
            result = self.__authenticate_password(username, password).json()
            if getSpHeaderValue(result, SUCCESS_KEY) == False:
                raise LoginFailedException(getErrorValue(result))
        else:
            raise LoginFailedException()

@elstevega , I might be overlooking something, but judging by the Traceback infomation you pasted above, it seems like your code is still using the original version of the personalcapital.py from pip install, because the file path showed as:

File "/usr/local/lib/python3.9/site-packages/personalcapital/personalcapital.py"

That would still cause the issue that we are tracking here in this issue #20.
Do you mind to double check your project folder and your python path. You may want to make sure your code is using the UnusualPi's updated personalcapital.py

@elstevega , I might be overlooking something, but judging by the Traceback infomation you pasted above, it seems like your code is still using the original version of the personalcapital.py from pip install, because the file path showed as:

File "/usr/local/lib/python3.9/site-packages/personalcapital/personalcapital.py"

That would still cause the issue that we are tracking here in this issue #20. Do you mind to double check your project folder and your python path. You may want to make sure your code is using the UnusualPi's updated personalcapital.py

@willyiwei I don't mind checking at all & thanks for your help. To recap what I've done:

edited /config/custom_components/personalcapital/sensor.py line 76 to
result = pc.authenticate_password(config.get(CONF_EMAIL),config.get(CONF_PASSWORD))

this took care of the initial SMS 2FA

Followed UnusualPi''s steps```

pip uninstall personalcapital -y
pip install git+git://github.com/UnusualPi/personalcapital.git@cd0c88f78912c928ad8fafcce05864a43d914d54
 - this placed code in /usr/lib/python3.9/site-packages/personalcapital - verified that the personalcapital.py here matches the github repo of UnusualPi

3) pip list shows both cloudscraper & personalcapital 1.01

4) restarted HA and am getting the same error I posted above

Logger: homeassistant.components.sensor
Source: custom_components/personalcapital/sensor.py:130
Integration: Sensor (documentation, issues)
First occurred: 2:40:34 PM (1 occurrences)
Last logged: 2:40:34 PM
Error while setting up personalcapital platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform
await asyncio.shield(task)
File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/personalcapital/sensor.py", line 130, in setup_platform
pc.login(config.get(CONF_EMAIL), config.get(CONF_PASSWORD))
File "/usr/local/lib/python3.9/site-packages/personalcapital/personalcapital.py", line 44, in login
raise Exception()
Exception

Any thoughts?

@elstevega , I might be overlooking something, but judging by the Traceback infomation you pasted above, it seems like your code is still using the original version of the personalcapital.py from pip install, because the file path showed as:

File "/usr/local/lib/python3.9/site-packages/personalcapital/personalcapital.py"

That would still cause the issue that we are tracking here in this issue #20. Do you mind to double check your project folder and your python path. You may want to make sure your code is using the UnusualPi's updated personalcapital.py

No problem & thanks for your response. Uninstalled/re-installed everything just to make sure.
I cleared out the session on the personal capital website & re-authenticated. gonna reboot the HA server tomorrow & see what happens

Sounds good, @elstevega. And I had a brief glance of your other reply in my email inbox and from there I got some hints about what your issue might be.
When you get a chance to check it again, you may want to pay attention to the two different python lib paths below:
/usr/lib/python3.9/site-packages/personalcapital.py
v.s.
/usr/local/lib/python3.9/site-packages/personalcapital.py
By default, I think pip install the packages into /user/loca/lib/python3.9/....
Make sure you replace this file in the correct lib path. I use virtualenv so my lib file is inside my venv/ folder instead of global /usr/local/lib
You can use this command line to check the information about your pip installed personalcapital package:
$ pip show PersonalCapital
Then it will give you a Location to your personalcapital package installation on your host.
Hope it helps.

Thanks @willyiwei  - looks like I'm surviving reboots & all is well - thanks to all here for lending a hand.

For anyone looking for the easiest way to get this working again - I forked this repo and got it working with the fixes mentioned above. You can switch over to https://github.com/sanghviharshit/sensor.personalcapital (add it as custom repository and reinstall personal capital using HACS)

I was using the fork from @UnusualPi but it looks like they deleted it. I made those same changes in my own fork here: https://github.com/banool/personalcapital..


CancelEventHandler is not available on RageMP

Same as FiveM, needs to use the copy-pasted code from .NET Core.
Originally reported in #87

This is a breaking change, sadly


sound/alsa: Release buffers even on RELEASE err

Summary of the PR
If a PCM_RELEASE control request comes from the guest and the state transition is invalid by spec standards, release the buffers anyway. This doesn't change the current behavior.
When dealing with an unrelated vhost-user issue, the state transitions became invalid because of unexpected errors in the virto-snd driver. Making this change made the issue obvious while debugging. Plus, the spec does not state rejecting a PCM lifecycle control request if the transition is not possible. (The possible state transitions are non-normative statements.)
Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[x] All commits in this PR are signed (with git commit -s), and the commit
message has max 60 characters for the summary and max 75 characters for each
description line.
[x] All added/changed functionality has a corresponding unit/integration
test.
[x] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[x] Any newly added unsafe code is properly documented.

Duplicate of https://github.com/rust-vmm/vhost-device/pull/487


Support calculating md5 / sha1 hash for each uploaded files

It is very handful if we could have file's md5 or sha1 after upload, for example, when using it to check duplicated files, as a global unique file name or as database primary key etc.. I tried to do it by using req.file().pipe but I found it's extremely hard without reading through the file again (without hacking the adapter). So I modified skipper-disk package and added this feature. When giving an option
{
  hash: 'md5'
}

In the returned results, it will show file's hash. Example:
[ { fd: '/srv/http/.tmp/uploads/d3cdcbd2-5b6a-4131-b9ce-2a5da00d4f9d.ini',
    size: 115,
    type: 'application/octet-stream',
    filename: 'desktop.ini',
    status: 'bufferingOrWriting',
    field: 'files[]',
    extra: { md5: '359bb6716c7fb00e02d4b681a0bce711' } } ]

if hash option is not given, it will load nothing so there's no perfomance impact when we not using it.
Since it's utilize the same pipe chain as skipper-disk's fs.createWriteStream, the overhead of hash calculation is minimal as well.

@caasjj I think you are right, I modified the code again


Select does not render at all if .browser-default is added V0.100.2

In the latest commit "v0.100.2", .browser-default in the select element does not render at all.
My code
<select name="homework_type" id="type_a" class="browser-default">
             <option value="" disabled selected>Select Type</option>
             <option value="Daily" selected>Daily</option>
             <option value="Test">Assessment</option>
             <option value="Assignment">Assignment</option>
</select>

<script>$(document).ready(function() {             $('select').material_select();         });</script>
It works fine in V0.99.0

Please fill out the issue template!

I just experienced the same issue. Interestingly enough, it only happens when you surround your select element with a div containing the "input-field" class. If you use the codepen to remove the class from the div tag, the select option appears. Re-add the class and select disappears.
Using Ubuntu 16.04 x64. Appears both in Chrome 61.0.3163.79 and Firefox 55.0.2.

@devansh1110 and @tomscholz , any chance we can re-open this issue? It still isn't fixed. See my comment above for more details.
Thanks!

@Dogfalo Could you reopen this and close #5294?

Temporary Workaround
For this reason, I've been working with Version 99.0 You can import it using
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/materialize/0.99.0/css/materialize.min.css">
<script src="https://cdnjs.cloudflare.com/ajax/libs/materialize/0.99.0/js/materialize.min.js"></script>
It works fine in V99.0.
Thanks,
Devansh

Materialize v0.100.2
Chrome Version 64.0.3282.186 (Offizieller Build) (64-Bit)
Opera Version:	51.0.2830.40
It is also not working for me, too.

The browser default select is not meant to go inside input-field class as is shown in the documentation example, however the change has been reverted in v1-dev (c0649e8c73d0700330bffae76bbe00553e7df642) due to the addition of select helper text system.


runtime: scavenger is too eager on Darwin

Currently on Darwin the scavenger is too eager and causing performance regressions. The issue mainly stems from the fact that in Go 1.14 the scavenger is paced empirically according to costs of scavenging, most of which comes from sysUnused which makes an madvise syscall on most platforms, including Darwin.
However, the problem on Darwin is that we don't just do MADV_FREE anymore, we do MADV_FREE_REUSABLE in the sysUnused path and MADV_FREE_REUSE in the sysUsed path (instead of sysUsed being a no-op). It turns out the source of the regression is mostly the fact that sysUsed is actually quite expensive relative to other systems where it's just a no-op and we instead incur an implicit page fault once the memory is actually touched. The benefits of using this API outweigh the costs, since it updates process RSS counters in the kernel appropriately and MADV_FREE_REUSABLE is properly lazy.
So since we don't account for sysUsed we end up scavenging a lot more frequently than we should to maintain the scavenger's goal of only using 1% of the CPU time of one CPU.
The actual size of the regression can be quite large, up to 5%, as seen in #36218, so we should fix this before 1.14 goes out.
The fix here is relatively simple: we just need to account for this extra cost somehow. We could measure it directly in the runtime but this then slows down allocation unnecessarily, and even then it's unclear how we should attribute that cost to the scavenger (maybe as a debt it needs to pay down?). Trying to account for this cost on non-Darwin platforms is also tricky because the costs aren't actually coming from sysUsed but from the page fault.
Instead, I think it's a good idea to do something along the lines of what we did last release: get some empirical measurements and use that to get an order-of-magnitude approximation. In this particular case, I think we should compute an empirical ratio "r" of using a scavenged page and sysUnused and turn this into a multiplicative constant, "1+r" for the time spent scavenging. So for example if sysUsed is roughly as expensive as sysUnused the factor would be 2.

Digging deeper, I think I know better why sysUsed is hurting performance so much, even though it only takes around the same amount of time as sysUnused: starting with this release, we might allocate across a scavenged/unscavenged memory boundary. If one allocates several pages at once and if even only one page in that range is scavenged, we call sysUsed on that whole region. This has no effect on systems where sysUsed is a no-op, since those systems just fault in those pages on demand.
I'm not sure what to do here. The patch I uploaded helps the problem a little bit, but to completely solve the problem would require figuring out exactly which pages are scavenged and only calling sysUsed on those, but that's somewhat expensive to keep track of in the new allocator.
Because the allocator doesn't propagate up which pages are scavenged (though it does know this information precisely!) currently we just do the heavy-handed thing. But, we could instead have the allocator, which actually clears the bits, do the sysUsed operation. It would then have precise information. Because syscall overhead is also a factor here, there would need to be a heuristic, though. For example, if >50% of the memory region is scavenged, just sysUsed the whole thing, instead of piece-by-piece.
Unfortunately, lowering the sysUsed operation down into the allocator like this is annoying for tests. In most cases it's safe to sysUsed something already considered in-use by the OS, but I don't think we make that part of sysUsed's API, though I suppose could. We also do already have a flag which does this so we can test scavenging, so maybe we should just use that?

On second thought, I can't really find evidence of the number of scavenged pages in an allocation being less than the size of the allocation in the above benchmarks. Perhaps this isn't a problem...

OK going to walk back my second thought. Turns out, yes, https://github.com/golang/go/issues/36507#issuecomment-573887201 is a real problem. It's slightly worse now, but only by a little bit. This is basically going back to Go 1.11 behavior where if any memory was scavenged you would just scavenge the whole thing.


Unclear instructions about folder structure

You wrote:

When you work through each project, make a new directory within Projects to keep your code away from the sample code. For example:

└── Projects/
    └── new-coder/
        # <-- snip -->
        ├── apis/     # sample code for tutorial #2
        ├── apis_workspace/ # your code for tutorial #2
        ├── dataviz/  # sample code for tutorial #1
        ├── dataviz_workspace/ # your code for tutorial #1
        ├── gui/      # sample code for tutorial #5
        ├── gui_workspace/ # your code for tutorial #5
        ├── network/  # sample code for tutorial #4
        ├── network_workspace/ # your code for tutorial #4
        ├── scrape/   # sample code for tutorial #3
        ├── scrape_workspace/ # your code for tutorial #3
        └── website/  # files that make newcoder.io

At this point, it's easy to understand the instructions and follow them.
But then, in the dataviz tutorial I see these instructions:
$ cd new-coder/dataviz
$ mkvirtualenv DataVizProj

Open up parse.py, found: new-coder/dataviz/tutorial_source/parse.py

I know it is a stupid question, but I just want to learn some cool stuff in Python. Extra difficulties are very demotivating. Could you please explain this stuff about folders? Should I:

create project_workspace folders and type the code following your tutorials? (but why downloading your code, then?..)
follow your tutorials and write the code without creating new folders?
read the code from the new-coder directory (downloaded from this repo) and write some other stuff?
use git, and branches, and merge stuff (and suffer)?
stop bothering and begin writing the code wherever I want?

Thanks :)

+1

The directory structure described above is intended to keep the code provided by authors separate from the code you write. The main reason one might want both is to be able to check that the end product actually works before bothering to read the tutorial. It can also be a nice sanity check if your version doesn't work at the end.
I realize I'm a bit late to the game here, but you might enjoy just coding along with the tutorial (they're pretty short) and then trying to do your own thing with the libraries you learned. That's a perfectly good way to go. Have fun!


[Bug] No Access via 5900 after Update

Current Behavior
get the following message when I open the Site: RFB 003.008

The following is displayed in the Docker log

Expected Behavior
Before the update, I always used port 5800 in the browser and was able to see the application there.
Steps To Reproduce
No response
Environment

OS: Synology DSM
OS version: 7.2.1 Update 3
CPU: AMD RyzenV1500B
Docker version: 20.10.23-1437
Device model: Samsung Galaxy Book 3 - Win 11
Browser/OS: Edge (Chromium)

Container creation

Container log
JDownloader
date,stream,content
2023/11/29 17:41:42,stdout,[xvnc        ]  Connections: blacklisted: <IP_ADDRESS>

2023/11/29 17:41:42,stdout,[xvnc        ] Wed Nov 29 17:41:41 2023

2023/11/29 17:41:42,stdout,[xvnc        ]  Connections: blacklisted: <IP_ADDRESS>

2023/11/29 17:41:40,stdout,[xvnc        ] Wed Nov 29 17:41:40 2023

2023/11/29 17:41:39,stdout,[xvnc        ]  Connections: blacklisted: <IP_ADDRESS>

2023/11/29 17:41:39,stdout,[xvnc        ]  Connections: blacklisted: <IP_ADDRESS>

2023/11/29 17:41:39,stdout,[xvnc        ] Wed Nov 29 17:41:39 2023

2023/11/29 17:41:38,stdout,[xvnc        ]  Connections: blacklisted: <IP_ADDRESS>

2023/11/29 17:41:38,stdout,[xvnc        ] Wed Nov 29 17:41:38 2023

2023/11/29 17:41:33,stdout,[xvnc        ]  ComparingUpdateTracker: (1:-nan ratio)

2023/11/29 17:41:33,stdout,[xvnc        ]  ComparingUpdateTracker: 0 pixels in / 0 pixels out

2023/11/29 17:41:33,stdout,[xvnc        ]  Connections: closed: <IP_ADDRESS>::56604

2023/11/29 17:41:33,stdout,[xvnc        ]  EncodeManager:          0 B (1:-nan ratio)

2023/11/29 17:41:33,stdout,"[xvnc        ]  EncodeManager:   Total: 0 rects, 0 pixels
"
2023/11/29 17:41:33,stdout,[xvnc        ]  EncodeManager: Framebuffer updates: 0

2023/11/29 17:41:33,stdout,[xvnc        ]               client?

2023/11/29 17:41:33,stdout,[xvnc        ]  VNCSConnST:  closing <IP_ADDRESS>::56604: reading version failed: not an RFB

2023/11/29 17:41:33,stdout,[xvnc        ]  Connections: accepted: <IP_ADDRESS>::56604

2023/11/29 17:41:33,stdout,[xvnc        ] Wed Nov 29 17:41:33 2023

2023/11/29 17:41:28,stdout,[xvnc        ]  ComparingUpdateTracker: (1:-nan ratio)

2023/11/29 17:41:28,stdout,[xvnc        ]  ComparingUpdateTracker: 0 pixels in / 0 pixels out

2023/11/29 17:41:28,stdout,[xvnc        ]  Connections: closed: <IP_ADDRESS>::56578

2023/11/29 17:41:28,stdout,[xvnc        ]  EncodeManager:          0 B (1:-nan ratio)

2023/11/29 17:41:28,stdout,[xvnc        ]  EncodeManager:   Total: 0 rects, 0 pixels"

Container inspect
No response
Anything else?
No response

blame on me. thx


Add a rake task to repopulate metrics

Adds a rake task rake etl:repopulate_master
This can be run across a range of dates and does the following:

Delete the metrics for each day.
Run the Etl::Master process for each day.

You can run the task as follows:
bundle exec rake etl:repopulate_master[2018-09-01,2018-09-30]

Or if you are using zsh:
bundle exec rake etl:repopulateviews\[2018-09-01,2018-09-30\]

 (3) Repopulate data from 1 Oct to 11 Nov 2018

Looks good, two small comments


Prevent  crash when the pad amount is negative

Description
If the start is less than the (count - lineLength) + 1 pad will be negative and cause the call to repeat on line 477 to crash with an uncaught RangeError. This caused Vite to crash when I was converting an app from Create React App to use Vite.
What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

This is my first time contributing to Vite (and nearly my first day using Vite), so please let me know if I've missed any steps or need to add anything.

LGTM, it would be great to have a link to the project for reference if it is public. Or an explanation of the setup at least if not.


SNOW-1360263: row_number window function raises a ValueError (index is not in list)

What version of Python are you using?
Python 3.10.4 (main, May 26 2022, 13:33:07) [GCC 4.8.5 20150623 (Red Hat 4.8.5-44)]

What are the Snowpark Python and pandas versions in the environment?
pandas==2.2.1
snowflake-snowpark-python==1.15.0

What did you do?
I'm trying to test a row_number() window function and it's raising a ValueError on this line:
_plan.py

from datetime import date

from snowflake.snowpark import Row, Window, Session
from snowflake.snowpark.functions import row_number, col

# create some test data
data = [
    Row(id=1, row_date=date(2024, 1, 1), value=1),
    Row(id=2, row_date=date(2024, 1, 1), value=1),
    Row(id=1, row_date=date(2024, 1, 2), value=1),
    Row(id=1, row_date=date(2024, 1, 2), value=100),
    Row(id=2, row_date=date(2024, 1, 2), value=1)
]

# create a local testing session and dataframe
mock_session = Session.builder.config("local_testing", True).create()
test_data = mock_session.create_dataframe(data)

# partition over id and row_date and get the records with the largest values
window = Window.partition_by("id", "row_date").order_by(col("value").desc())
df = test_data.with_column("row_num", row_number().over(window)).where(col("row_num") == 1)

results = df.collect()

What did you expect to see?
The results should include the following 4 records:

Row(id=1, row_date=date(2024, 1, 1), value=1),
Row(id=2, row_date=date(2024, 1, 1), value=1),
Row(id=1, row_date=date(2024, 1, 2), value=100),
Row(id=2, row_date=date(2024, 1, 2), value=1)

Instead of the above results, a ValueError is raised stating (1 is not in list).

Any update on this one? I am still encountering this issue on version 1.22.1


Deprecate span iterator?

While chasing a Regent vectorizer regression recently @magnatelee pointed out to me that the span iterator is now obsolete, being backed by the rect in domain iterator. Since this involves strictly more indirection, and since spans are one of the remaining places there is still some dim == 0 vs dim == 1 cruft, maybe it's worth deprecating them?
Even if we can't remove it (soon) from the C++ API, we've typically been more aggressive about the C API. Are there any C API users who are still using the span iterator today? If not I propose we just go ahead and remove it.

Yes please! :) It's marked deprecated in the C++ API and will be removed at some day in the future.

I've received no response, so I'll remove this right after we issue the release.

Removed in a22fe6f4fa8decf5455e00d81716f7ee4ef6772c.


Support Kafka Managed IO

Onboarding KafkaIO to be available via Managed API

R: @chamikaramj
R: @damccorm


Changes in the duplicated line are also applied to the parent line.

Hi!

I copy (duplicate) the string.
I make changes in the duplicate.
Changes in the duplicated line are also applied to the parent line.
Does not depend on the type of the selected command or plane (DCSFlightpanels_x64_4.4.30)

Checked it, it works fine on 4.4.30. Thank you!


Improve/consumer lti plugin

Purpose
Currently, an editor is not able to override values of is_automatic_resizing and inline_ratio fields when the plugin is based on a lti provider defined through RICHIE_LTI_PROVIDERS but it appears this feature could be useful is some cases.
From now, those fields defaults as follow:

Plugin linked to a lti provider

Use inline_ratio specified by user or try get value from lti provider settings and finally default to 0.5625
Use is_automatic_resizing specified by user or try get value from lti provider settings and finally default to True

Plugin not linked to a lti provider :

Use inline_ratio specified by user or default to 0.5625
Use is_automatic_resizing specified by user or default to True

New plugin options

Screenshot of a problematic use case
The video has not a 16:9 aspect ratio so the layout is broken and currently editor is not able to change the aspect ratio.

Screenshot with the suggested fix

Proposal

[x] Fix a localization issue related to rendering of inline_ratio percentage value
[x] Make inline_ratio and is_automatic_resizingfields overridable when the plugin relies on a lti provider.

@sampaccoud, I take your feedback into account. So from now, inline_ratio and automatic_resizing fields can be blanked then we use properties to retrieve the right value according the context.
Furthermore, in order to improve ux, I put those fields into a collapsed fieldset "Advanced settings".


Add Filters.matchPredicate helper, use it where appropriate.

This approach simplifies code and is generally faster, due to skipping
unnecessary dictionary lookups (see #2850).

👍

👍


Distribute via krew

Hi, could you consider listing you pgo client via krew?
https://krew.sigs.k8s.io/

Interesting idea, let me capture that in our backlog for discussion with the team.


Ui Issues - Associate/Add contact from address book

https://xd.adobe.com/view/e8259038-6a8a-4b4f-b88f-780e311438a5-bd76/screen/d08b1f1c-bdcc-4fa7-b7e6-f388848fb1a0/
Pls look at the next 3 screens

Verified this issue by Gourab in dev app v2.0.75(455)


Include conduit preview binary with Indexer release

We have a new make conduit target to build the conduit binary, this should be included in the release process.
There should be a new set of artifacts which start with conduit_.
Acceptance Criteria

Added to release pipeline
Binaries uploaded with rest of indexer binaries
Do we need deb/rpm creation?

Any testing required?
One potential test:

Use conduit to populate postgres.
Start an indexer reader pointed at conduit postgres
Run validator against this indexer reader to validate correctness

Jack, follow up with Will on testing requirements to ensure we feel comfortable with release binaries.

they included e2e and unit tests in the indexer to test the new binary too. So the new binary is covered with tests just like indexer.
Also there was a demo:

this is an in-house made tool
it connects to algod node
has a conduit.yml config file
runs in the background as a script
in config it has where to connect to algod node, where to export (for example slack), and what to filter (for example only transactions to a specific wallet id)

Closing ticket. Binaries for conduit were built and stored in our releases bucket. Any further work will be done through a different ticket


Support aiohttp 2.3.1 +

aiohttp 2.3 introduced explicit support for ssl.*Error
As well in upcoming 2.3.2 (bugfix release) will be private api changes for Connectors https://github.com/aio-libs/aiohttp/pull/2446
It guess aiosocks integration will be broken
@nibrag What about to move aiosocks connector to aiohttp itself?

Fixed https://github.com/nibrag/aiosocks/pull/21
@hellysmile

What about to move aiosocks connector to aiohttp itself?
aiosocks use python 3.5, aiohttp 3.4.2

master aiohttp uses python 3.5.3+

@hellysmile
In progress
https://github.com/aio-libs/aiohttp/pull/2539

@nibrag cool!


Explorer accessing inaccessible token list

Explorer accesses this on startup:
https://token-list.solana.com/solana.tokenlist.json

That endpoint is down.

@jacobcreech @jnwng is it time to take token-list.solana.com down, and delete it from Heroku?

How long has it been down? Probably time to move over to unified token list

No idea. I can't see logs older than 24 hours, so probably forever.

Haven't heard any complaints at this time on my side. Seems like it may be time to move

What do you mean by move? Move the app (I literally don't even know what we're talking about) or move on deleting it?

OH. Move over to the modern thing. Got it.


Error. Can't connect to RSS 1.0

Feed: https://nvd.nist.gov/feeds/xml/cve/misc/nvd-rss.xml
Grafana OSS v8.4.5 (4cafe613e)
No logs

@MeikelLP Thank you for submitting the issue. I will take a look and get back to you shortly.

@MeikelLP I am sorry for the delay. I added RSS 1.0 support and will release new version tomorrow.

Please let me know if there is anything else.

Hi, i get the same issue and if i browes to the url i see the first line like that:

whould this be fixed too?

@matzeeg3 Could you please send me the link to verify.

its hosted insite of a docker system i can open it for access but not pubic. how can i send it to you

@MeikelLP @matzeeg3 New version 1.5.0 released with support for RSS 1.0: https://github.com/VolkovLabs/volkovlabs-rss-datasource/releases/tag/v1.5.0
Please test and let me know if it works for you.
I will submit to Grafana repository review today.

@mikhail-vl Sadly I'm not an administrator at out company and I can't easily install a plugin from file. Would you mind updating the Plugin in the Grafana Store as well? https://grafana.com/grafana/plugins/volkovlabs-rss-datasource/

@mikhail-vl i just updated the datasource, still not working.

@MeikelLP Data Source was reviewed and updated in the Grafana Store: https://grafana.com/grafana/plugins/volkovlabs-rss-datasource/. Please let me know if it works for you.

@mikhail-vl  send the email. thanks a lot

@mikhail-vl It works fine :)


fix typo (with -> width)

Fix typo (with -> width) in error messages.

Closes because it's fixed at v<IP_ADDRESS>.


Native integration: show back navigation in Cordova & Electron apps

In the new app this will ideally be in the upper left corner of the title bar.
BoardZ1:

Von de Board.

Top left corner - he said.

Prio2 - the left corner ;)

Should be done soon.


feat(RPC): support a simple connection pool for RPC client.

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Sysbench benchmark shows we need a connection pool for RPC client.
Previous implementation just creates one connection for frontend node to compute node which is the bottleneck of our throughput during high concurrent batch query benchmark. CPU usage is limited to 30% for a 48C machine, after this PR, we can achieve almost 90%.

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)
#5386

I'm unsure whether it's the best practice to do this atop the client layer. Each Channel represents an HTTP2 connection in tonic, and it provides some built-in load-balancing mechanisms.
https://docs.rs/tonic/latest/tonic/transport/channel/struct.Channel.html#method.balance_list

Can it load balance the same end-point?

I'm unsure whether it's the best practice to do this atop the client layer. Each Channel represents an HTTP2 connection in tonic, and it provides some built-in load-balancing mechanisms.
https://docs.rs/tonic/latest/tonic/transport/channel/struct.Channel.html#method.balance_list

Can it load balance the same end-point?

I've not tested but I believe so. The Endpoint just describes the destination of the connection. We may have a try if it works well. :)

I'm unsure whether it's the best practice to do this atop the client layer. Each Channel represents an HTTP2 connection in tonic, and it provides some built-in load-balancing mechanisms.
https://docs.rs/tonic/latest/tonic/transport/channel/struct.Channel.html#method.balance_list

Can it load balance the same end-point?

I've not tested but I believe so. The Endpoint just describes the destination of the connection. We may have a try if it works well. :)

I'm unsure whether it's the best practice to do this atop the client layer. Each Channel represents an HTTP2 connection in tonic, and it provides some built-in load-balancing mechanisms.
https://docs.rs/tonic/latest/tonic/transport/channel/struct.Channel.html#method.balance_list

Can it load balance the same end-point?

I've not tested but I believe so. The Endpoint just describes the destination of the connection. We may have a try if it works well. : It doesn't work.

I'm unsure whether it's the best practice to do this atop the client layer. Each Channel represents an HTTP2 connection in tonic, and it provides some built-in load-balancing mechanisms.
https://docs.rs/tonic/latest/tonic/transport/channel/struct.Channel.html#method.balance_list

Can it load balance the same end-point?

I've not tested but I believe so. The Endpoint just describes the destination of the connection. We may have a try if it works well. :)

I have a try just now and It doesn't work.


Video player demo in Gallery doesn't test for network connectivity

The demo of video being landed in the Gallery (https://github.com/flutter/flutter/pull/13195), plays back videos from the network. If you run it and don't have a networking connection, nothing happens. It would be better if it checked for a connection, and somehow prompted that video requires one such.

cc @sigurdm

I think https://pub.dartlang.org/experimental/packages/connectivity can be used for this

Good idea!

Done in https://github.com/flutter/flutter/pull/13195


Extend middleware code sample to display var decla

Extend Localization middleware code sample to display the declaration of the variable 'supportedCultures'
When creating a new PR, please do the following and delete this template text:

Reference the issue number if there is one:
Fixes #Issue_Number

The "Fixes #nnn" syntax in the PR description causes
GitHub to automatically close the issue when this PR is merged.

Suggest reviewers if you know who should review the PR, using the GitHub Reviewers UI.

@AdrienTorris,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@AdrienTorris  Thanks! Nice improvement.


feat: add devlake-config update method

Signed-off-by: Yingchu Chen<EMAIL_ADDRESS>Pre-Checklist
Note: please complete ALL items in the following checklist.

[X] I have read through the CONTRIBUTING.md documentation.
[X] My code has the necessary comments and documentation (if needed).
[X] I have added relevant tests

Description

modified the definition of Connection and comment RenderAuthConfig
add devlake-config UpdateConfig

as we can see, when we changed the config, devlake processed requests from stream as expected

@warren830 Please take a look at this pr: #1181
I refactored the State/Resource in that pr.


Setting initial state property to undefined causes resulting proxy to drop the property

Optional type properties seem like they would be a fairly common requirement, but setting inital state to undefined causes the proxy to drop the property from it's internal state.
interface Store {
    a?: string;
}

const state = proxy<IState>({ a: undefined });

console.log(JSON.stringify(state));

// {}

This seems like an issue because a native js Proxy doesn't behave this way, instead keeping the undefined key and value in the target object.
This behaviour is also conflicting with the wiki page on persisting state using JSON.stringify(state) ( i.e. wiki link )
const state = proxy(JSON.parse(localStorage.getItem('foo')) || {
  count: 0,
  text: 'hello',
})

subscribe(state, () => {
  localStorage.setItem('foo', JSON.stringify(state))
})

When I attempt to persist / hydrate to and from storage, I commonly have missing properties, and instead have started to use null in place of undefined as that stays in the proxy, which feels like an unnecessary gotcha.
All in all, this seems like unintended behavio,ur and just wanted to raise it <3

https://codesandbox.io/s/zen-glitter-4e8wt5?file=/src/App.js
Looks like an issue with in.

Fixed by #440.
https://codesandbox.io/s/react-forked-owy91n?file=/src/App.js

Fixed by #440. https://codesandbox.io/s/react-forked-owy91n?file=/src/App.js

Thanks for the quick turn around on this! I see that code sandbox shows that JSON.stringify isn't serializing any undefined props, that still seems like an issue (well at least that was the core of this issue)

What do you mean?
undefined props don't seem to be serialized even without proxies.
$ node
Welcome to Node.js v14.18.0.
Type ".help" for more information.
> JSON.stringify({a:undefined})
'{}'

You're completely right 😅 sorry about that, I somehow remember doing tests and seeing undefined printed but that's just not part of the js spec so couldn't have happened. Ignore me 🙇🏻♂


docs: Add Breach to Barrier: Strengthening Apps with the Sandbox

Adds a blog post!

Thank you, @malept!

Thank you, all feedback implemented!


bug(mat-checkbox): Value gets removed when disable is present on Formcontrol

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Not 100% sure if this is a bug or intended feature but it causing me a bit of a headache.
I have a dynamically generated form to manage user permissions across a bunch of tools and has a few stipulations on how those permissions can be added or removed based on grantable/role permissions I want to be able to disable those mat checkboxes but still have a potential value for them however I noticed that if disable is added to the formcontrol when adding to a formarray the value is getting completely removed.
Is there a way around this, any help would be greatly appreciated.
Reproduction
reproduced stackblitz
https://stackblitz.com/edit/angular-wtakex?file=src/app.component.ts
Expected Behavior
Even if disable is added there should still be a value
Actual Behavior
Value is removed when disable is present on formcontrol
Environment

Angular: v15
CDK/Material: v15
Browser(s): chrome
Operating System (e.g. Windows, macOS, Ubuntu): macOS

This is how the forms module works, we don't have control over it in Material.


Fix bad error doing docker login in from non TTY

Signed-off-by: John Howard<EMAIL_ADDRESS>"Fixes" (well not really, as it's by design) https://github.com/docker/docker/issues/15272.
mintty does not present a terminal on stdin which can be used by console apps on Windows, hence will bomb out "The handle is invalid" when a Win32 console API is attempted to turn off echo on the console (actually a step early when it's trying to save the current state of the console, and performs a console API call on something which isn't a console). You see exactly the same type of error in Linux if you try something silly like cat | docker login.
This unifies things to give a clearer error message on both Windows and Linux by looking to determine if the stdin is a terminal in the docker login.
The solution to mintty not working for docker login is to not use it. The not being a terminal and incompatibility is well documented, and I added a bunch of links in the code for future reference.
The right solution is to use a shell which does provide a terminal such as cmd, powershell, lxss, git-for-windows bash (msys2 I believe). Just not mintty.
Screenshot below shows mintty above with new error message, and the experience under a few shells.
1 - mintty (fails)
2 - git for Windows 2.8 bash (works)
3 - PowerShell
4 - cmd

And for completeness, here's what you would see (without this fix) on Linux doing that cat | docker login example. With this PR, you'll get the same new error on Linux and Windows.

LGTM :+1:
/me just wondering about interactive logon vs login but meh :smiley:
/cc @thaJeztah

yeah, cannot come up with a better wording for that, so I guess it's ok
LGTM, but /cc @runcom to have one more set of eyes on this (there's quite a lot of logic in that function)

basically, here's before/after on linux;
Before this change;
echo "1234" | docker login
Login with your Docker ID to push and pull images from Docker Hub. If you don't have a Docker ID, head over to https://hub.docker.com to create one.
Username: inappropriate ioctl for device

echo "1234" | docker login --username=foobar
inappropriate ioctl for device

After this change;
echo "1234" | docker login --username=foobar
Error: Cannot perform an interactive logon from a non TTY device

echo "1234" | docker login
Error: Cannot perform an interactive logon from a non TTY device

One nit, otherwise LGTM

@vdemeester @thaJeztah Re interactive, yeah, I couldn't come up with anything better. However, I felt it was important to use the word interactive as you can still login using command line parameters. Just not interactively.

@jhowardmsft make sense :+1:

LGTM

Another weird gccgo error:
17:12:11 PASS: docker_cli_help_test.go:153: TestHelpExitCodesHelpOutput.pN52_github_com_docker_docker_integration_cli.DockerSuite	0.421s
17:12:26
17:12:26 ----------------------------------------------------------------------
17:12:26 FAIL: docker_cli_help_test.go:15: TestHelpTextVerify.pN52_github_com_docker_docker_integration_cli.DockerSuite
17:12:26
17:12:26 docker_cli_help_test.go:137:
17:12:26     c.Fatal(err)
17:12:26 ... Error: Should not have a blank line on ["cp"]
17:12:26
17:12:26
17:12:27
17:12:27 ----------------------------------------------------------------------

Doesn't look related, so merging.


add removeChangeListener api

Codecov Report

Merging #1462 into master will increase coverage by 0.07%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1462      +/-   ##
============================================
+ Coverage     48.37%   48.44%   +0.07%
- Complexity     1798     1800       +2
============================================
  Files           379      379
  Lines         10928    10930       +2
  Branches       1121     1121
============================================
+ Hits           5286     5295       +9
+ Misses         5254     5245       -9
- Partials        388      390       +2

Impacted Files
Coverage Δ
Complexity Δ

...rip/framework/apollo/internals/AbstractConfig.java
81.01% <100%> (+0.16%)
49 <1> (+1)
:arrow_up:

...ervice/service/ReleaseMessageServiceWithCache.java
85.71% <0%> (-1.2%)
24% <0%> (-1%)

...work/apollo/biz/message/DatabaseMessageSender.java
66.66% <0%> (+16.66%)
8% <0%> (+2%)
: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 ee4cfcc...a18ff9d. Read the comment docs.

Coverage increased (+0.09%) to 52.013% when pulling a18ff9d8ad4cc6bd0ebcc3c6a487c666add47cec on nobodyiam:remove-change-listener into ee4cfccc9e5718998835ff3a6d0835b547ef4d1c on ctripcorp:master.


Feature: get keys with prefix

Hi,
Would it be possible to add the ability to fetch a list of keys with a given prefix? It could be an optional parameter to the getAllKeys function.
For example, I would like to do something like this:
store.set(“foo”, “bar”)
store.set(“user.email”<EMAIL_ADDRESS>const userKeys = store.getAllKeys(“user”)
// [“user.email”]

Hey! Thanks for the feature request, but this is not possible to do on the native side because there is no way to filter keys in MMKV's request.
It's quite a light method, so I think the filtering should just be done on the JS side for better intention awareness by being more explicit in code.
Let me know what you think!

@mrousavy ACK. Thanks for the explanation. The getAllKeys() method returns a sorted array so I am basically just using sortedIndex from Lodash which IIRC is just a binary search.

Sounds good, yea. I don't think that'd be a perf. concern 👍


Use DeriveEntityModel without having a Relation enum

Summary
Right now, when I use the DeriveEntityModel Derive Macro, I need to have a enum Relation, which derives DeriveRelation.
Otherwise I get the error: cannot find type Relation in this scope.
Motivation
The reason why I would like to use the DeriveEntityModel macro without having the Relation enum is because I don't use the Relation enum.
I like to define my relations like this:

impl Related<super::homes::Entity> for Entity {
    fn to() -> RelationDef {
        Entity::belongs_to(super::homes::Entity)
            .from(Column::HomeId)
            .to(super::homes::Column::Id)
            .into()
    }
}

So there really is no need and it works... Right now I just have an empty Relation enum and that just feels a bit useless and adds unnecessary clutter.

Hey @LasseRosenow, welcome! Take the entity below as an example, you're saying that you want to get rid of the empty Relation Enum completely?
use sea_orm::entity::prelude::*;
use serde::{Deserialize, Serialize};

#[derive(Clone, Debug, PartialEq, DeriveEntityModel, Deserialize, Serialize)]
#[sea_orm(table_name = "posts")]
pub struct Model {
    #[sea_orm(primary_key)]
    #[serde(skip_deserializing)]
    pub id: i32,
    pub title: String,
    #[sea_orm(column_type = "Text")]
    pub text: String,
}

#[derive(Copy, Clone, Debug, EnumIter, DeriveRelation)]
pub enum Relation {}

impl ActiveModelBehavior for ActiveModel {}

This all of course would not be necessary if the Relation enum derive macro would work as documented.

See this screenshot from the documentation says that the whole first code block could be replaced by the one at the bottom which is using the macro, but I could not reproduce this.
When I only used the lower one with the macro, I still had to write the related<> traits as well. Is this a bug, or is the documentation wrong about that the enum with the macro replaces everything above?
Because if I could specify all my relations just with the Relation enum using macros, that would be cool as well and a good alternative to getting rid of the enum and doing everything with the related<> trait and there would be less boilerplate as well


Missing tag for version 0.3.18

v0.3.18 and v0.3.19 has been pushed: https://github.com/rust-lang/socket2/tags.


Reduce coupling to date.h by calling C++17 chrono functions

High Level Overview of Change
Reduce coupling to date.h by calling C++17 chrono functions
Type of Change
Minor cleanup.

Rebased to 1.7.0-rc4.

Rebased to 1.7.0.


Sound works ~14 times and then stops working

Hi man, awesome job with the library, it seems like it is the only one that actually does work with android and it's exciting to see possible manipulations with music via processing on android.
Nevertheless, there is some sort of a glitch that prevents the music from playing once it gets repeated, in my case around 14 times. I saw someone having the same issue on the processing forum: https://forum.processing.org/two/discussion/19302/using-android-mediaplayer-to-play-sound
Although in that case it was 7 times.
May I ask whether you would know where does the error come from or whether it is easy to fix? As processing does not throw any error whatsoever which seems suspicious.
Once again, thanks mate for your great work!

Hello ! The exactly same thing is happening to me

(Using Processing for Android, similair to Java)
It only works 1 time for me if I load the SoundFile in setup.
It works exactly 14 times if I load the SoundFile right before I play the sound.
This indeed only seems like the only library that works on Android, I've tried many.
It would be great if you could fix it!

Also, it doesn't matter if you use different files. There's a maximum of 14 playbacks. If you have 2 files, it doesn't go to 28.

code is something like:
music.play();
if(mousePressed) music.stop();'
And repeating that( of course only executing play() and stop() once, only when state changes)

So, i´ve decided to change library for playing sounds in processing for android.
This thread give me the solution :
https://discourse.processing.org/t/a-working-sound-playing-app-sketch-works-as-of-may-27-2018/451/18

If I load the resource again, it is limited to 14 plays.
If I don't load the resource again, it is limited to 1 play.
So that solution doesn't work

But yeah, MediaPlayer might work better. It's a lot more complicated though, so it would be nice if this library gets fixed, because of it's simplicity

@Aurelian24 Does this solution work for Android as well? Seems like it, looking forward to trying it out :)
In case of this library: I somehow managed to stuff three tracks into an app and miraculously the sounds would not crash. But if I changed the sounds to some other configuration with the same names or even the same sounds but converted differently, the sounds would... crash.
I have no idea what's the root of the problem but that must be an eerie one.


Preserve filesystem permissions of artifacts

closes #8

Hi @ellisto, thanks for pointing out that the zipfile contains the unix permissions; I was unaware.
A few points:

This adds yet another archive lookup by name in the loop; we should be using the ZipInfo APIs.
I'm a little hesitant to send the file mode bits straight from the archive to chmod; I feel like we should ignore bits like setuid

I included your commit (slightly modified for variable names) in #10. Care to take a look?

#10 looks great to me! Thank you!


bridge.external_interfaces not accepting vlan interfaces

Environment
I'm running Ubuntu 16.04.3 LTS with kernel Linux ubuntu 4.4.0-101-generic with packages from the ppa
driver: lxc
driver_version: 2.1.1
kernel: Linux
kernel_architecture: x86_64
kernel_version: 4.4.0-101-generic
server: lxd
server_pid: 6429
server_version: "2.20"
storage: zfs
storage_version: <IP_ADDRESS>-0ubuntu16
Issue description
When i create a unconfigured vlan interface via vconfig add ens3 30 i can't add that to a lxd  bridge by editing the network and adding it to 'bridge.external_interfaces'. I can add normal interfaces though. When adding a vlan interface i get the following error on saving:
Config parsing error: Invalid interface name 'ens3.30' Press enter to open the editor again
I can add the virtual interface to the bridge using brctl: brctl addif bridge_name ens3.30, which works fine but is not a persistent solution.
I would expect to be able to add vlan interfaces to lxd managed network bridges.
Steps to reproduce

Create network bridge
Create vlan interface
Try to add vlan interface to bridge.external_interfaces via lxc network edit bridge_name

Observed similar behavior in lxc network create for tunnel interfaces just while ago :-)
example:
# lxc network create vx339 tunnel.vx339.protocol=vxlan tunnel.vx339.group=<IP_ADDRESS> tunnel.vx339.id=339 tunnel.vx339.interface=xgb0.1000
error: Interface name contains invalid characters

xgb0.1000 is vlan interface on xgb0
When i change it to non-vlan interface everything is ok, except mtu is set to 1400 and not to mtu of the parent interface (9000 in my case, so right value have to be mtu minus 50 for header)
Should i open another issue?

It's likely the same check so same issue is fine.


[BUG]: Defaulting to Ethernet as a sort of lan bridge rather than wifi

Program version
3.4.0
Operating System
Windows
Type
GUI
Service
Crunchyroll
Command used
N/A
Show ID
All Shows
Episode
Download All
Console Output
N/A

Additional Information
I previous wrote a bug, but closed it -- it was that it was defaulting to a CH IP address, which was odd to me -- only for me to realize it was passing through my switch, and exiting out my other pc that had a VPN on -- rather than using onboard wifi, for some reason. I have to disable ethernet in ordered for it to function properly, or else it's slow, wrong region, etc.

This tool will use the default connection on your device. There is no way for us (as far as I know) to change that. You might need to change the default connection settings for this application/node.js.


HTML parser - capture with XPath

Would be nice to have in the HTML parser the possibility to capture data using XPath
thx

This is a pretty old ticket... But it's on the way now. It will be available on 1.5.14.


Bug: Scrolling Gets Weird on Field Selects

Issue Summary
Device: iPhone 6
In the admin interface, if you open up a page, like menu, or general, when you select a text field and try to scroll, it is very eratic, the cursor moves with the page, and bottom of page is cut off.

Steps to Reproduce

Log into the admin interface on an iPhone\
Open a any setting menu, like general
Select a text field, and then try to scroll

Technical details:

Ghost Version: Master
Client OS: Mac 10.10
Server OS: Mac 10.10
Browser: Safari Mobile
Database: SQLite

Looks like a bug in safari when using fixed position elements to me


Having issues regarding cloning a single branch of a repository

Where: Script 'C:\flutter\flutter\packages\flutter_tools\gradle\flutter.gradle' line: 1005 * What went wrong: Execution failed for task ':app:compileFlutterBuildDebug'. > Process 'command 'C:\flutter\flutter\bin\flutter.bat'' finished with non-zero exit value 1 * Try: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights. * Get more help at https://help.gradle.org BUILD FAILED in 2m 34s ********************************************************* WARNING: This version of flutter_tts will break your Android build if it or its dependencies aren't compatible with AndroidX. See https://goo.gl/CP92wY for more information on the problem and how to fix it. This warning prints for all Android build failures. The real root cause of the error may be unrelated. ********************************************************* Exception: Gradle task assembleDebug failed with exit code 1

Hi @Coder-Pritam,
Can you please provide your flutter doctor -v and flutter run --verbose logs, and a minimal complete reproducible code sample?
Thank you


Implementing Bigtable ColumnRangeFilter.

@tseaver I gave this the same treatment as in #1309, PTAL.

LGTM


Bring turbo back to business form

Fixes #259

Pull request checklist

[x] Your code contains tests relevant for code you modified
[x] You have linted and tested the project with bin/check

Please see the below clip to see how this will work.
https://user-images.githubusercontent.com/17806296/155901667-eb665605-d59a-4cc5-90c8-e1cd50f4efbe.mov

Quick glance and I'm wondering if it is possible to move the ID on the form to the existing line of code. That will keep the diff much smaller and be easier to reason about.

I will add a format based routing for both success and failure.
You are right about the form, I started with create-business-form and pivoted to simply business form due to the file partial. Will update as such.

Nice! Looking forward to seeing the results.

@joemasilotti Updated

Added tests for turbo and regular form submit. Also validated by manually disabling javascript

@joemasilotti  This one is ready for review again when you get a chance.

@joemasilotti Can you please validate the stripe flow? I am having some stripe issues testing it. Probably making some config mistake in Stripe

Ah, that's on me! I forgot to add the new price_ids structure for credentials. Check out the README on main to get the update.

@joemasilotti Thank you for that. I am able to get it to work now. Transactions are working fine without any issues.
Now the business owner is getting redirected to the pricing instead of the cold_messages_controller. It appears that the subscription is not getting activated for the business. I don't know enough about Stripe and Pay integration to be able to troubleshoot this effectively.

Closed in favor of #483. Turns out, the flow for redirecting to Stripe was removed a little while back!
Sorry for letting this sit for so long – I appreciate the contribution. And I'm glad we have this available if we need to follow the same flow again in the future.


Added some models and services

Moved to lots of classes to org.interledger.core.ilp.ledger package

Thanks @sappenin - missed the review somewhere!


Clarify the meaning of the pre-alpha status

Someone on the Kubernetes slack channel asked to clarify on what pre-alpha status means and if this means the operator it creates is not ready for production. I think we can add a note somewhere to clarify that the operator it generates is ready for production.

Closing as we moved over from pre-alpha.


Collation Support?

It would be nice, if sqlkata would support collation in WHERE and ORDER BY clauses.
For example, it should generate something like this:
SELECT [Column] FROM [Table]
WHERE [CallName] COLLATE Cyrillic_General_CS_AS = @CallName

Currently there is no plan to support engine specific expressions,
so you can build your own extension method for that (something similar to the below), I will close this for now and watch for other developers feedback, feel free to reopen if you have any input.
public static WhereCollate(this Query query, string column, string collation, object value)
{
  return query.WhereRaw("... = ?", value);
}