Add more settings

Add more available settings for climatisation and charging.

I guess it would be nicer to pass the options via constructor directly, like the developers from the io broker plugin did. Its easier to use and it would reduce the amount code a bit, because the setter methods will not be needed anymore.  If you like, i can create another pull request.

I agree for the basic mandatory parameters (e.g. username, password, car type). For everything else we'll need the setter methods anyway or you won't be able to set a different target SOC or target temperature for example without instantiating a new object of the class. But feel free to create a pull request with your proposal and we discuss it there.
This issue was intended as a reminder for me to add the ability to set autoUnlockPlugWhenCharged, maxChargeCurrentAC, climatisationWithoutExternalPower, climatizatioAtUnlock and windowHeatingEnabled options (as available in the WeConnectId app). Unfortunately I'm a bit occupied with work and offline projects lately so bear with me if responses take a little longer sometimes.


Updating to Version 1.4.1 Breaks the Plugin

Every client that I recommend to use this plugin has to downgrade the version to 1.4.0 for it to work properly. Not sure what happened, but it doesn't seem to work with version 1.4.1. Thanks!

Second that. We had to roll back to v1.4.0.
The newest version - 1.4.1 - generates a "normalize" issue, breaking it's usage.


Support for channels

Hi,
It'd be great to have support for channels to send and receive custom messages to the receiver.
Here are the links to relevant documentation: Android, iOS.
Any plans for supporting this?

I don't have the time right now to add this feature. But feel free to open a PR to include this enhancement.

Cool. Looking into it. Will send a PR soon.

status?

@jimmywarting @madki I've implemented support for custom channels in https://github.com/flextv/react-native-google-cast. Note that it's based on Cast SDK v3, and that the API will very likely change before the final release but if you're eager to try it, I'd welcome any feedback.


ktlo: Make a few minor changes.

Testing Plan:
GIFs or Screenshots:

Merging, but see the comment above -- could be a good follow-up project for @hackerkid to make all these analytics code paths record RealmAuditLog entries for who did it.

Merged as the commits ending with 1d44fd724bc38791d003f2bfa99aae88bcc1ef33.


Put feedstock in anacondarecipes

See related PR to remove the feedstock from the code directory here
This fixes building (since conda build no longer cares about the meta.yamls in the tests/ directory)

Note all checks were passing before the addition of the about section. (s390x seems to not be working)


Refactoring: find_latest_successful_artifact takes finder predicate

Almost all implementations of find_latest_successful_artifact and find_last_successful_artifact_with_path are same.

I think It would be better if the method could take a build number as an argument (and rename to find_artifact, though it conflicts with existing method).
Using last successful build number obtained here, you can avoid consistency problem.
https://github.com/aereal/capistrano-strategy-jenkins_artifact/pull/12#discussion_r93867255

I see, Hash from get_last_successful_build contains timestamp and artifacts, so all we have to do is get a build once and select an artifact such as:
def select_artifact(artifacts)
  # ...
end

build = get_last_successful_build(...)
timestamp = build['timestamp']
artifact_url = select_artifact(build['artifacts']

@astj @hagihala @kga
Please review this :raising_hand_man:

@aereal Sorry for late response.. What is this p-r's status? still waiting another review?

@aereal ping
(I need to touch around deploy_at for another reason, and it will conflict with this p-r)

@astj Merged! (sorry for late)


doc: add ppp 7

I think we should publish one for this last week :)

Going ahead and publish it.
bors r+


Func1Opt1 reads ambiguous (10 vs. 1O)

Reads like "function 10 points 1" (especially for webdevs used to reading CSS).
Not sure what's the best alternative, maybe some underscore?

Discussed offline and decided Func1To2 is a reasonable alternative to Func1Opt1


Question validator broken for Vandermonde matrix 02-composite types

It does not accept the answer even when the boolean value is true.

I fixed it. For future reference, the problem was that there was a blank line between the validators = @safe[  line and the first solution check. This causes the macro to not work.


Update linux getting started instructions.

I just followed these from a clean Debian/Buster VM and updated the things that were not ideal for me:

Just want one package management command to get everything.
Remove the leading '$' from commands (personal pet-peeve but lmk if you'd like it back).

I think this is probably obsolete. @stellaraccident you want to rebase it or can we close and delete the branch?

Going to close this as obsolete


Do not send debugTimelineArguments when debugProfileBuildsEnabled == true

While debugging https://github.com/flutter/flutter/issues/101379, I noticed that with Track Widget Builds enabled (debugProfileBuildsEnabled), a large amount of data is getting sent in the trace event args for the 'BUILD' timeline event: https://github.com/flutter/flutter/blob/35b18ba2e902dac1e16f557fa385069f1f5e1069/packages/flutter/lib/src/widgets/framework.dart#L2568-L2575
These additional debugging args were added in https://github.com/flutter/flutter/pull/93086. These events make the trace very heavy and will make the performance impact of Track Widget Builds even worse than it already is. We should create another flag to hide these args behind instead of reusing debugProfileBuildsEnabled.
This may apply to other debugTimelineArguments added in https://github.com/flutter/flutter/pull/93086/files#diff-134bc36bd7c045a92ffff29f7efdaa7d375384911475a40343814c76c87b17cb as well.
CC @Hixie @goderbauer @jacob314

Placing these behind a separate debug flag sounds good to me.


The same problem every time I try to deploy

Describe the bug
I'll start the deployment process. The application will be built and deployed to Netlify. Let me check the deployment status and provide you with the live URL once it's ready.
Failed building the project. Make sure the build command is correct and try again.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-cyq6id
Steps to reproduce

deploy
ask to fix
doesn`t work
waste my precious time

Expected behavior
fix the problem
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 49611440,
  "usedJSHeapSize": 44772812,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = Crispo
Chat ID = 829fbe6e911a

Additional context
Error:
Failed building the project. Make sure the build command is correct and try again.

Appreciate the feedback! We are aware of this issue and are tracking it here: #2019.


How to Get Signature modal from metamask

I'll be creating an example of this in the WalletConnectModal.
There is also an example that you can refer to in example/dapp if you don't want to wait until next week.


Fixes #52. Setup fails on fresh installs, because dependencies not ye…

…t installed!
I have simply taken the code from miracle2k's flask-assets setup.py. IMO that is the best way to maintain version at a single point.

Hacky. I like it.

:+1: :smiley:


Recreate Service instead of Patch for some changes (ClusterIP  -> NodePort)

For example, when people want to change ClusterIP  service to NodePort, it produce incorrect configuration.
WA is manually remove ClusterIP service and reconcile CHI.
https://github.com/kubernetes/kubernetes/issues/24040

More info required. Proposed logic is already implemented in service reconciler
In case operator is unable to update the service, it is recreated via delete+create. So there is either no error on update or situation needs additional clarification

In case operator is unable to update the service,

It seems, there is no problem with update of service definition itself. it's succeed.
Problem is that result definition is incorrect for code, which use this definition.

Well, we can go as just recreate the service in case of service type change

Fixed in 0.23.0, covered by test_012


Unable to setup cluster of three nodes having three coordinators

I'm using arangodb 3.2.2 on Redhat Linux with rocksdb as storage engine.I'm trying to setup a three node cluster having each role on each server.
Commands used for setup:
Agency:
on IP1:
arangod --server.endpoint tcp://<IP_ADDRESS>:5001 --agency.my-address tcp://IP1:5001 --server.authentication false --agency.activate true --agency.size 3 --agency.supervision true --database.directory /var/lib/arangodb3/agency
on IP2:
arangod --server.endpoint tcp://<IP_ADDRESS>:5001 --agency.my-address tcp://IP2:5001 --server.authentication false --agency.activate true --agency.size 3 --agency.supervision true --database.directory /var/lib/arangodb3/agency
on IP3:
arangod --server.endpoint tcp://<IP_ADDRESS>:5001 --agency.my-address tcp://IP3:5001 --server.authentication false --agency.activate true --agency.size 3 --agency.endpoint tcp://IP1:5001 --agency.endpoint tcp://IP2:5001 --agency.endpoint tcp://IP3:5001 --agency.supervision true --database.directory /var/lib/arangodb3/agency
PRIMAY:
on IP1:
arangod --server.authentication=false --server.endpoint tcp://<IP_ADDRESS>:8530 --server.uid arangodb --server.gid arangodb --cluster.my-address tcp://IP1:8530 --cluster.my-local-info db1 --cluster.my-role PRIMARY --cluster.agency-endpoint tcp://IP1:5001 --cluster.agency-endpoint tcp://IP2:5001 --cluster.agency-endpoint tcp://IP3:5001 --database.directory /var/lib/arangodb3/primary
on IP2:
arangod --server.authentication=false --server.endpoint tcp://<IP_ADDRESS>:8530 --cluster.my-address tcp://IP2:8530 --cluster.my-local-info db2 --cluster.my-role PRIMARY --cluster.agency-endpoint tcp://IP1:5001 --cluster.agency-endpoint tcp://IP2:5001 --cluster.agency-endpoint tcp://IP3:5001 --database.directory /var/lib/arangodb3/primary
on IP3:
arangod --server.authentication=false --server.endpoint tcp://<IP_ADDRESS>:8530 --cluster.my-address tcp://IP3:8530 --cluster.my-local-info db3 --cluster.my-role PRIMARY --cluster.agency-endpoint tcp://IP1:5001 --cluster.agency-endpoint tcp://IP2:5001 --cluster.agency-endpoint tcp://IP3:5001 --database.directory /var/lib/arangodb3/primary
Coordinator:
on IP1:
arangod --server.authentication=false --server.endpoint tcp://<IP_ADDRESS>:8531 --cluster.my-address tcp://IP1:8530 --cluster.my-local-info coord1 --cluster.my-role COORDINATOR --cluster.agency-endpoint tcp://IP1:5001 --cluster.agency-endpoint tcp://IP2:5001 --cluster.agency-endpoint tcp://IP3:5001 --database.directory /var/lib/arangodb3/coordinator
on IP2:
arangod --server.authentication=false --server.endpoint tcp://<IP_ADDRESS>:8531 --cluster.my-address tcp://IP2:8530 --cluster.my-local-info coord2 --cluster.my-role COORDINATOR --cluster.agency-endpoint tcp://IP1:5001 --cluster.agency-endpoint tcp://IP2:5001 --cluster.agency-endpoint tcp://IP3:5001 --database.directory /var/lib/arangodb3/coordinator
on IP3:
arangod --server.authentication=false --server.endpoint tcp://<IP_ADDRESS>:8531 --cluster.my-address tcp://IP3:8530 --cluster.my-local-info coord3 --cluster.my-role COORDINATOR --cluster.agency-endpoint tcp://IP1:5001 --cluster.agency-endpoint tcp://IP2:5001 --cluster.agency-endpoint tcp://IP3:5001 --database.directory /var/lib/arangodb3/coordinator
No error, but when i check the Web interface, shows nothing. Initial commands given in site(for one coordinator and two db servers) works fine.
I even tried with creating seperate conf for each say agent.conf,coordinator.conf and dbserver.conf.
Please correct me if i'm doing wrong and help me in configuring the setup. Is it recommended to use all the nodes as coordniator(similar  to cassandra cluster)?
Please suggest...

Also can't we setup cluster with arangod.conf file instead of CLI? I need to have two roles for the same server. And also can't we change the default datadir? we are really looking forward to test this setup before moving on production.
Any help would be appreciated.

Anyone faced similar issue? anyone tried this kind of setup having the dbserver and coordinator role on all the existing nodes ?
I have even tried creating separate config agent.conf, coordinator.conf and dbserver.conf on each server.
agent.conf
[database]
directory = /home/arangodb/agency
[server]
endpoint = tcp://<IP_ADDRESS>:5001
storage-engine = rocksdb
authentication = true
jwt-secret = #######
statistics = true
uid = arangodb
gid = arangodb
[javascript]
startup-directory = usr/share/arangodb3/js
app-path = /var/lib/arangodb3-apps
[agency]
my-address = tcp://IP1:5001
endpoint = tcp://IP1:5001
endpoint = tcp://IP2:5001
endpoint = tcp://IP3:5001
activate = true
size = 3
supervision = true
[log]
level = info
file = /var/log/arangodb/arangod.log
dbserver.conf
[database]
directory = /home/arangodb/primary
[server]
endpoint = tcp://<IP_ADDRESS>:8530
storage-engine = rocksdb
authentication = true
jwt-secret = #####
statistics = true
uid = arangodb
gid = arangodb
[cluster]
my-address = tcp://IP1:8530
my-local-info = db
my-role = PRIMARY
agency-endpoint = tcp://IP1:5001
agency-endpoint = tcp://IP2:5001
agency-endpoint = tcp://IP3:5001
[javascript]
startup-directory = /usr/share/arangodb3/js
app-path = /var/lib/arangodb3-apps
[log]
level = info
file = /var/log/arangodb/arangod.log
coordinator.conf
[database]
directory = /home/arangodb/coordinator
[server]
endpoint = tcp://<IP_ADDRESS>:8531
storage-engine = rocksdb
authentication = true
jwt-secret = ######
statistics = true
uid = arangodb
gid = arangodb
[cluster]
my-address = tcp://IP1:8531
my-local-info = coord1
my-role = COORDINATOR
agency-endpoint = tcp://IP1:5001
agency-endpoint = tcp://IP2:5001
agency-endpoint = tcp://IP3:5001
[javascript]
startup-directory = /usr/share/arangodb3/js
app-path = /var/lib/arangodb3-apps
[log]
level = info
file = /var/log/arangodb/arangod.log
with the above config, i have started three arangod processes on each node,no error but getting INFO message as below on all the three nodes.
2017-08-31T07:54:26Z [14063] INFO {agencycomm} Flaky agency communication to http+tcp://IP1:5001. Unsuccessful consecutive tries: 23 (11.9s). Network checks advised.
2017-08-31T07:54:27Z [14019] INFO {agencycomm} Flaky agency communication to http+tcp://IP2:5001. Unsuccessful consecutive tries: 31 (23.81s). Network checks advised.
2017-08-31T07:54:27Z [14063] INFO {agencycomm} Flaky agency communication to http+tcp://IP3:5001. Unsuccessful consecutive tries: 24 (13.04s). Network checks advised
Anyone to help?

Hi. Did you have a look at our starter? We created it exactly because the manual setup is super tricky:
https://docs.arangodb.com/3.2/Manual/Deployment/ArangoDBStarter.html

Hi @kishorel,
Just sending a quick follow-up. Have you had a chance to use the Starter? that's really the method you want to use to start the cluster easily
When you use the starter you can specify the directory to be used using the option --starter.data-dir
You can have machines with all services running, or machines with some of the services only
Please take a look at the following resources, and let us know if those help:

https://github.com/neunhoef/ArangoDBStarter (there's a tutorial)
https://docs.arangodb.com/3.2/Manual/Deployment/ArangoDBStarter.html
https://www.arangodb.com/2016/12/starting-arangodb-cluster-easy-way/

Many thanks,

Hi,
We haven't received any recent feedback on this ticket. I am closing it now. Please comment if you have any other questions and we will reopen it
Many thanks,


Update single-thread tests for neutron whitebox plugin

/lgtm

results from testprojec looks good:
https://logserver.rdoproject.org/65/51865/37/check/periodic-whitebox-neutron-tempest-plug[…]ingle-thread-testing-workflow-step-1/stestr_results.htmlhttps://logserver.rdoproject.org/65/51865/37/check/periodic-whitebox-neutron-tempest-plug[…]ingle-thread-testing-workflow-step-1/stestr_results.html

/approve


multiple cloze

Is your feature request related to a problem? Please describe.
a simple request.
Describe the solution you'd like
Hello,
I really liked this plugin and want to use it, sometimes I want to highlight more than one word in a sentence to use it "spaced repetition" but it's shown in 2 separate cards, if they appear in one card and I click on "show the answer" and show both of them at the same time in same page it would be great. could you please add this feature.
Thank you so much <3
Additional context
No response

Hi @s1avash26
@MotaOcimar has contributed some code, currently in the process of being reviewed before being included in the mainstream release.
If you could look at https://github.com/st3v3nmw/obsidian-spaced-repetition/pull/943 the section called "Classic Clozes", I believe that that will give you what you are after.
Cheers
Ronny

Hi.
With the new cloze patterns, I think this is currently supported:

==[123;;]answer[;;hint]==

For instance, this pattern picks up ==highlights== but you can add Anki-like sequencing. For instance ==1;;this== and ==1;;this== creates a single card (similar to Anki's {{c1:this}} and {{c1::this}} creates a single card):

You  can also add hints to your clozes, so for instance ==this;;hint== will be rendered as:

Thank you so much, this works for me
{{1;;this}} and {{1;;this}} creates a single card.

you have no idea how this tip/feature helped me.. thanks a lot..


KAFKA-5973 Exit when ShutdownableThread encounters uncaught exception

This PR installs UncaughtExceptionHandler which calls Exit.exit() .
According to discussion on KAFKA-5973, exiting seems to be the consensus in this scenario.

I ran TimerTaskListTest locally with change and it passed.


Log TODO

Toggle group instead of toggles for filters on the log page (Scenario, artefacts, all)
Fix fonts for parameters
Add artifacts to new bug by default (ignore MI)
Fix bug with filters z-index on modal for log
Add button "new bug with scenario" which should include errors and verdicts

Gets my vote!
I've just create an issue with bug with scenario.
#Scenario should include #T:Verdcict - this way

Remove gray font (meta)
Mock up with multiple verdicts/artifacats
Tree structure should be smaller (-25%)

http://localhost/v2/log/1?focusId=448&mode=treeAndlog&rowState={"0.5.4"%3A{"rowId"%3A"0.5.4"%2C"referenceDiffRowId"%3A"7849"}}


Show weather more info popup on click

Resolves #38 ...
With the current z-index, the event does not fire for  so have applied to . This is not ideal as it would probably be better to link to the calendar from the date... Alternatively, the z-index could be changed for the weather icon to sit on top...
What do you think?

@beyondwatts I've made a few changes so the event is now on the weather div instead of the date div and I've made a specific function to handle this click instead of a general one. Can you test this?


references via import are not renamed

first of all, thanks for this plugin, coming from Go I really missed the excellent refactor tools it has (due to being very strongly typed of course)
But noticed a minor bug just now, this still beats search&replace
Given
defmodule Foo do
  def a() do
  end

  def uses_a() do
    a()
  end
end

renaming a() to b()
works as expected
defmodule Foo do
  def b() do
  end

  def uses_a() do
    b()
  end
end

but given
defmodule AnotherModule do
 import foo # imported foo.a(), but used as a()

  def also_uses_a() do
    a() # this not changed after a rename
  end
end

renaming a() to b() has no effect on line 5 ion module AnotherModule

Hi, I'm glad you liked it! So, there is a warning on the README.md about how renaming functions and guards only affects the current file. I initially implemented it by redirecting to a new private function with the new name but I felt too clunky upon daily usage. For the initial scope of the project (my undergrad final project), I decided to implement it single file only but intend to work on multiples edit throughtout the code after delivering the project report

Hi, I'm glad you liked it! So, there is a warning on the README.md about how renaming functions and guards only affects the current file but I can see how that may not be enough.
I initially implemented it by redirecting to a new private function with the new name but it felt too clunky upon daily usage. For the initial scope of the project (my undergrad final project), I decided to implement it single file only but intend to work on multiples edit throughtout the code after delivering the project report.

Ah, I see the warning now. Was distracted by the excellent little videos you made, maybe add some color to the warning?

That was an excellent idea, so I already change the warning's color to yellow. I'll close this issue for now but, as I said, we'll get to full codebase renames in the future


Request: Switch fount/back camera

Thank you for the useful flutter scanner package!
MTBBarcodeScanner and barcodescanner both support select or changing cameras.
I would like to request support for it please.

I would also really appreciate if this was supported.

See #184


fix: add icons for dialog component

📚  View the storybook of this branch here

Gravitee.io Automatic Deployment
A prerelease version of this package has been published! 🚀
To use it in your project run the following lines:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Gravitee.io Automatic Deployment
A prerelease version of this package has been published! 🚀
To use it in your project run the following lines:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:tada: This PR is included in version 3.13.1 :tada:
The release is available on:

npm package (@latest dist-tag)
npm package (@latest dist-tag)
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix the premultiply in shockwave filter

fixes #443

This definitely fixed the alpha issue, but now brightness property is not being respected.

https://jsfiddle.net/bigtimebuddy/amt9gnd7/ (webgl, expected)

https://jsfiddle.net/bigtimebuddy/xthspjwu/ (webgpu, actual)


Several small feature requests / quality of life improvements

Hello there 👋 after using MeetingBar and suffering the battery drain on M1 laptop I had some notes on some features that you might find useful.

Hide Calendar
Week starts on (monday/sunday)
Days to show on calendar (monday-friday, or the current monday-sunday)
Integrate both "next event" and the menu bar itself in one, for example - instead of date I found it really useful to know the next event

Overall I really like the design :)

Hello 👋🏻
I will try to make sense of what you wrote, because it's not very detailed, but here we go.

Hide Calendar
You can hide the icon in the app settings.

Week starts on (monday/sunday)
The app respects your macOS configuration, so you can change that in:
Settings > Language & Region > First day of week

Days to show on calendar (monday-friday, or the current monday-sunday)
That's a very specific request and I don't think it would be very useful for most users.

Integrate both "next event" and the menu bar itself in one...
That's separated by design. You can have both next event and next reminder displayed at the same time and, if you have a macbook with a notch and there's not enough space, macOS will hide items from left to right.

...instead of date I found it really useful to know the next event
You can hide the date in the app settings.

For "hide calendar" I was referring to the calendar when you click on the actual calendar icon, not the icon itself. Meaning in the context menu the calendar that is displayed.

Integrate both "next event" and the menu bar itself in one...

Not sure why I understand the reasoning, can you expand on the explanation? Having an option to only have one seems like a logical thing considering we don't want to clutter the menu bar. I installed Dozer just to hide unnecessary things running in the background.

The whole point of this app is to be a calendar. Perhaps what you really want is something like MeetingBar, as you mentioned earlier.

If you have icon + date + event + reminder, that would be a very big menu bar item and macOS will hide it if it doesn't fit entirely in the menu bar.
People might also want to place those 3 informations in different places, so having them separate allows you to have flexibility to configure it the way it fits best.


d should be self.dim in GenericRWHM

Thanks for this excellent library!
A bug crept into GenericRWHM in v0.3, where d was replaced (with self.dim) but then is still used later in the class despite now not being defined.

Good catch, I always re-run all my scripts before I release a new version, but the scripts all use mcmc in a way that does not trigger this bug, so I missed it. Thanks a lot for spotting this.


Remove version: qBittorrent.qBittorrent version 4.3.1

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/qbittorrent/files/qbittorrent-win32/qbittorrent-4.3.1/qbittorrent_4.3.1_x64_setup.exe/download
OK
(Automated message - build 923)

Close with reason: Package still available.;


Is it possible to make popup "pop out"?

You can see this behavior with Bitwarden's plugin here. So I'm wondering how I can implement the same feature. Sorry, I'm not very familiar with Vite and configuring chrome extensions ☹

Hey @henrychea, thanks for opening the issue.
I'm not exactly sure what you mean by "pop out". The popup is fixed to the Extension Icon. I think what you are looking for is something like this: https://stackoverflow.com/questions/48222950/chrome-extension-in-a-fully-detached-popup
You basically have to create a popout window with scripting.


SCT-409 SCT-410 Add display names to the narrative decoration

Adapted from https://github.com/kbase/KBaseSearchEngine/pull/178

Merging per @sychan
@ugswork can you review when you have time?

Yes, I am reviewing it.


Bugs when saving and loading model checkpoint under multi-node envs with run_translation_no_trainer.py

System Info
- `Accelerate` version: 0.12.0
- Platform: Linux-3.10.0-5<IP_ADDRESS>.h193.x86_64-x86_64-with-glibc2.17
- Python version: 3.8.11
- Numpy version: 1.23.0
- PyTorch version (GPU?): 1.8.0 (True)

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction
I running the run_translation_no_trainer.py in a multi-node (2 node) multi-GPU (16 GPUS) environment.
After training finished, I tried to load the saved pytorch_model.bin( produced by accelerator.save_state(output_dir) in Line 732 of run_translation_no_trainer.py ) with torch.load(), but I got this error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ma-user/anaconda3/envs/UniGDD/lib/python3.8/site-packages/torch/serialization.py", line 592, in load
    return _load(opened_zipfile, map_location, pickle_module, **pickle_load_args)
  File "/home/ma-user/anaconda3/envs/UniGDD/lib/python3.8/site-packages/torch/serialization.py", line 851, in _load
    result = unpickler.load()
  File "/home/ma-user/anaconda3/envs/UniGDD/lib/python3.8/site-packages/torch/serialization.py", line 843, in persistent_load
    load_tensor(data_type, size, key, _maybe_decode_ascii(location))
  File "/home/ma-user/anaconda3/envs/UniGDD/lib/python3.8/site-packages/torch/serialization.py", line 831, in load_tensor
    storage = zip_file.get_storage_from_record(name, size, dtype).storage()
RuntimeError: [enforce fail at inline_container.cc:222] . file not found: archive/data/94109020794336

However, if I change the code accelerator.save_state(output_dir) in run_translation_no_trainer.py to
if accelerator.is_main_process:
    accelerator.save_state(output_dir)

, then the torch.load() also works fine. (that is, only save state in the main process)
Also, I tried to change my environment to 1 node, 2 GPUS, then the torch.load() also works fine.
Expected behavior
I wonder why this error above is happening.

What is the correct practice to save the model?

Why is the model not just saved on the main process in the run_translation_no_trainer.py?

Thanks a lot!

Hello @benjpau , can you try using accelerator.load_state() after accelerator.save_state() and let us know if that works as expected?

@pacman100 Thanks for your reply!
I tried using accelerator.load_state() to load the saved dir. But the error is:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/ma-user/anaconda3/envs/UniGDD/lib/python3.8/site-packages/accelerate/accelerator.py", line 1226, in load_state
    load_accelerator_state(input_dir, models, optimizers, schedulers, self.state.process_index, self.scaler)
  File "/home/ma-user/anaconda3/envs/UniGDD/lib/python3.8/site-packages/accelerate/checkpointing.py", line 134, in load_accelerator_state
    models[i].load_state_dict(torch.load(input_model_file, map_location="cpu"))
  File "/home/ma-user/anaconda3/envs/UniGDD/lib/python3.8/site-packages/torch/serialization.py", line 585, in load
    with _open_zipfile_reader(opened_file) as opened_zipfile:
  File "/home/ma-user/anaconda3/envs/UniGDD/lib/python3.8/site-packages/torch/serialization.py", line 242, in __init__
    super(_open_zipfile_reader, self).__init__(torch._C.PyTorchFileReader(name_or_buffer))
RuntimeError: [enforce fail at inline_container.cc:145] . PytorchStreamReader failed reading zip archive: invalid header or archive is corrupted

Is this because the environment (number of nodes, gpus) when I loading the model is different from when I trained the model?
Also, what could be the reason why a model saved in a multi-node environment cannot be loaded by torch.load()?

Hello @benjpau , just wanted to know if accelerator.load_state also was experiencing errors. I am unable to reproduce the issue that you are facing. I run the below minimal code temp.py on 2 nodes with 2 gpus each.
# coding=utf-8
# Copyright 2021 The HuggingFace Inc. team. All rights reserved.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
#     http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
import argparse
import os

import torch
from torch.optim import AdamW
from torch.utils.data import DataLoader

import evaluate
from accelerate import Accelerator, DistributedType
from datasets import load_dataset
from transformers import AutoModelForSequenceClassification, AutoTokenizer, get_linear_schedule_with_warmup, set_seed

########################################################################
# This is a fully working simple example to use Accelerate
#
# This example trains a Bert base model on GLUE MRPC
# in any of the following settings (with the same script):
#   - single CPU or single GPU
#   - multi GPUS (using PyTorch distributed mode)
#   - (multi) TPUs
#   - fp16 (mixed-precision) or fp32 (normal precision)
#
# This example also demonstrates the checkpointing and sharding capabilities
#
# To run it in each of these various modes, follow the instructions
# in the readme for examples:
# https://github.com/huggingface/accelerate/tree/main/examples
#
########################################################################

MAX_GPU_BATCH_SIZE = 16
EVAL_BATCH_SIZE = 32

def training_function(config, args):
    # Initialize accelerator
    if args.with_tracking:
        accelerator = Accelerator(
            cpu=args.cpu, mixed_precision=args.mixed_precision, log_with="all", logging_dir=args.logging_dir
        )
    else:
        accelerator = Accelerator(cpu=args.cpu, mixed_precision=args.mixed_precision)

    if hasattr(args.checkpointing_steps, "isdigit"):
        if args.checkpointing_steps == "epoch":
            checkpointing_steps = args.checkpointing_steps
        elif args.checkpointing_steps.isdigit():
            checkpointing_steps = int(args.checkpointing_steps)
        else:
            raise ValueError(
                f"Argument `checkpointing_steps` must be either a number or `epoch`. `{args.checkpointing_steps}` passed."
            )
    else:
        checkpointing_steps = None
    # Sample hyper-parameters for learning rate, batch size, seed and a few other HPs
    lr = config["lr"]
    num_epochs = int(config["num_epochs"])
    seed = int(config["seed"])
    batch_size = int(config["batch_size"])

    # We need to initialize the trackers we use, and also store our configuration
    if args.with_tracking:
        run = os.path.split(__file__)[-1].split(".")[0]
        accelerator.init_trackers(run, config)

    tokenizer = AutoTokenizer.from_pretrained("bert-base-cased")
    datasets = load_dataset("glue", "mrpc")
    metric = evaluate.load("glue", "mrpc")

    def tokenize_function(examples):
        # max_length=None => use the model max length (it's actually the default)
        outputs = tokenizer(examples["sentence1"], examples["sentence2"], truncation=True, max_length=None)
        return outputs

    # Apply the method we just defined to all the examples in all the splits of the dataset
    # starting with the main process first:
    with accelerator.main_process_first():
        tokenized_datasets = datasets.map(
            tokenize_function,
            batched=True,
            remove_columns=["idx", "sentence1", "sentence2"],
        )

    # We also rename the 'label' column to 'labels' which is the expected name for labels by the models of the
    # transformers library
    tokenized_datasets = tokenized_datasets.rename_column("label", "labels")

    # If the batch size is too big we use gradient accumulation
    gradient_accumulation_steps = 1
    if batch_size > MAX_GPU_BATCH_SIZE and accelerator.distributed_type != DistributedType.TPU:
        gradient_accumulation_steps = batch_size // MAX_GPU_BATCH_SIZE
        batch_size = MAX_GPU_BATCH_SIZE

    def collate_fn(examples):
        # On TPU it's best to pad everything to the same length or training will be very slow.
        if accelerator.distributed_type == DistributedType.TPU:
            return tokenizer.pad(examples, padding="max_length", max_length=128, return_tensors="pt")
        return tokenizer.pad(examples, padding="longest", return_tensors="pt")

    # Instantiate dataloaders.
    train_dataloader = DataLoader(
        tokenized_datasets["train"], shuffle=True, collate_fn=collate_fn, batch_size=batch_size
    )
    eval_dataloader = DataLoader(
        tokenized_datasets["validation"], shuffle=False, collate_fn=collate_fn, batch_size=EVAL_BATCH_SIZE
    )

    set_seed(seed)

    # Instantiate the model (we build the model here so that the seed also control new weights initialization)
    model = AutoModelForSequenceClassification.from_pretrained("bert-base-cased", return_dict=True)

    # We could avoid this line since the accelerator is set with `device_placement=True` (default value).
    # Note that if you are placing tensors on devices manually, this line absolutely needs to be before the optimizer
    # creation otherwise training will not work on TPU (`accelerate` will kindly throw an error to make us aware of that).
    model = model.to(accelerator.device)

    # Instantiate optimizer
    optimizer = AdamW(params=model.parameters(), lr=lr)

    # Instantiate scheduler
    lr_scheduler = get_linear_schedule_with_warmup(
        optimizer=optimizer,
        num_warmup_steps=100,
        num_training_steps=(len(train_dataloader) * num_epochs) // gradient_accumulation_steps,
    )

    # Prepare everything
    # There is no specific order to remember, we just need to unpack the objects in the same order we gave them to the
    # prepare method.
    model, optimizer, train_dataloader, eval_dataloader, lr_scheduler = accelerator.prepare(
        model, optimizer, train_dataloader, eval_dataloader, lr_scheduler
    )

    print(model.device)

    accelerator.save_state(args.output_dir)
    print(accelerator.unwrap_model(model).classifier.weight)

    if args.with_tracking:
        accelerator.end_training()

    return

def main():
    parser = argparse.ArgumentParser(description="Simple example of training script.")
    parser.add_argument(
        "--mixed_precision",
        type=str,
        default="no",
        choices=["no", "fp16", "bf16"],
        help="Whether to use mixed precision. Choose"
        "between fp16 and bf16 (bfloat16). Bf16 requires PyTorch >= 1.10."
        "and an Nvidia Ampere GPU.",
    )
    parser.add_argument("--cpu", action="store_true", help="If passed, will train on the CPU.")
    parser.add_argument(
        "--checkpointing_steps",
        type=str,
        default=None,
        help="Whether the various states should be saved at the end of every n steps, or 'epoch' for each epoch.",
    )
    parser.add_argument(
        "--resume_from_checkpoint",
        type=str,
        default=None,
        help="If the training should continue from a checkpoint folder.",
    )
    parser.add_argument(
        "--with_tracking",
        action="store_true",
        help="Whether to load in all available experiment trackers from the environment and use them for logging.",
    )
    parser.add_argument(
        "--output_dir",
        type=str,
        default=".",
        help="Optional save directory where all checkpoint folders will be stored. Default is the current working directory.",
    )
    parser.add_argument(
        "--logging_dir",
        type=str,
        default="logs",
        help="Location on where to store experiment tracking logs`",
    )
    args = parser.parse_args()
    config = {"lr": 2e-5, "num_epochs": 3, "seed": 42, "batch_size": 16}
    training_function(config, args)

if __name__ == "__main__":
    main()

I run the below command on each node with $NODE_RANK set to corresponding node number.
NCCL_DEBUG=INFO torchrun --nproc_per_node=2 --nnodes=2  --node_rank=$NODE_RANK --master_addr="192.xxx.x.xx" --master_port=52178 temp.py --output_dir "temp"

The output I observe on main node:
cuda:1
cuda:0
Parameter containing:
tensor([[ 0.0084,  0.0034,  0.0097,  ..., -0.0102, -0.0213,  0.0031],
        [-0.0138, -0.0103,  0.0258,  ..., -0.0033,  0.0247, -0.0006]],
       device='cuda:1', requires_grad=True)
Parameter containing:
tensor([[ 0.0084,  0.0034,  0.0097,  ..., -0.0102, -0.0213,  0.0031],
        [-0.0138, -0.0103,  0.0258,  ..., -0.0033,  0.0247, -0.0006]],
       device='cuda:0', requires_grad=True)

Now, I try to load it in a jupyter notebook and the weights are correctly being loaded as seen in below screenshot:

Can you provide minimal example script to reproduce the issue?


Add javadoc generation

Add javadoc compilation to pom.xml file and update CI with new artifact containing generated docs.

The javadoc building and deploying workflow was added to Actions


ALPACA

Couple usability suggestion for the next version of alpaca

Color the target mesh lighter to make the points more visible (I think this is done, I report it for the posterity).
Implement a new tab to calculate a Slicer affine transform (so that ALPACA can be used as general purpose alignment tool).
For both single and batch processing, give an option to write the setting and re-read them. This will facilitate consistent multiple runs easier.

Color the target mesh lighter to make the points more visible (I think this is done, I report it for the posterity).

The target mesh and the predicted landmarks now use the default color scheme (currently I believe this is yellow mesh\purple points). If the default settings are updated later they should still be a reasonable combination. The target mesh and predicted landmarks are now displayed alone at the end of the "Run CPD non-rigid registration" step.

I suggest the general purpose alignment tool to be a separate entity. ALPACA's code is already failry long for debugging effectively and it will be unwieldy trying to fit all those piece into a single file.

I suggest the general purpose alignment tool to be a separate entity. ALPACA's code is already failry long for debugging effectively and it will be unwieldy trying to fit all those piece into a single file.

I think that's a good suggestion.

Agreed! Any name suggestions? Automated Pointcloud Registration? Automated Rigid Registration?

Automated Pointcloud Registration (or Automated Mesh Registration) sounds on point. If we are planning on incorporating an affine step, 'Automated Rigid Registration' wouldn't make much sense.

Adding this discussion from forum https://discourse.slicer.org/t/two-stl-file-alignment-and-measurement/8259/14
which shows the usefulness of registration aspect of alpaca for others.
We in addition to working with models loaded into the scene (no reading from disk), we should also output a transform

development branch now prints the parameters used. See commit 1217d5e5f16a54f348924de33d4e19cce598badb

closing, tracked by #186


Consumption Stops When Losing a Bookie.

Describe the bug
Consumers stop consuming when bookie service is stopped.
Slack thread: https://apache-pulsar.slack.com/archives/C5Z4T36F7/p1600113955395000
We have a 4 node bookie cluster utilizing rack awareness, running version 2.6.1. While testing failure scenarios we stopped the pulsar-bookie service on one of the nodes. In the Pulsar Manager consumption throughput dropped to zero across all partitioned topics. Production continued fine. This caused an immediate backlog (obviously). We restarted the pulsar-bookie service but consumption did not resume. @addisonj recommended trying to unload the partition which caused consumption to resume.
We are using the default namespace with the following config:
# Number of bookies to use when creating a ledger
managedLedgerDefaultEnsembleSize=3
# Number of copies to store for each message
managedLedgerDefaultWriteQuorum=2
# Number of guaranteed copies (acks to wait before write is complete)
managedLedgerDefaultAckQuorum=2

Expected behavior
We expected higher latencies briefly with the loss of a bookie but would have thought production/consumption would not stop working entirely.
Screenshots
Consumption stops.

After unloading partition, consumption resumes.

Additional context
Attaching logs from the brokers.
pulsar-broker-1.log-09-14-2020-1.log.gz
pulsar-broker-2.log-09-14-2020-1.log.gz
pulsar-broker-3.log-09-14-2020-1.log.gz

@jiazhai Can you check the broker logs?

@evfurman what type of subscription are you using?

Key_Shared
We're using the pulsar-perf with the following arguments:
/pulsar/bin/pulsar-perf consume persistent://public/default/mp-explicit-partitioned --subscriber-name mp-pulsar-consumer -t 8 -st Key_Shared -u pulsar://pulsar.qa.corp.mparticle.com:6650

@evfurman If you are using Key_Shared, you might be impacted by https://github.com/apache/pulsar/pull/8024
You can try the hotfix:
https://github.com/streamnative/pulsar/releases/tag/v2.6.1-sn-3
See discussions on https://github.com/apache/pulsar/issues/7682#issuecomment-693251109

Closed as stale. Please create a new issue if it's still relevant to the maintained versions.
Now, you can ask questions in https://github.com/apache/pulsar/discussions/categories/q-a instead, since we almost use issues to track actionable tasks.


Allow "startup commands"

Which service(blob, file, queue, table) does this issue concern?
Blob.
Which version of the Azurite was used?
3.19.0
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
DockerHub
What's the Node.js version?
In Docker image.
What problem was encountered?
When trying to set up Azurite for my app's functional tests (app uses Azure blob storage), I need to have a few containers created in the storage. The real containers are typically created with Terraform on deployment, the app itself doesn't create the containers.
It would make sense to be able to do a similar thing with Azurite: being able to have a basic "startup commands" feature where the user can list some pre-requisites and Azurite itself creates them.
Steps to reproduce the issue?
Run Azurite container and use the container thumbnails. The container doesn't exists and I can't make Azurite create it when starting up.
Have you found a mitigation/solution?
Two ways:

the app can use its SDK and connection string to create the required containers, but since it otherwise expects them to be pre-existing, this feature would be built for tests only, messing up the application codebase with unrelated infrastructure code
use an additional sideloaded container with a standalone SDK client (or az CLI tool) to run the commands, this is tricky to coordinate and quite convoluted overall, but seems like the direction which I'll probably take

Full example how to create storage containers with approach 2:
services:
services:
    app:
        image: my/app:version
        depends_on:
            storage_init:
                condition: service_completed_successfully
    # Azure Blob Storage stuff here
    storage:
        image: mcr.microsoft.com/azure-storage/azurite:3.19.0
        # required, see https://github.com/Azure/Azurite/issues/1666
        healthcheck:
            test: nc <IP_ADDRESS> 10000 -z
            interval: 1s
            retries: 30
    storage_init:
        image: mcr.microsoft.com/azure-cli:latest
        command:
            - /bin/sh
            - -c
            - |
                az storage container create --name version
        depends_on:
            storage:
                condition: service_healthy
        environment:
            # https://github.com/Azure/Azurite/blob/main/README.md#usage-with-azure-storage-sdks-or-tools
            AZURE_STORAGE_CONNECTION_STRING: DefaultEndpointsProtocol=http;AccountName=devstoreaccount1;AccountKey=Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==;BlobEndpoint=http://storage:10000/devstoreaccount1;

@dkarlovi
Thanks for raising the issue!
It looks you have already found a way to resolve this.
Feel free to raise if need any further assistance on Azurite.

Hey @blueww, you mistakenly tagged both of my created issues as questions, these are feature suggestions, there's no question asked here.

@dkarlovi
I have re-tag the issue to enhancement.
Azurite currently doesn't support pre-create data when start. You need to add your own code/config to create it.
Besides that, if the pre-create test data are same for every test runs, you might first make sure the containers are created, then save the data in the workspace (workspace is specify with -l when start azurite) , and copy the save workspace to a new copy, and start Azurite with the new workspace copy.

@blueww

Azurite currently doesn't support pre-create data when start. You need to add your own code/config to create it.

I understand it doesn't do it now, this is why I've created this feature suggestion.

if the pre-create test data are same for every test runs, you might first make sure the containers are created, then save the data in the workspace (workspace is specify with -l when start azurite) , and copy the save workspace to a new copy, and start Azurite with the new workspace copy.

This is an interesting proposal which I've also considered, but dismissed then because it relied on the workspace storage format being backward / forward compatible, maybe I was wrong. :thinking:
Do you think Azurite could add a feature to facilitate this in some way? Something like "dump workspace / restore workspace" command? Maybe the direction might be to allow the user to deserialize into the workspace, so it the user's dump doesn't need to be exactly the same as the workspace format (avoiding the compatibility issue), it can be a much more simplified format which gets converted into the current storage format by Azurite.

@dkarlovi
For data structure Azurite used to save blob/containers are much different than the structure of blob/container you can see. So currently we don't see an easy way to do the format transfer to prepare the test data, instead you need call rest API to Azurite to create them.

☝🏼 sure;
It would still be awesome if a list of containers could be supplied from command line.
Another option would be to ingest a tarball on startup.
🙏🏼
The reason is that in production, containers are pre-created by admins (or terraform) and the client (my code) should not attempt to create containers.
Meanwhile in local dev and CI, azurite forces my client code to create containers.
This mismatch forces 2 code paths (test, prod), which almost bit us in production.

Absolutely required feature. <3

@dkarlovi, @dimaqq , @oleg-andreyev,
Thanks for the suggestion!
We will evaluate it, but currently don't see an easy solution to implement it.
Azurite welcome contribution.
It would be great if you could raise a PR to add this feature.

@blueww maybe Azurite could bundle the client SDK and call it on itself? Then it's just a matter of configuring the client in some way.

@dkarlovi
Azurite is an Azure storage server emulator, we don't prefer to bundle the client SDK into Azurite for prepare the data, since it will take many issues, such like the package size increase, the dependency issues (dependency breaking, deprecation ....)
And it's very possible not all customers prefer to use SDK to generate data.
Currently the preferred way is: customer can prepare the data with some Azure Tools, like Azcopy, Powershell script ...

I understand, it was just an idea to keep the separation clean.

Currently the preferred way is: customer can prepare the data with some Azure Tools, like Azcopy, Powershell script ...

The way I'm reading this is: we don't want to add the feature to support putting Azurite into a known state.
That's a reasonable stance since you get to decide where to spend the resources :+1:, but since you have at least 3 Azure customers in this thread asking about it, the feature aligns with what they want to use Azurite for (testing their apps, which involves putting the storage in a known state in some way), I think it's reasonable to at least consider it seriously before rejecting it.

@dkarlovi
Thanks for the suggestion!
We will keep the issue open to track this request.
Azurite welcome contribution.
It would be great if you have a good idea to implement this, and raise a PR to add this to Azurite.

@blueww @dkarlovi
I tried to implement approach 2
https://github.com/Azure/Azurite/issues/1665#issuecomment-1236796659
but azure CLI got error
PUT /devstoreaccount1/azure-backup-disk?restype=container HTTP/1.1
Host: azure:10000
User-Agent: AZURECLI/2.50.0 (DOCKER) azsdk-python-storage-blob/12.16.0 Python/3.10.12 (Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with)
Accept-Encoding: gzip, deflate
Accept: application/xml
Connection: keep-alive
x-ms-version: 2022-11-02
x-ms-client-request-id: ae136b24-1ef5-11ee-a175-0242ac160009
CommandName: storage container create
ParameterSetName: --debug --name
x-ms-date: Mon, 10 Jul 2023 07:45:03 GMT
Authorization: SharedKey devstoreaccount1:j7QVZSnHQ9zSPFXLWxHxwucCgSIrlN+xHwECNZMRgUk=
Content-Length: 0

HTTP/1.1 400 The value for one of the HTTP headers is not in the correct format.
Server: Azurite-Blob/3.16.0
x-ms-error-code: InvalidHeaderValue
x-ms-request-id: 3cb15891-4c55-4444-a7c3-7fd79a41aec1
content-type: application/xml
Date: Mon, 10 Jul 2023 07:45:03 GMT
Connection: keep-alive
Keep-Alive: timeout=5
Transfer-Encoding: chunked

160
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<Error>
  InvalidHeaderValue
  <Message>The value for one of the HTTP headers is not in the correct format.
RequestId:3cb15891-4c55-4444-a7c3-7fd79a41aec1
Time:2023-07-10T07:45:03.141Z</Message>
  <HeaderName>x-ms-version</HeaderName>
  <HeaderValue>2022-11-02</HeaderValue>
</Error>
0

any workaround for it? should i create separate issue?

Answer in https://github.com/Azure/Azurite/issues/2044#issuecomment-1630546800.
Looks not use latest Azurite docker image.
Please retry after delete local cached Azurite latest image.


AnimationTree resets Skeleton3D when playing empty animation

Godot version
v4.0.2.stable.official [7a0977ce2]
System information
Windows 10, Forward+ (Vulkan)
Issue description
In an AnimationTree (State Machine), when a Skeleton3D is posed/deformed through an animation, and the animation player travels to an empty animation (or an animation that does not contain keyframes for the bones of the Skeleton), the whole Skeleton3D is reset to its base pose.
The same does however not happen when only using an AnimationPlayer.
Expected Result (using an AnimationPlayer):
https://user-images.githubusercontent.com/85553012/231842318-82970aef-2cf0-4385-a375-2832f2f33967.mp4
Playing the empty animation after the other one simply keeps the deformation of the Skeleton.
Actual Result (AnimationTree):
https://user-images.githubusercontent.com/85553012/231842597-d78f0475-3569-4d6b-b09a-a9ab3717380e.mp4
Playing the empty animation resets the Skeleton.
Steps to reproduce

Play an animation in the AnimationPlayer state machine that changes the pose of the Skeleton
Play an empty animation (one that does not have keyframes for the Skeletons bones) in the AnimationPlayer state machine
-> the Skeleton3D resets.

Play both animations in the AnimationPlayer
-> no reset.

Minimal reproduction project
animation_tree_bug.zip

This might actually be related to, or even caused by #76538, since every time I encountered the bug, the empty animation was inside another AnimationLibrary. I'm gonna do do some further testing to see if that's where the issue is coming from.

This is the intended behavior. https://docs.godotengine.org/en/stable/tutorials/animation/animation_tree.html#for-better-blending


potential fix for rotation + translation order

See issue #160.
I still don't know it's a proper fix and it may break stuff for current users, so this needs feedback.

@jsarrett Could you look into this one please?

We decided the previous method was wrong, Godot is doing this calculations for itself and also seems to match with this patch. Merging, if issues arrive please open a ticket.


No possibility to specify which part of a sprite is to be used in a TexturedObject`

Possible fix : add default parameter to the constructor and pass them to the sprite initialization.

linked with #1


Setup base_url based on LIGHTSPARK_EXAMPLE_BASE_URL

Merge activity

May 6, 2:37 PM EDT: @zhenlu merged this pull request with Graphite.


While using git tfs checkintool

I ran
git tfs checkintool
and it shows the tool that allows me to choose the files, that I wish to checkin,
I choose to checkin only one file, and made a commit.
Now, tfs has only one file modified,
but my local "git tfs checkintool" or git tfs rchecking say nothing to commit

I tried overwrite with the -i flag but it doesn't work with the changeset.

That's how it's supposed to work. Your modified files are still tracked by git, so you haven't lost them. Essentially what you did is to merge TFS with your git branch and ignored all of the changes except to the one file. To get the other files back, you'll need to check out the git commit from before you ran git tfs checkintool.

oh ok thanks


Use Setfit to classify text in a very specific field of a business segment

I'm trying to use 'few shot solution' since I have few samples labeled of data and this data has very specific jargons, acronym and terminology.
Is there a specific strategy to use Setfit to accomplish a good accuracy  in this type of scenario?
Sorry if this is a very general question, but after reading all repositories I'm not confident yet this is the way.
thanks.

Hello!
To my knowledge, there is no special strategy for your task. Perhaps a good solution is to get a simple few-shot training script up and running and then experiment with various different (open) models from MTEB.
And otherwise, I've always enjoyed sentence-transformers/paraphrase-mpnet-base-v2 for SetFit. I think you should be able to reach reasonably good performance with SetFit for your task, even if it is a bit niche.

Tom Aarsen


Federated Directory Grouping

Proposed Work Item: Federation Directory Grouping
Proposal Editor: Joe Lamy
Work item Editor: TBD
Date: November 15, 2022
Version: 1
Domain: ITI
The Problem
Given the work in mCSD and the Topologies white paper to define document sharing topologies such as federation in more detail, there is a need to define how implementers may group these directory representations with addressing and routing behaviors of IHE document sharing profiles.
Key Use Case
Enable an mCSD Care Services Selective Supplier to describe not only its document sharing topology in a directory, but details about how requests and data will be routed, specific to grouped IHE profiles that support federation. Add normative routing requirements to be added to the existing document sharing transactions.
Enable an mCSD Care Services Selective Consumer to deterministically discover these details, answering questions such as:
•	Which federated Organizations’ data will be included in aggregate responses?
•	Which federated Organizations or related Practitioners may I target when using a transaction’s routing capability?
Standards & Systems
FHIR
Discussion
Proposed scope:
•	XCDR: targetable communities (Organization)
•	XDR: targetable intendedRecipients (Organization, Practitioner)
•	MHD ITI-65: targetable intendedRecipients (Organization, Practitioner)
•	XCPD: explicitly aggregated communities (Organization) (i.e. information identifiable as from those communities)
•	XCPD: implicitly aggregated communities (Organization) (i.e. information consolidated; not identifiable as from those communities)
•	XCPD: targetable communities (Organization)*
•	XCA: explicitly aggregated communities (Organization)
•	XCA: implicitly aggregated communities (Organization)
•	XCA: targetable communities (Organization)*
•	MHD ITI-66, ITI-67: explicitly aggregated communities (Organization)
•	MHD ITI-66, ITI-67: implicitly aggregated communities (Organization)
•	MHD ITI-66, ITI-67: targetable communities (Organization)
Scope would include potential additions/modifications to the mCSD Organization Affiliation Types code system to support specific needs of these groupings. See issue https://github.com/IHE/ITI.Topologies/issues/5
*This work item would not include in its scope the actual transaction federation capabilities (e.g. targeted XCPD), only the grouping of these capabilities with the directory.

https://github.com/IHE/IT-Infrastructure/blob/master/Proposals/IHE_ITI_Proposal_Federation_Directory_Grouping.docx

Seems this should be put on hold until the whitepaper gets closer to a defined set of needs. I think it is too soon to start work on solving a problem that is not yet clear.

Please itemize the specific changes this is asking for. Minimally defining the use-case need, or defining the specific items.

@jlamy Now that the whitepaper is done, should we review this again at the next face to face?

Related CP-ITI-1301, which is defining XCA: targetable communities.


After server restart, SwellRT returns 403 Forbidden to sign-in

After a SwellRT restart, sign-in calls return 403 Forbidden, despite credentials are sent correctly
POST /auth/signin?r=none HTTP/1.1
Host: swellrt:9898
Connection: keep-alive
Content-Length: 46
Origin: http://swellrt:8000
User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2490.71 Safari/537.36
Content-Type: application/x-www-form-urlencoded
Accept: */*
Referer: http://swellrt:8000/
Accept-Encoding: gzip, deflate
Accept-Language: en-US,en;q=0.8,es;q=0.6
Cookie: WSESSIONID=nji1q3jrszrzkr7d8xvifd5f

HTTP/1.1 403 Forbidden
Access-Control-Allow-Origin: http://swellrt:8000
Access-Control-Allow-Credentials: true
Access-Control-Expose-Headers:
Content-Type: text/plain; charset=UTF-8
Content-Length: 15
Server: Jetty(9.1.z-SNAPSHOT)

login forbidden

I can't reproduce. Plz provide more context info... e.g.
What is the current state of the client's session when the server goes down?
Does the sign-in call use the same credentials?

Sure:

What is the current state of the client's session when the server goes down?

Logged in

Does the sign-in call use the same credentials?

Yes same credentials.
The key point is removing session data from mongodb, so session does not exist anymore. Sorry for not giving enough details.

This issue happens on docker container restart (no just on jetty restart).
During docker restart, filesystem folder containing persisted jetty sessions (see swellrt's config property sessions_store_directory = _sessions) is deleted, hence persisted sessions are not restored.
Suggested solution is to configure the session store directory as a external folder to the container.

I am implementing this, but I think I would be better if they were stored in the mongodb with the rest of the data.
Maybe this is asking too much? :grimacing:

Wow, session manager is part of  the Jetty server.., it sounds horrible!
2016-02-09 17:36 GMT+01:00 Antonio Tapiador<EMAIL_ADDRESS>
I am implementing this, but I think I would be better if they were stored
in the mongodb with the rest of the data.
Maybe this is asking too much? [image: :grimacing:]
—
Reply to this email directly or view it on GitHub
https://github.com/P2Pvalue/swellrt/issues/96#issuecomment-181946913.

I think that current solution has problems from the sysadmin point of view, but... I am not going to push forward :smile:

Closed via 82a128c0ce4a2c0d73e48498f054741bf417bdbe


doc(README): remove redundant "more" in FAQs

Type of Change

[ ] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[x] Documentation
[ ] CI/CD

Description

Removed duplicate "more" word from the FAQs section in the REDME.md file
Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

There was a typo.
How did you test it?

No need for testing
Checklist

[ ] I formatted the code cargo +nightly fmt --all
[ ] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

@EliKalter can you please share you LinkedIn and Twitter handle? Would love to give you a shoutout for your contributions


Choose radio control library

Which radio library shall we use? RadioHead looks promising. Multiple specifications and implementations are waiting on this choice to be made.

Kai just ordered the nRF24L01 so whichever one supports that radio and seems easiest would be my bet.

From today's meeting: we're going to use RadioHead


Aux windows don't stack in order.

If user sets g:microdebugger_aux_windows = ['asm', 'openocd',  'variables', 'monitor'], then the windows are not stacked in such an order.

Fixed.


Html::beginForm() using cached CSRF token in some situations

I have been using a link that does a POST via AJAX to retrieve some data and it has been working fine; however I have noticed that if I remove a form from the page it no longer works and does not send a CSRF token in the headers for some reason.
I also have several hidden forms on the page that are there in both situations.
Here is a run down of the two situations:
Working Situation:
ALL CSRF's on the page are the same - the meta tag named csrf-token, the main form on the page as well as all the hidden forms, they all use the SAME CSRF token and this is the same token that is sent in the headers with the request.
... now I change the scenario so I can produce the non-working version.
Not Working Situation:
There is NO main form on the page in this situation. The meta tag named csrf-token has a different CSRF token from the last request, but for some reason all the hidden forms have the same CSRF token from the previous request/page load.
When I click on the link that POST's via AJAX no CSRF token is sent in the headers.
What I noticed:
The main form is loaded with ActiveForm::begin(...) whilst the hidden forms are loaded with Html::beginForm(...) and the meta tag in the header is generated with Html::csrfMetaTags.
Now I was doing some reading and it seems you have to pass the CSRF as a param when making AJAX requests so I guess that's why that's not working, not sure why it works when the main form is on the page though.
However, the main issue here is why are the hidden forms generated with Html::beginForm(...) using the previous page loads CSRF token?
Of note is that I restarted my browser and loaded the page without it having the main form on it and then reloaded it and both times the hidden forms had new and correct CSRF's.
So it seems it only uses a cached CSRF in them if there was one previously generated by an ActiveForm::begin() call?

@cyphix333, what's the status of this issue. Have you found a solution for it already?

Hey @dynasource .... well as I recall I just started making sure to always send the CSRF token from the meta tag named csrf-token and hence haven't had any problems; so I assume the cached CSRF issue is still prevalent, but it doesn't bother me any more since I started passing the CSRF manually when doing AJAX calls.


add tests for docker container#4146

To use this on Jenkins should be the build command:
docker build -f ./test/Dockerfile -t beaker/tests .
docker run beaker/tests


Missing containerBackgroundOpacity and styles

Hi,
It seems the PHP wrapper has been renewed. The old one had options for setting the 'containerBackgroundOpacity' and adding styles, this one doesn't. More might be missing, which I haven't noticed yet.
Any plan on making the PHP wrapper more complete? (the wrapper might be open source, but it is needed for a product that I bought).

I was wondering the same thing. We're missing a lot of options that are available via other means, such as dataLoadStartMessage, LoadMessage, etc.

Support for Messages and Events are already added.


Hook up auth provider

Coverage decreased (-0.3%) to 97.824% when pulling e49267fb9c7849fb737f1b1315d6869765d1f4c8 on CyberiaResurrection:HookUpAuthProvider into 0337535dfb26a47dbadbf9bce5960769f1d84182 on Algo-Web:master.

Coverage decreased (-0.1%) to 98.009% when pulling f9d4fe091a70e2d50242cf96452d52cc01ed5479 on CyberiaResurrection:HookUpAuthProvider into 0337535dfb26a47dbadbf9bce5960769f1d84182 on Algo-Web:master.

Coverage increased (+0.04%) to 98.19% when pulling f6208974c6cd43abbf534489991957e55e18555b on CyberiaResurrection:HookUpAuthProvider into 0337535dfb26a47dbadbf9bce5960769f1d84182 on Algo-Web:master.

Coverage increased (+0.04%) to 98.19% when pulling 23c2ec5a8ff7e689d4d85a2307669fb353f83ae5 on CyberiaResurrection:HookUpAuthProvider into 0337535dfb26a47dbadbf9bce5960769f1d84182 on Algo-Web:master.


[Feature] Speed up normalization

I have run a quick test, moving a 2M rows table from Snowflake to Snowflake.
It took ~4 min locally, with 3 min being spent at the "normalization" stage.
The data being already in a table, couldn't we request the schema to the DB directly and then use it when calling dlt?
I also tried to replicate the same tables with Sling and it took ~ 1 min to run.
If the normalization time can be cut down, the perf would then be quite close to the Sling one..

Hey, thanks a lot for taking the time and the benchmark here, much appreciated.
I am not much familiar with how dlt normalizes the data internally, I'll take a deeper look at this and have a conversation with the dlt folks on what we can do here.

Since this is an ongoing work and gets better with each release, I'll close this for now. Happy to continue the conversation and take suggestions on this, but we are limited by how dlt approaches things.


Text strings are replacing UI elements

Describe the bug
View Online and Download buttons are being replaced with text strings in the UI on the /object page in record details and on the /list page.
To reproduce
Steps to reproduce the behavior:

Go to dimes.rockarch.org/objects/DQCoJGgCRXHnsPjKGiaCsZ
See buttons in records details
Add this item to your list
Go to /list page
See that "View Online" button is replaced with string of text

Error message or screenshots

Should be:

And on My List page:

Should be:

Additional context
This seems to be a bug related to the lingui translations, because those text strings are present in /src/locales/en/messages.js. However, I'm not seeing it in local development.

I'm also seeing this phenomenon in the footer:

Running yarn extract and yarn compile (which extracts and compiles messages using lingui) appears to have done the trick.
I think there are a couple of things we should do as a followup:

Update the readme with a bit more information about when and how to extract and compile messages
Consider rolling this into git pre-commit hooks and/or our CI pipeline in general


How to cite napari-tomoslice?

I'd like to be able to cite napari-tomoslice.
Maybe creating a Zenodo DOI for the repository is the easiest way to achieve this?

Hi @GenevieveBuckley, this is a great suggestion and we added this into the roadmap for 1.0 release! :slightly_smiling_face:


Check if the aid of raids is required

https://github.com/Idrinth/IDotD/wiki/Server-Api#raids
Currently I'm sending back the autoincrement id, I'm unsure if it's actually required. This might also be a nice task to look up if @Algareb finds the time :)

found a reason it's required, closing this


Update description and logo for the operator

ref: https://issues.redhat.com/browse/ACM-6278

/lgtm


tf.image.decode_image cannot decode b64-encoded string in Python3.6

import tensorflow as tf
import base64

with open('~/001.jpg', 'r') as img:
	encoded_string = base64.b64encode(img.read())
	image_tensor = tf.image.decode_image(encoded_string, channels=3)
	print(image_tensor)

When run above code with Python 2, it is ok, the result is:
Tensor("decode_image/cond_jpeg/Merge:0", dtype=uint8)
However, when run with Python 3, it shows:
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 0: invalid start byte

It seems that API declare its contents must be byte string:

contents: 0-D string. The encoded image bytes.

This issue has completely nothing to do with tensorflow. This is a python usage issue.
https://stackoverflow.com/questions/42339876/error-unicodedecodeerror-utf-8-codec-cant-decode-byte-0xff-in-position-0-in/42340744

Hi, @ppwwyyxx  and @facaiy
Thanks for your reply. Solved it by explicitly adding the mode 'b' when open the image file:
with open('/home/lzhang/tmp/0000045/001.jpg', 'rb') as img:


Bug/dev 104 web3 instance singleton

BEFORE:
Web3Service singleton wasn't working as expected, no real singleton on web3 property.
AFTER:
Singleton is able now to maintain a single web3 property instance as long as the software is running and is also able to instantiate other providers than RPCPRovider.

Coverage increased (+0.3%) to 89.495% when pulling d7a692f4be84c140afbb1ae231b060ca57315459 on bug/DEV-104-web3-instance-singleton into da60b46b49ce31b852bb7808c9846c0d141e3a23 on master.


docs: migrate camera-management docs to here

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-examples/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[N/A] I have added unit tests for the new feature or bug fix (if not, why?)
[N/A] I have fully tested (add details below) this the new feature or bug fix (if not, why?)

Testing Instructions

These are merged and migrated from https://github.com/edgexfoundry/device-onvif-camera/blob/main/doc/using-demo-app.md

@bnevis-i can you merge this?


MeowyPlayer locale translation occasionally fails.

Meowyplayer fails to switch to other language even if OS's locale is not English.

It seems Fyne has bugs when loading the localization files: https://github.com/fyne-io/fyne/issues/5015
Fixed in the latest release.


Update parent_document_url of Whitehall attachments in Asset Manager

Blocked by https://github.com/alphagov/asset-manager/issues/507.

Superseded by #525. Closing.


[HOLD for payment 2023-10-02] [$500]  Workspace - In Invite new members the "Contacts" caption does not disappear when scrolling

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Prerequisites:
Log in to HT account.
Steps:

Open New Expensify app
Go to Settings
Go to WorkSpace
Open or create a WS
Go to Members
Tap the "Invite" button
Scroll up the contact list

Expected Result:
The caption "Contacts" should be above the list of contacts and when scrolling go up and disappear
Actual Result:
The "Contacts" caption remains in place when scrolling through the contact list and is displayed at the top of the contact list
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.58-2
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/ee887cda-87f8-47f0-bb46-6f56482abbb4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~010f0f40876a997e8c
Upwork Job ID:<PHONE_NUMBER>682036736
Last Price Increase: 2023-09-06
Automatic offers:

situchan | Reviewer | 26529846
mkhutornyi | Contributor | 26529849

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

This is not a deploy blocker. This came from #22622 which was deployed to production yesterday.

@hayata-suenaga for future issues, adding External label will do everything needed for external job. i.e. Help Wanted will be added automatically and auto-assign C+.
It seems C+ not auto assigned yet. I can take this if you agree.

How can get new members list?

still accepting proposals

send a message in #expensify-open-source slack channel about this issue

@mkhutornyi's proposal looks good. I think we should disable iOS sticky behavior in SelectionList to be consistent with OptionsList.

Did the approach turn out to be incorrect?

I believe yes.

nop I just completely missed that comment 😓

@hayata-suenaga bump on the above — thanks!

ops sorry I missed this 😓
@mkhutornyi can you provide the summary of dates so that we can check?

@mkhutornyi @situchan sent you both offers in Upwork for the bonus amount. Please let me know once you accept and I can get those paid. Thanks for your patience with this!

Both payments have been made! @situchan where are we at with the checklist above? Thanks!

[x] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/22622
[x] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/22622/files#r1348750497
[x] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x] Determine if we should create a regression test for this bug.
[x] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

No need regression test as this is super minor UI bug and not affect users.

Perfect! Thanks @situchan! Closing this!


fuubar_auto_refresh depends on gems loading order

Current Behavior
In a project that uses both Fuubar and pry-byebug, the fuubar_auto_refresh config is set to true.
Steps to Reproduce

Work in a project with a Gemfile that contains fuubar and pry-byebug.
Make gem "fuubar" appear before gem "pry-byebug" in Gemfile.
Add binding.pry to a spec.
Run the spec.
When it reaches the pry console, start typing, and notice that the Fuubar progress bar refreshes.
Ignoring the noisy refresh for a second, check the value of RSpec.configuration.fuubar_auto_refresh: it will be true whereas it is expected to be false in this context.

Screenshot

Expected Behavior
While I'm unsure whether there is a viable fix for this, the expected behavior would be that RSpec.configuration.fuubar_auto_refresh is false when it needs to be.
Remarks

I think commit 0cbd381bac7e920a0212f181ca7a5b65cba77c14 tried to fix the initial issue (#105), but that this fix is order-dependant.
A quick-fix for me is to add RSpec.configuration.fuubar_auto_refresh = false in a RSpec support file (eg. spec/support/fuubar.rb).

I think I've got a fix. PR coming shortly.

Here: https://github.com/thekompanee/fuubar/pull/109.


ebsnvme-id: open devices in read-only mode

Description of changes:
Opening block devices in read/write mode triggers a kernel change event, which leads to the child devices (corresponding with the partitions) being deleted and recreated.  Opening the fd in read-only mode still provides access to the NVME ioctl() interface, but avoids the unnecessary change event.
Issue #15
Tested on Debian unstable (kernel 5.15, python 3.9.9) and Amazon Linux 2 (kernel 4.14, python 2.7.18):
    admin@ip-10-0-0-75:~/amazon-ec2-utils$ git diff master
    diff --git a/ebsnvme-id b/ebsnvme-id
    index c6a31e2..90979f0 100755
    --- a/ebsnvme-id
    +++ b/ebsnvme-id
    @@ -113,7 +113,7 @@ class ebs_nvme_device:
                                            alen=id_len,
                                            cdw10=1)

    -        with open(self.device, "r+") as nvme:
    +        with open(self.device, "r") as nvme:
                 ioctl(nvme, NVME_IOCTL_ADMIN_CMD, admin_cmd)

         def ctrl_identify(self):
    admin@ip-10-0-0-75:~/amazon-ec2-utils$ stat --format="%i" /dev/nvme1n1p1
    598
    admin@ip-10-0-0-75:~/amazon-ec2-utils$ sudo python3 ./ebsnvme-id /dev/nvme1n1
    Volume ID: vol-019e7bd5b5b3861a0
    sdf
    admin@ip-10-0-0-75:~/amazon-ec2-utils$ stat --format="%i" /dev/nvme1n1p1
    598

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi Folks, approx when can this be merged?

@fred-lefebvre could you please help review this ?


https://developer.mozilla.org/en-US/docs/MDN/Feedback

Request type

[] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x ] Correction or update

Details
Some of the HTML output boxes don't show anything on this page: https://developer.mozilla.org/en-US/docs/Learn/CSS/Introduction_to_CSS/Simple_selectors
The attribute selectors page: https://developer.mozilla.org/en-US/docs/Learn/CSS/Introduction_to_CSS/Attribute_selectors has missing info, it shows the usage of the square bracket syntax, but not attaching it to a specific element (it might only show [href], instead of a[href], for example). Also the input code and output of the Substring value attribute selectors section is wrong.

We've got an all-new set of beginner's CSS docs covering selectors now; see https://developer.mozilla.org/en-US/docs/Learn/CSS/Building_blocks/Selectors. I think this should fix these issues.


Ability to set environment variables in exec

Currently it's only supported on run
(working on a patch)

Before you open a PR; note that there's a feature freeze on the exec command, see https://github.com/docker/docker/blob/master/ROADMAP.md#21-docker-exec
Also, realize that setting env-vars using exec, won't change env vars for the main process

@thaJeztah thanks for the explanation, hadn't noticed it before. Kind of frustrating once this is my first experiment in contributing to docker :)
Can this little patch be considered a freeze break? Or can it be seen as just maintenance?

We'll see :smile: thanks for opening the PR


what if FastICA doesn't converge?

this might be a more general problem with giving arguments to the underlying ICA algorithm.
for example:

increase n_inter
giving it a random state so you get exactly the same S3 model when you run it twice (nice for deployment)

I'm on it. Though I would argue that if the model doesn't converge after 200 iterations, it is probably a sign of something (there is no clear pattern in the data)


Remove ICA hooks

Description
They will be replaced by middleware, but I think it'd be best to remove it now so they don't hinder development
closes: #XXXX

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #480 (644c3c3) into interchain-accounts (e072e67) will increase coverage by 0.10%.
The diff coverage is 0.00%.

@@                   Coverage Diff                   @@
##           interchain-accounts     #480      +/-   ##
=======================================================
+ Coverage                78.50%   78.61%   +0.10%
=======================================================
  Files                      134      134
  Lines                    10338    10323      -15
=======================================================
- Hits                      8116     8115       -1
+ Misses                    1808     1794      -14
  Partials                   414      414

Impacted Files
Coverage Δ

...dules/apps/27-interchain-accounts/keeper/keeper.go
76.47% <ø> (-0.28%)
:arrow_down:

...odules/apps/27-interchain-accounts/keeper/relay.go
78.80% <0.00%> (+6.68%)
:arrow_up:


clj-http.headers.HeaderMap is not fully compatible with normal Clojure maps

Maps returned by header-map (HeaderMap instance) are not fully compatible with normal Clojure maps. These maps are part of the clj-http reponse data.
Not sure what if anything should be done to fix this.
=>
(clojure.set/rename-keys (into (clj-http.headers/header-map) {"Foo" "Bar"})
                         {"Foo" :foo})
{"Foo" "Bar"}

=>
(clojure.set/rename-keys (into {} {"Foo" "Bar"})
                         {"Foo" :foo})
{:foo "Bar"}

https://github.com/dakrone/clj-http/blob/3.x/src/clj_http/headers.clj#L105
https://github.com/dakrone/clj-http/blob/3.x/src/clj_http/headers.clj#L133

Hmm okay, we need to figure out what rename-keys uses that the HeaderMap needs to implement in order to get it to work.

rename-keys uses assoc internally. assoc behaves like this: HeaderMap converts keyword keys to Pascal cased string keys. So maybe this is just a feature of HeaderMap.
(require 'clj-http.headers)
(def a  (into (clj-http.headers/header-map) {"Foo" "Bar"}) )
(assoc a :foo "baz")
=> {"Foo" "baz"}

(assoc a :quux-xyz "baz")
=> {"Foo" "Bar", "Quux-Xyz" "baz"}

(assoc a "foob" "baz")
=> {"Foo" "Bar", "foob" "baz"}

Anyways, you can use (into {} ...) to convert HeaderMap to normal Clojure data structure.
(def a  (into (clj-http.headers/header-map) {"Foo" "Bar"}) )

(def b (into {} a))

(clojure.set/rename-keys b {"Foo" :foo})
=> {:foo "Bar"}


how to make update requests

store = OpenStruct.new(woocommerce_url: "https://alforriamoveis.com.br//wp-json/wc/v3/products/13409", token: "ck_6203256344errhello44", consumer_secret: "cs_d0203256344errhello44")
@prod = WooCommerce::API.new(
    store.woocommerce_url,
    store.token,
    store.consumer_secret,
      {
        wp_api: true,
        version: "wc/v3"
      }
    )

prod.post("//wp-json/wc/v3/products/13409", '{"name": "hello"}')
but this not worked
error code: rest_no_route

find how work..
substituinfo por...
store = OpenStruct.new(woocommerce_url: "https://alforriamoveis.com.br//wp-json/wc/v3/products/13409?", token:
...
....
response2 = @prod.put('/', {"name": "hello"})
....

You should use:
store = OpenStruct.new(woocommerce_url: "https://alforriamoveis.com.br/", token: "ck_6203256344errhello44", consumer_secret: "cs_d0203256344errhello44")
@prod = WooCommerce::API.new(
    store.woocommerce_url,
    store.token,
    store.consumer_secret,
      {
        wp_api: true,
        version: "wc/v3"
      }
    )

Since this library will compose this peace wp-json/wc/v3/, and then you should use @prod.get("products/13409").
Also not that there's no parameter name like in your example  '{"name": "hello"}.
You can find more example in the REST API documentation, there's complete examples for this library: https://woocommerce.github.io/woocommerce-rest-api-docs/?ruby#introduction


Remove ZStandard and redundant NULL checks

We now use LZ4 compression for the MatrixStore and remove NULL rows at build time.
Closes #1956

Coverage increased (+0.002%) to 75.863% when pulling 06acbacb46a50f051ea9ced7dbe3d014a1bcc949 on remove-old-matrixstore-code into f4ddfa8fc2feb2e52782977664a7150d562a9ebd on master.


Added OptionalExtensions

Please add it to the end of the list

@lfarah isn't it alphabetically ordered?

@RuiAAPeres No, it is by order of commit

Thanks

Oh, hi @dkhamsing :cake: !

:wink:


Initial version of MongoDB instrumentation

Uses the MongoDB APM subscriber infrastructure instead of hooks, so it doesn’t even require the opentelemetry extension.
The sub-project setup was blatantly stolen from the PDO module so I am 100% I am missing something.

Welcome, @lstrojny
I've taken a first pass at the code, it looks good to me.
I notice the tests are failing for php < 8.2, will be run in lower versions? If not, can you add exclusions to the workflow to disable tests against earlier versions? If possible, it would be good if it could work with as many versions of PHP as we support (7.4+, although most auto-instrumentations only support 8+)

@brettmc fixed. Supporting 7.4 and up is trivially possible so let’s do that for now. Enums in PHP 8.1 is probably the only interesting feature, especially for the OTEL core libraries. Adopting those will probably require another major version so we can kick that can down the road.

@lstrojny making progress. Integration tests are failing due to no mongodb available.
We can either look to disable those tests, or add a mongo (which is probably a better test).
I had a quick play and it looks like you can conditionally install mongo, so if you try adding a step into .github/workflows/php.yml like so, it may just work:
    - name: (optional) Setup mongodb
      uses<EMAIL_ADDRESS>      if: matrix.project == 'Instrumentation/MongoDB'
      with:
        mongodb-version: 6.0

We shouldn't even need a real MongoDB instance as the attempt to query should be good enough. Can you retry?

@lstrojny I did a little more investigation into mongodb. It should be available on localhost:27017, however in my testing there is a race condition: it looks like the tests are running before mongo is ready to accept connections.
I raised an issue against the action, but in the meantime I found that adding a replicaset does have a readiness check, so try adding this to the with section: mongodb-replica-set: test-rs

Released to packagist as v0.0.1


Us 3 d

Forgot to run ESLinter and had no space after a required file...

Linter sux.

please review and approve


[intro-to-pytorch] fc_model module missing

The fc_model.py file is not included in the folder. This prevents this notebook from executing without error. The module can be found here

Thanks, fixing it now.

Fixed here: 4951aca43927fb28546431ca866ebc2a1c56469c


WIP: Tests export nitrate

Attempt how to test export without nitrate. Very dirty :/

could be replaced by PR: https://github.com/psss/tmt/pull/582/


(WIP) Discrete logarithm

This is the work I started at ZuriHac. It implements Pollard's rho algorithm to compute the discrete logarithm. This has the deficiency that there are some solutions it cannot find, such as 2^x == 7 (mod 9), which I would like to resolve before this PR gets merged.

@Taneb just a gentle reminder that you intended to resolve remaining issues. Or we can merge as is and restrict inputs of discrete logarithm to primitive roots only.

Sorry, I've been busier than I expected! :sweat_smile: Going to do some work on this tonight, though

I think we can introduce a newtype
newtype PrimitiveRoot n = PrimitiveRoot (Mod n)

and amend the signature of Math.NumberTheory.Moduli.PrimitiveRoot to
isPrimitiveRoot
  :: KnownNat n => Mod n -> Maybe (PrimitiveRoot n)

Consequently, discreteLogarithm should allow only PrimitiveRoot as its second argument,
discreteLogarithm :: (KnownNat m, Integral b) => Mod m -> PrimitiveRoot m -> Maybe b

This eliminates the current issue with logarithms by non-primitive base (Pollard's algorithm supposes that the base is primitive) without compromising performance or correctness.

@Taneb any chance to finish it soon?

Closing, superseded by #130.

Sorry I neglected this issue! I'm glad it's been solved by someone, I had... less time and headspace for this than I'd have liked

It was a pleasure to work with you at ZuriHac.


Android-Java: 3.5 Android Explicit Intent Example

Description

Implicit vs explicit intent
What are Explicit Intent
eg by Calling one activity from another activity

Domain
Android Dev (Java)
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

/assign

@Rohit-2602 @VikasPandey121 Please check .

@Rohit-2602 @VikasPandey121 Please check.

If you created a PR on this issue then please tag the issue number on your PR like #2599 so that I can look into it,


Adds a bell

Resolves issue #13. Requires a browser that can handle Unicode6

Whistle is out of scope of issue #13.  Please open a new issue.  Unfortunately, it looks like upstream Unicode doesn't yet support whistling.

Adam's Family bell will work very good.

Thanks @sivinjski !


Beacon service is not FIPS compliant

Hello,
Looks like Beacon service is calling a version of SHA256 that is not FIPS compliant.
See thread I started in forums
http://www.dnnsoftware.com/forums/forumid/200/threadid/535732/scope/posts
.
Looks like the code in BeaconService.cs is calling SHA256Managed which is not FIPS compliant.  I see other code in DNN calling SHA256CryptoServiceProvider which is FIPS compliant.
Thanks
John

Sorry, eneterd this bug in the wrong location.


[WIP] HA - SNO operator implementation

This PR will contain HA - SNO operator implementation.
This operator is setting an HA Layer on 2 SNO clusters.
At the moment code is still WIP.
Feel free to review at this stage or wait for WIP to finish.

Closing this PR,
Code is moved to https://github.com/medik8s/ha-sno


sync demo profile from operator to classic installer

Bug description
i noticed istiocl x manifest installed demo profile varies differently than our istio-demo.yaml.  We need to have this and other profiles in sync to ensure a smooth experience.
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[x ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
consistent behavior on a given profile regardless how i installed it.
Steps to reproduce the bug
Follow this to install the demo profile: https://preliminary.istio.io/docs/setup/install/operator/
Version (include the output of istioctl version --remote and kubectl version)
1.3 RC3
How was Istio installed?
Follow this to install the demo profile: https://preliminary.istio.io/docs/setup/install/operator/
Environment where bug was observed (cloud vendor, OS, etc)
IBM Cloud KS
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

@irisdingbj could you look at this?


Multiple sites on same port with different hostname?

Is it possible to run multiple sites with the same port but different hostnames with dppm?
Thanks

To do this, you'll need a web server like Caddy, Nginx or Apache.
Each application/site must have its own port, but they can be forwarded through a reverse proxy that expose 80/443 (http/https).
The support of Caddy will come soon, I've just finished the API to handle its configurations.
Caddy has auto-https with Let's Encrypt, but we need of course a valid FQDN domain name, that will be checked.
Since we support a web server, PHP applications are also coming soon.

Yes, if it can just help me deploy multiple websites out of the box, it will really be very helpful. I can use it in production for deploying and managing multiple sites.


[GTA2] Compatibility with the v11.44 unofficial patch

Makes GTA2 compatible on modern Windows and stuff, but uses D3D rather than DDraw which your WS fix is hooking into.
http://gtamp.com/gta2/

You might as well close this then :)

After a new version release.

Okay, it's compatible, but menu is a bit messed up. Honestly, don't want to figure out why, so I'll leave it as is.


Install dependencies using npm instead of bootcss.com

In my experience during this day, the cdn.bootcss.com website is incredibly slow, taking the editors minutes to load. It would be more convenient to load the dependencies over npm or support another local source.
A developer could download these dependencies and add them manually, yet it would be easier to include them in the npm package or give documentation on where to download these dependencies.

The 2.1.0 has changed. Please see README.md


Fix Export bug

Fix export bug with trailing , when there are no tags
Adjusted code for RepoSense
Added ExportCommandIntegrationTest
Added comments

Pull Request Test Coverage Report for Build 493

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 64.295%

Totals

Change from base Build 488:
0.02%

Covered Lines:
2069

Relevant Lines:
3218

💛  - Coveralls