SICP JS: Rework program snippets

Description

This PR is dependent on source-academy/sicp#545.

Fixes #1830.
Reworked how snippet dependencies are hidden. Dependencies are now hidden by code folding instead. This solves the environment model not working with dependencies bug and the bug where some program snippets cannot be run when dependencies are hidden because the prepend was evaluated in a different scope to the main body.

In the image above, taken from the third program snippet in section 3.2.3, lines 2 to 15 are hidden. A user can click on the yellow button to reveal dependencies.

With this solution the only way to hide dependencies again is to close and reopen the snippet.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Code quality improvements

How to test

Ensure correct lines are highlighted when using env visualiser with and without code folding.
Ensure snippets with dependencies are hidden correctly and can be revealed by pressing the yellow button. The env viz should also work correctly when dependencies are hidden.

Checklist

[x] I have tested this code
[ ] I have updated the documentation

Pull Request Test Coverage Report for Build 974181707

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.03%) to 30.031%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/pages/playground/Playground.tsx
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/pages/playground/Playground.tsx
1
1.78%

src/pages/sicp/subcomponents/SicpWorkspaceContainer.tsx
1
11.76%

Totals

Change from base Build 974170028:
-0.03%

Covered Lines:
3199

Relevant Lines:
9806

💛  - Coveralls

Pull Request Test Coverage Report for Build 975272159

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.03%) to 30.031%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/pages/playground/Playground.tsx
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/pages/playground/Playground.tsx
1
1.78%

src/pages/sicp/subcomponents/SicpWorkspaceContainer.tsx
1
11.76%

Totals

Change from base Build 974255973:
-0.03%

Covered Lines:
3199

Relevant Lines:
9806

💛  - Coveralls

Pull Request Test Coverage Report for Build 975272159
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.03%) to 30.031%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/pages/playground/Playground.tsx
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/pages/playground/Playground.tsx
1
1.78%

src/pages/sicp/subcomponents/SicpWorkspaceContainer.tsx
1
11.76%

Totals

Change from base Build 974255973:
-0.03%

Covered Lines:
3199

Relevant Lines:
9806

💛  - Coveralls


Java client should support using custom HttpClient for testing

This will open up the test-ability of the client by allowing us to test code paths that depend on network calls that we will then be able to simulate.

https://github.com/aspnet/SignalR/commit/713e224591262ba52f871ae09ec66f95018e607b


Library Conversion: OpenSeq2Seq

OpenSeq2Seq is a toolkit for distributed and mixed-precision training of sequence-to-sequence (seq2seq) models, created by the researchers at NVIDIA.
OpenSeq2Seq is built using TensorFlow and provides all the necessary building blocks for training encoder-decoder models for neural machine translation, automatic speech recognition, speech synthesis, and language modeling. For documentation and TensorFlow 1.x installation instructions, check here.
The purpose of this ticket will be to migrate OpenSeq2Seq to the TF 2.0 API.

Convert all existing OpenSeq2Seq code.
Run unit tests.
Ensure processes are well understood.

https://github.com/tensorflow/addons/pull/72

Hi @dynamicwebpaige @qlzh727 @borisgin
Is Nvidia's OpenSeq2Seq still being migrated to TF 2.0 ?
I have re-implemented our AV speech toolkit for this latest version of tensorflow using tensorflow/addons/seq2seq (thanks @qlzh727 for all your help so far), yet I am facing a memory leak issue. Was wondering if I could have a look at some code samples in OpenSeq2Seq.
Thanks,
George


GEOMESA-2863 Re-throw scan errors from separate threads

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>

Merged as 5afc69759f86694aa8f7426be0864231d0a8a104


A way to disable vscode debugging logs

When using the debug option with external terminal such logs appear: "Debugger attached." "Waiting for the debugger to disconnect..." which polute the logs and doesn't deal well with clusters specially when the master process fails or has an error.

I think these messages come from node.js and not VS Code (or js-debug).
So I don't think that we can do anything here.
But @connor4312 can give you the definitive answer.

This is not possible to disable -- please file an issue for Node.js if you'd like to see this https://github.com/nodejs/node


FaceTec - Light mode and Dynamic dimming mode customizations [Native only]

FaceTec UI now detect lights level in the room by the device camera and uses device API to lighten / darken the screen. Also:

it changes design by filling the whole screen white/black. This couldn't be undone
it provides an option to customize the rest of UI for those particular modes

This isn't working on the all devices.
@vldkh was able to reproduce it on Windows 10 PC with webcam andGoogle Chrome v95
More info regarding those modes and how UI should look exactly you could find here:
https://dev.facetec.com/ui-customization#low-light-mode
https://dev.facetec.com/ui-customization#dynamic-dimming-mode
For now we made an attempt to override those modes on web. But we won't be able to do it on Android
What i suggest to do

[ ] remove our overrides @TomasGonzalez
[ ] check how it looks in comparison to the stock screen shots @patpedrosa  @vldkh
[ ] some app UI is shown over the popup - need to fix this @TomasGonzalez
[ ] check & fix the other app's UI @TomasGonzalez
[ ] close button isn't placed as on the screenshots - neet to fix this @TomasGonzalez
[ ] as the colors are lighten/darken we may need to think do we need the separate customizations for both modes or they're already looking fine @patpedrosa
[ ] design customization for the low light mode @patpedrosa
[ ] design customization for the dynamic dimming mode @patpedrosa
[ ] refactor our code to support applying customizations for the different modes (all platforms - web ios android) @johnsmith-gooddollar
[ ] fill & apply customization object for the low light mode (all platforms - web ios android) @TomasGonzalez
[ ] fill & apply customization object for the dynamic dimming mode (all platforms - web ios android) @TomasGonzalez

For now we need just 2,3,4,5 for the native only @TomasGonzalez @vldkh

@vldkh
please provide screenshots of how it looks vs regular mode
and @patpedrosa should provide input on what should be adpated

@sirpy @patpedrosa
Normal mode:

Black mode:

On the black mode, let's make the copy and the close icon white for readability.
"Get ready for your video selfie
Please frame your face in the small oval, then the big oval"
Also, I've noticed we've been using a lot of "First Letter Capitalization" across the app, including here and in feed cards. Let's remove that. We should use sentence case capitalization (or use user's input in cases like the feed).


Add harvester node driver to rancher

Problem:
Will need to add the default harvester node driver to the rancher server.
Solution:
Add default harvester node driver to the rancher server.
Related Issue:
https://github.com/rancher/harvester/issues/513
Test plan:

Check if the Harvester node driver exists in the Rancher and the UI is loaded successfully.

updated, thanks.


Misc fixes and tests.

Description

Added GET and HEAD /{index}/_source/{id} returning 404.
Added tests for GET /{index}/_refresh.
Added tests for POST /{index}/_mget.
Fixed all refresh options to allow boolean and string.
Added tests for POST /{index}/_update/{id}.

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

@Xtansia @nhtruong please?


Release yaml files

Hi,
Thank you for sharing your interesting work. I was wondering if there is an expected date on when you will be releasing yaml files regarding anything other than CORD? I want to reproduce the experimental results in my environment.

Hi, thank you for your interest. We will update it, but let me get a quick answer first.
The trained model configurations (e.g., input resolution, max_length, etc) are currently available at config.json; for example, see docvqa model's configuration. Plus, more details of important setups are also available at Appendix A.5 (https://arxiv.org/abs/2111.15664).
I will notify you when the update is done :)

Hi @rtanaka-lab, the files are updated in 3c6d818. Hope this helps :)


Audio recorded affected on some devices when 'configuration.providesAudioData' = true is used'.

When using:
let configuration = ARWorldTrackingConfiguration()
configuration.providesAudioData = true
sceneView.session.run(configuration)
On some devices (like iPad pro and iPhone 12) the volume for recording using AVFoundation's AVAudioRecorder is dramatically reduced. If anyone knows a solution for this I'd be very grateful.

There doesn't seem to be an answer yet and the issue is not related to SCNRecorder. Closing.


Unbalanced {{ breaks syntax highlighting

Unbalanced {{ breaks syntax highlighting
Fixes #419
Signed-off-by: azerr<EMAIL_ADDRESS>

mmm why are the colors changing?

I cannot reproduce it, could you shar eyour template content please.

Highlighting for double brackets seems to work across lines:

I assume this happens on the VSCode HTML TextMate side, but if an HTML tag is nested in double brackets, it will currently highlight as a normal tag.

This was caused by a vscode setting, but I lost it, can't reproduce anymore. nevermind.

This was caused by a vscode setting, but I lost it, can't reproduce anymore. nevermind.

No problem,do you think we can merge this PR?

@AlexXuChen @fbricon do you think we can merge this PR?

@AlexXuChen @fbricon do you think we can merge this PR?

I believe so, the behaviour is working as intended

Thanks guys


ci(renovate): migrate from config option excludePackageNames

:tada: This PR is included in version 2.2.26 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add notice about SCSS-Lint

scss-lint's readme starts with:

NOTICE: Consider other tools before adopting SCSS-Lint

The Sass core team is now building Sass in Dart instead of Ruby, and will no longer be maintaining the Ruby implementation unless a maintainer steps up to help. Since the SCSS-Lint project relies on the Ruby Sass implementation, this means it will eventually not support the latest Sass features and bug fixes.

Two alternatives worthy of consideration are stylelint and sass-lint, with sass-lint having closer feature parity to SCSS-lint.

The SCSS-Lint project will continue to accept pull requests and provide basic support on the issue tracker.

I suggest we add a short note next to the list entry:

scss-lint - Configurable tool for writing clean and consistent SCSS. Not recommended, consider stylelint instead.

Good idea!
How about we add deprecated instead? like:

scss-lint - Configurable tool for writing clean and consistent SCSS. (deprecated)

What do you think?

Sure, I'm make a PR later today :)


Android Exoplayer Project Causes DexException

Whenever I try to integrate the android-exoplayer project into my React Native project, I can't perform a ./gradlew assembleDebug which is required by react-native run-android
I get the following exception:
:app:compileDebugSources
:app:transformClassesWithDexForDebug
Running dex in-process requires build tools 23.0.2.
For faster builds update this project to use the latest build tools.
Dex: Error converting bytecode to dex:
Cause: com.android.dex.DexException: Multiple dex files define Landroid/support/v4/app/NavUtilsJB;
    UNEXPECTED TOP-LEVEL EXCEPTION:
    com.android.dex.DexException: Multiple dex files define Landroid/support/v4/app/NavUtilsJB;
        at com.android.dx.merge.DexMerger.readSortableTypes(DexMerger.java:596)
        at com.android.dx.merge.DexMerger.getSortedTypes(DexMerger.java:554)
        at com.android.dx.merge.DexMerger.mergeClassDefs(DexMerger.java:535)
        at com.android.dx.merge.DexMerger.mergeDexes(DexMerger.java:171)
        at com.android.dx.merge.DexMerger.merge(DexMerger.java:189)
        at com.android.dx.command.dexer.Main.mergeLibraryDexBuffers(Main.java:502)
        at com.android.dx.command.dexer.Main.runMonoDex(Main.java:334)
        at com.android.dx.command.dexer.Main.run(Main.java:277)
        at com.android.dx.command.dexer.Main.main(Main.java:245)
        at com.android.dx.command.Main.main(Main.java:106)

:app:transformClassesWithDexForDebug FAILED

FAILURE: Build failed with an exception.

Steps to reproduce:

Create a new React Native project react-native init Foo
Add "react-native-video": "git://github.com/react-native-community/react-native-video.git#master" to package.json dependencies
Add compile project(':react-native-video-exoplayer') to android/app/build.gradle
Add

include ':react-native-video-exoplayer'
project(':react-native-video-exoplayer').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-video/android-exoplayer')

to settings.gradle
5. Run react-native run-android and get exception.
What can be done to fix this?

I managed to solve a similar issue by adding compile 'com.android.support:support-v4:25.3.1' in dependencies section of build.gradle. See: https://github.com/google/ExoPlayer/issues/2744#issuecomment-297858018.

The fix from @perttuim fixed this for me. I'm not seeing this issue on 2.1.1 as we have upgraded ExoPlayer and apparently they fixed this.


Testing, please ignore. (qbzjqc)

This is a bug bounty test. Please ignore.. Please DO NOT APPROVE THIS! (qbzjqc)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.qbzjqc vpeku seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Nostalgy not working with Thunderbird 78

The add-on is still enabled by default after the upgrade, but the shortcut keys do not work anymore.

Duplicate of #85

Duplicate of #85

Yes, thank you for pointing that out to me. Closing this one...


late flushing

I think, this is wrong.
https://github.com/elastic/elasticsearch-perl/blob/5659396a3c6fcad01a85ecf92ff23804cf6eefe1/lib/Search/Elasticsearch/Client/7_0/Bulk.pm#L39-L50
You push something to the buffer and flush if it's greater than the limit. But shouldn't the limit be the limit, meaning: Not more than that?
Without testing I think, a correct way would be:
    while (@_) {
        my @json = $self->_encode_action( splice( @_, 0, 2 ) );

        my $size = $self->_buffer_size;
        my $added_size = 0;
        $added_size += length($_) + 1 for @json;

        $self->flush
            if ( $max_size and $size+$added_size > $max_size )  # Would the new size exceed the max_size
            || ( $max_count and $self->_buffer_count >= $max_count ) # is the current count equal or higher than the max_count?
            || ( $max_time  and time >= $self->_last_flush + $max_time );

        push @$buffer, @json;

        my $size = $added_size + $self->_buffer_size;
        $self->_buffer_count( $self->_buffer_count + 1 );
        $self->_buffer_size($size);
    }
    return 1;

The flush is needed to fill the buffer. We continue to push in the buffer until we reach this limit. I think this is correct.


feat: add WebSocket class interceptor

Closes #236
Closes #484
See this branch's README.md for the feature documentation.

Changes

Adds a WebSocket interceptor based on the globalThis.WebSocket class.
Introduces the client and the server APIs to represent the incoming client connection and the original server connection respectively in the connection listener of the interceptor.
Adds the required infrastructure of the interceptor/client/server/transport classes that describe the generic class and also the WebSocket class-relevant implementations of them, such as WebSocketClassClient, WebSocketClassServer, WebSocketClassTransport.

Todo

[x] Tests only: The ws module we use for testing doesn't pass the instanceof check on dispatched events in Node, causing some tests to fail (https://github.com/websockets/ws/issues/1818).

[x] Once this is fixed, unskip the tests at test/modules/WebSocket/WebSocketClass/exchange/websocket.server.connect.test.ts
This was caused by JSDOM. It polyfills global Event.

[x] Write WebSocket compliance tests.

[x] Default state
[x] Events
[x] Methods (.send() and .close() and its input)
[x] Setters (.onopen and etc)

[x] Some third-parties also add origin on the MessageEvent from the socket. Check if the interceptor should do that too (inspect the original WebSocket event).
[x] Emit the error event correctly. It's emitted in the following scenarios:

[x] Connection was closed with a > 1000 <= 1015 (https://www.rfc-editor.org/rfc/rfc6455#section-7.4.1)
[x] When sending data and being unable to buffer it anymore (the buffer is full, the connection is marked as full and closed). Such connection closures must "fire an event named error" (https://websockets.spec.whatwg.org/#dom-websocket-onerror).

I'll skip this; not sure we can implement it reliably.

[x] Close the actual server connection whenever the original (mocked) client is closed.
[x] Consider removing WebSocketClient#emit. It's non-standard.
[x] Support custom encoder/decoder for (a) JSON transfer; (b) to support custom event format (like SocketIO).

This may be out of scope. Consider this carefully.
Use bindings instead. I've implemented the socket.io-binding.

[x] Consider migrating to EventTarget-like public API: client.addEventListener().

Third-party support

[x] SocketIO

Proposed to expose higher-level encoding/decoding utilities https://github.com/socketio/socket.io-parser/issues/129

The API itself is complete. One thing remaining is proper test coverage, specifically addressing the bug in the ws module (the Event instance check) or migrating to a different server package to use for testing.

Released: v0.26.0 🎉
This has been released in v0.26.0!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


Add integration tests using Mountebank

We already have HTTP mocking tests using WireMock; let's add some end-to-end integration tests using Mountebank.

Closing this in favour of Snowplow Micro testing in future release


⚠ Turnos 24x7 has degraded performance

In f236f18, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 971 ms

Resolved: Turnos 24x7 performance has improved in e1df75e.


problem: Jacoco doesn't work on Java 15 yet

Codecov Report

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

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage     89.61%   89.61%
  Complexity      596      596
=========================================
  Files            75       75
  Lines          1926     1926
  Branches        302      302
=========================================
  Hits           1726     1726
  Misses          109      109
  Partials         91       91

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 69fb859...234b71b. Read the comment docs.


Wrong link to GetURL

Link to GetURL is currently:
https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/methods/devenv-geturl-method
That page does not exist.
Instead this page works
https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/methods-auto/system/system-geturl-clienttype-string-objecttype-integer-recordref-boolean-method

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

ID: e88aa7e3-9b56-f8ef-b63d-c31b7098539a
Version Independent ID: fb3c85ff-43fa-d496-6ef2-fdad0a24e23d
Content: SOAP and ODATA Web Services - Business Central
Content Source: dev-itpro/webservices/web-services.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: solsen

Thanks for your feedback, the topic contained an old link. This is fixed with the next refresh of the docs. Thanks.


StorageLoader: Patch JSON Path file for the com.callrail/call_complete event

Thanks! Putting this into the Blocked schemas milestone, as we cannot unfortunately merge this into master without breaking pipelines in the wild...

Migrated to https://github.com/snowplow/iglu-central/issues/533


tarball created by make dist is missing several files

On the develop branch, creating a tarball using the make dist target is missing several files, including for example ssl_compat.h
Run:
./autogen.sh
mkdir build
cd build
../configure --without-server
make dist
tar -zxvf voms-2.1.1.tar.gz
diff -qr voms-2.1.1 ..

will show for example several missing header files and also some documentation files (incl. manpages).

PR welcome :-) but I'll have a look myself soon.

@msalle please check

Hi, thanks!
I still see some missing files:

some .h and .c files (for example myproxycertinfo.[ch]), but hard to say which ones are needed.
I do think you need VOMSAC.wsdl (necessary for the server part currently) though: make distcheck fails on it.
some manpages, in particular voms.8 and voms-install-replica

* some .h and .c files (for example `myproxycertinfo.[ch]`), but hard to say which ones are needed.

myproxycertinfo.[ch] are not needed any more and I've removed them. Any other?

* I do think you need `VOMSAC.wsdl` (necessary for the server part currently) though: make distcheck fails on it.

I see it in the tarball. make distcheck for me (ubuntu 22.04) fails because there are three remaining files after uninstall.

* some manpages, in particular `voms.8` and `voms-install-replica`

I see voms.8 in the tarball. About voms-install-replica I'm trying to understand if it's still relevant or not; I see other two scripts, which are installed: voms_replica_{master,slave}_setup.sh


Allow compilation of HEAD_REF-only ports in manifest mode

Requires https://github.com/microsoft/vcpkg-tool/pull/24 to work, otherwise it does nothing.
Sort of addresses https://github.com/microsoft/vcpkg/issues/12504. More specifically, supports my desired use case:

I have a registry with very, very frequently touched package. It does not make sense for me to version this package in any way, I am the only one using it, and I only ever want to use the latest version. Therefore, I have it available on my vcpkg registry as a HEAD_REF-only port.
I have a project that consumes previously mentioned package, as well as many others from the official vcpkg registry. I do not want --head behavior for all those packages.
I want to use manifest mode, which means I cannot currently specify per-port build options (here's where #12504 comes into play)

With the given conditions, it is impossible to build my project. vcpkg will run the manifest install step, then hit my --head-requiring port, and abort.
I cannot use --head, because not all my vcpkg dependencies are built from source (discord-game-sdk). I also have no desire to be living on the bleeding edge for any third-party dependencies anyway.
Theoretically, this change should not break any existing behavior (unless people were relying on vcpkg not working in this situation). It takes a previously non-working scenario (HEAD_REF-only port, manifests, and no --head) and makes it work in a (in my opinion) reasonable way.

What does your PR fix? Somewhat related to #12504

Which triplets are supported/not supported? Have you updated the CI baseline?
I do not believe this has any affect for CI pipelines, but I have not studied them in detail. Please correct me if this is wrong.

Does your PR follow the maintainer guide?
To the best of my understanding, yes.

We are very hesistant to introduce any functionality that produces nondeterministic behavior, especially without loud, obnoxious messages for/from users.
Have you considered just defining VCPKG_USE_HEAD_VERSION yourself in your port?
set(VCPKG_USE_HEAD_VERSION ON)

Note: this is not a supported scenario, but it is a hack that may give you what you want.

Closing for now, let me know if the above solution doesn't work for you!


Can you share the training and testing codes of the 3dhp dataset?

I used the script provided by the P-STMO model(https://github.com/paTRICK-swk/P-STMO) to train and test the MHFormer model on the 3dhp dataset. The average value of PCK and AUC indexes obtained was 20 and mpjpe was 300, but the results were very unreliable.

Have u solve this probelm?

@ClimberY Have u solve this probelm?

No, I haven't found a solution yet.

Sorry, we do not have the plan to release this code. You can train our model in their repo, maybe it can work well.

@ClimberY Have u solve this probelm?

No, I haven't found a solution yet.

Have you solved this problem now?

@ClimberY have you solved this problem, I am getting same error i tried with frames=81,27 and 9. but at test time MPJPE is very large whereas train-MPJPE is decreasing

Sorry, we do not have the plan to release this code. You can train our model in their repo, maybe it can work well.

Hi @Vegetebird , thanks for your great work. May I ask if the 58.0 MPJPE results on MPI-INF-3DHP are cross-dataset, \ie, the model is trained on Human3.6M? According to what you said, it seems it is a result of training on 3DHP?


[Fix] Executing install.sh failed when ${password} and others contain @

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

sed -i ${txt} "s@^spring.datasource.password=.*@spring.datasource.password=${password}@g" conf/datasource.properties
${password}  is qwer@1234
Then one more @

while  sed -i ${txt} "s/^spring.datasource.password=.*/spring.datasource.password=${password}/g"  and  ${password}  is  qwer/1234, the same error

Each file is configured individually during deployment may be avoided.

Codecov Report

Merging #6733 (b10aacc) into dev (24bd8a1) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #6733      +/-   ##
============================================
- Coverage     37.98%   37.93%   -0.05%
+ Complexity     3241     3235       -6
============================================
  Files           643      643
  Lines         26409    26409
  Branches       2861     2861
============================================
- Hits          10032    10019      -13
- Misses        15453    15469      +16
+ Partials        924      921       -3

Impacted Files
Coverage Δ

...he/dolphinscheduler/common/enums/SqoopJobType.java
0.00% <0.00%> (-88.89%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...org/apache/dolphinscheduler/remote/utils/Host.java
37.77% <0.00%> (-2.23%)
:arrow_down:

...inscheduler/common/task/sqoop/SqoopParameters.java
74.00% <0.00%> (-2.00%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.11% <0.00%> (-1.41%)
:arrow_down:

Continue to review full report at Codecov.

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

@tracehh Is there an issue associated with this PR

@tracehh Is there an issue associated with this PR

@tracehh Yeah, maybe you should create a issue and target PR to it.

@tracehh Is there an issue associated with this PR

@tracehh Yeah, maybe you should create a issue and target PR to it.

ok, I create an issue : https://github.com/apache/dolphinscheduler/issues/6781

ok, I create an issue : #6781
I open another pr for this issue. @zhongjiajie

@tracehh Hi, you could just edit PR describe instead of open another PR, cause another PR would lost context about preview review.
I could reopen PR and edit describe like screenshot belove


Extra closing tag for <meta ... and <link ....

Using Sublime 3 - after i run the code formatter -  got extra html closing tag in head for <meta.. and <link
What should i do to not insert in code?
In inserted code you will see
Thanks!
<head>         <meta charset="utf-8">             <meta content="IE=edge" http-equiv="X-UA-Compatible">                 <meta content="width=device-width, initial-scale=1" name="viewport">                     <!-- The above 3 meta tags *must* come first in the head; any other head content must come *after* these tags -->                     <title>                         Bootstrap 101 Template                     </title>                     <link href="https://fonts.googleapis.com/css?family=Cabin|Source+Sans+Pro:300,400,600,700" rel="stylesheet">                         <!-- Bootstrap -->                         <link href="css/bootstrap.css" rel="stylesheet">                             <link href="css/main.css" rel="stylesheet">                                 <!-- HTML5 shim and Respond.js for IE8 support of HTML5 elements and media queries -->                                 <!-- WARNING: Respond.js doesn't work if you view the page via file:// -->                                 <!--[if lt IE 9]>       <script src="https://oss.maxcdn.com/html5shiv/3.7.3/html5shiv.min.js"></script>       <script src="https://oss.maxcdn.com/respond/1.4.2/respond.min.js"></script>       <![endif]-->                             </link>                         </link>                     </link>                 </meta>             </meta>         </meta>     </head>

+1

+1

+1

+1

ensure that your HTML code is correctly formatted and does not contain unnecessary closing tags.
Here's the corrected version of your HTML code with the redundant closing tags removed:

Bootstrap 101 Template

Make sure your HTML code follows this structure without redundant closing tags, and Sublime Text 3 should format it correctly without adding extra closing tags.
For more meaningful insights do check out Full Stack Web Development Courses


Provide specific scopes in contributing guidelines

This repository runs tests on Travis CI and AppVeyor. Although checking commit message, no one provides specific scopes in CONTRIBUTING.md.

Scope
The scope could be anything specifying place of the commit change. For example
Compiler, ElementInjector, etc.

Actually validate-commit-message.js checks scopes. If someone write wrong scopes in commit message, Travis CI and AppVeyor will fail. (me, too. at build #6536)
It's about time to provide specific scopes in CONTRIBUTING.md.
It looks like below.
### Scope
The scope could be anything specifying place of the commit change.
* @angular/cli
* @ngtools/json-schema
* @ngtools/logger
* @ngtools/webpack
* packaging
* changelog

Any thought?

A documentation update is in progress.

@clydin sounds good 👍


dart:mirror is not cross-platform

When working with Dart, there are some difficulties in solving the following task.
The task is to write a Chain class that will store a set of functions and execute them when necessary. The class, when executing the function chain, should trigger rollback actions for all the completed tasks in case of an error. This can be useful when you need to either execute all the functions or execute none if an error occurs.
When starting the chain, arguments can be passed to it. The chain stores these arguments. To call the functions from storage, the required arguments are fetched. The list of required arguments is determined based on dart:mirror, which extracts parameter names and information about them from the function's description. However, the solution based on dart:mirror is not cross-platform. Extracting names using the toString method for function signatures may not be very convenient, fast, and also is not cross-platform.
Is it possible to add a limited version of dart:mirrors in the Dart SDK that would be cross-platform and allow extracting the following information from functions: whether an argument is named or not, and the name of the argument?
A simplified code for working with chains is presented below.
import 'dart:mirrors';

/// Type for storing the arguments of the function chain.
typedef ChainArgs = Map<Symbol, dynamic>;

/// Used to describe the parameters of a function.
class FunctionParameter {
  /// The identifier of the parameter as specified in the function description.
  Symbol simpleName;

  /// Whether the parameter is named or not.
  bool isNamed;

  FunctionParameter(this.simpleName, this.isNamed);
}

/// Used to store a list with information about all function parameters.
class FunctionParameters {
  final List<FunctionParameter> params = [];

  /// Constructor that uses a non-cross-platform solution
  /// `dart:mirrors` to get the function parameter information.
  FunctionParameters(Function function) {
    var mirror = reflect(function) as ClosureMirror;
    for (var p in mirror.function.parameters) {
      params.add(FunctionParameter(p.simpleName, p.isNamed));
    }
  }
}

/// Class representing a task in the task chain.
///
/// There are two functions. The first one [run] is used to execute the task.
/// The second one [cancel] is used to undo the task in case of an error during execution.
class Task {
  /// Run the task for execution. For example, create a file.
  Function run;

  /// Undo the task. For example, delete a file.
  Function? cancel;

  /// Information about the parameters of the [run] function.
  FunctionParameters runParameters;

  Task(this.run, [this.cancel]) : runParameters = FunctionParameters(run);
}

/// Task chain.
class Chain {
  /// List of tasks to execute.
  final List<Task> tasks = [];

  /// List of completed tasks. This is used to call cancel functions only for
  /// tasks that have been executed.
  final List<Task> completedTasks = [];

  /// Add a task to the chain.
  void addTask(Task task) => tasks.add(task);

  /// Cancel all completed tasks.
  void cancel() {
    for (var t in completedTasks) {
      try {
        t.cancel!();
      } catch (e) {
        throw AssertionError("The cancel function should not throw exceptions");
      }
    }
  }

  /// Start executing the chain.
  void run(ChainArgs args) {
    try {
      for (var task in tasks) {
        // args contains arguments for the chain. All task arguments are stored
        // in a single Map. When running a task, the required arguments for calling
        // the function are extracted from this Map. This Map contains non-fixed
        // arguments, executed tasks can generate data, store it in this Map, and
        // the saved values can be used in other tasks.
        var params = task.runParameters;
        Map<Symbol, dynamic> namedParams = {};
        List<dynamic> posParams = [];
        for (var p in params.params) {
          // If the special parameter __context__ is specified, it is used
          // for accessing all arguments of the chain within the function.
          // It can change the state of the object with these arguments.
          if (p.simpleName == #$context) {
            if (p.isNamed) {
              namedParams[p.simpleName] = args;
            } else {
              posParams.add(args);
            }
          } else {
            if (p.isNamed) {
              namedParams[p.simpleName] = args[p.simpleName];
            } else {
              posParams.add(args[p.simpleName]);
            }
          }
        }

        // The function is called with the required arguments.
        Function.apply(task.run, posParams, namedParams);

        // The task is added to the completed list for rollback in case of error.
        // If the cancel function is not specified, it is considered that no
        // cancellation is needed for this task.
        if (task.cancel != null) {
          completedTasks.add(task);
        }
      }
    } catch (e) {
      // Cancel all tasks in case of an error.
      cancel();

      // Re-throw the exception.
      rethrow;
    }
  }
}

void main() {
  var c = Chain();
  c.addTask(Task((ChainArgs $context) {
    $context[#b] = 2;
  }));

  c.addTask(Task((int a, int b) {
    print('a = $a');
    print('b = $b');
    print('sum = $a + $b');
  }));
  c.run({#a: 1});
}

You can find extensive discussion on the long term plans for dart:mirrors support here: #44489
In summary, dart:mirrors is considered an unstable API and has too many inherit shortcomings (mostly around its affects on whole program tree-shaking) for it to be worth trying to add more support for it. The eventual goal is to replace it with something that addresses these issues and solves (most of) the same use-cases. But for now it's basically frozen as is.
The Dart team is currently working on implementing macros as a language-level metaprogramming solution that can be a replacement for dart:mirrors in many cases. Though it's not widely available yet, you can find more about that here:
https://dart.dev/language/macros#the-macros-language-feature
It's not as pretty but in the meantime if you need this code to work on all platforms you'll likely need the user to pass the list of FunctionParameters (or some similar representation that lets you derive the list) into the Task constructor for you.

Closing this out as there is no intended work here. @gmb15551 Feel free to comment on #44489 with any additional concerns if you feel macros won't solve your use-case and you need something else from dart:mirrors.


[test] Gpu workflow

Overview
This PR changes the GitHub Workflow to efficiently use self-hosted GPU runners.

Such tests will be triggered if a PR has the label read-for-testing.
Could you please try it, maybe find another account? I think only accounts with certain permission (like 'write') can assign labels.
Sure, will change the label name.

Yeah, only with certain permission (maybe triage access? based on the doc here ) can assign the labels.


Identity badge is not so trustworthy

          hmmm badge is not so trustworthy I can spoof any address and name myself as someone already knowed, otherwise code lgtm

Originally posted by @roiLeo in https://github.com/kodadot/nft-gallery/pull/8907#pullrequestreview-1810557868
What we can do to improve it
Same as polkassembly, show badge near verified identities (judgement)

would be cool to have this 🚀

yeah, but the person still had to pay 145 USD to create the identity

in the future, I would like to also distinguish between verified ID and unverified ID

for example, Twitter does it, you pay, you get a check, and you can have the same name, in our case the underlying address is what distinguishes

but yeah could be removed ~

replaced better

not sure with what


Do not fail when reading Plutus V2 cost model with 175 parameters instead of 185

Changelog
- description: |
    Do not fail when reading Plutus V2 cost model with 175 parameters instead of 185
# uncomment types applicable to the change:
  type:
  # - feature        # introduces a new feature
  # - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
  # - refactoring    # QoL changes
   - bugfix         # fixes a defect
  # - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  # - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

Context
Part of the fix for:

https://github.com/IntersectMBO/cardano-cli/issues/801

FYI: Because we now enforce the use of cabal-gild to format cabal files and, in order to reduce the amount of hassle, I have cherry-picked the appropriate commit into your PR. This will make the cabal-gild CI check pass for the PR. The commit should be discarded automatically as soon as you either rebase or merge the PR (since it is already in the main branch). Feel free to discard it if you want, but that will make the new required action to fail until you rebase.

FYI: I have rebased your branch because we have done changes to the formatting. I have made a copy of the unrebased branch here: backup/mgalazyn/fix/allow-reading-plutus-v2-175-param-costmodel

Parameterize cardano-api's alonzoGenesisDefaults on the era. If conway era is specified, add in the additional cost model parameters. If not don't add them. This should result in minimal changes.

The problem occurs when we provide alonzo genesis specification from a file. We're not using alonzoGenesisDefaults there, so it won't solve this.

When supplying an array, ledger calls mkCostModel which always retains the original values supplied. Plutus's mkEvaluationContext will then automatically adjust the cost model (either filling in missing values or truncating them).

That's correct. The whole purpose of this fix was to support flexible map format.

Our pretty printing function can potentially do some validation and suggest to the user if they are missing parameters from their particular cost model.

I don't understand which pretty printing function are you referring to.

The problem occurs when we provide alonzo genesis specification from a file. We're not using alonzoGenesisDefaults there, so it won't solve this.

Benchmarking is using create-testnet-data which will use alonzoGenesisDefaults if no Alonzo genesis file is created. This would fix their problem. @mkoura uses an array so they are excluded. I don't think anyone else is affected because they're not launching Babbage testnets with V2 cost models in the Alonzo genesis file so this is really for the benchmarking team.

I don't understand which pretty printing function are you referring to.

We would have to implement it.

The whole purpose of this fix was to support flexible map format.

Yes but this requires us to maintain code that we should not be maintaining. Benchmarking wants to be able to view the cost model as a json map. We can give them that whilst avoiding the additional gymnastics around decoding the cost model from a json map. In the future its possible for this situation to arise again and we can (and should) avoid it.

Yes but this requires us to maintain code that we should not be maintaining

I completely agree. I'll gladly remove this code when we won't need to support this edge case anymore.

Benchmarking wants to be able to view the cost model as a json map.

I understood that the use case is to be able to define and inspect cost model in JSON and not just only view it. I guess we could provide separate translation layer for them doing pretty much what's done in this decoding function. It still means that we have to fix ledger bugs in cardano-api either way.

FYI: I have rebased your branch because we have done changes to the formatting. I have made a copy of the unrebased branch here: https://github.com/IntersectMBO/cardano-api/tree/backup2/mgalazyn/fix/allow-reading-plutus-v2-175-param-costmodel


Make request IDs unique across chains

At the moment request IDs are not unique across chains. In the event that a node receives two request with identical IDs from different chains simultaneously (extremely unlikely), it will make a single API call for them. This may be an issue for future non-idempotent operation support (https://github.com/api3dao/airnode/issues/70). Also, it would be overall neat for request IDs to be unique across chains.

Closed by https://github.com/api3dao/airnode/pull/246


Consolidate bulk upload endpoints.

Test strategy:
Wrote tests for the Rails endpoints.
Tested the new Sample Upload flow by hand for both local and remote. Should test again on staging to make sure the entire pipeline works (it's flaky locally).
Also tested the existing upload flow works.

Thanks. Made a note of testing strategy in the description.


feat: guide the user when no api found

Inform the user when we can't connect to the API and give them the option to set a custom api address.

Early WIP on #711
License: MIT
Signed-off-by: Oli Evans<EMAIL_ADDRESS>

Still missing:

[ ] Redirect
[ ] Reload config

Requires:

[ ] https://github.com/ipfs-shipyard/ipfs-redux-bundle/pull/1

@olizilla I've made some changes. Please take a look to ipfs-redux-bundle.

If we can't connect to an instance, it redirects to /welcome.
It won't let you open any other page if there isn't a valid IPFS instance.

This makes our navigation tests fail on CI because there isn't an IPFS instance running. Suggestions? We could start a js-ipfs instance so it connects and does everything else.

Cache! So, you set an API address, reload the page, and it's still the same address.
You can always go to /welcome and change the API url.

@olizilla

Redirecting the url when ipfsReady is false feels like something that we should set up in redux.

It is on redirects bundle.

We should catch the condition that the welcome page bundle has loaded, but IPFS isn't ready and there is no error. It can just show a loading spinner on the welcome in that case.

Done.
When failed to connect.

After being successfully connected

@hacdias To complete this challenge please appease the CI gods. The tests should pass and the test coverage should not go down. Of note the navigation test that is failing was really a place holder for more exciting e2e tests. Do you accept this mission?

For this PR I can add tests with and without a node running. We can then think about more tests haha. Sounds good @olizilla?

@hacdias 💯

@olizilla made some updates and mocking but it seems to fail here: https://github.com/ipfs-shipyard/ipfs-webui/pull/715/commits/6a0f41dd6ff1caadef44c3e9a9c3910988415239#diff-0238dfb49a06ba796757bff9fcff0faaR41
There might be something on the files page that blocks the rendering? I can't see what it might be though. I tried adding empty functions for files.ls and files.stat but it didn't work.

@hacdias i'll take a look at this now

Thank you @olizilla


Add Screen session section to prompt

Description
Screen is a full-screen window manager that multiplexes a physical terminal between several processes, typically interactive shells.
https://www.gnu.org/software/screen/
There is a similar PR open for this functionality (#906) but I think that PR hasn't enough information in bash script definition neither into documentation. It also hasn't enough customization.
For that reasons I open this PR in order to improve this project.
Screen defines a $STY env var with format pid.tty.host or pid.sessionname (if defined). So prompt information is shown only when a session is opened. That check is defined using a regex match /^[0-9]+/ against $STY env var. If $STY is empty or it doesn't match regex section returns before define local 'screen_session'.
But if you don't want to show session name you can disable it using SPACESHIP_SCREEN_SHOW_SESSION=false.
Screenshot

Using SPACESHIP_SCREEN_SHOW_SESSION=false

Thanks for contributing!


Laradumps don´t work in Laragon?

Laradumps don´t work in Laragon?
I did several tests and the desktop app cannot detect the log
My .env
DS_APP_HOST=<IP_ADDRESS>
DS_APP_PORT=8000
DS_SEND_QUERIES=true
DS_SEND_LOGS=true
DS_SEND_LIVEWIRE_COMPONENTS=true
DS_SEND_LIVEWIRE_FAILED_VALIDATION=true
DS_AUTO_CLEAR_ON_PAGE_RELOAD=true
DS_AUTO_INVOKE_APP=true
DS_PREFERRED_IDE=phpstorm

Hi @quinhone,
I did not experience any issue. I have just tested with the latest LaraDumps version in a fresh Laravel Project.
My .env:

My .env:
DS_APP_HOST=<IP_ADDRESS>
DS_APP_PORT=9191
DS_SEND_QUERIES=true
DS_SEND_LOGS=true
DS_SEND_LIVEWIRE_COMPONENTS=true
DS_LIVEWIRE_EVENTS=true
DS_LIVEWIRE_DISPATCH=true
DS_SEND_LIVEWIRE_FAILED_VALIDATION=true
DS_AUTO_CLEAR_ON_PAGE_RELOAD=false
DS_AUTO_INVOKE_APP=true
DS_PREFERRED_IDE=vscode

Please confirm that you have the latest versions of LaraDumps Desktop APP and LaraDumps package?
I can only suggest that you remove (and backup) your App/config/laradumps.php and run artisan ds:init() again.
Also, you could try to set the APP_PORT to its default value (9191) and run php artisan config:clear.
🤠 Hopefully it works!

Closing due to inactivity.


Save SvgDocument to JPG - axis labels disappear & other issues

I have an SVG to convert to JPG using something like:
var svg = SvgDocument.Open(@"C:\Temp\chart.svg");
var image = svg.Draw();
image.Save(@"C:\Temp\chart.jpg", ImageFormat.Jpeg);

When I do this, the x-axis labels in the JPG are missing, the title of the SVG is not shown (though I think it's just pushed off the top of the image) and the axis legend format gets messed up.
SVG content is available below...
https://drive.google.com/file/d/0B8LTgi0clEPjdUpjU0VkN29pckE/view?usp=sharing
and the JPG produced by the code above is:
https://drive.google.com/file/d/0B8LTgi0clEPjSGxzZFVmWWwtZnc/view?usp=sharing

Looks like this has just been fixed in version 2.0. Upgrading from 1.7 to 2.0 fixed it for me


Remove QuantumCircuit.bind_parameters

Summary
This expires the deprecation on bind_parameters in favour of assign_parameters, which is a drop-in replacement with its defaults.

Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

0 of 0   changed or added relevant lines in 0 files are covered.
25 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.03%) to 87.531%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
5
92.17%

crates/accelerate/src/quantum_circuit/circuit_data.rs
8
90.71%

crates/qasm2/src/parse.rs
12
97.13%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
59154

Relevant Lines:
67581

💛  - Coveralls


Figure plotting

Hi Jacob and team,
Love your work and would like to contribute. From your current paper draft I saw that you are currently missing some figures of the data and so I started plotting the data.  Let me know if you are interested.
Best,
Ruben

So far I created two plots:
Figure 1: Time course of measured radiation
Figure 2: Variance of measured radiation
Figure_1_UV-C_on-off.pdf
Figure_2_Variance_UV-C_on-off.pdf

Hi Ruben!
Thanks so much for the contribution! Those look great! We're still figuring out what figures to put in the paper, but at a minimum this is really useful to have as a supplement. If you want to try to figure out how to make a pull request with these figures and the code to generate them, I'm happy to help. Alternatively, I can add them myself.

Mh, I tried requesting a pull from different branches (master and a new one figure_plotting) but nothing is displayed in the Pull Request tab.
This was the command I used:
git request-pull -p master https://github.com/TheoryDivision/covid19_biosafety_cabinet.git
I think you need to allow me to create pull requests?

Sorry for not getting back to you sooner, but it looks like you figured it out! Thanks so much for the contribution!


Verifying the read access of anonymous users prior to transmitting the Mutation result.

Why make this change?
Resolves #1464 for CosmosDB.
What is this change?
Verifying the read access of anonymous users prior to transmitting the Mutation result
How was this tested?


Tweak to multiple cache example

Believe line 116 should be  for i in sorted(eos, reverse=True): to cover the case where a multiple prompts finish at the same time and one closer to the front of the queue gets popped first.

The eos list is already in reverse order for that reason:
    eos = []
    ...
    for i in range(len(input_ids)):
        ...
            eos.insert(0, i)

You are right.   For some reason when I implemented this I was appending to the list, not inserting at the beginning.


Streamline and align the "Getting Started" docs to fit in the new structure

Problem Statement

The "Getting Started" documentation for sentry-electron platforms may not conform to the newly defined format, which includes the steps "Install", "Configure", "Verify", and the optional "Next Steps".

The "Verify" step is not always available and it is a very important step that helps the user to verify that the installation and configuration were done correctly.

New users onboarding with Sentry may find the existing documentation extensive and unclear. The "Getting Started" guide should cover the basics, while more advanced configurations shall live on the sentry-docs website.
Some copies and code snippets in the documentation may be outdated.

Solution Brainstorm
The task goals are as follows:

Thoroughly review all sentry-electron platform documentation, including less commonly used sections, and reorganize their content into structured headings: 'Install,' 'Configure,' 'Verify,' and the optional 'Next Steps.'
The onboarding documentation should strictly focus on the basics to help users get started. Advanced configurations should be available in the complete documentation on the sentry-docs website.
Essential information for new users should be included under the new 'Notes' section in the onboarding documentation.
Update code snippets and copies as necessary.
If a "Verify" step is missing, include it in the documentation.

Please refer to the "Frontendend Handbook" guide in our "Sentry" main repository for assistance in adding and updating the getting started docs: Frontendend Handbook.
Project: https://github.com/getsentry/team-sdks/issues/23

I added some extra detail for how verify native crash reporting:
https://github.com/getsentry/sentry/pull/55540


Updates to ResourceTracking Device model

Important: To be reviewed and deployed only alongside the relevant resource_tracking changes

This change is

src/components/sss/tracking.vue, line 661 at r3 (raw file):

          var c_label = ''
          var u_label = ''
          var c_driver = ((device.get("current_driver") || '') ? " " + device.get("current_driver") : '');

var c_driver = ' ' + ((device.get("current_driver") || '');

Comments from Reviewable

Reviewed 2 of 2 files at r1, 7 of 7 files at r2, 1 of 1 files at r3, 1 of 1 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Travis-CI: Drop Ruby 1.8.7/1.9.3 tests

This is really EOL since ages and the gems don't work anymore. There is
probably more to cleanup, but this is a low hanging fruit and reduces the
test matrix. Probably a good first step.

I should switch these modules to PDK actually…


Add automatic eBPF / OVS flow-capture based on CNI

Add an automatic selection of flow-capture method based on recognition of deployed CNI.
Suggestion:: for OVN-K8S the default will be OVS/N and for the rest of the CNI's it will be eBPF.

@jotak ^^^

closing as eBPF is now the default for all


Investigate AirFlow chip failure

Conclusion:

The receiver board on one aircon unit was broken. AirconCtrl PCBA and FW works fine.


Fixed compiler warning for Berlin 10.1

Berlin 10.1 also issues the same warning, so I made this small change since some organizations use acceptance policies of 0 warnings in the CI process.

This is still resulting in a warning on XE7
D:\CI_WS\Ws\978186\Source\DUnitX\Source\DUnitX.Timeout.pas(170): warning W1036: Variable 'elapsedTime' might not have been initialized
I'll take a look at this today and see what needs to be done as I have XE2-10.4 installed on my dev machine.

Closing this in favour of the change I just pushed - updated DUnitX.inc to define DELPHI_XE100_DOWN rather than DELPHI_XE10_DOWN. Thanks for bringing it to my attention.


Identifiers validation failed for dataset Programa Integral de Protección de tortugas marinas. Temporadas 2007 a 2012

Identifier validation failed for the dataset Programa Integral de Protección de tortugas marinas. Temporadas 2007 a 2012:

Crawler attempt: 324
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 9801
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 9801 total records; 9801 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3740001/step

New IDs sample:

Old IDs sample:

0568d8e93a1249339a06ff91037a10be
057ca139551029393a8157d631f01f68
0bc3cc2b4c670d069dd7452bcf25cb56
15d5b2efec0f641b9d02f942b4c4f7f0
1eb0566cb63f5043e9c8cbd3ff8d4acc
2f8b988bfcd940519e49e7db5f9650ff
32e74972676836597822378595a39cc3
397e79744cc75939084fc3744fd7f10a
411a1438dcd43c11b663ff319fb6b520
41b33b8e9f49cd6c4f2793d48296f746

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Programa Integral de Protección de tortugas marinas. Temporadas 2007 a 2012 : https://doi.org/10.15468/hpkcx6.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Programa Integral de Protección de tortugas marinas. Temporadas 2007 a 2012:

Crawler attempt: 325
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 9801
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 9801 total records; 9801 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3740970/step

New IDs sample:

Old IDs sample:

0568d8e93a1249339a06ff91037a10be
057ca139551029393a8157d631f01f68
0bc3cc2b4c670d069dd7452bcf25cb56
15d5b2efec0f641b9d02f942b4c4f7f0
1eb0566cb63f5043e9c8cbd3ff8d4acc
2f8b988bfcd940519e49e7db5f9650ff
32e74972676836597822378595a39cc3
397e79744cc75939084fc3744fd7f10a
411a1438dcd43c11b663ff319fb6b520
41b33b8e9f49cd6c4f2793d48296f746

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Programa Integral de Protección de tortugas marinas. Temporadas 2007 a 2012 : https://doi.org/10.15468/hpkcx6.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Programa Integral de Protección de tortugas marinas. Temporadas 2007 a 2012:

Crawler attempt: 327
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 9801
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 9801 total records; 9801 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3741601/step

New IDs sample:

Old IDs sample:

0568d8e93a1249339a06ff91037a10be
057ca139551029393a8157d631f01f68
0bc3cc2b4c670d069dd7452bcf25cb56
15d5b2efec0f641b9d02f942b4c4f7f0
1eb0566cb63f5043e9c8cbd3ff8d4acc
2f8b988bfcd940519e49e7db5f9650ff
32e74972676836597822378595a39cc3
397e79744cc75939084fc3744fd7f10a
411a1438dcd43c11b663ff319fb6b520
41b33b8e9f49cd6c4f2793d48296f746

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Programa Integral de Protección de tortugas marinas. Temporadas 2007 a 2012 : https://doi.org/10.15468/hpkcx6.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.


Can I run a planespotting model locally?

I'm new in tensorflow. I have only small keras experiences, so I don't know about tensorflow itself well...
I'm trying to train and learn a tensorflow planespotting model locally. Training is done by reading the readme file. I get saved_model.pb file and variables as result. But I got stuck on inference.
import tensorflow as tf
import numpy as np

# DATA
DATA = './samples/airplane_sample.png'

# MODEL
SAVED_MODEL_DIR = './1559196417/'

def decode_image():
    img_bytes = tf.read_file(DATA)
    decoded = tf.image.decode_image(img_bytes, channels=3)
    return tf.cast(decoded, dtype=tf.uint8)

def main1():
    with tf.Session(graph=tf.Graph()) as sess:
        tf.saved_model.loader.load(sess, [tf.saved_model.tag_constants.SERVING], SAVED_MODEL_DIR)
        img = decode_image()
        result = sess.run(['classes'], feed_dict={'input': img})
        print(result)

def main2():
    model = tf.contrib.predictor.from_saved_model(SAVED_MODEL_DIR)
    pred = model({'image_bytes': [decode_image()], 'square_size': [tf.placeholder(tf.int32)]})
    print(pred)

if __name__ == "__main__":
    main2()

Above is a code written by me but it doesn't work. Could you give me some advice to me?

Closing the issue that I'm not interested in anymore.


Document some recently-deployed features

Updates documentation with a few small features we added recently.

LGTM.


universal-android-debloater: Add version 0.5.1

Closes #11256

[X] I have read the Contributing Guide.

/verify

Error log from failed checks:
INFO: Checkver

Log of Output:

    universal-android-debloater:
    The remote server returned an error: (404) Not Found.
    URL https://api.github.com/repos/0x192/universal-android-debloater/releases/latest is not valid

INFO: Checkver done

INFO: Autoupdate

INFO: Autoupdate done

running checkver locally with ...\apps\scoop\current\bin\checkver.ps1 -App universal-android-debloater -Dir . works fine, so I'm not sure how to reproduce this.
Maybe it is because the releases on github are marked as prerelease?
Help wanted.

/verify

Maybe it is because the releases on github are marked as prerelease?

Yes

@HUMORCE I don't believe this actually works. You changed the downloaded file name to uad.exe, but didn't change the shortcut file name.
S:\
❯ scoop i universal-android-debloater
Installing 'universal-android-debloater' (0.5.1) [64bit] from extras bucket
uad_gui-windows.exe (7,3 MB) [=========================================================================================================================================================================] 100%
Checking hash of uad_gui-windows.exe ... ok.
Linking S:\Apps\Scoop\apps\universal-android-debloater\current => S:\Apps\Scoop\apps\universal-android-debloater\0.5.1
Creating shortcut for Universal Android Debloater (uad_gui-windows.exe) failed: Couldn't find S:\Apps\Scoop\apps\universal-android-debloater\current\uad_gui-windows.exe
'universal-android-debloater' (0.5.1) was installed successfully!

so creating the shortcut fails

😦 I will fixes it when home

@leumasme Done. https://github.com/ScoopInstaller/Extras/commit/f7f7a635e3366ce6049192712847c4286b94fc16


Link to permanent Slack signup page

Slack lets you create permanent signup links now, so I think we should use one of these.

If this works for us, we can actually shut down this app completely: https://dashboard.heroku.com/apps/slackin-citizenlabs


Hotfix/users security

Closes #231
Changes

rename allUsersUsernamesOnly publication for explicitness
make sure only admin users can access /users route
bump version to 0.38.1

@frenchbread fixed issues.

@brylie 🕶  awesome


[eas-json] Switch from @hapi/joi to joi, types now included in package

Why
Fixes a few warnings on installing eas-cli:
╰─$ npm install eas-cli
npm WARN deprecated<EMAIL_ADDRESS>Switch to 'npm install joi'
npm WARN deprecated<EMAIL_ADDRESS>Moved to 'npm install @sideway/formula'
npm WARN deprecated<EMAIL_ADDRESS>Moved to 'npm install @sideway/address'
npm WARN deprecated<EMAIL_ADDRESS>Moved to 'npm install @sideway/pinpoint'

How

Install the joi package
Notice that the @types/joi package is out of date
See that bumping from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>would include types, so do that
Run tests

Test Plan
Run tests

nm, too many other places depend on @hapi/joi to make this a quick driveby


Submission with automatic title

Issue Description
I'm trying to submit a submission to subreddit, but I'd like Reddit to handle title assignment. According to the documentation it should be possible to simply invoke subreddit.submit(url=<URL>), but it seems to not work and all I get is the following error
ipdb> sr.submit(url=link)
*** TypeError: submit() takes at least 2 arguments (2 given)

Is it possible in the API to send a submission without a title and let Reddit handle the title's assignment?
System Information
PRAW Version: praw==4.1.0

Python Version: 2.7.12
Operating System: Ubuntu 16.04

The feature to select a title on the website (fetch_title) is not made available through the API. Even on the website it requires you to select "use suggested title". With PRAW a title is required. You can, however, copy and use their get_title function directly if you want to auto-fill the title in the same way that they do:
https://github.com/reddit/reddit/blob/40625dcc070155588d33754ef5b15712c254864b/r2/r2/lib/utils/utils.py#L255
Make sure to follow whatever is necessary in their license if you're going to copy some of their code: https://github.com/reddit/reddit/blob/master/LICENSE


chore(dependabot) Create dependabot.yml

Summary
This will make a dependabot.yml file which will scan for latest versions of the github actions used in this repository.

Please review @jackkav

Bumping for tomorrow

Closing for now, we are considering a new approach to dependency management. Thanks for you contribution.


Add payment contract

[x] Add payment contract
[ ] Create builder
[ ] Use payment contract
[ ] Add specs

#15 #16 #17 were extracted from this PR, thanks for contribution @vsevolod


We are featured on XKCD

We... sort of :-P

It could be runtime.js, too... ;-)

[Something]... now I'm curious

Curious... or feeling epic? ;-)


docs: add explanation on what exactly this plugin does

@mesqueeb Thank you for your contribution!
I will take the opportunity to make this a FAQ section. Surely there will be more interesting questions in the future.


Allow using a different PrismJS Theme

I have added support to choose from the builtin PrismJS Themes, with Okaidia still being the default. Technically it works by serving the css files for each and then adjusting the link.href dynamically when a theme is chosen, which is the same thing the https://prismjs.com/ website is doing.

Let me know your thoughts, if this is something you could imagine to merge. I tried following existing coding conventions, but am happy to change anything if it helps adoption. I would also love to "remember" the chosen theme e.g. in localStorage, but since that's not done yet for anything else I thought I'd ask first.
Thank you for the tool, I :heart: it.

Thanks, I'll check it. BTW, you should add your name in the copyright notice of the new .js file.

No idea why the 3.10 tests fail, as I haven't touched any of the backend, maybe you can have a look as currently it's not possible for me to easily test 3.10 on my machine :bow:

Regarding the failing 3.10 test, it's a know issue that I haven't been able to resolve yet. I guess I need to mark this test as skipped until I have enough time to investigate it.

Sorry for the late reply. I've checked your PR. Looks intetesting. The only minor issue that I've noticed is that a mouse cursor inside the theme selection dropdown is a textual one (a vertical line) instead of a pointer (an arrow). Could you check this?

Good catch, I did not notice that. It was because I did not use any href on the anchors and bootstrap 3 styles them a little differently then. It is probably debatable whether for javascript-anchors a href="#" or href="javascript:void(0)" or just omitting it and using some additional css is the best approach, I picked the latter for now. Let me know if there is more I can do 👋

I'm having some trouble fixing the conflicts, likely cause I am using a different nodejs version to run the npm dependency solver and build command (node 17.x seems bugged on windows, had to downgrade to 16.x). What nodejs and npm versions are you using exactly? Maybe using the same version and running the commands again will fix it.

Could you merge the current master into this? FYI: I'm using node v16.17.0 with the latest changes.

@romanvm I did my best to resolve all conflicts, on my side the build looks good, using the same node version as you v16.17.0. I have added all build artifacts from the frontend to git, which means the themes are there as well, I hope that was correct. I would love to add that the web UI "remembers" my last theme choice, maybe via localStorage, what do you think?

Hi. I'm sorry but I have to respectfully decline your PR. Currently I have very little time and resources to maintain this project and I don't wan't to introduce additional complexity that do not affect core functionality.

Hi Roman, not a problem at all. Stay safe 🫂


Allow the import of instance specific host functions

Is your feature request related to a problem? Please describe.
We have:

An application with a single wasm runtime (we support wazero, and other similar engines)
Many modules authored by third parties implementing a shared interface
Many instances of each of those modules
Performance constraints that mean we really do not wish to recompile the module for each instance

Each instance needs a single instance-specific host function to be imported to the instance.   The host function references instance specific stuff, so we cannot import the same HostModule instance into multiple wasm module instances.
Describe the solution you'd like
A means to import instance specific host functions to multiple wasm module instances of a module that only needs to be compiled once.
Other wasm runtimes allow definition of a wasm-instance specific ImportObject, through which stuff like this can be done.
Alternatively, perhaps a func similar to runtime.InstantiateModule that allows us to pass in a set of compiled HostModules, like the below signature?
InstantiateModule(
  ctx context.Context,
  compiled CompiledModule,
  hostModules ...CompiledModule,
) (api.Module, error)

Describe alternatives you've considered
If performance wasn't an issue we could just re-init a new runtime and re-compile per instance that we require, but sadly that is too expensive.  It could also be that there is a way to do this that I have not found, but I've been looking for a while.

A colleague spotted that you guys handle this at the runtime level via https://github.com/tetratelabs/wazero/tree/main/examples/multiple-runtimes - closing the ticket :)


Add new supernova and cataclysmic classes

This PR adds new supernova and cataclysmic classes that are used in the ZTF BTS and CLU catalogs.

The test passes when I open a PR in my forked repo (https://github.com/bfhealy/timedomain-taxonomy/pull/1) - maybe this is a permissions issue?


sampler2DArray NEW_IDENTIFIER for texture 2d array

Hi, I'm having trouble using SAMPLER2DARRAY and have been stuck for a while. I have tested it in a very simple shader. But it seems that the sampler2DArray is not recognized by bgfx. I also tried SAMPLER2D and SAMPLER3D and those are fine.
my_fragment_shader.sc
#include <bgfx_shader.sh>
SAMPLER2DARRAY(u_texture0, 0);
void main()
{
gl_FragColor = vec4(1,1,1,1);
}

Error msg from compiler:
95: uniform sampler2DArray u_texture0;
24:                                         ^
Error: (95,24): error: syntax error, unexpected NEW_IDENTIFIER, expecting '{'
I look at bgfx_shader.sh and it seems that SAMPLER2DARRAY macro is defined a couple of places depending on the BGFX_SHADER_LANGUAGE_* I tried both Vulkan and OpenGL for the renderer type of my program, but both have the same compile issue. I also checked BGFX_CAPS_TEXTURE_2D_ARRAY is enabled.
Any thought what may go wrong? Many thanks! I appreciated your attention.

Using the new version of shaderc solved this issue for me

@thegabman  Hi thanks for the info. Which exact version do you use? I tried BGFX_API_VERSION (91) and this issue still persists on my end.

I run into this issue if no specifying --profile which defaults to essl = 2.
Try adding --profile 120 to your command if you're not using essl.

@bwrsandman  Thank you for providing this solution


[SR-12425] Improving diagnostics for key path application on non-convertible root type

Follow up to #31140
This improves diagnostics when applying a key path with a root type that cannot be applied to an instance base that has another non-convertible type.
cc @xedin @hamishknight

Resolves: SR-12425
Resolves: rdar://problem/62200986

Right, all adjusted I think :)

@swift-ci please smoke test

@swift-ci please smoke test

The macOS failure is unrelated, so let us try again.

@swift-ci please smoke test

@LucianoPAlmeida Could you please cherry-pick this to 5.3 branch as a simple commit?

@LucianoPAlmeida Could you please cherry-pick this to 5.3 branch as a simple commit?

Sure! :)


Add Feature Importance Plot Function

Add feature importance plot function
Add document and example of feature importance plot function

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

@yanyachen , it seems failed our tests.

@chivee The error seems a MD5MismatchError from "conda install atlas". The pull request is a R plot function. It's very strange if it affect the Python installation process.

@yanyachen reported to https://github.com/conda/conda/issues/4739


adapt vllm distributed module to sglang

Motivation

Modifications

Move vllm distributed module (v0.6.4.post1) to sglang, and current model is stilll using vllm.distributed module
Checklist

[ ] Format your code according to the Contributor Guide.
[ ] Add unit tests as outlined in the Contributor Guide.
[ ] Update documentation as needed, including docstrings or example tutorials.

What to do Next

[ ] Install sgl-kernel for custom allreduce and replace vllm.distributed to sglang.srt.distributed
[ ] Add tensorrt allreduce module

QQ why not use the v0.6.4.post1

I see pyproject.toml require vllm>=0.6.3.post1

This is to maintain compatibility, 0.6.3.post1 uses torch 2.4, and 0.6.4.post1 uses torch 2.5.1. The current main branch is compatible with both torch 2.4 and torch 2.5.1. Regarding the distributed part, I suggest directly updating to v0.6.4.post1

ok

isort....................................................................Failed
- hook id: isort
- files were modified by this hook

Fixing /home/runner/work/sglang/sglang/python/sglang/srt/_custom_ops.py

I think the test failed is caused by pr #2266


redirect->route wrong docu or default values

hey, currently the redirect function not work like the docu describe it. Link
it result in an
Argument 3 passed to CodeIgniter\HTTP\Response::redirect() must be of the type integer, null given
because route has null as default $code value which is not valid ( but suggested by the documentation )
so what behavior is the right one ?
(personally i would remove the default value of $code ( and switch $params and $code ) )
public function route(string $route, array $params=[], int $code=null, string $method='auto')

Please provide a code sample how you're using this. The docs you linked to are for the common function redirect() while it looks like samples you're showing are calling it directly on the Response object, so I'm a bit confused.

// Go to a named/reverse-routed URI
return redirect()->route('named_route');

thats not a sample -> its the route function definition. just to make it easier to understand the problem ( PS: it should show that the default parameter are invalid ( and they are used by the redirect function ) )

fixed by #1147


Add sparkline plots prototype

Add a prototype displaying a multiple time series dataset in sparklines: Each entity in a separate small plot.
This visualization only works for a small number of lines.
Support both column-wise and entity-wise orientation:
column-wise orientation displays separate plots for each column, with each plot containing all the entities,
entity-wise orientation displays separate plots for each entity, with each plot containing all the columns.

Depends on #29 for data to use during development.


@scharifa: Actually, it is ok when I train the code. But test phase, the error is throw out. I change the KEY_FRAME_INTERVAL in config.py and min_diff and max_diff. Hope it is helpful for you.

@scharifa: Actually, it is ok when I train the code. But test phase, the error is throw out. I change the KEY_FRAME_INTERVAL in config.py and min_diff and max_diff. Hope it is helpful for you.
Originally posted by @Feywell in https://github.com/msracver/Flow-Guided-Feature-Aggregation/issues/16#issuecomment-442284201

Excuse me，Where are the min_diff and max_diff？I could not find them.

hello，
how do you install mxnet?
build from source or use pip?


Crash RxJava

In the production app, we have many crashes on the RxJava:
java.lang.IllegalStateException: Exception thrown on Scheduler.Worker thread. Add onError handling.
at rx.internal.schedulers.ScheduledAction.run(ScheduledAction.java:52)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:422)
at java.util.concurrent.FutureTask.run(FutureTask.java:237)
at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:152)
at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:265)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
at java.lang.Thread.run(Thread.java:841)
Caused by: rx.exceptions.OnErrorNotImplementedException
at rx.Observable$31.onError(Observable.java:7134)
at rx.observers.SafeSubscriber._onError(SafeSubscriber.java:154)
at rx.observers.SafeSubscriber.onError(SafeSubscriber.java:111)
at rx.internal.operators.OperatorMerge$MergeSubscriber.innerError(OperatorMerge.java:429)
at rx.internal.operators.OperatorMerge$MergeSubscriber.access$800(OperatorMerge.java:93)
at rx.internal.operators.OperatorMerge$InnerSubscriber.onError(OperatorMerge.java:552)
at rx.internal.operators.NotificationLite.accept(NotificationLite.java:147)
at rx.internal.operators.OperatorOnBackpressureBuffer.pollQueue(OperatorOnBackpressureBuffer.java:159)
at rx.internal.operators.OperatorOnBackpressureBuffer.access$000(OperatorOnBackpressureBuffer.java:29)
at rx.internal.operators.OperatorOnBackpressureBuffer$2.onError(OperatorOnBackpressureBuffer.java:94)
at rx.internal.operators.OperatorMap$1.onError(OperatorMap.java:49)
at rx.internal.operators.OperatorMap$1.onNext(OperatorMap.java:58)
at rx.internal.util.ScalarSynchronousObservable$1.call(ScalarSynchronousObservable.java:41)
at rx.internal.util.ScalarSynchronousObservable$1.call(ScalarSynchronousObservable.java:30)
at rx.Observable$1.call(Observable.java:145)
at rx.Observable$1.call(Observable.java:137)
at rx.Observable$1.call(Observable.java:145)
at rx.Observable$1.call(Observable.java:137)
at rx.Observable.unsafeSubscribe(Observable.java:7304)
at rx.internal.operators.OperatorMerge$MergeSubscriber.handleNewSource(OperatorMerge.java:188)
at rx.internal.operators.OperatorMerge$MergeSubscriber.onNext(OperatorMerge.java:158)
at rx.internal.operators.OperatorMerge$MergeSubscriber.onNext(OperatorMerge.java:93)
at rx.internal.operators.OperatorMap$1.onNext(OperatorMap.java:55)
at rx.internal.operators.OperatorUnsubscribeOn$1.onNext(OperatorUnsubscribeOn.java:52)
at rx.internal.operators.OperatorSubscribeOn$1$1$1.onNext(OperatorSubscribeOn.java:76)
at rx.internal.operators.OnSubscribeRedo$2$1.onNext(OnSubscribeRedo.java:231)
at rx.internal.operators.OperatorDoOnEach$1.onNext(OperatorDoOnEach.java:84)
at rx.internal.operators.OperatorMap$1.onNext(OperatorMap.java:55)
at rx.internal.operators.OnSubscribeTimerPeriodically$1.call(OnSubscribeTimerPeriodically.java:51)
at rx.Scheduler$Worker$1.call(Scheduler.java:120)
at rx.internal.schedulers.ScheduledAction.run(ScheduledAction.java:47)
... 7 more
Caused by: java.lang.NullPointerException
at android.net.Uri$Builder.appendQueryParameter(Uri.java:1493)
at com.snowplowanalytics.snowplow.tracker.Emitter.requestBuilderGet(Emitter.java:406)
at com.snowplowanalytics.snowplow.tracker.Emitter.performEmit(Emitter.java:316)
at com.snowplowanalytics.snowplow.tracker.Emitter.access$lambda$6(Emitter.java)
at com.snowplowanalytics.snowplow.tracker.Emitter$$Lambda$7.call(Unknown Source)
at rx.internal.operators.OperatorMap$1.onNext(OperatorMap.java:55)
... 26 more
Caused by: rx.exceptions.OnErrorThrowable$OnNextValue: OnError while emitting onNext value: com.snowplowanalytics.snowplow.tracker.utils.storage.EmittableEvents.class
at rx.internal.operators.OperatorMap$1.onNext(OperatorMap.java:58)
... 26 more

Assigning to Josh...

Hi @lixiaoyi could you please copy and paste your tracker and emitter object creation here so I can see what could be causing it?

This crash happen on 0.3.0:
/snowplow/
compile 'com.snowplowanalytics:snowplow-android-tracker:0.3.0'
compile 'com.snowplowanalytics:snowplow-java-tracker-core:0.2.0'
compile 'io.reactivex:rxandroid:0.24.0'
I don't know if it still the crash on the 0.4.0.
But for now, I will try to use the classic lib on the next release instead of RxJava, until you find something to fix:
compile 'com.snowplowanalytics:snowplow-android-tracker-classic:0.4.0'
This is the code we used in 0.3.0:
sInstance = new SnowPlowManager(context.getApplicationContext());
        Emitter emitter = new Emitter
                .EmitterBuilder(SNOWPLOW_COLLECTOR_RELEASE, context.getApplicationContext())
                .method(HttpMethod.GET)
                .build();

        final Subject subject = new Subject();
        // Screen resolution
        WindowManager wm = (WindowManager) context.getSystemService(Context.WINDOW_SERVICE);
        Display display = wm.getDefaultDisplay();
        Point size = new Point();
        display.getSize(size);
        subject.setScreenResolution(size.x, size.y);

        sAppId = context.getString(R.string.snowplow_app_id);

        tracker = new Tracker
                .TrackerBuilder(emitter, NAME_SPACE, sAppId)
                .platform(DevicePlatforms.Mobile)
                .subject(subject)
                .build();

Hi @lixiaoyi so having a new look at your error code it appears to be crashing when adding a null value to the request builder: at android.net.Uri$Builder.appendQueryParameter(Uri.java:1493)
Now this should not be possible but it could be that 0.3.0 let some values in that it should not.  If possible would you be able to do a debugging run and grab the contents that it attempts to add to the GET request?  Just to make sure that 0.4.0 does not contain the same fault.
You also do not need to include the snowplow-java-tracker-core as a dependency anymore.  This was removed in 0.3.0.

The request is made inside the sdk, we don't know when there is a null value here
I will upgrade to 0.4.0
But there is no code changed from 0.3.0 to 0.4.0 in the implementation of method com.snowplowanalytics.snowplow.tracker.Emitter.requestBuilderGet
I don't understand why it could NullPointerException here.

While there is no code changes in the requestBuilderGet there have been many updates to the actual payload construction.  The error has to do with what is happening during event building.  A null value is finding its way into the payload to be sent.  When the payload is retrieved to be sent on to the collector it is being built into a query string.  It is throwing a null pointer trying to build that query.
As far as I can see there is no fault with RxJava, just with the request builder here.
What is bizarre is that it should be impossible to build a Tracker Payload with a null value in it.  Could you potentially also paste in your tracker.track... call.  It could be a particular event constructor not behaving itself as well.

Yes I saw the updates of the payload construction.
I will update to 0.4.0 RxJava version, and we will see on next release.
Because we never reproduce it in the beta app. Thank you.

No worries @lixiaoyi !  Please do let us know if you run into any other problems with the Android Tracker.
As a side note as well, as long as you pass your application context to the Subject it will populate the screen dimensions automatically for you.  So you do not need to do it yourself if you do not want to.

Closing as non-reproducible...


Virt-API Permission Denied

What happened:
virt-api CrashLoopBackOff with the following error logs:
kubectl -n kubevirt logs virt-api-657ccf77-tblws
W0110 15:37:57.513783       1 client_config.go:618] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
W0110 15:37:57.515784       1 client_config.go:618] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
panic: mkdir /tmp/certsdir744381112: permission denied

goroutine 1 [running]:
kubevirt.io/kubevirt/pkg/virt-api.(*virtAPIApp).Execute(0xc000332420)
	pkg/virt-api/api.go:178 +0x5b9
main.main()
	cmd/virt-api/virt-api.go:37 +0x79

What you expected to happen:
virt-api pod running
How to reproduce it (as minimally and precisely as possible):
Try to create Kubernetes cluster in Debian 11 with kernel 5.10.0-27-amd64 x86_64
Additional context:
Add any other context about the problem here.
Environment:

KubeVirt version (use virtctl version): v1.0.0 and v1.1.1
Kubernetes version (use kubectl version): v1.28.3
VM or VMI specifications: N/A
Cloud provider or hardware configuration: Baremetal
OS (e.g. from /etc/os-release): Debian 11
Kernel (e.g. uname -a): Linux 5.10.0-27-amd64 x86_64
Install tools: N/A
Others: N/A

I think the issue is for my side. Hint to close


Fixed reloading of CollectionBricks

If the CollectionBrick has bricks with heights that are calculated using AutoLayout, the sizeChangedHandler was called during layoutSubviews, which caused a reload but the height wasn’t yet calculated
Fixed by adding a flag that indicates if the CollectionBrick is currently calculating the layout
Fixes #28

Current coverage is 93.76% (diff: 100%)

No coverage report found for master at 3de6a7b.

Powered by Codecov. Last update 3de6a7b...3a9b215


request: new release of dep

This is a request for a new release of dep to be cut. We use dep to vendor dep itself over at CockroachDB, and we'd like to pick up some of the fixes since 0.4.1 was released in January!

yup, coming out this week! i suspect you're going to be very pleased with bits like this: https://github.com/golang/dep/pull/1912#issuecomment-402371810

Awesome! Thanks for all your hard work, @sdboyer. It's much appreciated. dep is a real step up over the past Go dependency managers.


Readonly repositories should make SCM input readonly also

Have a readonly repository. Notice the nice message in the SCM input box.
However the input box should also be readonly, so the placeholder text is always visible and that users can not type in it.
Since the input box is just an editor you should be able to easily set that as an editor option.
fyi @joyceerhl

Closing since there's no more high demand for readonly repositories.

Suggestion: RemoteHub could simply hide the input box, since there is SCM API for that.


cleanup old users accounts

there's a lot of old accounts in language depot, we would like to delete those we deem inactive or unused.
To start with we could delete  accounts older than 6 months, without any projects, of which there are ~17,000
we already have backups of this data, but users can easily recreate their account if they need.

cleanup done, 17,454 accounts without projects deleted. In addition to our backups I made a simple local backup of those users, so I can easily restore them if it causes problems.


fix: messages type generation not watching locale files

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
This has not been tested with restructureDir: false, I expect it might trigger generation twice 🤔

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have added tests (if possible).
[ ] I have updated the documentation accordingly.

This relies on chokidar being installed (nuxt uses it too), should chokidar be added to dependencies or devDependencies? What do you think @userquin


Albany, CA

To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Albany Resident for Defunding the Albany Police
To Mayor Pilch, the Albany City Council, City Attorney, and the Social and Economic Justice Commission:
My name is [YOUR NAME], and I am a resident of Albany. These past weeks, our nation has been gripped by protests calling for a rapid and meaningful reconsideration of the role of policing in communities as well as an end to racism and anti-Blackness in America. Our neighbors have been at the forefront of much of this action.
At the same time, Albany Police has seen an increase in its share of funding over the years, with a worrying increase in the budget allocated for equipment. The 2019-20 budget allocated 30% to the police and only half of that to community development and services. The 2020-2021 budget proposes a ~10% increase in operations.
I call on you to slash the PD budget and instead meaningfully reallocate funds towards social programs and resources that support housing, jobs, education, health care, child care, and other critical community needs. I demand the discontinuation of the use of general fund dollars to pay for settlements due to police misconduct and negligence. I demand a budget that supports community wellbeing, rather than empowers the police forces that tear them apart.
We can be a beacon for other cities to follow if only we have the courage to change.
Sincerely,
[YOUR NAME] [YOUR ADDRESS] [YOUR EMAIL] [YOUR PHONE NUMBER]

on it now


redirect vtcombo stderr to test stderr

We previously redirected vtcombo's output to a file. This was
probably to prevent a spammy output on the test. But troubleshooting
becomes hard if the vtcombo crashes on Travis (or docker).
So, it may be better to live with the spam.

LGTM

YES PLEASE :)


Delete recorded audio from the list of audio files

Hello,
I am creating this issue to make the delete icon on improve the digital stethoscope functional.

I have closed this issue with the following pull request.
https://github.com/Mboalab/Outreachy_May_2023-August_2023-Internships/pull/200