Fix BinExport's "tight loop" feature extraction.

idx.target_edges_by_basic_block_index[basic_block_index] is of type List[Edges]. The index basic_block_index was definitely not an element.

Checklist

[x] No CHANGELOG update needed

[x] No new tests needed

[x] No documentation update needed

thanks!

something seems off here, investigating that this reports functions like this as having a tight loop:

bummer that mypy didn't catch this, oh well

something seems off here, investigating that this reports functions like this as having a tight loop:

There was a bug in the second line of the patch, it should be:
    target_edges = idx.target_edges_by_basic_block_index[basic_block_index]
    if basic_block_index in (e.source_basic_block_index for e in target_edges):

note e.source... versus e.target....
target_edges contains a list of all edges that terminate at the given basic block. So we need to inspect the list for edges that also source from that block.
I've pushed a fix to the branch.


SideMenuContainer: Uncaught TypeError: Cannot read property 'side' of null

ionSideMenuContainer.created always results in Uncaught TypeError: Cannot read property 'side' of null
When removing next code in meteoric_ionic.js, the problem seems to be solved:
Template.ionSideMenuContainer.created = function () { this.side = this.data.side || 'both'; };
Moreover in the app under meteoric/demo, this piece of code does not exists.

This was fixed over the weekend but I haven't pushed a new version to Atmosphere yet. I'll plan on doing that today, but in the meantime you can symlink the repo into your packages directory if you want.


How do you modchart on Psych engine?

What is your question?
How do you modchart on psych engine like agoti and those mods?

There are Four Functions for this:
noteTweenX(tag:String, note:Int, value:Dynamic, duration:Float, ease:String)
noteTweenY(tag:String, note:Int, value:Dynamic, duration:Float, ease:String)
noteTweenAngle(tag:String, note:Int, value:Dynamic, duration:Float, ease:String)
and noteTweenAlpha(tag:String, note:Int, value:Dynamic, duration:Float, ease:String)
tag is a random string for if you want to do something else after the tween has finished. (use function onTweenCompleted([whatever tag you used]))
note is the direction of notes.
Opponent's notes:
0- Left
1- Down
2-Up
3-Right
BF's notes:
4-Left
5-Down
6-Up
7-Right
value is the result of the tween (if you do noteTweenX('example', 0, 50, 1, 'linear'), the opponent's left arrow's x value will be 50.)
duration is how long it takes for the tween to finish.
and ease is the tweening method. (for example, 'linear' and 'quartIn')


IntelliSense for data files does not work on subfolders

When trying to read data files via df = pd.read_csv(), it appears that intellisense does not show subfolders and I have to explicitly trigger intellisense again after there are no matches. It would be nice if I didn't have to explicitly trigger this and it just worked automatically.
https://user-images.githubusercontent.com/35271042/136071529-1149082d-fd54-4e4b-b9ce-58ea65373b09.mp4

This PR will partially resolve this:
https://github.com/microsoft/vscode-jupyter/pull/8247
if you add ' or " to your trigger list.
However after you hit tab on say "./data/, you have to use ctrl+space again (or delete the last /) to get the completions to come up again. Not sure how to get VS code to do autocompletions again when just typing? Maybe delete the final / on the end when inserting the text so the user has to hit / again? (/ becomes a trigger character automatically when strings cause completions)

@rebornix @misolori do you have any suggestions on how to get the secondary directories to work?

We use intellisense for paths in HTML where the suggest widget doesn't go away, which look like files as opposed to intellisense suggestions (notice the file icons vs symbol icons in your gif)
https://user-images.githubusercontent.com/35271042/141827268-856beebe-a7a3-496d-877d-e966ea3c4af4.mp4
y

Thanks. I can look at how the html extension does it. Might be the 'tab' key is a trigger character too.


[BEAM-4229] Downgrade BigQuery to fix Spanner

Both Spanner and BigQuery depend on google-cloud-core. The spanner library requires 1.2.0, the BigQuery library requires 1.27.0, and we include 1.0.2 in our build_rules.gradle. Downgrading the BigQuery library fixes the issue. We can't upgrade spanner because of a conflict with gRPC and Protobuf. This shouldn't be a problem as we are only using this BigQuery library for schema conversion, our actual IO uses an even older library.

Follow this checklist to help us incorporate your contribution quickly and easily:

[X] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[X] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.

[X] Run ./gradlew build to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

run java precommit

This is an impossible change due to various version incompatibility.


feat(stepped-progress-bar): added inverted mode

добавлен inverted-режим

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.005%) to 80.306%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
7856

Relevant Lines:
8875

💛  - Coveralls


NotImplementedError: Layers with arguments in __init__ must override get_config

Related to #32662
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Google Compute Engine (GPU) running Intel(R) Xeon(R) CPU @ 2.30GHz
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.0.0
Python version: 3.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the current behavior
Model is unable to be saved using model.save()
Describe the expected behavior
Model is saved successfully.
Code to reproduce the issue
Code to reproduce this issue can be found on the related Stackoverflow page: https://stackoverflow.com/q/58678836/12315223
Other info / logs
log.txt

This issue has been resolved thanks to a "hacky solution".
Solution:
https://stackoverflow.com/a/58680354/12315223
I'll leave this issue open in-case a better solution is provided on Stackoverflow or here. Feel free to close it here if no solution better than mine is available.

Closing this issue since its resolved. Feel free to reopen if necessary. Thanks!


fix(seed): Fixes database seeding issue

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #17257
Description

Added an environment variable to ignore a section of home.js when
seeding using npm run only-once.

Best first timer PR ever! Thanks a lot @korydondzila

:sparkles: :sparkles: AWESOME. :sparkles: :sparkles:
Thanks @korydondzila !!
Happy Contributing!


mask rcnn damage detection - Are the damages mapped in - 'via region data.json'? file

Hi,
mask rcnn damage detection - Are the damages mapped in - 'via region data.json'?

Yes the via_region_data.json should have annotations for the area of damage

But how to create JSON file through program or manually. Because I have test model with own dataset but it give me error.
How to create json file.


Specify API stability level (IDFGH-2658)

In the documentation, specify whether APIs are stable, and if so, to which extent.
I know there are some functions kept for backwards compatibility that end up getting removed, but is there a repository wide policy on this? What about the order of enum constants, and the order of struct parameters, etc? As in, are new ones always appended?

Hi @KaeLL, good point that this should be documented.
Until this information is added to the docs, here are the answers to your questions:

Within one major version of IDF (e.g. 3.x), the policy is to not remove functions, type definitions, enum constants, defines, header files, and to not change function signatures. There are some exceptions to this rule:

A definition may be removed when a function/type/constant could never have been used successfully. For example, when related functionality was never implemented or was broken.
Functions and types defined in examples subdirectory are not considered part of the API, and may be changed at any time. This also applies to the components included with the examples (such as esp_modem, led_strip and others).

At the same time, we may deprecate functions and types (using a deprecated attribute) and header files (using a preprocessor warning). The functions/types/header files should still work after being deprecated. Deprecated definitions may be removed in the next major version.

We don't have a rule about keeping the order of enum constants and of structure members. We don't make changes to the ordering in patch versions (e.g. 3.3.x), but we don't guarantee that the order will be the same between minor versions. So if you are, for instance, distributing a pre-compiled library for ESP-IDF, this library may need to be recompiled for different minor versions.

We'll add this information to https://docs.espressif.com/projects/esp-idf/en/latest/versions.html#versioning-scheme. Please let me know if there is anything else you think should be mentioned.

We don't have a rule about keeping the order of enum constants and of structure members. We don't make changes to the ordering in patch versions (e.g. 3.3.x), but we don't guarantee that the order will be the same between minor versions. So if you are, for instance, distributing a pre-compiled library for ESP-IDF, this library may need to be recompiled for different minor versions.

This could be a potential bug if you don't guarantee that the order of enum between versions.
The user application may save the enum value as user setting in NVS.
And it even can cause breakage within esp-idf components. e.g. a9b0336ac5f5c20876da868ffa7adc0d38971ef5

Well, that's new.
To be honest, the argument for stable structs declarators' order is just because I like my short initializer lists.
In the end, this is just another example of Hyrum's Law.


🆕 Software Suggestion | Add globalupload.io

Basic Information
Name: Globalupload
Category: Uploader decentralized
URL: https://globalupload.io/
I find this website interesting where you use hash and IPFS technology to upload your files permanently.
Take a look and tell me about it.

It's interesting with a quick glance, but IPFS is very far from private and I don't think this brings anything new over the already listed options.
We do list IPFS in self-contained networks or similar section though.

I like the idea, but it's not open-source, otherwise I would have suggested to add as a "Worth Mentioning" in self-contained network, there are other self-contained networks' utilities listed there.


feat: Adding Active state in any pages to navbar list..

Description
We need to enhance the user experience on our website by adding an active state to the navigation bar for all pages. Currently, when users navigate through our site, it's not always clear which page they are currently on. By implementing an active state in the navigation bar, we can visually highlight the current page, making it easier for users to understand their navigation context.
Screenshots

Ready to Work?

[X] I want to work on this issue

assigned to you.

Bro, I tried to implement the isActive() function but can't get the expected output. It was maybe I'll didn't have enough knowledge in .astros.

Hey, I want to work on this issue ? Also we need to add an hover effect to navbar elements!!

Hey, I want to work on this issue ? Also we need to add an hover effect to navbar elements!!

assigned to you @Pawank06

@Pawank06 any updates?

@Shoyebaktar-shirol any update?

if its still open i would like to work on this

@Ganesh107 I've assigned you.

sorry no idea on astro, I should have checked before asking sorry!!

#201 I think this fixes the issue

@shaurya5 Merged, however it only adds highlighting when hovering over, not an actual colour change when you are on that page.

This is working fine in development but failing in production. I'll check on that.

Hi, can you review #202?

fixed in #201


RedHat.virtio-win version 0.1.215

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

I'll close it for now


Upstream cherry picks

looks like you need to run black, maybe do this on the upstream changes then cherry-pick back in again

Done


Fixed copy and paste errors in Background Services paragraph

Learn Build status updates of commit 05362dc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/mac-install-with-intune.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 26c2516:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/mac-install-with-intune.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


upload image from cordova

Hi there thanks for this great piece of work I've uploaded all files working just fine.
I have one question though I've tried to add the url for upload in my hybrid app in view to upload the image via cordova.
you have mentioned in your readme file the following.
UPLOAD_CODE
If set, will show users a code field in the upload form. If it doesn't match your setting, files won't be uploaded.
If enabled, the Upload API will need the variable upload_code via GET (eg: https://pictshare.net/backend.php?getimage=**https://www.0xf.at/css/imgs/logo.png**&upload_code=YourUploadCodeHere)
what if I am uploading the image from my phone there will be no url per se.
in my file I have the path of the image only.
Any help will be much appreciated

You can post the image as a multipart, add a part with the name postimage with your image as a stream, and POST it to http://HSOT/backend.php.
Here are a example code for Cordova with this plugin:
var uri = encodeURI('http://HSOT/backend.php');
var fileUri = // here the LOCAL uri of your image (the result from the camera or gallery)
var ft = new FileTransfer();
var options = new FileUploadOptions();
options.fileName = 'myPicture';
options.fileKey = 'postimage';
ft.upload(fileUri, uri, function(result) {
    console.log(result);
});

Thanks for your swift response.
I've used your code almost similar to mine I get success callback but nothing has been uploaded I don't know whether the cordova script or the php script.
here's my version still i get upload successful but nothing has been uploaded.
`		function uploadPhoto(imageURI) {
alert("imageURI: " + imageURI);
$('#cameraPic').attr('src',imageURI);
//$("#cameraPic").rotate(90);
	var options = new FileUploadOptions();
	options.fileKey = "postimage";
	var userid = '123456';
	var imagefilename = userid +'generatedimg' + ".jpg";
	alert(imagefilename);
	options.fileName = imagefilename;
	options.mimeType = "image/jpg";
	var ft = new FileTransfer();
	var url = encodeURI('http://site/pictures/backend.php');
	ft.upload(imageURI, url, win, fail, options, true);
	}`

everything is showing fine in each alert ...
Thanks

@hmdadou the FileUpload is a mess, please add a console log in win, fail, and see the server log to see if something happens.

At last there is a light at the end of the tunnel, the problem I had I've already added upload code as soon as I put it false it worked.
Though I've tried to add the code in here
options.fileKey = "postimage&upload_code=YourUploadCodeHere";
didn't work any idea where I could try to add it?

your upload_code must be in your imageURI variable, something like this:
var finalUrl = imageURI + '&upload_code=' + superSecretCode;

Thanks for the suggestion unfortunately didn't work, I actually tried it i get success from cordova but nothing has been uploaded to the server without the upload code works fine.

Please add your working code @hmdadou

I couldn't manage to make it work with the upload code but it is working without it.

Note that the upload code is checked in PHP with $_REQUEST, meaning you can use it as GET or as POST variable.
Also note that @aVolpe made a small mistake in his code above (i edited his reply) and it should look like this:
finalUrl = encodeURI('http://site/pictures/backend.php?upload_code=' + superSecretCode)


RFE: app proxy support for find-fruit

It would be nice to funnel that script through BurpSuite.

Done! Thanks for your contribution.


Libraries-feature [from Mocris File-Explorer] missing (in the Files-app)

What feature or improvement do you think would benefit Files?
The libraries-feature is a very neat feature, because it is very useful.
Requirements
Hence, 'i' (& certainly many other present File-explorer-users) will merely be able to move-on from its sluggish performance (& crazy Mocris-managements' GUI-decisions) WHEN the libraries-feature (in ie. "Files") can be made available.
Files Version
<IP_ADDRESS>
Windows Version
11
Comments

This already exists, Right click the blank space in the sidebar.

Thanks.


Update Node.js to version 20

Update Node.js to version 20

this is updated in our prs, we updated the packages etc accordingly and released the GH on monday,


Fix task latency measurement for better batching behavior

While benchmarking (#204) Clipper, @nishadsingh1 discovered that the latency measurements used to estimate container throughput and therefore batch size include the time a task spends in the model queue waiting to be executed. As a result, we underestimate batch sizes as the queues start to fill up.
@nishadsingh1, you have a fix for this right?

Yep, it's demonstrated here: https://github.com/ucbrise/clipper/pull/204/files#diff-b833f65e61cd8ad63fd6e579acbb3096R459.
I'll get a PR up with the fix + other general benchmarking refactoring when #191 is merged in to avoid a rebase

This will be fixed by #225


I have updated SECURITY.md

I would rather prefer bugs and vulnerabilities being documented here rather unorganised on discord server.
And since it's a static web application, It's hard for any security vulnerability are rare and the common ones are mainly performance issue or bugs related to some action in the application.

I would rather prefer bugs and vulnerabilities being documented here rather unorganised on discord server. And since it's a static web application, It's hard for any security vulnerability are rare and the common ones are mainly performance issue or bugs related to some action in the application.

Ohk, will try to find bugs and fix them in the future.

No I mean you wrote, "If you find a security vulnerability in the Win11react project, kindly send a message to any of the SUPER MODS on the Discord Server instead of posting a public issue in GitHub."
That should be changed to something generic.

No I mean you wrote, "If you find a security vulnerability in the Win11react project, kindly send a message to any of the SUPER MODS on the Discord Server instead of posting a public issue in GitHub."
That should be changed to something generic.

Sorry but can't understand what you want to say :( I mean what do you mean by generic here?

Actually, I wrote so because that happens in most of the Open-source projects. They ask the contributors to report the bug to them on email or stuff and not publically. So that's why I wrote so.

Something as simple as "Check if your issue exist before creating another" and some more in an ordered fashion. It really doesn't matter as long as the message is right.

Oh Ok! got it now

Something as simple as "Check if your issue exist before creating another" and some more in an ordered fashion. It really doesn't matter as long as the message is right.

Can you please merge it now? I wanna be a part of the community!!

vro in 7th line (discuss something) ig it shouldn't be under the 'Reporting vulnerablity' subHeading

gg


rbd: use rados namespace for manager command

Currently, we have a bug that we are not using rados namespace when adding the ceph manager command to remove the image from the trash. This PR adds the missing rados namespace when adding the ceph manager task.
without fix the image will be moved to trash and no task will be added to remove from the trash. it will become ceph responsibility to remove the image from trash when it will cleanup the trash.
workaroud: manually purge the trash
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>

@Madhu-1 Thanks ! , its good to mention the side effect of this bug in the commit as well.

@Madhu-1 will take this in 3.4.0 . agree?

@Madhu-1 will take this in 3.4.0 . agree?

Yes we can take it if everything goes smooth

@Madhu-1 Thanks ! , its good to mention the side effect of this bug in the commit as well.

Done!

@Madhu-1 Thanks ! , its good to mention the side effect of this bug in the commit as well.

Done!

I think this is an important fix, in absense of it , a system has to wait for ceph's purge timeout and may be in between cluster becomes full..etc situations can pop up! updating the PR with the sideeffect and marking for the release take! :+1:

Does this fix #1728?

No both are different problems. This is specific to rados namespace and other one is generic problem

/retest ci/centos/upgrade-tests

/retest ci/centos/upgrade-tests

upgrade-tests-cephfs: failed with #2264
upgrade-tests-rbd failed to create configmap (logs):
Jul 27 08:39:26.893: INFO: Running '/usr/bin/kubectl --server=https://<IP_ADDRESS>:8443 --kubeconfig=/root/.kube/config --namespace=cephcsi-e2e-590f689e create -f -'
Jul 27 08:39:34.533: INFO: rc: 1
Jul 27 08:39:36.534: INFO: Running '/usr/bin/kubectl --server=https://<IP_ADDRESS>:8443 --kubeconfig=/root/.kube/config --namespace=cephcsi-e2e-590f689e create -f -'
Jul 27 08:39:37.708: INFO: rc: 1
Jul 27 08:39:37.708: INFO: will run kubectl (create) again (10 seconds elapsed)
Jul 27 08:39:37.708: FAIL: failed to create vault configmap failed to run kubectl: error running /usr/bin/kubectl --server=https://<IP_ADDRESS>:8443 --kubeconfig=/root/.kube/config --namespace=cephcsi-e2e-590f689e create -f -:
Command stdout:

stderr:
Error from server (AlreadyExists): error when creating "STDIN": configmaps "ceph-csi-encryption-kms-config" already exists

error:
exit status 1

Made a small change in e2e.

/retest ci/centos/upgrade-tests-rbd

/retest ci/centos/upgrade-tests-rbd

Error from server (AlreadyExists): error when creating "STDIN": configmaps "ceph-csi-encryption-kms-config" already exists

/retest ci/centos/mini-e2e-helm/k8s-1.20

/retest ci/centos/mini-e2e-helm/k8s-1.20

`failed to create vault statefulset failed to run kubectl: error running /usr/bin/kubectl --server=https://<IP_ADDRESS>:8443 --kubeconfig=/root/.kube/config --namespace=cephcsi-e2e-5347f172 create -f -:
Command stdout:
stderr:
Error from server (AlreadyExists): error when creating "STDIN": serviceaccounts "rbd-csi-vault-token-review" already exists
Error from server (AlreadyExists): error when creating "STDIN": clusterroles.rbac.authorization.k8s.io "rbd-csi-vault-token-review" already exists
Error from server (AlreadyExists): error when creating "STDIN": clusterrolebindings.rbac.authorization.k8s.io "rbd-csi-vault-token-review" already exists
`

/retest ci/centos/mini-e2e/k8s-1.20

/retest ci/centos/mini-e2e/k8s-1.20

`Jul 27 18:05:17.172: FAIL: failed to create vault statefulset failed to run kubectl: error running /usr/bin/kubectl --server=https://<IP_ADDRESS>:8443 --kubeconfig=/root/.kube/config --namespace=cephcsi-e2e-4b896a81 create -f -:
Command stdout:
stderr:
Error from server (AlreadyExists): error when creating "STDIN": serviceaccounts "rbd-csi-vault-token-review" already exists
Error from server (AlreadyExists): error when creating "STDIN": clusterroles.rbac.authorization.k8s.io "rbd-csi-vault-token-review" already exists
Error from server (AlreadyExists): error when creating "STDIN": clusterrolebindings.rbac.authorization.k8s.io "rbd-csi-vault-token-review" already exists
error:
exit status 1`

/retest ci/centos/mini-e2e/k8s-1.19

/retest ci/centos/mini-e2e/k8s-1.19

`Jul 27 18:20:30.039: INFO: >>> kubeConfig: /root/.kube/config
Jul 27 18:20:30.184: INFO: attach command running after restart, runningAttachCmd: rbd-nbd --id cephcsi-rbd-node -m rook-ceph-mon-a.rook-ceph.svc.cluster.local:6789 --keyfile=/tmp/csi/keys/keyfile-290608680 attach replicapool/csi-vol-d7b4d211-eefe-11eb-b59c-4a221eddfd05 --device /dev/nbd0 --try-netlink --reattach-timeout=300
Jul 27 18:20:30.187: INFO: ExecWithOptions {Command:[/bin/sh -c echo 'Hello World' > /var/lib/www/html/test] Namespace:rbd-694 PodName:csi-rbd-demo-pod ContainerName:web-server Stdin: CaptureStdout:true CaptureStderr:true PreserveWhitespace:true Quiet:false}
Jul 27 18:20:30.187: INFO: >>> kubeConfig: /root/.kube/config
Jul 27 18:20:32.224: INFO: stdErr occurred: /bin/sh: 1: cannot create /var/lib/www/html/test: Read-only file system
Jul 27 18:20:32.224: FAIL: failed to write IO, err: command terminated with exit code 2, stdErr: /bin/sh: 1: cannot create /var/lib/www/html/test: Read-only file `


Created the main file to run the REST services

REST URL for GET and POST methods

This may conflict with the server.py file I created

Ya I just saw that....ill check the server fle
On Wed, Oct 11, 2017 at 6:44 PM, Brenton Durkee<EMAIL_ADDRESS>wrote:

This may conflict with the server.py file I created
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nyu-devops-golf-team/wishlist/pull/13#issuecomment-335970035,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aea0tCLhpB0IMekNcxloxf0ucf3cwRhfks5srUTHgaJpZM4P2OKO
.

This pull request is not required as another issue had taken care of the main file to run the APIs


[Failing Test]: beam_PerformanceTests_PubsubIOIT_Python_Streaming failing due to #25617

What happened?
Tests failing. Error message in Dataflow UI (not shown in log)
Could not load main session. Inspect which external dependencies are used in the main module of your pipeline. Verify that corresponding packages are installed in the pipeline runtime environment and their installed versions match the versions used in pipeline submission environment. For more information, see: https://beam.apache.org/documentation/sdks/python-pipeline-dependencies/
Traceback (most recent call last): File "/usr/local/lib/python3.7/site-packages/apache_beam/runners/worker/sdk_worker_main.py", line 117, in create_harness _load_main_session(semi_persistent_directory)
File "/usr/local/lib/python3.7/site-packages/apache_beam/runners/worker/sdk_worker_main.py", line 342, in _load_main_session pickler.load_session(session_file)
File "/usr/local/lib/python3.7/site-packages/apache_beam/internal/pickler.py", line 65, in load_session return desired_pickle_lib.load_session(file_path)
File "/usr/local/lib/python3.7/site-packages/apache_beam/internal/dill_pickler.py", line 313, in load_session return dill.load_session(file_path)
File "/usr/local/lib/python3.7/site-packages/dill/_dill.py", line 368, in load_session module = unpickler.load()
File "/usr/local/lib/python3.7/site-packages/dill/_dill.py", line 472, in load obj = StockUnpickler.load(self)
File "/usr/local/lib/python3.7/site-packages/dill/_dill.py", line 462, in find_class return StockUnpickler.find_class(self, module, name) ModuleNotFoundError: No module named 'hamcrest'

Issue Failure
Failure: Test is flaky
Issue Priority
Priority: 1 (unhealthy code / failing or flaky postcommit so we cannot be sure the product is healthy)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

CC: @psobot @tvalentyn

other integration tests get hamcrest through https://github.com/apache/beam/blob/master/sdks/python/scripts/run_integration_test.sh#L219 , not sure why PubsubIOIT doesn't, perhaps it's launched differently.

Yes, load test / performance test is an executable python (having if ... __main__ block). hamcrest is only used in assert locally so that specific test actually does not need to save main session.


Packet: Allow to specify kubernetes version

Upgrade kubernetes to v1.14.3 and extract version to variables.

We update the kubernetes version via terraform-render-bootkube, usually. Like here: https://github.com/kinvolk/terraform-render-bootkube/commit/a80eed2b6ac489243a6454dc2f46b17eefa7d84d
Would it be okay for you to update the kubernete version via the bootkube repo but not have a variable?

In this way I removed duplication of usage kube version. You can change in one place, and everything related would be updated. It would remove human factor, that you should change in multiple places same value.

Sure, but just to be clear: this shouldn't be changed without changing terraform-render-bootkube repo too, and changing the bootkube.tf in the module to use the new commit, IIUC.
As this is easy to script, I'm not sure it is worth and don't want to confuse the user thinking he can change the kubernetes version just changing that variable (as bootkube change is needed too, IIUC). It will also add more diff to typhoon, so not sure this is a clear win.
Let me ask others what they think and get back to you.

@rata here is bootkube changes https://github.com/pubnative/lokomotive-kubernetes/blob/master/packet/flatcar-linux/kubernetes/bootkube.tf#L33

updated PR

I'm not sure this will work either, as when updating kubernetes we might have to do some changes on bootkube.
Furthermore, an example is just around the corner: it is needed to change bootkube for the 1.14 to 1.15 upgrade. And with this path it will break.
The changelog here says hyperkube binary should chenge their names: https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.15.md#apps (see the PR here: https://github.com/kubernetes/kubernetes/pull/76953). Therefore, we need to do something like this on terraform-render-bootkube when updating the kubernetes version: https://github.com/poseidon/terraform-render-bootkube/commit/62df9ad69cc0da35f47d40fa981370c4503ad581. If not, cluster creation will be broken.
I think the upgrade path that works, as I said, is: update terraform-render-bootkube, update the reference here to the new version terraform-render-bootkube, update the manifests too. This can be auto-generated/scripted.
Am I missing something?
What is the intention of the patch? To simplify changing the kubernetes version used?

I see, so new bootkube would work only with version v1.15 and higher.
But still it is not blocker to use variables for kube version. With this feature this repo is more independent from changes in terraform-render-bootkube. You would not need to update everytime module dependicies, and in same time have a new or old version of kube, with new changes of bootkube.

What lead you to write the patch? To simplify changing the kubernetes version used?

I tried to upgrade from kube 1.14.1 to 1.14.3, and spent some time to find all places where I should apply changes. Then I came to bootkube module, that has different version.
I use Don't Repeat Yourself practice in the code. We require changes in one place, for related things.

I see, so new bootkube would work only with version v1.15 and higher.

No, if the new bootkube, if that change is made, will work with older Kubernetes versions too (all maintained versions). Just that those commands were depprecated and completely removed in 1.15, but the new commands already worked in older kubernetes versions. (there is a deprecation period in which kubernetes releases support both, until one is fully removed, to allow users to transition easily).
So, a new bootube with that change will work fine, even if we are using an older k8s version.

But still it is not blocker to use variables for kube version. With this feature this repo is more independent from changes in terraform-render-bootkube. You would not need to update everytime

The thing is: the repo is not totally independant and needs to be in sync with terraform-render-bootkube right now.

What lead you to write the patch? To simplify changing the kubernetes version used?

I tried to upgrade from kube 1.14.1 to 1.14.3, and spent some time to find all places where I should apply changes. Then I came to bootkube module, that has different version.
I use Don't Repeat Yourself practice in the code. We require changes in one place, for related things.

I see a point on switching kubernetes releases on an easier way. But I must say I don't think it is really repeating ourselves if we can just do it with a script. Also, having a variable exposed to the user means the user can change it. But that will fail and create a bad user experience, the user might specify 1.15.0 and fail. I do not want to expose that to the user if it won't work reliably.
As there might be breaking changes (always some change is there) in new k8s releases and need to adapt, usually you support up to some version. Other tools, like kops, use compatibility matrix for example: https://github.com/kubernetes/kops#compatibility-matrix.
Right now we are doing some even simpler: this installs a particular kubernetes version (v1.14.1, for example). That is it for now.
I see value on changing kubernetes releases, for example, but right now it needs to be in sync with terraform-render-bootkube. Maybe it is easier to support just chaning patch releases... But not sure we want to go down that route now, and just install one version for now?

can we close this PR ?

@miry friendly ping? :)

@rata Sure, if you think it is ok, we can close.


Update: KDE.kile version 2.9.93

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 471
Microsoft Reviewers: Open in CodeFlow

/azp run


Add ::new! and ::run! class methods for actions

Also: change the behavior of the #run! instance method to no longer check for valid params (which should now be done via ::new! instead) and to start checking for an absence of failure conditions.

Closing in favor of https://github.com/davidrunger/active_actions/pull/103 (to change the branch name).

Closing in favor of https://github.com/davidrunger/active_actions/pull/103 (to change the branch name).


[BUG]: Wrong CUDA path on Linux

Description
The path for the CUDA drivers on Linux is wrong. Not sure, but with recent versions of CUDA drivers NVIDIA change the path.
On my Debian installation the path is
➜  ~ which nvcc
/usr/bin/nvcc

Reproduce

cd examples/rust/ntt/ && cargo run

Expected Behavior
Please provide a clear and concise description of what you expected to happen.
Environment
Please complete the following information:
OS + Version: Debian 12
Cargo Version: cargo 1.79.0
GPU type: NVIDIA A2000 6GB

Hi, @FilipLaurentiu
What does nvcc --version say to you?

I would suggest to use a symbolic link for now. We will address the issue.

Hi, @FilipLaurentiu What does nvcc --version say to you?

➜  ~ nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2022 NVIDIA Corporation
Built on Wed_Sep_21_10:33:58_PDT_2022
Cuda compilation tools, release 11.8, V11.8.89
Build cuda_11.8.r11.8/compiler.31833905_0

it's a standard installation following this


Does not work in IE11

Current behavior
In IE11, cash does not work bacause of Expected identifier error.
This error is caused by remaining class syntax in ./dist/cash.esm.js (I'm using webpack v4 and it imports esm version of cash).
https://github.com/kenwheeler/cash/blob/fbe3dc0966653b67e464f32f5efa3b1f5e3a68c4/dist/cash.esm.js#L16
class syntax seems to be appeared from 3.0.0( https://github.com/kenwheeler/cash/commit/1eaddca7a63332410ba9d3f01d304137baafa97c#diff-33b3d582ba732231f8b49c1ba1fcf7f8R16 ). Is it what should be?

Expected behavior
Works well in IE11.

I have the same issue.
Following is a minimum example.
app.js
import $ from 'cash-dom';

$('#cash').on('click', (e) => {
  $('#body').toggle()
});

webpack.config.js
const path = require('path');

module.exports = {
  entry: path.resolve(__dirname, 'app.js'),
  module: {
    rules: [
      {
        test: /\.js$/,
        use: [
          {
            loader: 'babel-loader'
          },
        ],
        exclude: /node_modules/
      }
    ]
  },
  resolve: {
    extensions: [
      '.js'
    ]
  },
  output: {
    filename: 'bundle.js',
    path: path.resolve(__dirname, 'dist')
  }
}

index.html

package.json
{
  "scripts": {
    "build": "webpack --mode development"
  },
  "dependencies": {
    "@babel/core": "^7.4.0",
    "cash-dom": "^3.1.0"
  },
  "devDependencies": {
    "babel-loader": "^8.0.5",
    "webpack": "^4.29.6",
    "webpack-cli": "^3.3.0"
  }
}

npm install && npm run build

This may fix by changing pacco plugin settings (enable babel or typescript target to es5 ).

The esm build is meant to be used in esm-supporting environments. If you want to use it in IE11 I think you'd have to configure webpack to transpile it to ES5 via babel or something. But I'm not hugely familiar with webpack in this regard, am I missing something?
Alternatively I think one could instruct webpack to load cash/dist/cash.min.js file when requiring cash, but that would be more of a workaround then the proper way of doing this.

@fabiospampinato Thanks, It makes sense :)
Exactly as you said, IE11 is not esm-supporting environment and importing cash/dist/cash.min.js works in IE11. I have not tried to transpie with babel yet but it probably works too.

I'm closing this then. It will be helpful to other users if you guys shared a webpack configuration that handles the transpilation properly.

@fabiospampinatoThanks, I understand. I'm grateful for your support.

Using resolve.mainFields like mainfilelds: ["main", "module"] may be the optimal solution for this issue.
ref: https://github.com/webpack/webpack/issues/1979

@fabiospampinato I'd like to reopen this. Webpack relies on package.json of various packages to be able to resolve the module. It has a sensible default for web target: browser followed by module and finally main.
cash advertises support for "modern browsers (IE10+)" and its package.json contains just main and module.
"main": "./dist/cash.js",
"module": "./dist/cash.esm.js"

The proposed solution (@makotot) suggests changing the Webpack configuration for every package to look at main field first and module field next, which would work for cash, given the contents of its package.json file.
However, adding a web field to point to dist/cash.js would work just as well, and would work with no global changes required on Webpack configuration.
"web": "./dost/cash.js",
"main": "./dist/cash.js",
"module": "./dist/cash.esm.js"

@ipavlic Oh cool, do you happen to know where this "web" property is documented?

@ipavlic Wouldn't this mean though that dist/cash.js will basically always be the file that will be used by webpack? I'm not sure this will work if the file is being imported as 'import $ from 'cash-dom'` for instance 🤔 Unless maybe webpack can just figure things out?

Sorry, it's browser, not web. Webpack documents resolve at https://webpack.js.org/configuration/resolve/

resolve.mainFields
[string]
When importing from an npm package, e.g. import * as D3 from 'd3', this option will determine which fields in its package.json are checked. The default values will vary based upon the target specified in your webpack configuration.
webpack.config.js
module.exports = {
//...
resolve: {
mainFields: ['browser', 'module', 'main']
}
};

@ipavlic Wouldn't this mean though that dist/cash.js will basically always be the file that will be used by webpack? I'm not sure this will work if the file is being imported as 'import $ from 'cash-dom'` for instance 🤔 Unless maybe webpack can just figure things out?

I think it just works.

Another alternative is to polyfill in Webpack:
entry: ["@babel/polyfill", 'your-entry-file.js']

Facing the same issue here. @fabiospampinato could you please merge and release #291 ? I can confirm that it's the correct solutions for this issue.

Right now, I have to use import $ from 'cash-dom/dist/cash.js' instead of import $ from cash-dom`

@limonte I'm quite busy right now, I can't properly test this at the moment. I'd be happy to merge the change if it allows the library to be imported under all the following scenarios:

CommonJS environment (require)
ESM in Node.js (import)
ESM in TypeScript (import)
Script modules (script[type="module"])

If somebody could provide me some sample configs/projects that show that these all work, so that I could just try them very quickly, I'll probably be able to merge the change in faster.

I've build a thing for checking that cash can be properly imported under the most common environments (attached below, it may come in handy in the future).
This issue has ben fixed by https://github.com/kenwheeler/cash/pull/291. Basically the only problem was when importing cash via require and when bundling that with webpack. I'm not quite sure why that was happening to be honest.
The bottom line is that importing cash should now work everywhere, and when explicitly importing cash it won't pollute the global namespace. If you want $ to be globally available just do it yourself: window.$ = $.
New release coming soo (I want to take care of a few other issues first).
cash-dom-import-test.zip

Isn't zip too old-school? :) Why not the repo under your username?

I don't know it's probably garbage, but I didn't want to just delete it.


No error checking

Errors in the changelog in other mods causing this mod to error in the loading of the file

Fixed with PR1


Should put and get do implicit conversions?

When we call put and get we expect the cast types to match exactly:
https://github.com/shogun-toolbox/shogun/blob/93a649bb9bacfd342875eae40115d37e74d1f0f8/tests/unit/base/SGObject_unittest.cc#L486-L487
However, does this make sense? In C++ it is valid for me to call a function with an implicit conversion:
void f(double a);
int main()
{
    f(1);
}

Similarly, it is valid to implicitly cast a return type (so a getter can return int, but we store it as float64_t). So the test from above would fail, because the types would be casted implicitly, if we were using a getter (rather than SGObject::get):
float64_t data = get_int(); // no compiler error

From a technical perspective this is possible to implement. My question is: would it bring more bugs than it would avoid? For example, on one hand a user might lose information by casting float to int, but on the other the program won't crash after 10 hours of training because of the type mismatch. I would also not want shogun to become like JS where everything has some result because the type system is too flexible.

I'd prefer if we had a more relaxed typing with the parameter framework. Then we also wouldnt have to do these horrendeous workarounds in swig that I added to avoid problems with this.
E.g. in java everything is a float64, in octave matrices with one element automatically get converted to scalars, 1.0 gets converted to int, etc.
I'd basically expect the same implicit casting as C++ compilers do?

I guess that means every arithmetic type can be casted to any other arithmetic type?

what happens in c if you do int a = 3.4?

It depends. With -Wconversion you get a warning and I guess if you combined that with -Werror you get a compiler error. Without that warning the floating point type is floored, ie a=3

I see....mmmh. Probably we would want to follow the behaviour that we have in our build, i.e. without those flags?

Yes, I agree. I think that at least all integer types should be implicitly converted between them using the safe conversion helpers we already have. The same for floating points. Just from int to float I am not sure about...

What I would do then is to have everything that doesnt loose information automatically. And then the ones which might loose information can simply print a runtime warning from shogun


adding lordswoodtrust.co.uk to the school list

as my daughters email domain is lordswoodtrust.co.uk, adding this to your school list

@alikny Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.

Hello! I'm not familiar with GitHub so excuse me If I'm doing this wrong. But please add the domain @rootsivy.edu.pk to your list. I've attached proof as well as the folder containing the .txt file.

Here are the two most used URLs: https://www.ivycms.com.pk, https://rootsivyintschools.edu.pk/index.html
IMAGE of MS Teams account showing roots ivy offers computer science.

IMAGE of MS Teams Email of the teams showing the school officially uses that domain:

Folder:
rootsivy.txt


<IP_ADDRESS>

test stubs for all three attitude models; small pattern + rule changes

@thareUSGS this new commit applies fixes from your comments.


Newly created Addresses via in-app CLI do not appear in receive list

Actual behavior:
When creating a (legacy) DFI address using the in-app CLI, the newly created address does not appear in the "receive" list of the app:
$ getnewaddress "name" legacy
When using $ dumpprivkey <address> a private key gets printed.
Restarting the app doesn't bring up the new address, either.
Expected behavior:
The new address should appear in the "receive" list
How to reproduce it (as minimally and precisely as possible):
In-App CLI:
$ getnewaddress "name" legacy
--> Results in an <address>
$ dumpprivkey <address>
--> Results in a private key
Navigate to "Balances" -> "DFI" -> "RECEIVE" -> no new <address> in List
What are your environment parameters?

DeFi Wallet Version: 2.5.1
Operating System (OS): MacOS 11.3, MacBook Air with M1 chip
Current free storage capacity: 853 GB

Anything else we need to know?
Same behavior when creating a regular (non-legacy) address

Hi @DerFuchs , if you want to create an address via CLI:
In CLI:

Unlock your wallet if you've a passphrase
getnewaddress — this would be your newly created address
dumpprivkey <dULR...>

To view it in app:
4. Go to Receive > NEW ADDRESS
5. Toggle-off Automatically generate a new address

Alternatively, you can do it via in app:  Go to Receive > NEW ADDRESS

For how-to enquiries, you can ask the community via DeFiChain Telegram (EN)


Task secret management

The current mechanism used to pass environment variables to task scripts stores the values in plain text as part of the instance's cloud-init startup script.
Instead, it should be using permission_set from #550 and:

SecretsManager for aws
Key Vault for az
Secret Manager for gcp
Secrets[^1] for k8s

This brings the high-level debate of whether automate the whole permissions set and secret manager creation process or leave it to the user as part of a separate lifecycle, similarly to #299.
[^1]: Semantically equivalent, but insecure by default: securing them is not our responsibility, but the cluster administrators'

Example
terraform {
  required_providers {
    aws = {
      source = "hashicorp/aws"
    }
    iterative = {
      source = "iterative/iterative"
    }
  }
}

provider "aws" {
  region = "us-east-1"
}

provider "iterative" {
}

resource "iterative_task" "example" {
  cloud          = "aws"
  permission_set = aws_iam_instance_profile.example.arn

  environment = {
    SECRET_ID  = aws_secretsmanager_secret.example.arn
    AWS_REGION = data.aws_region.current.name
  }

  script = <<-END
    #!/bin/sh
    sudo apt-get update
    sudo apt-get install awscli --yes
    echo reading the secret value...
    aws secretsmanager get-secret-value --region $AWS_REGION --secret-id $SECRET_ID --query SecretString --output text
  END
}

resource "aws_secretsmanager_secret" "example" {
  name = "example"
}

resource "aws_secretsmanager_secret_version" "example" {
  secret_id     = aws_secretsmanager_secret.example.id
  secret_string = "v3rys3cret" # don't hardcode this here, use a variable instead
}

resource "aws_iam_instance_profile" "example" {
  name = "example"
  role = aws_iam_role.example.name
}

resource "aws_iam_role" "example" {
  name               = "example"
  assume_role_policy = data.aws_iam_policy_document.example_assume_role.json
}

resource "aws_iam_role_policy" "example" {
  name   = "example"
  role   = aws_iam_role.example.id
  policy = data.aws_iam_policy_document.example.json
}

data "aws_iam_policy_document" "example_assume_role" {
  statement {
    actions = ["sts:AssumeRole"]

    principals {
      type        = "Service"
      identifiers = ["ec2.amazonaws.com"]
    }
  }
}

data "aws_iam_policy_document" "example" {
  statement {
    actions   = ["secretsmanager:GetSecretValue"]
    resources = [aws_secretsmanager_secret.example.arn]
  }
}

data "aws_region" "current" {
}

I like this idea, and it would be the creating credentials that would need to access the various secret managers?

If we automate the whole process as part of the task lifecycle, it would manage the following additional resources:

Permission set with access to:

Object storage: read/write
Secret storage: read
Instances: terminate

Secret storage + secrets

Going to use aws terminology
I think we discussed that we should not edit, create, or delete roles.
I feel that if a user creates permission_set with access to Secrets Manager the connotation is that their code/script is going to call that service.
If they list an arn  to SecretsManager in the hcl the connotation is that we query that service.

I think we discussed that we should not edit, create, or delete roles.

I don't recall discussing this, but we're on the same page: if task created roles, it would need administrator-level permissions, and that's a clear “no” for teams.

I feel that if a user creates permission_set with access to Secrets Manager the connotation is that their code/script is going to call that service.
If they list an arn to SecretsManager in the hcl the connotation is that we query that service.

Then, do you propose to add a new secret_storage field taking an ARN or whatever?

I think that a secrets block could have some value. Where keys get populated as env vars.
secrets {
  SNOWFLAKE_USER = "arn"
  SNOWFLAKE_PASS = "arn"
  GITHUB_BOT_PAT = "arn"
}

However I don't really see this being a highly demanded feature, it would be nice and I'm sure that it could/would get used, but you can also just use the environment to pass the arn and code just load it as well...
I feel that there could be some confusion about who has access to the secrets, ie the creating user's credentials or the tasks assigned credentials. Maybe I am overcomplicating this?

What if we use the environment argument for everything?
Unencrypted by default
resource "iterative_task" "example" {
  ···
  environment = {
    NAME = "value"
  }
}

Encrypted on demand
resource "iterative_task" "example" {
  ···
  key = "arn:"
  environment = {
    NAME = "value"
  }
}

When users provide a key identifier (e.g. ARN to SecretsManager or KMS), the environment file is (en|de)crypted with that key.

I like this idea! Now we just need some log masking for them as well 😅

See also mozilla/sops


[013] As an experimenter, I want to be able to generate QR codes

Completed and pull request is up


Allow FillFlow layouts to be centered in their parent

Automated calculation of MaximumSize.
Needs a visualtest, but PRing for feedback since I'm not sure this is the correct way to achieve this.
Example:
https://streamable.com/p43d4
Used to maintain content in the center of its parent rather than assuming the full width and making Anchor/Origin non-useful.

As pointed out by @smoogipoo , FillFlowContainer already handles these scenarios if all children are set to an Anchor with Anchor.x2 present (centered). This addition is not required.


Increase max brush entity limit

The engine supports 512, and since bmodels aren't actually stored/communicated in CS even though the limit was checked against a CS limit, there should be no issues in increasing this to the same value as engine uses.
I will still need to test how this behaves if someone tries to for example add 500 brush entities with a model2 key set, because registering models is still limited by CS.

I will still need to test how this behaves if someone tries to for example add 500 brush entities with a model2 key set, because registering models is still limited by CS.

This should actually be fine, the server never let's you register more than MAX_MODELS anyway, so this would just error with configstrings overflow.
https://github.com/etjump/etjump/blob/85e707a6fcf5f5bc9ff285efc66c7286735a7b55/src/game/g_utils.cpp#L81-L115


STOP sign dataset

Hello,
I would like to compare with your method in the gtsrb STOP sign database. Could you please direct me to the exact version you tested in your paper? Thanks.
Pram

Hi Pramuditha,
I've send you an e-mail with a link to the adversarial attacks we generated for the ICML paper experiment.
Cheers,
Lukas

Hi Pramuditha,
I'm curious about the method of generating STOP adversarial attacks.Do you use the foolbox or other tools?Could you please direct me to the exact version you or author tested in experiment? Thank you very much.
chunyun

Hello,
I have used this dataset for experiments in a unpublished work. I obtained it from Lukas. Thanks.
Best,
Pramuditha

From: chunyuner<EMAIL_ADDRESS>Sent: Sunday, November 3, 2019 11:07 PM
To: lukasruff/Deep-SVDD<EMAIL_ADDRESS>Cc: PramuPerera<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [lukasruff/Deep-SVDD] STOP sign dataset (#6)
Hi Pramuditha,
I'm curious about the method of generating STOP adversarial attacks.Do you use the foolbox or other tools?Could you please direct me to the exact version you or author tested in experiment? Thank you very much.
chunyun
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/lukasruff/Deep-SVDD/issues/6?email_source=notifications&email_token=AG6NJ4AWLF4NZFEEUGFRFRDQR6NXXA5CNFSM4GAK6XLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC6G3GI#issuecomment-549219737, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AG6NJ4EN5MI6BI56PBKRFCLQR6NXXANCNFSM4GAK6XLA.


build query from json

Hello,
Thank you for this package. The query builder is good to construct it at the server, but it would be great to have a way to build query from a specification, like json object. say, we have a query struct
type filter struct {
	fieldName string
	op        string
	value     interface{}
}

type Query struct {
	filter []filter
	order  []order

	limit  int
	offset int

	keysOnly bool
}

if, there is a way to build a query, from this struct, then we can construct query from a json/client request. Do you think it is viable, and any plan for it. Else, how do you typically handle a query from client?
thanks again,
bsr.

Sorry, it is not planed.
I have a question. What is the client? golang or browser(javascript)?

javascript. I use with a react application, and like to build the query from the json payload. I may do the validation of the payload (like filter fields) at the server. It may be nice to have generated code to map this  to a proper smg search query.

Hi. I am closing this as I found a solution outside this package. I use a recursive structure to model the query and accept it as json from client. Them map it to gae query and use it. thanks for providing this package and the examples been quite useful.


cors issue in videojs

Can i play m3u8 video by this url i already fetch cors issue in reactjs

You can if you self-host CORS Anywhere. m3u8 URLs are blocked on the public demo because of previous abuse.

please explaing it i dont
Have any idea about this...

See the README for instructions on self-hosting.


docs: 🇩🇪 added german locale link to config.ts;

Description
Adding german docs translation, following the guidelines using the vite-docs-template.

Closes #14336.

Repository: https://github.com/rojadesign/vite-docs-de
Live App: https://vite-docs-de.netlify.app/

What is the purpose of this pull request?

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

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

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

Thanks a lot, @rojadesign! The work done in the translation looks good to me but I don't speak German so I'll let this one open for a bit to let others check it out. I'll contact you next week in Discord and if things look good, we can transfer the repo to the vitejs org.
Did you manage to get the ryo-cho bot setup? I don't see PRs in your repo from it.

@patak-dev thanks for your feedback, sounds good!
Yes, I did it like the example in the contributing guide, but I also noticed there were no PRs coming in. Can you tell right away what the problem is? Otherwise I'll look into it again.

Thanks for checking it out @Shinigami92!

Yes, I did it like the example in the contributing guide, but I also noticed there were no PRs coming in. Can you tell right away what the problem is? Otherwise I'll look into it again.

@mrcego could you help us here to see if you see a diff from the setup you did?

Sure, will be back later with news.

Look like ryu-cho config is well done.
@rojadesign I'm just curious: Have you created repo access token?

Ah, that's it! No, for whatever reason I forgot. Will take care of it, thank you!

@mrcego I tried both options using a machine user as well as the default github-action bot. Both run fine but neither create pull requests. 😞 I compared my configuration to the other translations and don't see much difference to mine. I also made sure the repo settings allow github actions to create pull requests, which was disabled before by default. I'm stuck a bit now and don't know what more I could try. Maybe @tony19 can help out since he wrote the docs template?

@mrcego I tried both options using a machine user as well as the default github-action bot. Both run fine but neither create pull requests. 😞 I compared my configuration to the other translations and don't see much difference to mine. I also made sure the repo settings allow github actions to create pull requests, which was disabled before by default. I'm stuck a bit now and don't know what more I could try. Maybe @tony19 can help out since he wrote the docs template?

The only thing come to my mind is wait until some new PR is coming. That's happened before and in that point ryu-cho start to works.
Question: what happen if you try to start job manually?

@rojadesign Thanks for the translations!
I think something is wrong with your ACCESS_TOKEN. Despite the green check, your ryu-cho action is actually failing with bad credentials.
Also be aware that ryu-cho will fail to create a PR when there's a merge confict: https://github.com/vuejs-translations/ryu-cho/issues/9

Question: what happen if you try to start job manually?

@mrcego thanks for the input! I switched to the default github actions bot (with GITHUB_TOKEN) and started ryu-cho again in debug mode, but there seems to be everything fine.

Also be aware that ryu-cho will fail to create a PR when there's a merge confict: [vuejs-translations/ryu-cho#9]
(https://github.com/vuejs-translations/ryu-cho/issues/9)

@tony19 thanks for the info! Maybe there are merge conflicts, I will check this further.


Allow Fractal to be able to send jobs to different slurm clusters within a federation

From the resource infrastructure point of view, running things with the current Drogon architecture means making luigi send jobs to either a dedicated CPU VM or to a physical workstation that has a Quadro RTX 8000 for processing segmentation with pre-trained neural networks.From the resource infrastructure point of view, running things with the current Drogon architecture means making luigi send jobs to either a dedicated CPU VM or to a physical workstation that has a Quadro RTX 8000 for processing segmentation with pre-trained neural networks.
From the perspective of the lab, it would be useful to expand our minislurm cluster in order to add the gpu workstation to it as a new node so we could run with the same resource capacity as Drogon currently.
One logical step is then to make the large FMI cluster and our minislurm part of the same federation, so that jobs can be sent either to our local or to the institute wide slurm depending on some basic parameters (e.g. queue size, resource demands, etc).
I have been checking the slurm documentation and the slurm.conf docs it looks like creating a federation of clusters should conceptually not be so difficult as long as ideally all of the related clusters run the same version of slurm (which will be our case very soon as we are currently updating the clusters). Each cluster keeps its own database. Creation of a federaion is done with the sacctmgr  command and in practice Fractal would need to add a -M,--clusters=<cluster_list> and the --cluster-constraint=[!]<constraint_list> options.
This would be very useful for us, as we would then have the flexibility to run Fractal within our own slurm resources or be able to outsorce larger jobs to the large cluster without the need of having two independent Fractal installations.
I do see however that this can be thought right now as a rather more specific implementation case, so here is the important question: How much work would this mean for the fractal-server? Is there anything in the current architecture that would prevent us from having this road available?

Are those options (-M,--clusters=<cluster_list> and --cluster-constraint=[!]<constraint_list>) related to sbatch or to sacctmgr?
Briefly: whatever you can do with a sbatch script should be "easily" doable within fractal:

In the current version, the sbatch script is generated by parsl, which lets you add any additional configuration (see parameters scheduler_options in SlurmProvider).
In a possible future version of fractal-server, where we could use a simpler SLURM interface, it is reasonable that we'll always be able to provide additional sbatch parameters.

Some other questions:

Do the two different clusters share a filesystem? At least in the current version, this is strictly needed.
Do the two different clusters share users? Otherwise we will need to associate one fractal user to two cluster users (one per cluster)

We already discussed this in calls, but for completeness:
Those should be sbatch options, right?

Do the two different clusters share a filesystem? At least in the current version, this is strictly needed.

Yes they do

Do the two different clusters share users? Otherwise we will need to associate one fractal user to two cluster users (one per cluster)

Share users as in usernames or credentials? The setup would be done in a way that the users have the same name on both clusters. And the fractal user on both should have sudo rights to impersonate them. So I think that should be fine as well.

Share users as in usernames or credentials? The setup would be done in a way that the users have the same name on both clusters. And the fractal user on both should have sudo rights to impersonate them. So I think that should be fine as well.

Warning: this is not what I understood from the call with @gusqgm.

My understanding was that the "Joel" users of the two clusters in the federation are the same one, and not two different ones with the same name. This point is not blocking, but let's clarify it.
More importantly, we are not planning to have multiple fractal instances, one running on each cluster of the federation. Rather, we have a single fractal server running, from a place where the user would be able to submit a job to the federation (and then it's SLURM that selects the appropriate cluster to use). In this situation, there is no "fractal user on both", since each human user at FMI is only associated to a single fractal user.

My bad. @gusqgm Can you confirm? Probably me having misunderstood the discussion on Wednesday!

So, to clarify some points:
The utilization of a federation means that we have fractal server running somewhere in the common storage, and this single instance will be able to communicate to any member of a federation, since with the -M command it is possible to choose which cluster of the federation should execute the job. There is the possibility that fractal itself knows to which cluster of the federation the job should be sent, as well as the scenario where slurm should be able to select the cluster based on specific parameters of the job in question.
From my current understanding of this approach, I do agree with what @tcompa wrote above is correct: there should be just one user that fractal should be able to impersonate.
Once we start trying this here I will be able to give more information. 👍

After long discussions at FMI, we're not currently continuing the federation approach, but aim for just running on the main cluster. Thus, we can close this issue for the time being.


[crypto] SuiAddress derivation via sha3

We currently mix sha2 (pubKey->address) and sha3 (objects/transaction) and this is an attempt to unify our hash-digest functionality into a single behavior and scheme.
This is very important for a) auditing purposes b) consistency c) less dependencies (-sha2)

@huitseeker back to you, per your recommendation, a slightly more generic test has been introduced that will help us spot any changes to the SuiAddress derivation algorithm in the future.


Gazebo

Hi @bergercookie,
I have cloned to the repository and launched the follow_waypoints by using the command "roslaunch follow_waypoints follow_waypoints.launch". I got the following output:
process[dynparam_Ubuntu_14007_4263002704199055716-2]: started with pid [14025]
[INFO] [1638600216.207114]: Connecting to move_base...
[dynparam_Ubuntu_14007_4263002704199055716-2] process has finished cleanly
log file: /home/user/.ros/log/803124a0-54ba-11ec-a955-c025e9292bff/dynparam_IT075882_Ubuntu_14007_4263002704199055716-2*.log
Could you please guide me how can I get the gazebo output after that?
Thanks for the help.

Hi there,
I 'm not sure what you are trying to accomplish to be honest.
Have you also executed move_base alongside follow_waypoints.launch? This package works by sending goals to move_base querying it for the latest position and sending new goals when the robot is within reach of the already followed goal.

Hi @hsaleem1, I'll close this one for now. Feel free to respond though  and we can potentially reopen this.


New nodes are not provisioned for StatefulSet with podAntiAffinity topologyKey: failure-domain.beta.kubernetes.io/zone

Description
Observed Behavior:
When I deploy a simple StatefulSet with podAntiAffinity using the label failure-domain.beta.kubernetes.io/zone as the topologyKey, if one of the pods in my StatefulSet needs a new node, it does not get provisioned. Looking in the karpenter logs, it seems to have an issue with this key as it prints out:
unsatisfiable topology constraint for pod anti-affinity, key=failure-domain.beta.kubernetes.io/zone (counts = us-east-1a: 1 , podDomains = topology.kubernetes.io/zone Exists, nodeDomains = topology.kubernetes.io/zone Exists

Expected Behavior:
According to the docs under well known labels - failure-domain.beta.kubernetes.io/zone should be mapped to the stable equivalent (topology.kubernetes.io/zone), so nodes should continue to be provisioned when necessary for these pods in the StatefulSet.
Reproduction Steps (Please include YAML):

Deploy the following StatefulSet:

apiVersion: apps/v1
kind: StatefulSet
metadata:
  labels:
    run: curl
  name: curl-1
spec:
  selector:
    matchLabels:
      run: curl
  replicas: 3
  podManagementPolicy: OrderedReady
  template:
    metadata:
      labels:
        run: curl
    spec:
      affinity:
        podAntiAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            - labelSelector:
                matchExpressions:
                - key: run
                  operator: In
                  values:
                  - curl
              topologyKey: failure-domain.beta.kubernetes.io/zone
      containers:
      - args:
        - sleep
        - "3000"
        image: curlimages/curl
        name: curl
        resources:
          requests:
            cpu: 1000m
            memory: 1000Mi
          limits:
            memory: 5000Mi
        securityContext:
          runAsUser: 1001
      dnsPolicy: ClusterFirst
      restartPolicy: Always
      terminationGracePeriodSeconds: 1
status: {}

If none of the pods needed a new node and all get scheduled, continue to deploy the StatefulSet in different namespaces until eventually a new node needs to be provisioned to fit one of the pods.
Once you have a pending pod, note in the karpenter logs or on the pod events that there is an error for unsatisfiable topology constraint for pod anti-affinity, key=failure-domain.beta.kubernetes.io/zone
Repeat the process, but change the topologyKey to topology.kubernetes.io/zone
Once you have a pod that needs a new node, note that karpenter nominates a nodeClaim for that pod and the new node comes up as expected

Versions:

Chart Version: 0.36.2
Kubernetes Version (kubectl version): 1.27

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Is there are reason for using the deprecated label?

We don't have to use the deprecated label and our issue is fixed by using topology.kubernetes.io/zone.
Reported this in case others run into the same issue because it was a pain point in debugging this issue since the docs say that the deprecated label works but it does not.

same issue


Variable Maps (post choice manipulation)

Is your feature request related to a problem? Please describe.
I find myself writing cheats that have some level of choice, and currently the only way to give choice is to duplicate the individual cheats.
A simple example is firewalld which can have ephemeral settings (i.e. won't survive a restart) or to make settings permanent simply add --permanent to any command.
This is a somewhat contrived example, because it's easy to simply type --permanent at the end of any navi printed command, but let's assume the --permanent must go in a specific place of the arguments (maybe the middle), then this becomes much more cumbersome. The options are either two cheat lines:
# add port permanently
firewall-cmd --permanent --add-port=80/tcp

# add port
firewall-cmd --add-port=80/tcp

Or I can edit the navi printed cli manually, which again if the change needs to happen in the middle, is a pain.
Describe the solution you'd like
Giving navi the ability to do something with a selection after the user picked something through FZF would be awesome.
For example, the above would turn into a single cheat (I'm using ~ to mean "after the user has picked a value for the variable...arbitrary an unimportant, it could be anything):
# add port
firewall-cmd <PERM> --add-port=80/tcp

$ PERM: echo 'permanent: yes_permanent: no' | tr '_' '\n'
$ ~PERM: awk '{ if ($0 ~ /y/ ) print "--permanent"; }'

Which would allow the user to picked if it's permanent or not.
Describe alternatives you've considered
See above.
Additional context
Maybe FZF already contains such a feature? I'm not aware of one though.
Also how a user would specify the "after" selection commands is totally up to you. I used ~ in the example, but honestly it doesn't matter.

It would also be fine if the above example used some form of --- maybe with a new option like --map i.e.
# add port
firewall-cmd <PERM> --add-port=80/tcp

$ PERM: echo 'permanent: yes_permanent: no' | tr '_' '\n' --- --map "awk '{ if ($0 ~ /y/ ) print \"--permanent\"; }'"

Hi! I really understand your use case but I think I'm not prepared enough to add a new syntax structure at the moment, considering:

I want the syntax to be as simple as possible
I definitely want to avoid breaking-changes in the future

For example, I regret having created the --- --column <n> syntax because, had I created a $ ~VAR-equivalent, this logic wouldn't need to be together with the custom fzf options. It would be much more elegant, indeed.
I'll probably implement something along the lines of what you're suggesting but I prefer not to do it now.
In the meantime, you can workaround the problem:
# Middle argument
<is_permanent> && arg="--permanent"
echo mycmd <port> foo "${arg:-}" bar

$ is_permanent: echo 'true false' | tr ' ' '\n'
$ port: echo '80 81 82' | tr ' ' '\n'

It's not very concise but it's an implementation over a simple, easy syntax.
Let me know if that covers your use case.

I totally understand that! The workaround kinda works (I didn't know you could have multiple lines of script), but still not perfect as the final command only evals the "${perm:-}" when not using navi's --print. I can live without this feature, it's ok, just wanted to propose it. And if it gets added in the future awesome :+1:
Also, FWIW so long as navi's major version is increased I don't think breaking changes are a bad thing. I rarely assume a command line program to maintain full backwards compatibility across major version increases.

I think I'll implement the --- --map <arbitraty code> in the near future and mark it as experimental so that users will be aware that it may break at any moment.
Then, once we have more data on what would be an elegant API, we can revisit it. WDYT?

Some complexities arise, though.
Consider this case:
--- --map <arbitraty code> --column 2

Should we apply --map after or before --column?

Then, once we have more data on what would be an elegant API, we can revisit it. WDYT?

Great! I think that's a good plan!

Should we apply --map after or before --column?

I would say after. In my mind, the map is applied as the "final" step prior to being inserted into the command line. Reason being if map was applied before, why even use the --column? I can select a column as part of the map. However if it's applied after I can use --column to save some awking.


Add cluster-autoscaler v1.25.0

What this PR does / why we need it:
Add cluster-autoscaler v1.25.0 for Kubernetes 1.25 clusters.
Which issue(s) this PR fixes:
xref #2406
What type of PR is this?
/kind feature
Does this PR introduce a user-facing change? Then add your Release Note here:
Add cluster-autoscaler v1.25.0 for Kubernetes 1.25 clusters

Documentation:
NONE

/lgtm
/approve


GLFW error 65543: GLX: Failed to create context: BadValue (integer parameter out of range for operation)

Generated my_scenic_app app
GLFW error 65543: GLX: Failed to create context: BadValue (integer parameter out of range for operation)
12:12:11.458 [error] dirty close
Versions and Environment
Elixir:
Elixir 1.7.2 (compiled with Erlang/OTP 19)

Erlang:
Erlang/OTP 21 [erts-10.0.5] [source] [64-bit] [smp:12:12] [ds:12:12:10] [async-threads:1] [hipe]

Eshell V10.0.5  (abort with ^G)

Scenic:
* parse_trans 3.2.0 (Hex package) (rebar3)
  locked at 3.2.0 (parse_trans) 2adfa4da
  ok
* mimerl 1.0.2 (Hex package) (rebar3)
  locked at 1.0.2 (mimerl) 993f9b0e
  ok
* metrics 1.0.1 (Hex package) (rebar3)
  locked at 1.0.1 (metrics) 25f094de
  ok
* unicode_util_compat 0.3.1 (Hex package) (rebar3)
  locked at 0.3.1 (unicode_util_compat) a1f612a7
  ok
* idna 5.1.2 (Hex package) (rebar3)
  locked at 5.1.2 (idna) e21cb58a
  ok
* gettext 0.16.0 (Hex package) (mix)
  locked at 0.16.0 (gettext) 4a7e9040
  ok
* ssl_verify_fun 1.1.1 (Hex package) (rebar)
  locked at 1.1.1 (ssl_verify_fun) 28a4d65b
  ok
* scenic_sensor 0.7.0 (Hex package) (mix)
  locked at 0.7.0 (scenic_sensor) a12aed75
  ok
* combine 0.10.0 (Hex package) (mix)
  locked at 0.10.0 (combine) eff8224e
  ok
* elixir_make 0.4.2 (Hex package) (mix)
  locked at 0.4.2 (elixir_make) 332c649d
  ok
* certifi 2.3.1 (Hex package) (rebar3)
  locked at 2.3.1 (certifi) d0f42423
  ok
* hackney 1.13.0 (Hex package) (rebar3)
  locked at 1.13.0 (hackney) 24edc8cd
  ok
* tzdata 0.5.19 (Hex package) (mix)
  locked at 0.5.19 (tzdata) 7962a399
  ok
* timex 3.3.0 (Hex package) (mix)
  locked at 3.3.0 (timex) e0695aa0
  ok
* scenic_math 0.7.1 (Hex package) (mix)
  locked at 0.7.1 (scenic_math) 131ef7cf
  ok
* scenic 0.7.0 (Hex package) (mix)
  locked at 0.7.0 (scenic) fa93041f
  ok
* scenic_driver_glfw 0.7.0 (Hex package) (mix)
  locked at 0.7.0 (scenic_driver_glfw) e9051384
  ok
* scenic_clock 0.7.0 (Hex package) (mix)
  locked at 0.7.0 (scenic_clock) 645bfd0f
  ok

OS:
Distributor ID: Ubuntu
Description:    Ubuntu 18.04.1 LTS
Release:        18.04
Codename:       bionic

Linux olafura-GS73-Stealth-8RF 4.15.0-32-generic #35-Ubuntu SMP Fri Aug 10 17:58:07 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Steps to reproduce
sudo apt-get install pkgconf libglfw3 libglfw3-dev libglew2.0 libglew-dev
mix archive.install hex scenic_new
mix scenic.new my_scenic_app
cd my_scenic_app
mix scenic.run

https://github.com/boydm/scenic/issues/31

This might be a problem on my end I'm getting:
name of display: :1
X Error of failed request:  BadValue (integer parameter out of range for operation)
  Major opcode of failed request:  155 (GLX)
  Minor opcode of failed request:  24 (X_GLXCreateNewContext)
  Value in failed request:  0x0
  Serial number of failed request:  85
  Current serial number in output stream:  86

From glxinfo, I was using Steam on this before so something has messed up.

Now after a reboot it works perfectly. The nvidia driver has two versions loaded at the same time or something like that, at least the tools weren't agreeing with the driver.


snap: Add user facing support documentation

On the snapcraft forum, some common support questions are beginning to surface. We spend time debugging and answering the questions when they should really be documented. Here'a a few:

Can't install snap

Is it available in this region (some countries are geo blocked for support or political reasons)
Sometimes the snap is only available for certain architectures
Maybe the snap is private

Snap is outdated

How do I find out who maintains a snap?
Where do I file a bug against a snap?

A central support document which has some common support questions and answers we could point to (and would turn up on internet search) might be useful?

Hello, Can I work on this if nobody has taken it up?

Hi @iyissa! Thanks for your interest in this. And yes, of course. That would be really helpful.
We do have a Fix common issues page, which might be the best place to add this kind of information - what do you think? And if this seems suitable, would you prefer for me to move that document into this repository so we can work on it here?
Big thanks to @popey for creating this issue.

Hello @degville
I agree the Fix common issues page is the best place to add this kind of information. Sure, moving the document into this repository will be great for us to be able to work on it.


Feature.GetSequence() always returns a nil error value

https://github.com/cachemoi/poly/blob/10b8f398c4272700dd27a177af244d6b531d2582/io/genbank/genbank.go#L129-L157
Should either not have an error in its signature or actually return an error.

Hi!
You don't mind if I'll take that? Actually, I already did all the corrections in my branch locally, just don't know what's the correct flow further, I'm just learning to contribute. Should I just provide this issue's ID in commit message to link my changes?
And what if I have minor corrections to 'CONTRIBUTING.md' about URLs actualization, could I bring them with?

If you could the other two changes in a separate or, that'd be great.
This is currently blocked by @Koeng101 's PR! Try and merge it into his ioToBio PR instead.

Other proposed changes are just 2 lines, sure need a separate pr?, like:
`-You can learn how from this free series, How to Contribute to an Open Source Project on GitHub.
+You can learn how from this free series, How to Contribute to an Open Source Project on GitHub.
-You can also check out these tutorials.
+You can also check out this tutorial.
`
First link was working but with 2 redirections, latest is the final destination at the moment.
Second one (makeapullrequest.com) just invites to go via the same egghead.io/courses link, maybe could've been removed as a duplicate then?

Try and merge it into his ioToBio PR instead

Sure, will try.

Ah gotcha. I'm generally opposed to drive-by changes, but this seems fine since it's really small and in documentation files only.
Thank you so much for taking a crack at this issue by the way!

Hi again, sorry for being lost for some time. Does the same function in io/polyjson have to be ~un-errored:) too? I see that it's basically doing the same, but has some unclear todo action.

~un-errored:) too? I see that it's basically doing the same, but has some unclear todo action

That's... uh unclear I think. Personally I think we should nuke polyjson - it doesn't accomplish much that Genbank does better. But basically, you are correct that it theoretically should be un-errored. I think polyjson it hasn't gotten a lot of love because nobody uses it (they just use JSON version of genbank)


Add --offline parameter for yarn install

This adds the ability to use the --offline parameter for yarn install commands. It has updates for unit tests as well.

thanks!


Script for BERT-base training to reproduce the results

Hi Team! Firstly, thank you for the nice paper and approach. Secondly, while browsing through the paper of MRL, it is mentioned that the BERT-Base model was trained for MLM task on English Wikipedia and BookCorpus in Appendix C and values are provided in Table 7. However, I was not able to locate the script that trains the BERT-based model in the repository to reproduce the results. It will be really helpful if you can locate the script. Thanks in advance.

Hello, the BERT-Base model from the original MRL work was trained internally at Google and is not open sourced. We can provide BERT-Base models trained at AI2 for our followup AdANNS work (ref: Qingqing Cao) on Natural Questions with MRL, following the DPR codebase.
MRL-NQ models are available on Huggingface


RHOAIENG-15041: Show validation warnings only after the first form submission

https://issues.redhat.com/browse/RHOAIENG-15041
Notebook-based images to use for tests (commit 3679179af4af9842f066761a3f964462d0d5ddcc):
ghcr.io/caponetto/opendatahub-io-elyra/workbench-images:jupyter-trustyai-ubi9-python-3.11-20241203-3f95611-sha-3679179a

ghcr.io/caponetto/opendatahub-io-elyra/workbench-images:cuda-jupyter-tensorflow-ubi9-python-3.11-20241203-3f95611-sha-3679179a

Demo running on the image above:
https://github.com/user-attachments/assets/26aa41d2-633b-48d8-97d1-2c880f31301d

from the video, looks to me it behaves allright now
/lgtm


set security policies from-zone syntax not recognized

Thanks for your report, it looks like a bug. I'll take a look.

Should be fixed in v0.1.7.


drop uncurated symbols from the navigator index (#890)

Explanation: Remove symbols that were not curated by AutomaticCuration from the navigator index, to prevent them from appearing outside of any module.
Scope: QOL fix for clients of the LMDB navigator index, e.g.  Xcode's documentation window.
Issue: rdar://125855186
Original PR: https://github.com/apple/swift-docc/pull/890
Risk: Low. The symbols were already being dropped from the JSON representation of the navigator index (used by Swift-DocC-Render) without issue.
Testing: Automated tests were updated to reflect the new behavior.
Reviewer: @d-ronnqvist @patshaughnessy

@swift-ci Please test

@swift-ci Please test


Ability to customise primary key

Motivation
The current primary key rules are very strict:
graphql-migrations

Can only use ID scalar, which maps to Knex increments (an auto-incrementing integer).
Field name must be id.

Graphback

Can only use ID scalar, which maps to Knex increments (an auto-incrementing integer).
Field name must be id.
Generation engine does not allow ability to pass custom ID from server/client.

Strategy
A new strategy will need to be considered to increase the flexibility of primary key customisation.
graphql-migrations
Issues arise when changing a field with Knex to increments from something else.
This creates an integer primary key
table.integer('id').primary();

This creates an auto-incrementing primary key. Note that is does not require .primary().
table.increments();

TL;DR this needs further investigation
This difference makes it a challenge since .increments always assumes a primary key, so Knex will try to create two primary keys.
Graphback
This should be more straightforward. When a developer uses the ID scalar then Graphback will not pass the responsibility to provide the ID to the server and client. If the developer wants to set these they can use Int, String etc.

I see this as very low priority over the other changes. The problem we have here can be resolved by better validation.

Adding documentation should be enough for the moment


Looking for a way for pagination integration with swift/ios...

Looking for a way for pagination integration with swift/ios...
I recently saw the following post: https://www.sitepoint.com/paginating-real-time-data-cursor-based-pagination/
Does Flask-sqlalchemy support this? If not, how should one go about using the orm to do this sort of cursor based pagination so that it can be used in an API for pagination.

You can use pagination with anything, it's not swift specific.  I'm not sure how the link is related either.  Flask-SQLAlchemy is not going to support every pagination scheme under the sun, but it does support anything that SQLAlchemy does.


Bugfix: nvd3 tooltips do not disappear

Addresses https://github.com/apache/incubator-superset/issues/3471
NVD3 tooltips remain in DOM when switching to another viz in explore view, and sometimes they aren't hidden. Added a MutationObserver to detect viz type change and then perform cleanup of tooltips.

Coverage remained the same at 69.442% when pulling ca449f8087548f0286f7433af06f99fab3763bd8 on Mogball:mogball/bugfix/nvd3_tooltips into c3c9ceb1cc3fc70670257f8f652f083cde2ea14d on apache:master.

Coverage remained the same at 69.442% when pulling ca449f8087548f0286f7433af06f99fab3763bd8 on Mogball:mogball/bugfix/nvd3_tooltips into c3c9ceb1cc3fc70670257f8f652f083cde2ea14d on apache:master.

Notice: this issue has been closed because it has been inactive for 201 days. Feel free to comment and request for this issue to be reopened.


perf(sqllab): Rendering perf improvement using immutable state

SUMMARY
The main rendering in SqlLab is managed by the queryEditors state from redux(state machine).
Each queryEditor state has been updated whenever an editor configuration has updated.
Since most components observes entire queryEditor state not specific attributes, any queryEditor updates trigger multiple repainting even if no visual changes impacted.
For example, when a user types any single character in the ace(sql) editor, it triggers the update for selectedText value in the active queryEditor state. This single change will trigger the following components repainted (though this change won't impact the left and bottom panels at all)

This unrelated repainting job causes the performance delay especially when table list is humongous.

The delay can be found in the following performance analysis due to the unnecessary repainting/redux post processes.

This commit fixes this performance issue by optimizing observers in each component to subscribe only related items.
To minimize the regression issue from existing persistence state, this commit keeps the existing queryEditors state but make it immutable during the tab is active and only updates when the tab is switched. It introduces unsavedQueryEditor state which stores all these active changes.
With this update, the page rendering performance improved 3.5x faster(from 4.5s to 1.3s) and verified all unnecessary tasks cleaned.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before (4.5s):
https://user-images.githubusercontent.com/1392866/181118443-f8949bc1-1d07-4dcb-8a65-aec1ff024fee.mov
After (1.3s):

TESTING INSTRUCTIONS

Go to SqlLab
Choose a database and a schema that includes a large set of table list
Open chrome console > Performance tab
Select Caption settings on right menu
Change CPU throttling to 6x slowdown
type a string in the sql editor and then watch the refresh icon on the left panel (it will blink during repainting)

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[x] Introduces new feature or API
[x] Removes existing feature or API

/testenv up

/testenv up

/testenv up

/testenv up

@eschutho I updated the commit. Can you review the update? thanks!

@eschutho @michael-s-molina Can we merge the PR?


Change behaviour of Top Grades to make it more useful

Describe the bug
The main use case of Top Grades I see is as follows: Someone decides they want to play all of the songs they've only gotten a B on and try to do better, or maybe play songs they haven't played before.
When you sort by Top Grades, find a song to play, and play it, here is the issue.
If you improve your score on the song, when you return to the song wheel you will have been moved from the group of grades the song was previously in to the new group the song is in.
As a result, the above use cases are very painful to do.
Although this isn't non-intuitive behaviour (your currently selected song remains the same after finishing a song, like all other sort), it makes Top Grades all but useless, as using it involves
a) Scroll to the group of songs you want to improve
b) Play a song (doing better on it)
c) Scroll forever through the new group you're in to make it back to where you were before
With how long c can take for very large song libraries (I have many many hundreds of songs I've gotten a A on), it is literally faster to go pack-through-pack in group sort to find specific letter grades to improve than to use the sort mode exclusively made for that purpose.
Expected behavior
I think it would be more useful if you stayed "near" where the song was before improving your score on it.
Perhaps the cursor could be made to select the song that had previously been right after it, or the like.

Utilize the up+down shortcut to exit an opened musicwheel folder.
Will reopen if you make a better argument


ISPN-15659 Hot Rod Client RemoteCache#computeAsync() can lose the result of remapping processing when executed concurrently against a new entry

https://issues.redhat.com/browse/ISPN-15659

It's :green_heart:

Thanks @jabolina


fix: clean extra file & fix agents use zhipu

Description

Clear web screenshots,  tools/scripts
Update assets/schema  dbgpt.sql
Fix agent chat with zhipu glm-4

How Has This Been Tested?
No need.
Snapshots:
Include snapshots for easier review.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have already rebased the commits and make the commit message conform to the project standard.
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] Any dependent changes have been merged and published in downstream modules

Nice!
Run with zhipu proxyllm.
LLM_MODEL=zhipu_proxyllm
ZHIPU_MODEL_VERSION=glm-4
ZHIPU_PROXY_API_KEY={your zhipu api key}
PROXY_SERVER_URL=xxxxx