RNN Variables mistakenly (?) placed on GPU when doing multi tower training.

I'm trying to do "data parallelism" following the cifar-10 multi gpu example. Main difference is that I am using tensorflow's rnn abstractions. When I run the code ( mgf.tar.gz ) on a CPU machine it works but when I run it on a machine with 1-4 gpus it crashes, reporting that some of the GRU/LSTm variables have not been initialized (on the gpu).
Traceback (most recent call last):
  File "trainMultiGPU.py", line 211, in <module>
    tf.app.run()
  File "/usr/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 43, in run
    sys.exit(main(sys.argv[:1] + flags_passthrough))
  File "trainMultiGPU.py", line 207, in main
    train(args)
  File "trainMultiGPU.py", line 171, in train
    sess.run(init)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 766, in run
    run_metadata_ptr)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 964, in _run
    feed_dict_string, options, run_metadata)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1014, in _do_run
    target_list, options, run_metadata)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1034, in _do_call
    raise type(e)(node_def, op, message)

tensorflow.python.framework.errors_impl.FailedPreconditionError: Attempting to use uninitialized value RNN/GRUCell/Gates/Linear/Matrix
         [[Node: RNN/GRUCell/Gates/Linear/Matrix/_52 = _Send[T=DT_FLOAT, client_terminated=false, recv_device="/job:localhost/replica:0/task:0/cpu:0", send_device="/job:localhost/replica:0/task:0/gpu:0", send_device_incarnation=1, tensor_name="edge_149_RNN/GRUCell/Gates/Linear/Matrix", _device="/job:localhost/replica:0/task:0/gpu:0"](RNN/GRUCell/Gates/Linear/Matrix)]]
         [[Node: init/NoOp_1/_62 = _Recv[client_terminated=false, recv_device="/job:localhost/replica:0/task:0/cpu:0", send_device="/job:localhost/replica:0/task:0/gpu:0", send_device_incarnation=1, tensor_name="edge_164_init/NoOp_1", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"]()]]`**

I've tried this both with the vanilla and dynamic variations of the rnn functions and with LSTM as well as GRU.
What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?
Issue #1390
Environment info
Operating System:
RHEL 7.2
Installed version of CUDA and cuDNN:
(please attach the output of ls -l /path/to/cuda/lib/libcud*):

[tal@E8A2-DL380G90-01-1G multi_gpu_lstm_fail]$ ls -l /usr/local/cuda/lib64/libcud*
-rw-r--r--. 1 root root   558720 Dec  6 10:55 /usr/local/cuda/lib64/libcudadevrt.a
lrwxrwxrwx. 1 root root       16 Dec  6 10:55 /usr/local/cuda/lib64/libcudart.so -> libcudart.so.8.0
lrwxrwxrwx. 1 root root       19 Dec  6 10:55 /usr/local/cuda/lib64/libcudart.so.8.0 -> libcudart.so.8.0.44
-rwxr-xr-x. 1 root root   415432 Dec  6 10:55 /usr/local/cuda/lib64/libcudart.so.8.0.44
-rw-r--r--. 1 root root   775162 Dec  6 10:55 /usr/local/cuda/lib64/libcudart_static.a
-rwxr-xr-x. 1 root root 79337624 Dec 13 10:51 /usr/local/cuda/lib64/libcudnn.so
-rwxr-xr-x. 1 root root 79337624 Dec 13 10:51 /usr/local/cuda/lib64/libcudnn.so.5
-rwxr-xr-x. 1 root root 79337624 Dec 13 10:51 /usr/local/cuda/lib64/libcudnn.so.5.1.5
-rw-r--r--. 1 root root 69756172 Dec 13 10:51 /usr/local/cuda/lib64/libcudnn_static.a

If installed from binary pip package, provide:

A link to the pip package you installed:
export TF_BINARY_URL=https://storage.googleapis.com/tensorflow/linux/gpu/tensorflow_gpu-0.12.0rc1-cp27-none-linux_x86_64.whl
The output from python -c "import tensorflow; print(tensorflow.__version__)".
[tal@E8A2-DL380G90-01-1G multi_gpu_lstm_fail]$ python -c "import tensorflow;

print(tensorflow.version)"
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcublas.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcudnn.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcufft.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcurand.so locally
0.12.0-rc1

If possible, provide a minimal reproducible example (We usually don't have time to read hundreds of lines of your code)
I've attached a tar with a minimal reproducing example. Unfortunately I am behind a corporate proxy and can't push to github.
[mgf.tar.gz - reproducible example
To recreate the example simply run

python trainMultiGPU.py
It should work fine on a CPU only machine. On a machine with gpus you should see the above traceback.

What other attempted solutions have you tried?

I've tried variants of rnn (dynamic, regular, bidirectional)
I've tried LSTMCell and GRUCell
I've tried passing the scope explicitly to the call to rnn
I've tried wrapping the call to rnn with a device placement on the cpu. This prevents the traceback but I think it places the OP on the cpu as well as processing becomes incredibly slow.

Logs or other output that would be helpful
(If logs are large, please upload as attachment or provide link).
device_placements.txt
nvidia-smi.out.txt
TRACEBACK.txt

Looked at your code. Aren't you doing the same thing as Issue #1390, i.e running the op to initialize all variables before starting queue runners?

I tried both ways, it doesn't seem to make a difference.
eg running like in #1390
 sess = tf.Session(config=tf.ConfigProto(
             allow_soft_placement=True,
             log_device_placement=False))

     # Start the queue runners.
     tf.train.start_queue_runners(sess=sess)
     init = tf.global_variables_initializer()
     sess.run(init)

or how I posted
  sess = tf.Session(config=tf.ConfigProto(
              allow_soft_placement=True,
              log_device_placement=False))

      # Start the queue runners.
      init = tf.global_variables_initializer()
      sess.run(init)

      tf.train.start_queue_runners(sess=sess)

Doesn't make a difference. Either way the same error appears

Looking at the output, it looks like a Send is consuming the variable before it is initialized, which suggests that the variable is being read as part of another initialization that is occuring on CPU.
Without being able to debug into your graph too much, I would try to find which GPU variable is being read on CPU, and to make sure that you are using https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/ops/variables.py#L515 (initialized_value) if you are trying to initialize a value based on the value of another variable, during the same initialization op.

Thanks @vrv .
As far as I can tell, I'm not initializing any variable based on another one and I only have a single call to an init opp.
Your suggestion is at the limits of my understanding of TF.  Could you point me at the right docs to understand how to go about "try to find which GPU variable is being read on CPU, " ?
Thanks

Quick update:
I removed the following lines and it worked!
        variable_averages = tf.train.ExponentialMovingAverage( args.decay_factor, global_step)
        variables_averages_op = variable_averages.apply(tf.trainable_variables())

I'm not sure why, but I suspect it has somehting to do with the variables that ExponentialMovingAverage creates, per the docs:

Creates a new ExponentialMovingAverage object.
The `apply()` method has to be called to create shadow variables and add
ops to maintain moving averages.

Closing this as it looks like you resolved it yourself. Thanks @talolard for reporting back.


Remove data sources field from forms

The data source for an entry should be automatically dictated by a user's selection in the datasources button (see the README for the latest description of the desired datasources workflow)

Changes for maintaining DataSource are in 6987cbafc4136f68d7b3e034971251dd2cf3ec40, d0a9f3238876dfeb4769d89d461bdb4e7b9380ad, d9e247fcde230a9e1917462fd93dd4c82aba72e1, and c03bd9aa68dccaf96a15b7ce7550e8f2030ef641.
The user must now select a DataSource before C&F and MU functionality is enabled. The selected value will be used in all transactions.
In order to provide a visual reminder of what DS has been selected, I am leaving the DataSource field on all forms for now. But they are in a disabled state with the text in italic to emphasize this. Some sort of reminder is necessary, and this was the quickest thing to implement.


Start a new conversation

Hello guys,
I not use this lib in my project, but I use to learn and develop it.
I am with a new problem: Start a chat with a number that not contact me any time is with the next problem:
model : createdLocally : true id<EMAIL_ADDRESS>__proto__ : Object name : "ModelCreateError" stack : "ModelCreateError: t↵    at t.n (https://web.whatsapp.com/app.1c6cccbeae5d099f4d1e.js:19:20325)↵    at new t (https://web.whatsapp.com/app.1c6cccbeae5d099f4d1e.js:20:2887)↵    at https://web.whatsapp.com/app.1c6cccbeae5d099f4d1e.js:9:31640↵    at x (https://web.whatsapp.com/app.1c6cccbeae5d099f4d1e.js:30:31007)↵    at MutationObserver.I (https://web.whatsapp.com/app.1c6cccbeae5d099f4d1e.js:30:30499)"
I use this code:
<EMAIL_ADDRESS>How I can start a new chat without use my cellphone?
Obs: I try save the contact in my cellphone, but the problem persist

You’re using an old version, probably from pip. That version doesn’t work since may 2018. Checkout the version in this repository and apply manually the patch of PR #380 because last Friday WP changes the chatID scheme and the @mukulhase doesn’t approved the PR yet.


Rewrite and restructure browser tests --> python tests

The "Wagtail" suite of browser tests have now all been converted to unit tests that use a test database. A lot of things have been rewritten and moved around, but generally there is no new test coverage nor have any tests been (functionally) removed.
There is no longer an initial_test_data as the data is created for each test that needs it, although I make use of initial_data as Andy set up in this PR: https://github.com/cfpb/cfgov-refresh/pull/2269
Testing
As before, you can run gulp test, but this just runs gulp lint and the unit tests now. It is faster to do the following:
tox -e fast
gulp lint
You can also still run integration (which need to be updated as well) and content tests like so, but as before, they will fail unless you have the relevant prod data locally.
gulp test:acceptance --suite=integration
gulp test:acceptance —suite=content
Review

@chosak
@rosskarchner
@willbarton
@grapesmoker

Todos

There are two tests commented out in test_organisms.py relating to the e-mail in Contact Info, as they are failing and they need to be looked into further. They both make the following assertion:
self.assertContains(response<EMAIL_ADDRESS>
Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[x] Passes all existing automated tests
[x] New functions include new tests
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

@richaagarwal I think overall this looks fantastic, and a lot better than the tests that relied on initial_test_data.py.
I can't run them locally, however, for the same reason that Travis fails:
  File "/Users/bartonw/repos/platform/cfgov-refresh/cfgov/cfgov/test.py", line 9, in <module>
    from scripts import initial_data, initial_test_data
ImportError: cannot import name initial_test_data
ERROR: InvocationError: '/Users/bartonw/repos/platform/cfgov-refresh/.tox/fast/bin/python manage.py test'

It looks like initial_test_data.run() is removed in test.py, but the import isn't.
With that import removed, it works as advertised.

@willbarton Thanks! And good catch; for some reason they were still running fine for me. Removing the import now.

@richaagarwal that did it. 👍

Updated the spacing.  Going to merge now but still open to additional feedback. Thanks all for your input!

Should the atomic components in the /integration/ directory retain the /molecules/, /organisms/, etc. subdirectories that are used in the implementations (https://github.com/cfpb/cfgov-refresh/tree/master/cfgov/unprocessed/js/molecules, https://github.com/cfpb/cfgov-refresh/tree/master/cfgov/unprocessed/js/organisms)?


Update test maintainers

@ekouts should use the two initials format for the maintaners name or go with the @<username>?

@ekouts should use the two initials format for the maintaners name or go with the @<username>?

hm maybe @<username> makes more sense, I will fix the files I am editing..


Rebind dropdowns when switching to uk address

This resolves an issue when navigating back to the address page from the 'change' link during the signup process

Well, in the event of failed validation the viewmodelbuilder isn't called. That's the problem.
The builder does create the lookups but the data isn't posted back so isn't bound the model again on post, longer term we may want to change this approach and have that data posted.


Doc addition to OutOfScope > Domain based forwarding for HTTP

Feature Proposed

Domain based forwarding for HTTP

Perhaps (rather??) point to Caddyserver.com that is very easy to setup reverse proxy to point to rathole's server/upstream... and do the LetsEncrypt certificates as added benefit/additions without additional configs

I don't think link any other applications here is a good idea.

Suggestion, but then why list nginx which is complex?
I see rathole as simple/easy, and this is a quick/simple/easy way to achieve just that.
Any case, was a suggestion only. If you don't like it, welcome to close ticket as "Not going to implement" :)

Anyone is free to choose their preferred application for processing at the application layer, but the purpose of the documentation is simply to provide an explanation and an example, listing all available options may only make the document more cumbersome and hard to read. And yea, I'm not against the idea that people should have more options, feel free to use your favorite one :)
By the way, I don't think NGINX is complex to configure. Besides, we don't always need web application to generate certificates for us, in many cases, we wanna use our own certificates or support devices that do not support HTTPS using a pure HTTP network. (For example, when using some Linux distributions, you need to first install ca-certificates, openssl or that kind of things over HTTP before using HTTPS.)

I'm fine with the improvement to the docs since people will ask less questions. It may not suit well in a section called out of scope. However we don't have faq or cookbook. So feel free to bring up a PR about caddy.
---Original---
From: @.>
Date: Thu, Mar 16, 2023 19:40 PM
To: @.>;
Cc: @.***>;
Subject: Re: [rapiz1/rathole] Doc addition to OutOfScope > Domain basedforwarding for HTTP (Issue #232)
I'll respectfully disagree when you can do (without ca-certificates and without openssl) just download a single binary and execute caddy reverse-proxy --from example.com --to localhost:9000 or have this single configfile:
example.com { 	reverse_proxy localhost:5000 }
That said, I do have sites I don't front with Caddy but with nginx for application reasons.
Again: in the spirit of "out of scope: here is simple/quick option"
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Do we have the docs already? I would like to expose apache, caddy or nginx, but don't know how.
The introductory docs assumes frp knowledge/experience. I came here to use it for web http and expose it as https just like what ngrok does.
Newbie questions like ours that can be answered by the docs will allow us to make use of this tool and recommend it to others that would naturally attract a community to this project.
Can someone point me to the example of how this is use for any of the three?
Thanks.


⚠ ClamAV has degraded performance

In b994c06, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 451 ms

Resolved: ClamAV performance has improved in 2f231c0 after 14 minutes.


feat: added Make targets for forwarder containers

Description of the changes
Added missing make targets for container build.
PR Review Checklist
Author

[ ] the PR should focus on a single subject. Change only relevant files to the problem you’re working on
[ ] clean and format the code
[ ] add unit tests for your changes and make sure all unit tests are passing
[ ] add a risk label after carefully considering the "blast radius" of your changes
[ ] describe the intent of your changes in the description. don't just rewrite your code in prose
[ ] assign at least one reviewer
[ ] address the feedback
[ ] add a "ready for review" label. Only PRs with this label will be reviewed.

Reviewer

[ ] review code for readability
[ ] verify that high risk behavior changes are well tested
[ ] check license for any new external dependency
[ ] ask questions about anything that isn't clear and obvious
[ ] approve the PR when you consider it's good to merge

Pull Request Test Coverage Report for Build 205837473

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 55.791%

Files with Coverage Reduction
New Missed Lines
%

pkg/metrics/sampler/sampler_routine.go
2
94.74%

pkg/integrations/v4/logs/cfg_watcher.go
4
64.18%

Totals

Change from base Build 199558837:
0.0%

Covered Lines:
8835

Relevant Lines:
15836

💛  - Coveralls


Make component

Additions include:

Bash script scripts/make_component.sh

JSON field commit.committer-ts is added to generated file release (see mx source).

Batch file scripts/make_component.bat

variable GIT_DESCRIPTION is updated to reflect change in bash script.
JSON field commit.committer-ts is added to generated file release.
property Bundle-RequireCapability in generated file MANIFEST.MF has a new filter (java_version=<...>) (see mx source).

NB.  Variables GIT_SHORT_COMMIT (--format="%h") and GIT_UNIX_TIMESTAMP (--format="%ct") could be merged (generated file MANIFEST.MF uses --format="%ct").

Thanks! Squashed and merged the two last commits, and pushed them via https://github.com/hpi-swa/graalsqueak/commit/eeb73a7ea33a25adc399738896c35e0c18fdcbb1 on dev.


Compilation failure on s390x

We've gotten a report of hacl-star-raw failing to compile on s309x. The problem seems to be that -mavx -mavx2 are being passed even though these options aren't available. @msprotz mentioned that this currently happens on any platform that we don't explicitly detect as not being x86, so maybe it would be worth changing that default?
Additionally, in this specific case, the reporter mentioned that vector intrinsics could still be used, but we'd need to bundle the z intrinsics too and pass -march=z13 -mzvector.
CC @franziskuskiefer

Thanks for the report Victor! Essentially my proposal is to:

disable 128, 256 and inline assembly (like we currently do for x86) by default, UNLESS
we find an armv8 chip that supports NEON OR we find an x64 chip

One thing we could improve is to change
-include Makefile.config

to
Makefile.config:
        ./configure

in the dist/gcc-compatible/Makefile so that people who are used to hitting "make" still get a decent version compiled

Essentially my proposal is to ...

This is definitely the safer default and should avoid issues with less common/tested platforms.
We have all the mechanisms in place in configure  but don't use it for avx. We should resolve this TODO and make sure configure is called before the rest of the Makefile is processed.
https://github.com/project-everest/hacl-star/blob/3612beeaacf11bbaef7c0bba8b24e005924c2f1f/dist/Makefile.tmpl#L83-L85

@victor-dumitrescu can you check if #406 solved this issue and close it after confirming?


s/DockerRunCommand/DockerRunBuilder/

Fixes #68

jenkins/superCI comment

Status: failure, return code = 1
Result url: https://da.gd/gscpL
Comment: See details for logs

rebayz'd

jenkins/superCI comment

Status: failure, return code = 1
Result url: https://da.gd/DtUQy
Comment: See details for logs

jenkins/superCI comment

Status: success, return code = 0
Result url: https://da.gd/ZzFq4
Comment: See details for logs


pdf will not run because it can't find  waterfall crossing_all.JPG

Output created: Bulkley2022.html
Warning: A runtime exception has occured while executing JavaScript
  Runtime exception message:
    ReferenceError: $ is not defined
    at http://<IP_ADDRESS>:3732/Bulkley2022.html:32186:9
Error in force(expr) :
  Failed to generate output. Reason: Failed to open http://<IP_ADDRESS>:3732/data/photos/2022091003/crossing_all.JPG (HTTP status code: 404)
In addition: There were 50 or more warnings (use warnings() to see the first 50)

> pscis_all %>%
+   distinct(site_id) %>%
+   arrange(site_id) %>%
+   # head() %>% #test
+   pull(site_id) %>%
+   purrr::map(fpr_photo_amalg_cv)
Error in magick_image_append(image, stack) :
  rsession-arm64: NegativeOrZeroImageSize `' @ error/image.c/CloneImage/794

tried to build amalgamated photos but won't run.  first impression is that it is because there are directories that are not present.
setdiff( pscis_all %>%
distinct(site_id) %>%
arrange(site_id) %>%
# head() %>% #test
pull(site_id) ,
list.dirs('data/photos', full.names = F, recursive = F)

)

`[1] 198110`

setdiff(  list.dirs('data/photos', full.names = F, recursive = F),
      pscis_all %>%
        distinct(site_id) %>%
        arrange(site_id) %>%
        # head() %>% #test
        pull(site_id)

)
`[1] "1493"   "198066" "198115" "198117" "admin" `

right - makes sense I can't run fpr_photo_amalg_cv because I don't have the raw photos required to make the crossing_all.JPGs.  Still looks like 198110 is missing photos though.
Need to get a solution to cleanly access those onedrive photos eventually - will file a separate issue
@Mateo9569 - can you deal with 198110 and run the fpr_photo_amalg_cv section so we get those waterfall photos back and close if it will build the pdf? you don't need to run the tools::compactPDF section of the run.R script for now if you still need to setup ghostscript.  just need to see if it will build

just needed to filter out the sites I didn't have photos for https://github.com/NewGraphEnvironment/fish_passage_bulkley_2022_reporting/blob/3d7b259beb5b2a843648c0881a06162bf55eda2c/scripts/01_prep_inputs/0120-photos.R#L402
pscis_all %>%
  distinct(site_id) %>%
  arrange(site_id) %>%
  # put this here to test if it is the problem
  filter(site_id != 198110) %>%
  filter(site_id != 123750) %>%
  filter(site_id != 198116) %>%
  # head() %>% #test
  pull(site_id)  %>%
  purrr::map(fpr_photo_amalg_cv)

was able to see which ones were missing by running https://github.com/NewGraphEnvironment/fish_passage_bulkley_2022_reporting/blob/3d7b259beb5b2a843648c0881a06162bf55eda2c/scripts/01_prep_inputs/0120-photos.R#L383


[BUG] Usage of ResourceManager with multiple subscriptions not possible

Describe the bug
We are using the Azure SDK to manage Azure resources in different subscriptions. So far it works as we need but there is a huge blocker for us. The problem is to use the one ResourceManager with different subscriptions. Our current workaround is to create for each subscription a new ResourceManager BUT that only works if all resources in one request to Azure API are from the same subscription.
For example:

Create Virtual Machine in Subscription A with an Image from SharedImageGallery which is also in Subscription A works.
Create Virtual Machine in Subscription A with an Image from SharedImageGallery which is in Subscription B leads to error 404 (Resource not found in Subscription A). Heres the problem. Even if the ResourceId of the image is completely correct and the permissions are there, the ResourceManager will automatically replace the SubscriptionId in the ResourceId with the one of the ResourceManager. This way the ResourceId is changed so that the requested resource is not available. The same call with Azure CLI works without any problems with the same identity.

Exception or Stack Trace
Failed to create virtual machine with exception: Status code 404, "{"error":{"code":"ResourceGroupNotFound","message":"Resource group 'rg-example' could not be found."}}"
To Reproduce
Steps to reproduce the behavior:
Look in Bug description
Code Snippet
AzureProfile profile = new AzureProfile(AzureEnvironment.AZURE); ClientSecretCredential credential = new ClientSecretCredentialBuilder() .clientId(clientId) .clientSecret(clientSecret) .tenantId(tenantId) .authorityHost(profile.getEnvironment() .getActiveDirectoryEndpoint()).build();
 azure = AzureResourceManager .configure() .withLogLevel(HttpLogDetailLevel.NONE) .authenticate(credential, profile) .withSubscription("A");
azure.virtualMachines() .define("name") .withRegion("region") .withExistingResourceGroup("rg-example") .withExistingPrimaryNetworkInterface(networkInterface)             .withGeneralizedWindowsGalleryImageVersion("/subscriptions/B/resourceGroups/rg_gallery/providers/Microsoft.Compute/galleries/example_gallery/images/example-image/versions/0.0.1") .withAdminUsername(vmDto.getUserName()) .withAdminPassword(vmDto.getPassword()) .withComputerName(vmDto.getComputerName()) .withOSDiskStorageAccountType(StorageAccountTypes.fromString(vmDto.getStorageType())) .withOSDiskSizeInGB(vmDto.getStorageCapacity()) .withSize(vmDto.getSkuName()) .withSystemAssignedManagedServiceIdentity();
Expected behavior
Virtual Machine creation should just work as with Azure CLI

See https://learn.microsoft.com/azure/azure-resource-manager/management/authenticate-multi-tenant
For SDK, provide the token from 2nd subscription via AzureResourceManager.configure().withAuxiliaryCredential(...) (it basically adding the x-ms-authorization-auxiliary header of 2nd subscription for you)

@weidongxu-microsoft for now it seems to work, at least for this use case. thanks!


[New Port Request] [kompute]

Library name
kompute
Library description
General purpose GPU compute framework built on Vulkan to support 1000s of cross vendor graphics cards (AMD, Qualcomm, NVIDIA & friends). Blazing fast, mobile-enabled, asynchronous and optimized for advanced GPU data processing usecases. Backed by the Linux Foundation.
Source repository URL
https://github.com/KomputeProject/kompute
Project homepage (if different from the source repository)
No response
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
No response

I'm testing my port privately. Will open a PR when ready.


Replace does not work on mac

Hi,
I am trying to use Replace but it does nothing when I select it from the menu or use the shortcut.
Mac version 10.11

@lzantal
Would love to get more steps here. Did you open a text file before trying to replace in it? Other folks at the office use mac and it works for them.

Hi,
I checked it on another mac and it works there. So I deleted it and reinstalled it and now it works.
Not sure what was going on, but it appears to be working now, including the shortcut.

Thank you!


Step 19, Codelab Function not in Github Code

The last part of Step 19, "Update the paging Flow creation", refers to a SearchRepositoriesActivity.initAdapter function that exists nowhere in the prior steps nor in the Github code for this exercise.
It appears that the significant parts of this codelab initAdapter function  are changes merged into the coroutine at the end of SearchRepositoriesActivity.ActivitySearchRepositoriesBinding.bindList

Thanks again for reporting this @joec58! Fixing.

Fixed.


Searching available plugins in ROS2

In ROS1, you could do it by doing as the below link.
http://wiki.ros.org/pluginlib#pluginlib.2Fpluginlib_groovy.Querying_ROS_Package_System_For_Available_Plugins
What command line should  I use to do the same in ROS2? (rospack is not avaiable in ROS2)
Thanks in advance!

Upvoting

any luck on this?

Not yet

Not exactly the same, but I found that moveit has a node to list the available plugins. ros2 run moveit_ros_move_group list_move_group_capabilities.
Maybe you can use the C++ code here for inspiration: https://github.com/moveit/moveit2/blob/5548ef152a9b4fdbf87551b4a4291ba7ea1883d9/moveit_ros/move_group/src/list_capabilities.cpp


Incorrectly triggering dragenter and dragleave events in v-for loop - continuation to #4479

Version
2.6.10
Reproduction link
https://jsfiddle.net/Eggon666/d6htzpr0/12/
Steps to reproduce
Drag the buttons (on Windows, Chrome). Watch console.log reports on number of button on which dragenter/dragleave event was triggered - for buttons 3-5 the number might be incorrect. The bug sometimes needs a few drags before it triggers.
What is expected?
First dragenter/dragleave event triggering inside the dragged element should trigger only on the element that is being dragged (unless the pointer moves over another element).
What is actually happening?
Dragenter and dragleave events are triggered on an incorrect (i.e. preceding sibling) element from v-for loop together with or instead of the events of the actually dragged element (DragTarget). The problem seems to be related to v-for specifically, I haven’t encountered it on non-v-for’ed elements. It appears on Chrome only.
Bug behavior is erratic. There are 5 elements rendered with v-for:

The problem does not always occur on the first drag

Buttons 1 and 2 always work fine

Button 3 sometimes works fine and sometimes does not trigger any dragenter/dragleave event at all

Buttons 4 and 5:

a.	sometimes work fine (rarely),
b.	sometimes they trigger events only on the preceding element (eg dragging No 4 triggers dragenter/leave on No 3)
c.	sometimes they trigger events on both the dragged element and the preceding element. In such case the order is: 1) PrecedingSibling.DragEnter 2) DragTarget.DragEnter 3) PrecedingSibling.DragLeave 4) DragTarget.DragLeave
d.	However, if you start the drag and don’t move the pointer outside of the DragTarget the bug is very unlikely to occur! If it occurs it never triggers PrecedingSibling only, it triggers both elements: 1) PrecedingSibling.DragEnter 2) DragTarget.DragEnter 3) PrecedingSibling.DragLeave (the latter is therefore not connected with DragTarget.DragLeave, but happens automatically regardless)
e.	On the other hand, if you move the pointer out of the element immediately after drag start, the bug is very likely to occur.
f.	When the dragenter is triggered on the DragTarget again, i.e. when you move the pointer out of the element and come back into it – the sibling event is not triggered anymore, just on the very beginning of the drag.
Bug is present on all 3 versions of Vue in js fiddle and on Vue 2.6.10 in my project.

This is the same problem as was submitted in unresolved issue #4479. I can reproduce the behavior from the jsfiddle mentioned in #4479 on my PC. I hope this submission can shed some more light on the problem, because I noticed some more nuances.
Bug behavior on my local project (Vue 2.6.10) is slightly different than the one from my fiddle described above. The setup is slightly different as well (v-for is used on components which have some nested divs in them). Bug behavior:

The problem does not always occur on the first drag

Once the problem occurs for the first time it occurs on every consecutive drag – except for element no 2 which can behave correctly again.

Dragging element triggers dragenter/dragleave triggers on both the dragged element and on the preceding v-for sibling (sibling first) or just on the preceding sibling. When the dragenter is triggered on the DragTarget again, i.e. when you move the pointer on the child div or when you leave the element and come back into it – the sibling event is not triggered anymore, just on the beginning of the drag.

The difference between js fiddle and local behavior is mainly (but not only) frequency of particular behaviors. On the local setup:
a. second v-for element is also affected, although can react correctly sometimes (in fiddle second is not affected),
b. third element very rarely ignores the drag (in fiddle it is much more frequent)
c. starting from third element rendered by v-for, once the bug occurs, it will occur every time (although will still randomly oscillate between triggering on preceding sibling only or two elements together),
d. the fact of not moving the cursor out of the DragTarget does not prevent the bug from happening (in contrary to jsfiddle description point 4.d ). This might be caused by children divs?
Problem occurs on my two computers (Win7 HP and Win7 Professional), only on Chrome (74.0.3729.157). Mozilla and IE work fine. On one of the computers I removed Chrome with all extensions and history and reinstalled it. The problem persisted. Kaspersky’s Chrome extension is unfortunately automatically loaded on Chrome install, though not activated. Turning off Kaspersky on Windows did not prevent the extension from loading, and I don’t know how to prevent it other than uninstalling Kaspersky which I’d rather not do. Removing the Kaspersky’s extension from Chrome does not change the situation though.
PS. This is my second submission of this issue. Previous was closed by the bot, because I used "reference in new issue" option from my comment to #4479. I thought that linking them this way would be more useful than going through the template. Please consider adding explicit clarification on how to submit issues "reopening" previously closed issues  and/or a issue reference field to the template. I read the documentation, but didnt find it. Sorry for inconvenience.

I swear it's the most detailed issue I have ever seen.

but I can't reproduce the bug too,Maybe It's not a bug about vue.

Haha, hope that's a good thing though. :)
For any future persons seeking answer I was able to prove this is not Vue indeed. I reproduced the bug without any Vue involvement in this fiddle:
https://jsfiddle.net/Eggon666/jd4ztfyh/6/
I thought it was Vue, because it started appearing when I rewrote my project to Vue from vanilla JS and in the vanilla code I didnt experience it (I did check it obviously). Also when I googled for an answer Vue issue #4479 came up and no "general js" bugs, so it looked too obvious. Sorry for the trouble. At least it's clear now Vue's innocent.
Cheers!

Incorrect fiddle in the previous comment. The right one:
https://jsfiddle.net/Eggon666/p8e7donq/37/

Hi! any news on this issue? Im having same problem. Thanks in advance


IDF v5.0 problem with Host Driven Claiming (python issue) (MEGH-4255)

I'm trying Host Driven Claiming. It works with IDFv.4.4.3, but does not work with 5.0
F:\GitHub\esp-rainmaker\cli>python rainmaker.py login
[ERROR]:Descriptors cannot not be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:
 1. Downgrade the protobuf package to 3.20.x or lower.
 2. Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates

I downgraded protobuf to 3.20.0.
Now, login command goes but problem is with claim command:
F:\GitHub\esp-rainmaker\cli>python rainmaker.py claim COM6 --addr 0x7a0000
[ERROR]:module 'esptool' has no attribute 'FatalError'
module 'esptool' has no attribute 'FatalError'

Also tried a few different python versions and each trying ends with [ERROR]:module 'esptool' has no attribute 'FatalError'
Python 3.11 have more issues so i left it. Now trying with 3.7.4 and no luck.

@jacek12345
We are working on this issue and get it fixed soon.


Cant create Default tags

when i try to create a volume with Name tag it just ignore the variable and move to the next tag.
is it in purpose? if so can you change it cause this is very valuable tag we need.

Hi In readme you can see
https://github.com/mtougeron/k8s-aws-ebs-tagger/blob/main/README.md#ignored-tags

is it possible to change it?
i dont understand the purpose of it

Let me dbl check my notes on this. I'm pretty sure it was to not conflict with the tags assigned from the ebs controllers (both in-tree & csi).

Yeah, that was the issue. The aws-ebs-csi-driver already adds the Name tag to the EBS volumes. I'm open to the idea of allowing overriding of that value though.

Yeah, that was the issue. The aws-ebs-csi-driver already adds the Name tag to the EBS volumes. I'm open to the idea of allowing overriding of that value though.

i think it will be great, like to use this tool to tag all, not to use both thing for the same purpose

Created a PR for this but I'm going to think on the approach overnight. I'll update you tomorrow.

Created a PR for this but I'm going to think on the approach overnight. I'll update you tomorrow.

great thanks!

![Screen Shot 2022-03-27 at 12 30 22](https://user-images.githubusercontent.com/71871067/160275488-59963895-513d-43aa-9249-46fb8df5aa00.png)
still doesnt work for me, am i doing something wrong?
also tried with "Name".

@mtougeron FYI

Did you set the new --allow-all-tags flag? In hindsight, I probably should have mentioned that allowing all tags was flagged based.

Did you set the new --allow-all-tags flag? In hindsight, I probably should have mentioned that allowing all tags was flagged based.

no, where should i put it?
didnt see any info about this flag

It's a command line flag for the application. If you're using the helm chart, I believe it'd be
extraArgs:
  'allow-all-tags': ''

It's a command line flag for the application. If you're using the helm chart, I believe it'd be
extraArgs:
  'allow-all-tags': ''

where should i put it?
in the vpc i want to tag? in the k8s-aws-ebs-tagger?
can you provide a yaml file for example?

You would need to create a values file (e.g., myvalues.yaml) with the following content:
extraArgs:
  'allow-all-tags': ''

and when you run helm install k8s-aws-ebs-tagger --values myvalues.yaml mtougeron/k8s-aws-ebs-tagger
the deployment would end up looking like:
      containers:
        - name: k8s-aws-ebs-tagger
          args:
            - --lease-id=$(POD_NAME)
            - --allow-all-tags

You would need to create a values file (e.g., myvalues.yaml) with the following content:
extraArgs:
  'allow-all-tags': ''

and when you run helm install k8s-aws-ebs-tagger --values myvalues.yaml mtougeron/k8s-aws-ebs-tagger
the deployment would end up looking like:
      containers:
        - name: k8s-aws-ebs-tagger
          args:
            - --lease-id=$(POD_NAME)
            - --allow-all-tags

great sir it worked!
thanks alot!!
now i am waiting for the templating part, will be great to use this tool in our envs


DPJTH-1 Add unit tests to HSM sample

Do not merge until there is team agreement that testing is sufficient
Changes in this pull request
Add several unit tests to the HSM sample
Possible additional tests:

Test of LunaHsmManager.getCertificateChain()
Test of LunaHsmHsmLoginParameters

Fulfills  DPJTH-1
Checklist for approving this pull request
(PR creator amend this with more conditions if necessary)

[ ] There are unit tests for new/changed code, or a good explanation why this was not possible.
[ ] All CI builders have indicated success (Give them a few minutes to notice the pull request.)
[ ] The Pull request title has the JIRA issue numbers separated by spaces (if any), a space, and then a short, but descriptive summary.
[ ] Commit messages are well formed: A note about Git commit messages
[ ] New public packages, classes, and methods are documented. (Strongly consider documenting private classes and methods.)

Blockers
Add a checkbox for yourself with your @name to this list if you are holding this PR open for review. PR Shepherd, please hold off merging this PR until these are all checked:

[x] <add your name here with an @>

Build succeeded.

I'm wondering if we should have more distinction between how we name Mocks, Fakes, and Stubs. See discussion: http://stackoverflow.com/a/346440

This is looking really good so far. The overall functionality of the HSM sample, HsmManagerFactory, and the LunaHsmManager appear to have tests covering them.

As mentioned in the pull request message, I see two things that could still be tested:

LunaHsmManager.getCertificateChain(): Verify that LunaHsmManager interacts appropriately with a certificate provider. This is worth testing, but I'm doubtful I can complete it in time for the sprint end.
LunaHsmLoginParameters: Is just a property holder class; there is not currently any logic in it to test.

One question I'm posing with this PR is whether we want to get those tests in before saying we're satisfied that the card is complete.

The integration tests don't run, of course, because I don't have access to hardware. They should always be run in CI. The question is how do we solve this problem...

I advocate for option 1 in @datalogics-kam's list, but again, that will take time to implement.

One way to figure out if we've done enough testing is to run a coverage test. I loaded EclEmma into my Eclipse, and ran the unit tests only (because I can't run the integration tests), and here's sort of a snapshot of what the coverage looks like.

It looks like the security package needs more testing.
Also, FontUtils.java has no coverage at all; it also has no references. I'm wondering if we shouldn't drop it from the project altogether.

I advocate for option 1 in @datalogics-kam's list, but again, that will take time to implement.

Option 1 is a great solution, and I would also like to see us running Jenkins on a machine that has HSM access.

Bear in mind that some of the coverages are low, like HSMSignDocument shows 58%, but that's not too bad, because the part that is covered is the juicy logic, and the part that isn't covered is main(), so it helps to open the source in the editor and look over the line-by-line coverage.

??? How can HsmManager have 81.2% coverage and LunaHsmManager have 0.0%? HsmManager isn't even code: it's an interface!

??? How can HsmManager have 81.2% coverage and LunaHsmManager have 0.0%? HsmManager isn't even code: it's an interface!

It successfully initialized its enum:

Build succeeded.

Build succeeded.

Build succeeded.

Build succeeded.

Looks good to me and builds successfully on my machine. Some stellar work here @datalogics-robl!


Sacred Heart of Jesus occuring with the Nativity of John the Baptist

The Congregation for Divine Worship and the Discipline of the Sacraments released a decree (found via Wiki, where they tell this (the text below is taken from Wiki):

Friday following the second Sunday after Pentecost: Sacred Heart of Jesus – Solemnity (In 2022 this solemnity will coincide with another solemnity [24 June, the Nativity of John the Baptist]. The Holy See has indicated that the Sacred Heart should be kept on Friday 24 June and the Nativity of John the Baptist transferred to Thursday 23 June, except in instances where John the Baptist is a patron saint, where the reverse should apply.)

The exception will be possible/easier to implemented when we implement #201.

We may have an exception in Canada, especially in the Canadian province of Quebec.
Here, Saint-Jean-Baptiste Day is a public and national holiday, fixed on June 24.

We may have an exception in Canada, especially in the Canadian province of Quebec.
Here, Saint-Jean-Baptiste Day is a public and national holiday, fixed on June 24.

I didn’t say there is no exception in any calendar world-wide, but that is will be easier/better to implement the exception rule when we will define the titles (esp the patronage titles) within the celebration definition.
However, if you come up with a way to work this around until #201 is implemented, I’m all good with it. :wink:

It wasn't a criticism, I was just sharing an example I had in mind. 😥

It wasn't a criticism, I was just sharing an example I had in mind. disappointed_relieved

Actually, I didn’t consider your comment as a criticism, but my comment might sound a bit harsh, sorry for that.


Wrap norelativenumber setting in a feature check.

This avoids an error message with vim versions < 7.3, which don't
support the relativenumber option.

So true. How's that? Is it better to amend the commit in a pull request or add another one?

LGTM. Merging now.
Thanks!


Add subcommand to config command

example
config git pull
config provision cui

Support zsh completion.

https://qiita.com/xtetsuji/items/433607abb46118ed200b

ws which is shortcut of workspace looks better.


Can I use it for Angular application?

Of cause you can, as long as you integrate remote-redux-devtools in your redux store.
And sorry for delay of response.

can you please provide sample for it? In my organization we can't install
chrome extension "redux-devtools". So using your vscode extension want to
debug an application.
On Wed, Jun 26, 2019 at 10:11 PM JK<EMAIL_ADDRESS>wrote:

Of cause you can, as long as you integrate remote-redux-devtools in your
redux store.
And sorry for delay of response.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jkzing/vscode-redux-devtools/issues/19?email_source=notifications&email_token=ACNTKY6BLKQKRYBOHTZAAM3P4OL25A5CNFSM4HZUQKZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYUD3WQ#issuecomment-505953754,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACNTKY3XUUOX5ZPHHOTCRYDP4OL25ANCNFSM4HZUQKZQ
.

--
Thanks and Regards,
Himanshu Rana

@himanshu-a-rana  See here https://github.com/jkzing/vscode-redux-devtools#quick-start

Optional, if you don't have an remotedev server running, you need to start one. (Find documentation here or see our example)

Additionally, you also need to compose remote redux devtool middleware into your redux store:

import { createStore, applyMiddleware } from 'redux'
import { composeWithDevTools } from 'remote-redux-devtools'

const composeEnhancers = composeWithDevTools({
  realtime: true,
  name: 'Your Instance Name',
  hostname: 'localhost',
  port: 1024 // the port your remotedev server is running at
})

const store = createStore(
  yourReducer,
  composeEnhancers(
    applyMiddleware(/* put your middlewares here */)
  )
)

Thanks a lot. Will check and get back to you with results.
On Fri, Jun 28, 2019, 2:42 PM JK<EMAIL_ADDRESS>wrote:

@himanshu-a-rana https://github.com/himanshu-a-rana See here
https://github.com/jkzing/vscode-redux-devtools#quick-start

Optional, if you don't have an remotedev server running, you need to
start one. (Find documentation here
https://github.com/reduxjs/redux-devtools/tree/master/packages/redux-devtools-cli
or see our example http:///example/simple-redux)

Additionally, you also need to compose remote redux devtool
middleware into your redux store:

import { createStore, applyMiddleware } from 'redux'import { composeWithDevTools } from 'remote-redux-devtools'
const composeEnhancers = composeWithDevTools({
realtime: true,
name: 'Your Instance Name',
hostname: 'localhost',
port: 1024 // the port your remotedev server is running at
})
const store = createStore(
yourReducer,
composeEnhancers(
applyMiddleware(/* put your middlewares here */)
)
)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jkzing/vscode-redux-devtools/issues/19?email_source=notifications&email_token=ACNTKY6KOJLVQEVJ4LMM42DP4XIWNA5CNFSM4HZUQKZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYZRKKQ#issuecomment-506664234,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACNTKY4PR4OZZYX43WBZQELP4XIWNANCNFSM4HZUQKZQ
.


docs: add TypeDocs auto generated API Reference

fixes #132

Codecov Report

Merging #136 (de4d1d4) into main (5364df1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #136   +/-   ##
=======================================
  Coverage   84.57%   84.57%
=======================================
  Files           9        9
  Lines         188      188
  Branches       44       44
=======================================
  Hits          159      159
  Misses         19       19
  Partials       10       10

Impacted Files
Coverage Δ

libs/ts-rest/core/src/lib/client.ts
80.00% <ø> (ø)

libs/ts-rest/core/src/lib/paths.ts
66.66% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM, however, can you update or create a README somewhere on how to update these API docs? So future maintainers can reference it to auto-update the docs when they make changes? IE, any npm commands to run to update it, etc.


Status of nRF5340 support

Hi, I've noticed that there is an nrf5340 branch on this repository, but I don't know how complete the nRF5340 implementation is, or if there are any plans to merge it to master in the near future. If someone could clarify this it'd be helpful. Thank ou.

It seems the application core was added in https://github.com/nrf-rs/nrf-hal/pull/366 which is available on https://crates.io/crates/nrf5340-app-hal i.e.:
nrf5340-app-hal = "0.14.1"

v0.15.1 includes a HAL for both the Application and Network core, so closing this :tada:


multiple instances of ICal-Objects in different scripts not possible

Hello,
I have a PHP script which opens 3 other PHP scripts in iframes.
I want 2 of those 3 to create an ICal Object (different instances) In both scripts I use:
<?php
    require("./ics-parser-master/class.iCalReader.php");
    $ical = new ICal($icsUrl);
    $events = $ical->events();
?>

The problem I encounter is that on one script the $events array and the $ical itself are empty.
This randomly changes when refreshing the site (the one that houses the iframes). Sometimes the instance of the one script is empty, sometimes the other one is empty. But they are never filled at the same time.
I hope it is a mistake on my side, if so please guide me.
This is really crucial to me.

Not entirely sure what you are trying to achieve. Sounds like you're getting unwanted overlap between objects. Have you tried using different variable names for each of your iCal objects?
require("./ics-parser-master/class.iCalReader.php");

$ical1 = new ICal($icsUrl1);
$events1 = $ical1->events();

$ical2 = new ICal($icsUrl2);
$events2 = $ical2->events();

This question is perhaps better suited for being asked on Stack Overflow

I think there is a misunderstanding.
I am not talking about the same file. But different ones. Please reread my initial post.
in file1.php I have like
 <?php
    require("./ics-parser-master/class.iCalReader.php");
    $ical = new ICal($icsUrl);
    $events = $ical->events();
?>

and file2.php is like:
 <?php
    require("./ics-parser-master/class.iCalReader.php");
    $ical2 = new ICal($icsUrl);
    $events = $ical2->events();
?>

They are running on the same server and both files reside in seperate iframes on the same html page. When i refresh this page strange behaviour described in my initial post occurs.

Okay I found another detail about this issue. It seems like it only occurs with URLs as arguments for the new ICal() object creation. IF you use a file as Argument it works as expected. So chances are high that there is a problem with the ics-url as filename implementation in the class.
I have a working example here in this gist: https://gist.github.com/Bratzmeister/b6ce2e99007f2bf1aa5c
I think they are pretty self explanatory: You put them in the same directory the ics-parser-master directory resides in and  open test.html.
Please correct me if I am wrong.


VM: Inconsistent return value for extracted setters

class A {
  static var x;
}

class B {
  var x;
}

main() {
  var f1 = A#x=;
  var f2 = (new B())#x=;
  print(f1('value'));  // 'value'
  print(f2('value'));  // null
}

Looks like the extracted instance setter should also return 'value'.
16.18.7 Ordinary Member Closurization
...
The closurization of setter f on object o is defined to be equivalent to
(a){return u.m = a;} if f is named m =, except that iff identical(o1, o2) then
o1#m = == o2#m =

It doesn't seem to make sense to fix this unless dart2js and the analyzer care to implement this language feature.


Add threshold to quorum algorithm so it stops the execution

As signal persistence among detrended light curves is a great indicator of good candidates, we might want to add a limit in the quorum algorithm so that it stops sherlock when there is not enough consensus.

Done.


fix: bump json-schema-viewer and json-schema-tree version

Elements Default PR Template
In general, make sure you have: (check the boxes to acknowledge you've followed this template)

[x] Read CONTRIBUTING.md

Other Available PR Templates:

Release: https://github.com/stoplightio/elements/compare?template=release.md

[ ] Read the release section of CONTRIBUTING.md

Fixes: https://github.com/stoplightio/elements/issues/2419
When resolving references in an allOf, if the reference had a description or summary sibling, it was getting overwritten by the  description or summary in the referenced object, if the same key existed. Starting in OAS 3.1, description and summary became supported reference siblings.
This PR bumps versions of packages that have been updated to not overwrite description and summary ref siblings for allOf objects to stay inline with OAS 3.1
Image from modified static-html app with the latest version bumps using a modified version of the OAS file provided in the reported issue :

https://deploy-preview-2481--stoplight-elements-demo.netlify.app/?spec=https://raw.githubusercontent.com/brendarearden/garbage/main/schemas/differentRefSiblings.yaml#/operations/operationId see live preview here of fix

Don't you want to release that package too?

Release looks good too!


SITL logging

TBD: logging concept, sdlog port, etc.
Useful for CI testing

Closing this as we now use rosbag for logging. sdlog port could be something that comes along with more Linux porting later.


onediffx compile_pipe error

when use onediffx with the following code,
################################
import torch
from diffusers import StableDiffusionXLPipeline
from onediffx import compile_pipe
pipe = StableDiffusionXLPipeline.from_pretrained(
"stabilityai/stable-diffusion-xl-base-1.0",
torch_dtype=torch.float16,
variant="fp16",
use_safetensors=True
)
pipe.to("cuda")
pipe = compile_pipe(pipe)
#################################
I got these error:
Loading pipeline components...: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████| 7/7 [00:18<00:00,  2.61s/it]
Setting ds_accelerator to cuda (auto detect)
WARNING [2024-02-23 07:28:52] /home/notebook/code/group/songxin/AIGC/onediff/src/onediff/infer_compiler/transform/custom_transform.py:49 - Failed to import register_diffusers from /home/notebook/code/group/songxin/AIGC/onediff/src/infer_compiler_registry/register_diffusers. e=RuntimeError('Failed to import diffusers.models.unet_2d_condition because of the following error (look up to see its traceback):\nFailed to import transformers.models.clip.modeling_clip because of the following error (look up to see its traceback):\nduplicate validator function "deepspeed.runtime.zero.offload_config.DeepSpeedZeroOffloadOptimizerConfig.set_pipeline"; if this is intended, set allow_reuse=True')
WARNING [2024-02-23 07:28:52] /home/notebook/code/group/songxin/AIGC/onediff/src/onediff/infer_compiler/transform/custom_transform.py:49 - Failed to import register_diffusers from /home/notebook/code/group/songxin/AIGC/onediff/src/infer_compiler_registry/register_diffusers. e=RuntimeError('Failed to import diffusers.models.unet_2d_condition because of the following error (look up to see its traceback):\nFailed to import transformers.models.clip.modeling_clip because of the following error (look up to see its traceback):\nduplicate validator function "deepspeed.runtime.zero.offload_config.DeepSpeedZeroOffloadOptimizerConfig.set_pipeline"; if this is intended, set allow_reuse=True')
WARNING [2024-02-23 07:28:52] /home/notebook/code/group/songxin/AIGC/onediff/src/onediff/infer_compiler/transform/custom_transform.py:49 - Failed to import register_diffusers from /home/notebook/code/group/songxin/AIGC/onediff/src/infer_compiler_registry/register_diffusers. e=RuntimeError('Failed to import diffusers.models.unet_2d_condition because of the following error (look up to see its traceback):\nFailed to import transformers.models.clip.modeling_clip because of the following error (look up to see its traceback):\nduplicate validator function "deepspeed.runtime.zero.offload_config.DeepSpeedZeroOffloadOptimizerConfig.set_pipeline"; if this is intended, set allow_reuse=True')
WARNING [2024-02-23 07:28:52] /home/notebook/code/group/songxin/AIGC/onediff/src/onediff/infer_compiler/transform/custom_transform.py:49 - Failed to import register_diffusers from /home/notebook/code/group/songxin/AIGC/onediff/src/infer_compiler_registry/register_diffusers. e=RuntimeError('Failed to import diffusers.models.unet_2d_condition because of the following error (look up to see its traceback):\nFailed to import transformers.models.clip.modeling_clip because of the following error (look up to see its traceback):\nduplicate validator function "deepspeed.runtime.zero.offload_config.DeepSpeedZeroOffloadOptimizerConfig.set_pipeline"; if this is intended, set allow_reuse=True')
WARNING [2024-02-23 07:28:52] /home/notebook/code/group/songxin/AIGC/onediff/src/onediff/infer_compiler/transform/custom_transform.py:49 - Failed to import register_diffusers from /home/notebook/code/group/songxin/AIGC/onediff/src/infer_compiler_registry/register_diffusers. e=RuntimeError('Failed to import diffusers.models.unet_2d_condition because of the following error (look up to see its traceback):\nFailed to import transformers.models.clip.modeling_clip because of the following error (look up to see its traceback):\nduplicate validator function "deepspeed.runtime.zero.offload_config.DeepSpeedZeroOffloadOptimizerConfig.set_pipeline"; if this is intended, set allow_reuse=True')
Exception ignored in: <function Library.del at 0x7f05d967dcf0>
Traceback (most recent call last):
File "/opt/conda/lib/python3.10/site-packages/torch/library.py", line 136, in del
AttributeError: 'NoneType' object has no attribute 'remove'
torch version: 2.0.1+cu118
diffuser version: 0.25.1
transformer version: 4.27.1
How can I fix these errors, thank you very much!

@West2022 Please disable DeepSpeed.

@West2022 Please disable DeepSpeed.
Thanks your reply extremely.
But how to disable DeepSpeed?

deepspeed.runtime.zero.offload_config.DeepSpeedZeroOffloadOptimizerConfig

pip list | grep deepspeed

to see whether DeepSpeed is installed
pip uninstall deepspeed

to uninstall deepspeed
@West2022

@West2022 Any new progress?

No new progress. Feel free to reopen.

I have this same issue, no deep speed installed.
diffusers                     0.26.2
transformers                  4.37.2
 python examples/text_to_image.py
Exception ignored in: <function Library.__del__ at 0x7f1d523d1670> Traceback (most recent call last):   File "/opt/conda/envs/pytorch/lib/python3.9/site-packages/torch/library.py", line 139, in __del__ AttributeError: 'NoneType' object has no attribute 'remove'

Update, this error only happens when the scripts completes, so it is not blocking

Update, this error only happens when the scripts completes, so it is not blocking

Thanks!
This looks like an object destruction problem. We will try to reproduce the problem.

Has this issue been fixed?
use oneflow_compile from onediff.infer_compiler
/python3.8/site-packages/torch/library.py", line 136, in __del__ AttributeError: 'NoneType' object has no attribute 'remove'

@cchen-reese Can you provide an example for it so we can reproduce this?

too old to follow, please feel free to reopen it.


Legacy connection string for Postgres <=9.4, fix for later versions

Use Case
We use Postgresql 16 for PuppetDB, and the class puppet_operational_dashboards::profile::foss_postgres_access only has compatible configuration for legacy Postgres 9.4 and earlier.
Describe the Solution You Would Like
We have changed this config to a parameter as follows, and can now pass in the correct parameter using Hiera.
Hiera entry:
puppet_operational_dashboards::profile::foss_postgres_access::auth_option: 'map=puppetdb-telegraf-map clientcert=verify-full'
Revised code snippet:
class puppet_operational_dashboards::profile::foss_postgres_access (
String $auth_option    = 'map=puppetdb-telegraf-map clientcert=1',
...
) {
...
postgresql::server::pg_hba_rule { "Allow certificate mapped connections to puppetdb as ${telegraf_user} (ipv4)":
type        => 'hostssl',
database    => 'puppetdb',
user        => $telegraf_user,
address     => '<IP_ADDRESS>/0',
auth_method => 'cert',
order       => 0,
auth_option => $auth_option,
}
postgresql::server::pg_hba_rule { "Allow certificate mapped connections to puppetdb as ${telegraf_user} (ipv6)":
type        => 'hostssl',
database    => 'puppetdb',
user        => $telegraf_user,
address     => '::0/0',
auth_method => 'cert',
order       => 0,
auth_option => $auth_option,
}
...
}
}
`
Additional Context
https://stackoverflow.com/questions/36927928/purpose-of-clientcert-1-mean-in-postgresql-configuration

Could you reformat this or can you put this into a PR for investigation purposes?


Unable to pass dynamic flags at runtime with strict set to false in command class

We have one custom oclif command for test which is internally using Jest. It just acts as a wrapper on jest and passes all flags and arguments to jest directly.
In the previous version of<EMAIL_ADDRESS>we were able to pass the dynamic argument at runtime by setting strict to false in the command class.
my-command test --watch

when we try to do the same with the<EMAIL_ADDRESS>it gives us the following nonExistentFlags error although we have set strict to false in the command class.
  oclif: { exit: 2 },
  code: undefined,
  parse: {
    input: {
      argv: [Array],
      context: [Test],
      '--': undefined,
      flags: [Object],
      args: {},
      strict: false
    },
    output: {
      argv: [Array],
      flags: [Object],
      args: {},
      raw: [],
      metadata: [Object],
      nonExistentFlags: [Array]
    }
  },
  flags: [ '--watch' ]
}
 ›   Error: Nonexistent flag: --watch
 ›   See more help with --help
error Command failed with exit code 2.

as you know jest has a lot of flags so it's difficult for us to define each and every flag in its wrapper
To Reproduce
Steps to reproduce the behavior:

Create an oclif command.
Set strict to false.
Run the command with a flag that is not defined in the command
See error Nonexistent

Expected behavior
As we have set strict to false in the command class, it should run with a dynamic flag provided at runtime.
Screenshots

Environment :

OS & version: MacOS Ventura 13.5
Shell/terminal & version: Bash 3.2.57(1)-release (x86_64-apple-darwin22)
Node v18.16.0

Thanks in advance.

@RohitChaturvedi05 Here's how one option for how you could do this:
import {Command, Flags} from '@oclif/core'

const getDynamicFlags = (argv: unknown[]) => {
  return (argv as string[]).filter(a => a.startsWith('-'))
}

export default class Hello extends Command {
  static description = 'Say hello'

  static examples = [
    `$ oex hello friend --from oclif
hello friend from oclif! (./src/commands/hello/index.ts)
`,
  ]

  static flags = {
    from: Flags.string({char: 'f', description: 'Who is saying hello', required: true}),
  }

  static strict = false

  async run(): Promise<void> {
    const {flags, argv} = await this.parse(Hello)
    this.log('flags', flags)
    this.log('dynamic flags', getDynamicFlags(argv))
  }
}

~/repos/oclif/hello-world (main*) » bin/dev hello --from Oclif -- --watch --foo
flags { from: 'Oclif' }
dynamic flags [ '--watch', '--foo' ]

Important things to note in the example are:

strict is set to false
The argv array from this.parse is used to get everything passed in after the passthrough (--)
If you also define args for your command you'd need to filter those out from argv as well

Thanks, @mdonnalley for suggesting this solution.
This comes with some restrictions. Our scripts are used similarly as react-scripts which are added in package.json.
/ /package.json "test": "my-command test" }
Now to run this command consumer runs them like this yarn run test
Here we can not pass arguments like this yarn run test -- --watch  as (--) this is not supported by yarn.
To fix this consumer need to update the package.json, which defeats the purpose of having dynamic flags
//package.json "test": "my-command test -- --watch"

Thanks @mdonnalley , will consider this solution as we don't have any other alternative.


Update README.md

笔误

Thank you :)

Thank you for your open source project & blog too.


Stats - time to discover and transfer new cids to the gateway and their respective dag size

"do we have metrics on "time to discover and transfer new hashes to the gateway and their respective dag size? this would be a cool stat for the WebUI as well, which means that the ipfs-api should have a way to get stats from Gateway specific things."

@diasdavid

cc @kyledrake – I think we have some sorts of stats for gateways already as a prior art (?)


Saving Vizro report as HTML

Question
Hi, I am using Vizro to generate visualizations and would like to know if there is a way to save the report in HTML format. My goal is to save these reports as artifacts in MLflow. Could you provide an example or guidance on how to achieve this?
Code/Examples
No response
Which package?
vizro
Code of Conduct

[X] I agree to follow the Code of Conduct.

Hello @Yairama and thank you for your question! The answer is that there's no built-in way of doing this but it is on the roadmap as a future feature. But, depending on exactly what you want to save, you may already be able to achieve what you need.
Please could you explain exactly what you'd like to save? e.g. just the graphs? one page or multiple pages? the filters/parameters controls? It's already possible to save plots to png from your browser by clicking the little "Download to png" icon in the top right corner of each graph.

Hello @antonymilne ,
Thank you for your response! I'm content with saving an HTML page containing multiple graphs divided into various sections, or several pages (each section as a page) with their respective graphs. There is no need for filters; I'm only interested in ensuring the plots remain interactive.
P.S.: Nice cat.

Hi @Yairama, sorry for the very delayed reply - I completely lost track of this and was then reminded about it again today.
Here's what I've come up with as a possible solution that I think should work pretty well for now🤞The biggest (only?) catch is that you cannot easily download the entire dashboard into a single HTML file; instead you need to download each page individually.
This page from the plotly docs is a good reference. Note:

the file downloaded file will be quite large since it includes plotly.js but that can be changed using the include_js argument
you might like to use Jinja template(s) for your report (e.g. to loop through graphs) rather than just a simple Python string format like I did here with REPORT_TEMPLATE

Link to py.cafe example with the following code.
from dash import dcc, callback, State, Input, Output
import dash_bootstrap_components as dbc
from dash.exceptions import PreventUpdate

import vizro.plotly.express as px
from vizro import Vizro
import vizro.models as vm
from vizro.managers import model_manager
from vizro.models import Page
from vizro.models.types import capture
import plotly.graph_objs as go
import plotly.io as pio

df = px.data.iris()

# Template that graph HTML is injected into
REPORT_TEMPLATE = """
<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
</head>
<body>
{graphs}
</body>
</html>
"""

class PageWithDownloadReport(vm.Page):
    def pre_build(self):
        super().pre_build()

        # Fetch all graphs on the page. There will be a nicer way to do this in future
        graph_ids = [
            model_id
            for model_id in model_manager._get_model_children(self.id)
            if isinstance(model_manager[model_id], vm.Graph)
        ]

        # Build a callback to take in all graph on the page and output an HTML report
        @callback(
            output=Output("download", "data", allow_duplicate=True),
            inputs=[
                Input(f"download_button_{self.id}", "n_clicks"),
                tuple(State(graph_id, "figure") for graph_id in graph_ids),
            ],
            prevent_initial_call=True,
        )
        def download_report(n_clicks, graphs):
            if not n_clicks:
                raise PreventUpdate

            # Convert each graph to HTML, concatenate all the resulting divs together and then inject into
            # REPORT_TEMPLATE.
            graphs_html = "".join(pio.to_html(graph, full_html=False) for graph in graphs)
            report_html = REPORT_TEMPLATE.format(graphs=graphs_html)
            return dcc.send_string(report_html, filename=f"report_{self.id}.html")

        # Add download button to controls.
        self.controls.append(
            vm.Button(id=f"download_button_{self.id}", text="Download report"),
        )

page_1 = PageWithDownloadReport(
    title="Page 1",
    components=[
        vm.Graph(figure=px.histogram(df, x="sepal_width", color="species")),
        vm.Graph(figure=px.scatter(df, x="sepal_width", y="sepal_length", color="species")),
    ],
    controls=[vm.Filter(column="species")],
)

page_2 = PageWithDownloadReport(
    title="Page 2",
    components=[
        vm.Graph(figure=px.scatter(df, x="sepal_width", y="sepal_length", color="species")),
    ],
    controls=[vm.Filter(column="species")],
)

dashboard = vm.Dashboard(pages=[page_1, page_2])
app = Vizro().build(dashboard)

# Download component to handle downloaded reports:
app.dash.layout.children.append(dcc.Download(id="download"))

app.run()

FYI @petar-qb

Thanks a lot @antonymilne for the backup solution !!
Won't it be possible to keep the filters, buttons,...inside the html report ?
Basically, an integrated solution of this feature would be a real plus for vizro. For medium-sized applications, sharing an html file is oftent  a faster solution with no security constraints, compared with more traditional deployment solutions. Some libraries (such as Datapane) offered this practical feature.

@antonymilne Thanks for the answer, testing it!
Additionally, it would be a future feature?

Thanks for the comment @Bebio95! To be honest this feature has gone down in priority now that we have py.cafe which enables you to easily share not just static HTML but a fully functioning dashboard. This definitely does not work for all use cases (e.g. with a free account your dashboard will be public for a start) but we really like it as an alternative to traditional deployments.
However, static export is still something that's on our roadmap and I am always thinking about it and have repeatedly gone back to it and played around with some options. And this time I think I may have actually found something that works great without us doing anything! 🙌
Check out https://github.com/gildas-lormeau/SingleFile. It's a browser extension that works great on Vizro (and Dash too I guess).

the default options seem to work well out of the box so no need to fiddle with anything
downloading a whole dashboard is possible using Batch URL download with two gotchas:

links between the pages don't work correctly. I opened an issue to see if they would add a feature to help with this - see https://github.com/gildas-lormeau/SingleFile/issues/1591
controls from pages that you don't already have open will be reset to their original values instead of persisted. We can probably change this but the right path forward depends on the answer to https://github.com/gildas-lormeau/SingleFile/issues/1591

no scripts run in the downloaded HTML so it's not possible to e.g. change theme, collapse nav panel or zoom around graphs. The reason for this is explained in their FAQ. Unfortunately enabling these interactive elements doesn't work right away because you'll just see the Dash "Loading..." screen in your HTML file. Possibly there's some ways around this - would need more investigvtion

Some alternative that's also worth knowing about:

https://gofullpage.com/ - if all you need is a static picture then this seems great
just doing "Save as..." ➡ "Web page, Single File" in Chrome generates MHTML that works surprisingly well and doesn't need you to install any extension. The only catch I see here is that font files aren't included, which means that nav icons and the nav panel arrows don't render correctly. Possibly we could fix this but it would have downsides too, and the only advantages I see of this route compared to SingleFile is that you don't need to install an extension
there seems to be other extensions out there, e.g. https://chromewebstore.google.com/detail/save-webpages-offline-as/nfbcfginnecenjncdjhaminfcienmehn?hl=en, but I didn't try all of them. Maybe some are better suited to Vizro than SingleFile, but for now SingleFile looks like a great option to try anyway

Let me know what you think! I've really spent a lot of time thinking about this problem over the last year and I'm super excited to have found SingleFile so I hope it works for you!

Hello @antonymilne,
First of all, I want to really thank you for your work and all the time you have dedicated to this topic.
I will have a look at the different solutions proposed and I'll give you a feedback shortly.
Thks


⚠ Bofrost.de has degraded performance

In 155f5ee, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1339 ms

Resolved: Bofrost.de performance has improved in d664d75 after 9 minutes.


fixes #5

fixes #5

I guess there is a new field consequently while listing all apps validation fails. I don't know what this filed is for and couldn't found it in the documentation. Maybe documentation is not updated to reflect the changes yet. Because of this left is as Optional. Thanks for the awesome code by the way

I accidentally changed in same branch so will explain that too here:
in api.py line 207 it seems that we don't need to check for all chars to be below 128, it is only the first 3 one that is above 128. Others seem fine. I'm using this library daily and this haven't stumbled across another char that is above 128


youtube-dl --external-downloader doesn't work with a Full Path

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2<IP_ADDRESS>
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

Description

Using the "--external-downloader" argument doesn't work with a full path...
youtube-dl.exe --external-downloader=D:\aria2\aria2c.exe
However, if I put "aria2c.exe" in the folder above youtube-dl.exe, then this command does work...
youtube-dl.exe --external-downloader=..\aria2c.exe
I'm well aware that I can change the PATH variable, and just use "aria2c" instead of a Full Path, but I was just curious on why using a Full Path doesn't work? Was not allowing full paths the intended behavior? Or is this just an oversight?

On Windows (at least), you need to use double-quotes, and backslashes must be escaped. The following works for me:
youtube-dl --external-downloader "C:\\Program Files\\aria2\\aria2c.exe" https://www.youtube.com/watch?v=vwnXqmgNBEw

Forward-slashes also work on Windows (again with double-quotes), if you prefer:
youtube-dl --external-downloader "C:/Program Files/aria2/aria2c.exe" https://www.youtube.com/watch?v=vwnXqmgNBEw

Don't forget to specify your --external-downloader-args, if any.

I was already using double-quotes, but not escaped backslashes. However, I just tried using double-quotes AND escaped backslashes like you mentioned, but it still doesn't work.
I noticed that you have installed aria2c on your system, that means that "aria2" is probably listed in your PATH environment variable which is why it's probably working for you. Go remove the "C:\Program Files\aria2\aria2c.exe" from your envorinment PATH and try it again. I'm testing this on a system where aria2c is only being used by calling the EXE directly.

Hmm, that would be weird, because according to what you're saying, youtube-dl would have to be taking a fully-specified command line argument, and explicitly stripping away the path part, to obtain just the executable name -- that it would then launch, now dependent on the environment PATH?
Anyway, @dstftw please don't let my observation affect whether you look into this...
p.s. I assume your aria2c is the statically linked version (i.e., not the mingw version), so there's no issue of it somehow not finding its own libraries....

Yeah, my aria2c.exe binary works just fine, no issues there, and like I said in the OP, it works just fine if I use "--external-downloader aria2c", it just doesn't work when I use the full path. I know I can add my aria2c path to my PATH environment variable, but it doesn't make sense why youtube-dl isn't accepting the full path with "--external-downloader". Seems like it's probably just a simple oversight to me.
For example, the batch script below works just fine and aria2c is correctly being called from my "D:\Temp\aria2c" folder...
@echo off
set path=%path%;D:\Temp\aria2c
youtube-dl.exe --external-downloader=aria2c http://www.website.com/video.mp4
However. this batch script below doesn't work when I use the full path with "--external-downloader" instead of using the %path% variable...
For example, the batch script below works just fine and aria2c is correctly being called from my "D:\Temp\aria2c" folder...
@echo off
youtube-dl.exe --external-downloader="D:\Temp\aria2c\aria2c.exe" http://www.website.com/video.mp4
None of the syntax's below are working....
"D:\Temp\aria2c\aria2c.exe"
"D:\\Temp\\aria2c\\aria2c.exe"
"D:/Temp/aria2c/aria2c.exe"

The --external-downloader is not designed to take a path, it takes a name and checks that against the internal list of supported downloaders. (I'm not saying this couldn't be changed, I'm just saying what the current implementation does).
The option certainly needs to understand which tool it is given on the command line, in order to apply the correct options.

That's exactly what I figured the issue was. Thank You for confirming.

yt-dlp has an implementation to allow full paths https://github.com/yt-dlp/yt-dlp/commit/7f7de7f94dfeacb0eb78cb9487dfcaf5707da381 https://github.com/yt-dlp/yt-dlp/commit/e1eabd7beb4cc83338a7422546ae1c9ae8b2097f
This doesn't work for ffmpeg since it has it's own option


Close

Closes #920

This pull request introduces 1 alert when merging d8a83a0a579ebbbec7324ffb36ad16ca2badb6ec into 782adad75a8b6b19014ac771575d5ffac360913a - view on LGTM.com
new alerts:

1 for Unused variable, import, function or class

Comment posted by LGTM.com


No data fron https://cloud.alphaess.com/

I get no statistics, only realtime.
I know this is probably not an integration error, but can somebody confirm this or is it just me?

I'm certainly seeing my own statistics correctly.  The image you attached was from the AlphaCloud service.  Is that showing statistics for today or not?  If AlphaCloud doesn't have statistics then this integration will not either - I can only pull what is available on Alpha.

Yes, the screenshot is from today.
Okay thank you, then maybe the time settings of my Alpha are incorrect.
Yesterday statistics ends at 22:30.

Yeah, its definately a time offset. Marked as fixed


Watch-Only Wallets

Please enable us to import a pub or something along these lines to watch other wallets like our hardware cold-storage from inside Trust-Wallet without the need to switch apps all the time.

Importing by public key is possible, this may partly satisfy your use cases.
Importing the private key (extended or not) or recovery phrase is not supported, and it would not be a good idea for security: private key should not leave the hardware wallet in any form.
Another issue is that currently Trust Wallet does not support multiple wallet addresses for the same coin, and there are no plans to support this in the foreseeable future.
See the section  'Method 4: Import via Public Key' section of the guide:
https://community.trustwallet.com/t/how-to-import-a-wallet/87#method-4-import-via-public-key

Yeah I get for ETH it is possible but why not for lets say BTC / LTC.
Also I don't want to export my private Key from the hardware-wallet, but only the xpub. That way I could see the balance but don't loose security.

No current plan for BTC XPub import.
No plans for supporting multiple addresses for a coin.  Closing.


Updated travis build scripts (from rspec-dev)

See: https://github.com/rspec/rspec-dev/pull/201

I will look at errors on 2.4.3 and 2.5 but meanwhile, I don't think you want this change for Rubocop?

I don't think you want this change for Rubocop?

I think @xaviershay was working on reconciling the rubocop setting changes.  For now, undo that change in this branch.

Does this change https://github.com/rspec/rspec-core/pull/2524/commits/d9be90fede2d49a7af0fb7dd2a99513cad26d7dc should be implemented in rspec-dev?

Does this change d9be90f should be implemented in rspec-dev?

Sure!  thanks for doing that.

Thanks!


Add documentation for choosing a bourbon version

This is in reference to @lukemelia's request for docs in https://github.com/yapplabs/ember-cli-bourbon/pull/21#issuecomment-364006157 and a blatantly transparent attempt to butter you up so I can ask for a release with the changes in #21 😜
If these weren't the docs you wanted let me know and I'm happy to write some more!

 Thanks!

Published as 2.0.0

Fantastic. Thanks!


Add Flask-Executor to app.py

This pull request adds Flask-Executor to the app.py file. It also includes the necessary changes to the requirements.txt file to add the flask-executor package as a dependency.

Using Flask-Executor, it's possible to have a lightweight queue that can store jobs and retrieve based on the Flask state of the application


Pending Transfers flow

For external users: Pending transfers should be shown only if the current user is the "To" user.

#149


CI: Switch To Centralised Workflows

:heart: Thank you for your contribution!
Description

Change branches type from string to array
Switch to centralised workflows

Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[ ] I'm aware of the code of conduct.
[ ] I've created logical separate commits and followed the commit message format.
[ ] I've added relevant test cases.
[ ] I've added relevant documentation.
[ ] I've marked translation strings.
[ ] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[ ] I've NOT included third-party code (copy/pasted source code or new dependencies).

If you have added third-party code (copy/pasted or new dependencies), please reach out to an architect.

Frontend

[ ] I've followed the CSS/JS and React guidelines.
[ ] I've followed the web accessibility guidelines.
[ ] I've followed the user interface guidelines.

Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

Closing in favor of https://github.com/inveniosoftware/invenio-app-rdm/pull/2773


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2222

Questions ?
Please refer to the Backport tool documentation


HDDS-10095. Move SCM tests to org.apache.hadoop.hdds.scm package hierarchy

What changes were proposed in this pull request?
SCM tests are split in two package hierarchies:

org.apache.hadoop.hdds.scm
org.apache.hadoop.ozone.scm

This PR moves tests from the second one (.../ozone/...) to the first one (.../hdds/...), since production code is also in that hierarchy.
https://issues.apache.org/jira/browse/HDDS-10095
How was this patch tested?
CI:
https://github.com/adoroszlai/ozone/actions/runs/7481069440

Converted to draft because there are some pending PRs changing the SCM tests being moved by this PR.

Thanks @kaijchen for the review.


docs(POC): cards on landing page; tags, categories

Relates to Jira: CDAAS-2375
https://deploy-preview-1997--armory-docs.netlify.app/cd-as-a-service/
Taxonomy categories/tags only appears on CDaaS product -> the "taxonomy cloud" partial is commented out in the other product sets

moved to docs-cdaas repo


Error building northstar binary

Hi everyone,
whilst building the northstar binary, the process fails with the following error:

Would you know the root cause of the error?
Compiling on darwin.
Thanks
Thomas

Hello @thomashoefkens-vw
Northstar doesn't support Darwin and runs on Linux only. It relies on Linux features which showed up to be to hard to emulate on Darwin (which we tried in the past).
cheers,
Felix

Hello @thomashoefkens-vw
Northstar doesn't support Darwin and runs on Linux only. It relies on Linux features which showed up to be to hard to emulate on Darwin (which we tried in the past).
cheers,
Felix

Hi Felix, understood, thanks for that.

Hi Felix, understood, thanks for that.


Docs: Include ZK Deprecation notice and information

Include information on ZK deprecation in documentation and provide timelines for removal. With AK 3.5 ZK is marked deprecated for new clusters.  This section aims to provide information on what deprecation means, timelines, and path to migration with KRaft.
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

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

I updated the sentence suggested by @ijuma

Merged to trunk and cherry-picked to 3.5 branch. Will take care to get the changes into kafka-site, too.


Not able to change screenshot tool from Print Screen button.

Microsoft PowerToys version
0.75.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
Trying to disable any of the tools that is currently assigned to the Print Screen button, so I can use the button in another software in parallel.
✔ Expected Behavior
Be able to release the usage or biding of the Print Screen key in the keyboard to any other software.
❌ Actual Behavior
Currently, the software binds its own screenshot tool to the printscreen key and does not offer to disable this behavior as a toggle or something.
Other Software
Using in parallel with SnagIt, having conflicts with it.

Are you trying to remap PrintScreen key using PowerToys Keyboard Manager?
/needinfo

@davidegiacometti Hi! I recently installed the PowerToys and it has its own screenshot tool that took over the PrintScreen key. I tried to find ways to disable that screenshot tool, or even to rebind using the keyboard mapping tool that you mentioned, but I have not been successful yet.
Sorry that I haven't responded earlier, I thought my initial data that I provided made it clear that I wanted to unbind the PrintScreen key to use it on another app not related to PowerToys.
I find so useful that there are many toggles here and there to enable and disable tools within the PowerToys, so, perhaps we could simply add a toggle that allows disabling that screenshot tool or its key binding.
Let me know if I can help somehow.
Thanks!

Can you post a screenshot of the screenshot tool (😄) that is opened when you press the print screen key?
/needinfo

Can you post a screenshot of the screenshot tool (😄) that is opened when you press the print screen key?
/needinfo

Sure!
It is the TechSmith Snagit tool, which is always open and has PrintScreen bind to it, but after installing the PowerToys, it replaced it to another one.
Let me know if you still want to see a screenshot of the Snagit tool.
The case was marked as closed, can we re-open it?
Thanks

Are you on Windows 11?
It seems that the PrintScreen now opens the Windows built-in Snipping Tool.

Are you on Windows 11?
It seems that the PrintScreen key now opens the Windows built-in Snipping Tool.

Yes! I'm on Windows 11. Sorry for the delay, I probably missed the notification.
It might  be the snipping tool like you said. I have never used it before (not really familiar with it), but it only started "to exist" (per say) after I installed PowerToys. Before it, the PrintScreen would do a sient copy of the screen into the clipboard, so I could paste it anywhere, and on top of that I have Snagit installed to allow me cropping parts of the screen and etc.
Thanks

Reopened since the bot closed it.
PowerToys doesn't come with a snipping tool so I would like to see a screenshot of the PowerToys one.
If you close PowerToys does TechSmith Snagit tool come back?
/needinfo

I tried closing PowerToys but it comes simply hides on the systray. If I force it through task manager, it re-opens once or twice then nevermore (I guess it works as a service?). Even after killing it, Snagit failed to assign that key back to it, it still using the snipping tool (even after I closed the snipping tool using the task manager).
I attempted to uninstall the powertoys, and the snipping tool ia not coming up anymore, and Snagit is work again. Perhaps something gets hard coded in the registry for key binding?


Phi 2 2.7B Support

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
Please close this if it is already supported, but I would like to request phi 2 support with axolotl.
✔ Solution
Implement phi 2 ft support
❓ Alternatives
No response
📝 Additional Context
No response
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

Could you try see if you can use the phi 1 config and point to the phi2 model?

Hey it works for me
https://huggingface.co/cognitivecomputations/dolphin-2_6-phi-2/blob/main/configs/phi-dolphin-qlora.yml

I tried using the existing phi config for phi2 and it looks like there might be an issue with save_safetensors config. I set it to False explicitly and this allowed my run to complete. If I don't set anything (assuming it picks up true for this config) I get a runtime error,
Some tensors share memory, this will lead to duplicate memory on disk and potential differences when loading them again

@stangirala , yes I think Eric had this issue before. Turning off safetensor seemed to fix it. Thanks for linking that issue.

Closed thanks to @ehartford 's shared yaml above!


Rephrase "Is the following text human written or machine generated?"

Instead of

Is the following text human written or machine generated?

I recommend

Is this {story|news article|speech} written by a person or a machine?

Done.


Lexical BlockFeature: minRows/maxRows fails validation

Link to reproduction
No response
Describe the Bug
I have Block which allows editors to create a multi-column layout. It consists of a Select and three conditional fields being an Array with minRows and maxRows corresponding to a selected option.
Having this Block outside Lexical works just fine. Now if otherwise: condition works perfectly in order to show or hide an Array UI component: Add/Remove buttons are properly updated as I change the layout. However as soon as I invoke validation, it fails as if there would be only one Array field with no conditions, i.e. having properly set up 2 rows fails with an error caused by a minRows: 3, or 4 rows — by maxRows: 2 etc. It also breaks validation of the same Block outside Lexical at the same time.
To Reproduce
Block fields:
fields: [
	{
		label: 'Layout',
		name: 'layout',
		type: 'select',
		options: [
			'1/1',
			'1/2',
			'2/1',
			'1/1/1',
			'2/1/1',
			'1/2/1',
			'1/1/2',
			'1/1/1/1',
		],
		defaultValue: '1/1',
		required: true,
	},
	{
		...BlockColumns,
		admin: {
			condition: (data, siblingData) =>
				['1/1', '1/2', '2/1'].includes(siblingData.layout),
		},
		minRows: 2,
		maxRows: 2,
	},
	{
		...BlockColumns,
		admin: {
			condition: (data, siblingData) =>
				['1/1/1', '2/1/1', '1/2/1', '1/1/2'].includes(
					siblingData.layout
				),
		},
		minRows: 3,
		maxRows: 3,
	},
	{
		...BlockColumns,
		admin: {
			condition: (data, siblingData) =>
				['1/1/1/1'].includes(
					siblingData.layout
				),
		},
		minRows: 4,
		maxRows: 4,
	}
]

Where BlockColumns in an array:
export const BlockColumns: ArrayField = {
	type: 'array',
	name: 'columns',
	interfaceName: 'BlockColumns',
	label: 'Spalten',
	labels: { singular: 'Spalte', plural: 'Spalten' },
	admin: {
		initCollapsed: true,
	},
	fields: [...],
};

Payload Version
2.0.14
Adapters and Plugins
richtext-lexical, db-mongodb, bundler-webpack

Hey @pooledge just a heads-up:
You're using multiple fields with an identical name. Because this has a lot of negative side-effects (despite kinda working here), this will not work in a future version and will cause an error to be thrown in the config sanitization process.
You could use a custom validate function, but that would not cause the number of columns to change in the admin UI. For that, you could use a custom React component.
The optimal way to handle this in the future would be for the minRows and maxRows properties to accept a function - that way, they could be conditional depending on the layout field's value. A PR for that is definitely accepted, if you'd like to prepare for the future version! Otherwise, feel free to make a feature request for this in GH Discussions :)


getInitialProps query is undefined during next export

When no query is given, getInitialProps({query}) is undefined when running next export. It's {} in the browser and when running on the server.
Version: 3.0.0-beta3

Good one. Will fix it.


Add pxt_blockly_functions to blockly.html for debugging uncompressed …

…pxt_blockly, and alphabetize scripts.

Did not know this file existed :)


Update readme

Hi this is what it looks like from a markdown editor:

Lemme know what you think. I could have added more links to the functions within the subpackes but thought it would be overkill. If you think something else ought to be mentioned, etc, then by all means.
Also, I'd like to edit your godoc if you're cool with that: gizmo, you notice that there are so many backticks that are quite useless, also, some other errors are here or there.
Thanks!

Coverage remained the same at 47.67% when pulling 4b6a1c96ef3a193d7bb7bdecc6b36bf07513fc35 on jMuzsik:update_README into 6fdbe2c1590e782aedb01eb3510fc0d9580bf3bd on NYTimes:master.

This is a great start, thanks!!
A couple bits I'd like to add:

include a small note and link for each pubsub sub package
include the server/kit package and the KitServer to match what you've done with the server package

And definitely feel free to clean up the doc.go's as well! I believe we were trying to keep some of this stuff duplicated there, but much of it is unkept.
Thanks again!

Ok, I forgot about the image change, it's not important. And this is the next draft.
I mainly just copied the server/kit README.md that you have already and added the issue part at the bottom of the README as you had within it.


"Unsupported Realm file format version" crash using Realm 0.95.3

We had been using XCode 7 and Realm 0.95.3 with no issue. Lately we have moved up to XCode 7.3 and have experienced spate of crashes when trying to initialize Realm and the error is:
Error Domain=io.realm Code=2 "Unsupported Realm file format version" UserInfo={NSFilePath=XXX, NSLocalizedDescription=Unsupported Realm file format version, Error Code=2}
It's happening when I'm trying to initialize RLMRealm with configuration as below:
let filesToMigrate = [
            RealmMigrator.pathForRealmWithName(Constants.Source.RealmDatabaseFileName.tools)
        ]

        for path in filesToMigrate {
            let config = RLMRealmConfiguration.defaultConfiguration()

            config.path = path
            config.schemaVersion = version

            config.migrationBlock = {(migration, oldSchemaVersion) -> Void in
                if oldSchemaVersion == 0 {
                    migration.enumerateObjects(Photo.className()) {
                        (oldObject: RLMObject?, newObject: RLMObject?) -> Void in
                        guard let newObject = newObject else { return }
                            newObject["sortId"] = 0
                    }
                }
            }
            RLMRealmConfiguration.setDefaultConfiguration(config)

            let _ = try! RLMRealm(configuration: config)
        }
    }

Realm version: 0.95.3
Xcode version: 7.3
Dependency manager + version: Cocoapods 0.39

Realm 0.95.3 is a very old version. We've seen some reports of bugs in older versions when compiled with newer versions of Xcode that are fixed in newer versions of Realm. I'd strongly suggest upgrading to our latest release, v2.0.

Please let us know if you continue to have problems after updating to a current version of Realm.