backend-common: discovery implementation that supports split backends

Feature Suggestion
The PluginEndpointDiscovery interface in the backend currently has a single implementation, SingleHostDiscovery. This implementation results in all communication happening internally with each deployed backend instead. It does support horizontal scaling of the catalog in that you can have multiple instances without issues, but the load may not be as well distributed as if all requests were routed to all instances.
The SingleHostDiscovery does not support splitting up the backend plugin deployments in any way. Every instance must have each plugin present for the intra-plugin communication. As a part of making it easier to do these kind of backend split, we propose to add a new PluginEndpointDiscovery implementation that falls back to the PluginEndpointDiscovery implementation, but that can also be configured to support split backend deployments.
Possible Implementation
The routing could be driven either through code or static configuration, but most likely it's best if the primary method is through static configuration, since that's a bit easier to share between instances.
The configuration might look for example like this:
backend:
  routing:
    hosts:
    - baseUrl: http://backstage-charm.internal
      plugins:
      - catalog
      - search
    - baseUrl: http://backstage-strange.internal
      plugins:
      - techdocs
      - scaffolder
    - baseUrl: http://backstage-higgs.internal
      plugins:
      - auth
    fallback:
      baseUrl: http://backstage-tau.internal

Want to stress that this is just an example to get discussions going, it's not necessarily the way I suggest we implement it. In this example the configured baseUrl of each hosts entry would be used whenever one wants to communicate with any of the listed plugins, with the fallback configuration being used if no other hosts match.
We may also allow configuration of the external base URL, but generally it's something we'd recommend agains and instead make sure you have a proxy in front that takes care of the routing. On that topic we could potentially power up the proxy plugin to allow it to consume this configuration to handle routing of API paths, but likely a future thing in that case.
Naming of the implementation is very much TBD, but I suggest that we make it the new default implementation and let it handle the current single host use-case too. Couple of ideas:

HostDiscovery
EndpointDiscovery
PluginRouting
PluginEndpointDiscoveries.fromConfig
EndpointRouting

Context
Split plugin deployments is something we want to make a lot easier than it is today. In our view it is an important feature that improves security and makes it easier to scale things independently, distribute load, and split up ownership and lifecycles of the various backend plugins.

PluginEndpointDiscovery
SingleHostDiscovery

#11836 should be considered as part of this

I'll not i NB o  v js
On Fri., Oct. 7, 2022, 10:58 github-actions[bot], @.***>
wrote:

This issue has been automatically marked as stale because it has not had
recent activity. It will be closed if no further activity occurs. Thank you
for your contributions.
—
Reply to this email directly, view it on GitHub
https://github.com/backstage/backstage/issues/11064#issuecomment-1271316425,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACFMJIW57TELHQZFGB5S4NLWB7Q4NANCNFSM5UIFDCDA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi, @Rugvip, I have met the same issue and thanks for raising this point.
I see this issue closed at Feb, so which version of @backstage/backend-common contain the change, I can't find related information, pls help to indicate, thanks.


draft - support multitenancy for stateless session

Fixes #766

@blafond you need to rebase to the latest changes in main, or the build on CI will fail

This PR is obsolete now. @blafond is working on the new one


Kernel size can't be greater than actual input size

I'm trying to convert LLM replies to sound via Piper TTS. The call sometimes fails with:
Traceback (most recent call last):
  File "/home/pi/Projects/GitHub/hopper-chat/hopper-llama.py", line 285, in <module>
    wav = tts.tts(text=f"Here. {reply}")
  File "/home/pi/.local/lib/python3.9/site-packages/TTS/api.py", line 276, in tts
    wav = self.synthesizer.tts(
  File "/home/pi/.local/lib/python3.9/site-packages/TTS/utils/synthesizer.py", line 398, in tts
    outputs = synthesis(
  File "/home/pi/.local/lib/python3.9/site-packages/TTS/tts/utils/synthesis.py", line 221, in synthesis
    outputs = run_model_torch(
  File "/home/pi/.local/lib/python3.9/site-packages/TTS/tts/utils/synthesis.py", line 53, in run_model_torch
    outputs = _func(
  File "/home/pi/.local/lib/python3.9/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/TTS/tts/models/forward_tts.py", line 689, in inference
    o_en, x_mask, g, _ = self._forward_encoder(x, x_mask, g)
  File "/home/pi/.local/lib/python3.9/site-packages/TTS/tts/models/forward_tts.py", line 408, in _forward_encoder
    o_en = self.encoder(torch.transpose(x_emb, 1, -1), x_mask, g)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1520, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/TTS/tts/layers/feed_forward/encoder.py", line 161, in forward
    o = self.encoder(x, x_mask)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1520, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/TTS/tts/layers/feed_forward/encoder.py", line 71, in forward
    o = self.res_conv_block(o, x_mask)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1520, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/TTS/tts/layers/generic/res_conv_bn.py", line 123, in forward
    o = block(o)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1520, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/TTS/tts/layers/generic/res_conv_bn.py", line 79, in forward
    return self.conv_bn_blocks(x)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1520, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/container.py", line 217, in forward
    input = module(input)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1520, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/TTS/tts/layers/generic/res_conv_bn.py", line 42, in forward
    o = self.conv1d(x)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1520, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/conv.py", line 310, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/home/pi/.local/lib/python3.9/site-packages/torch/nn/modules/conv.py", line 306, in _conv_forward
    return F.conv1d(input, weight, bias, self.stride,
RuntimeError: Calculated padded input size per channel: (4). Kernel size: (7). Kernel size can't be greater than actual input size

The best I can tell, the model needs sentences longer than 2 words, or it throws this error. For example, I ask the LLM for a joke, and it returns the following that's split into 3 sentences.
 > Text splitted to sentences.
['Here.', "Why don't scientists trust atoms?", 'Because they make up everything!']

I'm using the en_US-lessac-medium model.
Any ideas on how to fix this? I'm assuming that I can force the LLM to give me 3+ word sentences and/or check the sentences manually in Python, but I have to imagine that a TTS engine should be able to handle 1-word sentences.

Wow...oops. I'm mixing my TTS engines. I was using Coqui TTS before transitioning over to Piper, and it looks like my code is still calling the Coqui engine. Single-word sentences work with Piper. Closing the issue.


Docs should clarify "click on add" and ...

docs should discuss the fact that when you click add there are two choices in a drop down.

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

ID: d7cafac9-f288-9348-1915-0236405c9794
Version Independent ID: ac2ef813-051c-10bd-fd56-e1cf346c30d5
Content: Set up an apex domain with Azure DNS in Azure Static Web Apps
Content Source: articles/static-web-apps/apex-domain-azure-dns.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@rrelyea Thank you for your feedback! We will review and update as appropriate.

@rrelyea the document aim is to create a show steps on how to Set up an apex domain with Azure DNS and not with custom domain on other DNS(external Provider). Hence adding a suggested screenshot is not necessary.
For the second option "custom domain on other DNS (external Provider)" this document outline the process

Just trying to point out ambiguity/possible confusionin the instructions.
Less ambiguity = good.
No need for screenshot...but reducing ambiguity would improve customer experience.
The hardest part of my move (from GitHub pages to azure static sites) was custom domains!

Select Next

There is no Next, the button is Add.  It also does not mention selecting a DNS zone from the list (required field) in the previous step.
Another confusing part is that "Set up a custom domain with Azure DNS in Azure Static Web Apps" comes before this step when going through the instructions, but that wasn't working without an Apex domain.  I'm waiting for my apex domain to validate and will try the previous step again.  Maybe I'm the only one confused by this.


Suggestions on how to mock data for NoSQL database

My application is data-driven but developed with Non-relational Database.
I think using the current package doesn't mirror the application code. So, I'm wondering if there's any solution or suggestion on how to approach the mocking.

Hey, @rowlandekemezie. Could you please share some more info on your NoSQL data structure?
The usage of this package resembles describing a document store, and I can imagine it being useful for your use-case as well.


ConnRecord tags - their_public_did and invitation_msg_id

Signed-off-by: Shaanjot Gill<EMAIL_ADDRESS>
resolve #1542

Would there be a way to add a process (perhaps via a startup parameter?) to run through the existing connection and add this tag?  Asking hypothetically -- perhaps not specifically for this change.  Eventually, we're going to run into a situation that this is needed by some that have deployed ACA-Py.  It would be nice to know of any approach for this in case someone does need it.


Frontend: Sending a login request to the backend and receiving the returned bearer token

In the Frontends api file implement a new API call that request the backend for login authorization when the user logs in.
Then store the returned login bearer token to the browsers localStorage.
Check the browsers localStorage for the token automatically when starting the app to see if user needs to login.

They need to be in the backendAPI.tsx file.
/authentication/login should be a POST request with the username and password in the body. It should return the JSON web token from succesful login.
/authentication/logout should be a DELETE request that deletes the token from the system.
/authentication/checkToken could be a GET request that checks that the token is in the system, is valid and is related to the current user.


v2 - determine if PROXY_PRESENT is still needed

Describe what problem your feature request solves
We have an env var in version 1.6.0 for proxies: PROXY_PRESENT
Describe the solution you'd like
Determine if PROXY_PRESENT is needed for version 2.0 onwards
Additional context

I misunderstood what this config was used for.  In previous iterations, that was set for the passport/cookie auth configuration.  v2 no longer uses cookie auth, but instead relies on bearer token auth.  I don't know if we would need this setting, or how we would configure it.  I don't have any environments suitable for testing.

Thanks for looking at this - we can close this as it is  not needed


Fix custom death message not working

Perhaps you forgot to execute invokedFailAction = true;

Thanks for the fix!


texstudio: 4.8.2 -> 4.8.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/texstudio-org/texstudio/releases.
meta.description for texstudio is: TeX and LaTeX editor
meta.homepage for texstudio is: https://texstudio.org
meta.changelog for texstudio is: https://github.com/texstudio-org/texstudio/blob/4.8.3/utilities/manual/source/CHANGELOG.md
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.8.3 with grep in /nix/store/p9lw37l8h25fifb7rf0xp1nvgksmdk28-texstudio-4.8.3
found 4.8.3 in filename of file in /nix/store/p9lw37l8h25fifb7rf0xp1nvgksmdk28-texstudio-4.8.3

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

texstudio

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/p9lw37l8h25fifb7rf0xp1nvgksmdk28-texstudio-4.8.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A texstudio https://github.com/r-ryantm/nixpkgs/archive/fa9140e51116668941846052b77dbd82ec1fed2a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/fa9140e51116668941846052b77dbd82ec1fed2a#texstudio

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/p9lw37l8h25fifb7rf0xp1nvgksmdk28-texstudio-4.8.3
ls -la /nix/store/p9lw37l8h25fifb7rf0xp1nvgksmdk28-texstudio-4.8.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

texstudio

Maintainer pings
cc @ajs124 @Chaddai for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 343390 run on aarch64-darwin 1

1 package built:

texstudio


UP updates to Developer Resources page

Problem statement
The UP change communication strategy IDed ways we can updated and leverage the Developer Resources page more to help our users as the product transitions.
What you need to know

...

Acceptance criteria

[x] Page updates drafted, reviewed, and approved by content owner
[ ] Next step planned (Content handed off to eng or design as necessary)

To do

[ ] ...

@chris-kuryak @jillian-hammer FYI

Doc approved and ready for eng ticket @chris-kuryak

#12970 is follow-on ticket


feat: Added contour roi interpolation tool

Context

Creates a contour tool that allows to create interpolated contour ROIs between contour drawn slices by FlyWheel.io
This allows to quickly create annotation in multiple slices for a tissue. Upcoming sculpt tool along with this feature will make editing the interpolated shapes easier.

Changes & Results

Add new interpolated contour ROI tool.
Introduced 'autoGenerated' property in an annotation to indicate whether the annotation was manually created or automatically generated.
Added events for annotation label changes and contour interpolation completion.
Allows suppressing event triggers on adding/removing automatically generated annotations.
Allows separate tool style for auto generated annotations if needed.

Testing

Open example file named 'contourROITool'.
Follow the instructions given in example file to generate interpolated contour ROI.
There is a ContourROITool_test file to run tests on the tool.

Checklist
PR

[x] My Pull Request title is descriptive, accurate and follows the
semantic-release format and guidelines.

Code

[x] My code has been well-documented (function documentation, inline comments,
etc.)

[x] I have run the yarn build:update-api to update the API documentation, and have
committed the changes to this PR. (Read more here https://www.cornerstonejs.org/docs/contribute/update-api)

Public Documentation Updates

[x] The documentation page has been updated as necessary for any public API
additions or removals.

Tested Environment

[x] "OS: Windows 10 with WSL (Ubuntu)
[x] "Node version: 16.14.2
[x] "Browser: Chrome 120.0.6099.110

Since already one MR merged related to this MR. I'am closing this MR.


Ignore POM defined repositories during Aether resolution

Configure Aether to not use repositories definied in pom.xml for dependency
resolution of transitives. This prevents Aether from reaching out to external
repos which might be blocked by corporate firewall.
Signed-off-by: Gunnar Wagenknecht<EMAIL_ADDRESS>

I'm behind a corporate firewall and was wondering why Aether still insisted on going out to Maven Central. This solved it for me.


Can Samurai be installed in an unofficial Windows integration pack?

Can Samurai be used in unofficial Windows bundles? SAM2 on my computer is an add-pack that runs on Windows (a version with gradio :https://github.com/sujanshresstha/SAM2-in-video)
Because the official asks for this image, I have the above questions.

We have never test nor deploy on Windows system. But we don't expect there will be any difference between SAMURAI and SAM 2 in terms of installation.


Live Demo Not Working

It's giving 404
Where Can I see it hosted, If possible then also add a link in links of repo which you can find on the right side.

Live Demo will me added in a week.


RC transmitter modification for tamiya version

Hi, I made JetRacer of tamiya version.
Then, I followed the instruction in docs/examples.md Example 1 - Basic Motion.
When I run the line of car.steering = 0.3 in notebooks/basic_motion.ipynb, my car doesn't response.
The comment in the notebook said:

If the car does not respond, it may still be in manual mode.  Flip the manual override switch on the RC transmitter.

However, my RC transmitter ( which is TTU-08 also in the photo of docs/tamiya/bill_of_materials.md) doesn't have override switch.
It seems like the hardware setup for latrax (at the Phase 4 of docs/latrax/hardware_setup.md) has RC transmitter modification.
Does tamiya version also have to do same modification like latrax version to make it manual mode?

Hi. I was reading, it uses channel 3 switch to change from manual to "Nvidia" mode.
Channel 3 is connected to SEL which is the input of the PIC10 at the PWM multiplexer. So that pin detects a pwn threshold to let channel M1 to M4 or S1 to S4 pass to OUT1 to OUT4 respectively.
In this manual you will see what is channel 3 switch:
manuals.hobbico.com › futPDF
Resultados de la Web
1M23N31504 - You have reached ftp.hobbico.com
If this does not work for you you can by pass the multiplexer and connect the PCA PWM driver ch0 to the steering wheel servo and ch1 to the ESC, but you won't have command on the car by your transmitter.
Let me know whatever you got.

Thank you for your advice.
I think I didn't have enough knowledge about RC transmitter.
My  Tamiya RC Transmitter has only 2ch which cannot send SEL signal to the receiver.
I bought combo version of NSX RC car with Tamiya TT02 and RC transmitter which is different from the one in bill of materials.
I should have bought Futaba 3PV 3ch 2.4GHz transmitter.
As you told me, I bypass the multiplexer and switch "Nvidia mode" and "Manual mode" with just replacing the cable from RC transmitter with the one from motor driver by hand.
It is not convenient to replug cable every time after collecting data to switch to "Nvidia mode" but it works at least.

Sorry for late reply and thank you for your kind advice.
I followed your advice but I couldn't switch "Nvidia mode" and "manual" mode from ch2 of the driver.
As you mentioned, SEL may not receive valid signal from pca9685.
I cannot use oscilloscope right now so not sure the exact reason of this  but I will try the 2nd option which directly connects GPIO pins to SEL pin.

@romybompart @hygradme
I find the  way
pca96855 can  make signal  below

The multiplexer considers the RC signal on the SEL channel to be valid if it has a 10–330 Hz pulse
rate and 0.5–2.5 ms pulse width. The other inputs (S1–S4 and M1–M4) are unrestricted.

first follow @romybompart  's way
and  add some  code in
class NvidiaRacecar(Racecar):
def __init__(self, *args, **kwargs):
    super(NvidiaRacecar, self).__init__(*args, **kwargs)
    self.kit = ServoKit(channels=16, address=self.i2c_address)
    self.steering_motor = self.kit.continuous_servo[self.steering_channel]
    self.throttle_motor = self.kit.continuous_servo[self.throttle_channel]
    self.manual = self.kit.continuous_servo[self.manual_channel]

    **self.kit._pca.frequency= 100
    self.kit._pca.duty_cycle = 0x7fff**

and  then set  the  value
car.manual = 0  #  will  control by  your trasmitter
or
car.mamual = 1 #  will control  by your pca96855
thank you

I am having the same issue. I followed this method and my race car is still not moving.


Use moment to render dates everywhere

Also changes dates to be rendered like April 29, 2017.

"1 May, 2017" feels odd, I don't recall seeing this anywhere. "1 May 2017"
looks more balanced and is nicer when adding the time as we need only one
(or zero) separator.
Nitpick, granted, but I do think it's more standard-looking :)


Search.SequenceNo as text

On display, the GPV is ordering Search.SequenceNo as text.

Tested with HAIG.  Searches are sorted on SequenceNo.  Verified by inspecting code.


"Expand/collapsed" Card option not saved

Hi,
When you "Expand/collapsed" - "s key" the card, the option is not saved so you have to do it each time you login to the board.
thanks,

Yes, at the moment it's saved in your session not in the database.

This now saved into the db.


Output Question

I've successfully ran your tool on 1.16tb of google takeout photos (took 10hrs on my underpowered NAS but made it through). Can you help me understand the folder structure after everything is complete? Both AlbumsProcessed and PhotosProcessed seem to be close to the same size (~500gb a piece). Are these after deduplication scan? Do both directories have the same photos in them but is the photos directory just a flat directory and the albums directory has the same content but in their original directories? Id love to see a results.json file that lists all the stats once completed that explains things a bit. I need to know what directory i need to upload to immich and i dont want to do another dedupe scan. Any help would be great.

Your PhotosProcessed should now hold all the photos from the Photos from * folders, while AlbumsProcessed should contain the contents of the rest of the folders. There shouldn't be any duplicates between them two. Does this answer your question?
If unsure, you could run the tool again with the latest version as the structure was changed a bit. Now the albums are all inside output/<album> plus the merged output/Photos. Though I don't think this should be necessary.


Conflicts with pytest and WSL

Recently, GitHub added WSL to the virtual environment used in GitHub Actions (actions/virtual-environments#1081). As a result, C:\Windows\System32\bash.exe is created. This is NOT an issue for users of setup-msys2 with the default path-type or with strict. However, it might be conflictive for users of mode inherit.
Moreover, in Python, when pytest is used along with check_call(['bash',...]), it is resolved to C:\Windows\System32\bash.exe and it fails if the user has not explicitly configured WSL before. A workaround is to use which from shutil. See https://github.com/actions/virtual-environments/pull/1081#issuecomment-661524144.
Curiously, the problem is found in MINGW terminals only, not in MSYS. See https://github.com/msys2/setup-msys2/runs/892341986. I think it is because https://github.com/msys2/setup-msys2/runs/892341986?check_suite_focus=true#step:6:7 and https://github.com/msys2/setup-msys2/runs/892341966?check_suite_focus=true#step:6:7 (platform msys vs win32).
/cc @The-Compiler

I'd guess cygwin has it's own exe lookup and thus ignores the system directory. Since our bash is always first in our PATH, things shouldn't be a problem independent of pathtype.
Not sure if there is anything we can do here, except suggest users to delete %SYSTEM32%\\bash.exe if it is a problem.

@lazka, I did not check whether this is a problem with pytest only, or with any Python script which uses check_call or other resources from subprocess.

Not sure if there is anything we can do here, except to suggest users to delete %SYSTEM32%\bash.exe if it is a problem.

Using shutil.which('bash') instead of 'bash' solves the issue. If this is reproducible with any Python script that uses subprocess.* and 'bash', I believe we should add a comment in the README. If this is not specific to the virtual environment, but it can be reproduced in any host with MSYS2 and WSL, it might need to be documented somewhere else.

@lazka, I did not check whether this is a problem with pytest only, or with any Python script which uses check_call or other resources from subprocess.

I'd guess any API which calls CreateProcess() internally. But, the question is why would any native Windows app call "bash.exe"? I'd guess that this is a corner case and/or hack in most cases.

@lazka, the use case is using pytest in MINGW32/MINGW64 to handle multiple different test scripts. Some are makefiles, other are python scripts, other are shell scripts. See https://github.com/msys2/setup-msys2/commit/c167722af9b757063fd9c0e21f6aeb689e9d076a. So, it is not a native Windows app, but a multiplatform testsuite composed mainly of a pytest script and shell/bash scripts.

This is the result if bash is not called:

https://github.com/msys2/setup-msys2/commit/f01e00cfeb5344c03a54b1a3d419afc7a82addcb
https://github.com/msys2/setup-msys2/runs/899776128?check_suite_focus=true

Ref msys2/msys2-devtools#1

I guess the only thing we can do here is add a section about bash.exe and tar.exe in the README.

The WSL's bash.exe is also get confused with a meson build. CI runs System32/bash.exe while calling find_program() with a shell script. Here is the project's meson.build file https://github.com/Genymobile/scrcpy/blob/master/server/meson.build

I've created https://github.com/msys2/msys2.github.io/issues/334 so we document this at some point.
Nothing that needs fixing in setup-msys2, so closing this.


Possibility of extracting address fields ?

Good morning,
First of all, a BIG thank you for writing this superb lib (camt_parser) to process these CAMT XML files which are really painful.
Among all the libs I tested, only yours, written in Ruby, worked well but above all can be modified very easily to have the columns you want for extracting the CAMT file !
In order to simplify the life of our financial secretary, who is not a computer scientist, I created a very small WEB service that is very simple to use:
https://github.com/zuzu59/zcamt2csv
This works great, except we need to be able to extract the transaction proxy address columns:

                       <Dbtr>
                            <Nm>Toto, Tutu</Nm>
                            <PstlAdr>
                                <StrtNm>Avenue de Milan</StrtNm>
                                <BldgNb>26</BldgNb>
                                <PstCd>1007</PstCd>
                                <TwnNm>Lausanne</TwnNm>
                                <Ctry>CH</Ctry>
                            </PstlAdr>
                        </Dbtr>

and

                        <Cdtr>
                            <Nm>Association MakeRevolution</Nm>
                            <PstlAdr>
                                <AdrLine>Chemin du Closel 2</AdrLine>
                                <AdrLine>1020 Renens VD</AdrLine>
                            </PstlAdr>
                        </Cdtr>

And this is where my limits in Ruby are exceeded !
Would it be possible, please, to add in your camt_parser lib the possibility of extracting address fields ?
THANKS
Sincerely
Ch. Zufferey

Good morning Christian,
thank you for your very kind and polite request. 😃
Definitely the best one I have received on Github so far.
Yes, it is definitely possible to extend camt_parser accordingly.
Based on the code you linked to, you are currently using name and iban from the transactions object.
It provides the convenience of automatically selecting the respective creditor or debitor that would probably be of interest.
Based on whether the entry is a credit or a debit it selects the Creditor or Debitor
The creditor or debitor allow access through attr_reader to an underlying XML data object which would already expose the data without any changes that have to be made to camt_parser:
      statement.entries.each do |entry|
        related_party = entry.transactions[0].credit? ? entry.transactions[0].debitor : entry.transactions[0].creditor

        street_name = related_party.xml_data.xpath('PstlAdr/StrtNm/text()').text

        # ...
      end

So it would be possible for you to use that approach to get to the data.
However, I also see the value in extending camt_parser here and providing convenience functions for the correspondding access.
Right now I would imagine a convenient access like:
address = entry.transactions[0].postal_address
# for unstructured address data
address.lines
# for structured address data
address.street_name
address.building_number
address.postal_code
address.town_name
address.country

With that all the data would then be conveniently accessible, and also distinctly per related party by going through creditor and debitor if needed.
Would you have anything else in mind here or is this already sufficient for your use case?
Best,
Tobias

Hi Christian,
please have a look at https://github.com/viafintech/camt_parser/pull/65
This adds what I had in mind based on your request. Your feedback would be appreciated.
Best,
Tobias

Good evening,
Sorry for the late reply but I was out walking all day with my family!
And thank you for responding so quickly, BRAVO for the support and responsiveness!
I studied the code:
https://github.com/viafintech/camt_parser/blob/feature/64-add-postal-address-support/lib/camt_parser/general/postal_address.rb
It seems correct to me and meets my expectations.
I'll try modifying my Gemfile to work with the branch:
https://github.com/viafintech/camt_parser/tree/feature/64-add-postal-address-support
But since the Ruby language is very new to me, it will take me a little time.
I will keep you informed of my tests soon.
THANKS
Sincerely
Ch. Zufferey

Good evening as well,
no explanation necessary. Fortunately text patiently waits for people to read it. 😄
I would not have expected a response over the weekend.
Considering that you will need to probably deal with cases where address lines as well as the individual fields are present, I would assume that you might want to get the address into a single column in your CSV file?
If so, I might have a suggestion on how to do this in Ruby and could provide you with a pull request or a code snippet based on  your code.
Best,
Tobias

Good evening,
So, as expected, I spent a lot of time to successfully create my Docker container (Gemfile) with your test repository (modification of my Gemfile):
https://github.com/viafintech/camt_parser/tree/feature/64-add-postal-address-support
But finally I got there \o/
Except that I have problems using the 'postal_address' fields!
Nothing to do, I tried everything in every direction, I must not have understood how to use them.
My test code, lines 32 and 33, can be found here:
https://github.com/zuzu59/zcamt2csv/blob/master/app/app.rb
Could you please point out my mistake?
And yes, I would like to be able to retrieve the 'postal_address' fields in detail in order to be able to create a db with the addresses of the principals
thanks again
Sincerely
Ch. Zufferey

Good morning Christian,
line 32 is how it would have to be used.
The way it is currently implemented is that

postal_address would only be null if the PstlAdr field does not exist in the XML content
lines on the postal_address object is only set if the XML content contains AdrLine fields as in your second example

                       <Cdtr>
                           <Nm>Association MakeRevolution</Nm>
                           <PstlAdr>
                               <AdrLine>Chemin du Closel 2</AdrLine>
                               <AdrLine>1020 Renens VD</AdrLine>
                           </PstlAdr>
                       </Cdtr>

This should work to just get it into the file somehow independent of whether the address data would be provided structured or unstructured.
statement.entries.each do |entry|
  address = nil
  postal_address = entry.transactions[0].postal_address
  if postal_address != nil
    address = [
      postal_address.lines,
      street_name,
      building_number,
      postal_code,
      town_name,
      country
    ].flatten.join(' ')
  end

  csv << [
    entry.value_date,
    entry.amount,
    entry.sign,
    entry.additional_information,
    entry.transactions[0].remittance_information,
    address,
    entry.transactions[0].transaction_id,
    entry.transactions[0].name,
    entry.transactions[0].iban,
  ]
end

If this does not work for you, I would ask you to provide an anonymized example of an XML file, so that I may take a look at whether I got something wrong about where the address is found in the XML file.

Hello,
So, I used your sample code to retrieve the address fields and it works fine. I had to make some small corrections because the 'postal_address' table was missing in the concatenation of the 'address' variable.
I abandoned the idea of being able to retrieve the details of the address fields because the bank does a bit of nonsense with these addresses at the CAMT level and it was not usable.
So that's going well, I have everything I need to be able to work well and I can therefore validate the pull request :-)
I am quite surprised, not knowing the Ruby language, to have managed relatively easily to use your dev branch (feature/64-add-postal-address-support) to validate the tests on my code. A former developer who recently retired, I know quite a few computer languages, but unfortunately not Ruby.
So I thank you very much for very quickly adding this new possibility of retrieving the address field in camt_parser!
thanks again
Sincerely
Ch. Zufferey

Good evening,
happy to hear that!
Then I'll merge the branch tomorrow and release a new gem version so that you do not have to rely on the feature branch.
Best,
Tobias

Good morning,
I released it in v2.16.0.
I would recommend that you update the Gemfile to use that version instead of the branch
Best,
Tobias


feat: add protobuf 3.24.0RC1 release

update protobuf with release 3.24.0RC1

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


Cody (for vscode) can't handle filenames with special characters on Windows

Version
1.52.0
Describe the bug
When I ask Cody to modify a svelte file, (e.g. src/routes/+layout.svelte), it generates a new one instead called routes+layout.svelte. So it looks like it chokes on the "+" symbol in the file name. Svelte uses this filename format to function properly, so it can't be omitted.
Expected behavior
Cody should understand filenames with leading special characters like: src/routes/+[filename].svelte
Additional context
Windows 10 - It looks like an escape character issue between Windows path format (backslash) and the "+" character. I'm assuming that when used on macOS or Linux, it's not an issue. In the above example, I'm using forward slashes, because even this website sees the backslash as an escape character :)

Hey @u84six
Thank you for providing this detailed issue report. This helps a lot to pin down the problem.


getting error on ios build time

/Users/jenkins_slave/Library/Developer/Xcode/DerivedData/Arbella_Mobile-blqxbvfriaheghbgxrjolhtcocdy/Build/Intermediates.noindex/ArchiveIntermediates/Arbella_Mobile/IntermediateBuildFilesPath/Arbella_Mobile.build/Release-iphoneos/Arbella_Mobile.build/Objects-normal/arm64/Binary/Arbella_Mobile
ld: warning: Could not find or use auto-linked library 'swift_Concurrency'
ld: warning: Could not find or use auto-linked library 'swiftFileProvider'
ld: warning: Could not find or use auto-linked library 'swiftDataDetection'
Undefined symbols for architecture arm64:
"_swift_FORCE_LOAD$_swiftDataDetection", referenced from:
_swift_FORCE_LOAD$swiftDataDetection$_react_native_in_app_review in libreact-native-in-app-review.a(RNInAppReviewIOS-e1790b5a904d9f60d9459fdc5fb17c8d.o)
(maybe you meant: _swift_FORCE_LOAD$swiftDataDetection$_react_native_in_app_review)
"_swift_FORCE_LOAD$_swiftFileProvider", referenced from:
_swift_FORCE_LOAD$swiftFileProvider$_react_native_in_app_review in libreact-native-in-app-review.a(RNInAppReviewIOS-e1790b5a904d9f60d9459fdc5fb17c8d.o)
(maybe you meant: _swift_FORCE_LOAD$swiftFileProvider$_react_native_in_app_review)
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

My react native version is 0.63.2

try to add this lines in your podfile:
post_install do |installer|
installer.pods_project.targets.each do |target|
target.build_configurations.each do |config|
config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = "arm64"
config.build_settings["EXCLUDED_ARCHS"] = "arm64”
end
end
end

clean build
run ‘’’npx pod-install’’’

no, still not fixed, error was coming during jenkins build,
after your solution , new error is coming
1 error generated.
fatal error: module map file '/Users/jenkins_slave/Library/Developer/Xcode/DerivedData/Arbella_Mobile-blqxbvfriaheghbgxrjolhtcocdy/Build/Intermediates.noindex/ArchiveIntermediates/Arbella_Mobile/BuildProductsPath/Release-iphoneos/react-native-in-app-review/react_native_in_app_review.modulemap' not found

make sure it build successfully or not before run on another machine.
try to run your project . xcworkspace in your machine with xcode.
clean build and run npx pod-install.
or make sure file that jenkins run it to make a build  ios, it’s extension is . xcworkspace

no, still not fixed, error was coming during jenkins build,
after your solution , new error is coming
1 error generated. fatal error: module map file '/Users/jenkins_slave/Library/Developer/Xcode/DerivedData/Arbella_Mobile-blqxbvfriaheghbgxrjolhtcocdy/Build/Intermediates.noindex/ArchiveIntermediates/Arbella_Mobile/BuildProductsPath/Release-iphoneos/react-native-in-app-review/react_native_in_app_review.modulemap' not found

try to edit those lines to be empty string, strongly this sol. solve your second issue ..
config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] = ""
config.build_settings["EXCLUDED_ARCHS"] = "”

Hey @MinaSamir11  I'm facing a similar issue.
Project Details
"react-native-in-app-review": "^3.2.3",
"react-native": "0.63.3",
Device Details
Xcode Version 13.2.1 (Intel)
It runs completely fine on the simulator only when I try to release a build on test flight I am getting this error.
module map file '/Users/Abc/Library/Developer/Xcode/DerivedData/Project-dmcgxnkjieidszfshxleawrnjeog/Build/Intermediates.noindex/ArchiveIntermediates/Project/BuildProductsPath/Release-iphoneos/react-native-in-app-review/react_native_in_app_review.modulemap' not found

@liqiankl
1.) This library uses Swift code, so make sure that you have created the Objective-C bridging header file.
If not, open your project in Xcode and create an empty Swift file.
Xcode will ask if you wish to create the bridging header file, please choose yes.
For more information check Create Objective-C bridging header file.
2.) add this lines to your podfile
 use_flipper!()
  post_install do |installer|
    react_native_post_install(installer)
        installer.pods_project.build_configurations.each do |config| config.build_settings["EXCLUDED_ARCHS[sdk=iphonesimulator*]"] =  ""
        end
  end

OR
  use_flipper!()

  post_install do |installer|
    react_native_post_install(installer)
      installer.pods_project.targets.each do |target|
          target.build_configurations.each do |config|
            config.build_settings["ONLY_ACTIVE_ARCH"] = "NO"
          end
      end
  end

OR
if you are working with M1 chip add this
Add arm64 under Architectures->Excluded Architectures


Flyout doesn't match intended design

If you load the default toolbox, eg with tests/horizontal_playground.html, you see a flyout as below:

This has the following issues:

The top margin is too small.
The blocks are not aligned as they would be if they were stacked together.

The intended design is shown in this screenshot from the blog post:

Look at line 1145 of core/flyout.js (I don't know how to link lines in GitHub :frowning:).
Through testing, this.margin is the margin on the bottom - when it changes, the blocks stay the same.
This leads me to believe that the blocks aren't being centered inside the flyout.

Still new to this so correct me if I'm wrong.
Lines 1156 - 1174 in core/flyout.js control where the block gets placed within the flyout:
if (this.height_ != flyoutHeight) {
    for (var x = 0, block; block = blocks[x]; x++) {
      var blockHW = block.getHeightWidth();
      if (block.flyoutRect_) {
        block.flyoutRect_.setAttribute('width', blockHW.width);
        block.flyoutRect_.setAttribute('height', blockHW.height);
        // Blocks with output tabs are shifted a bit.
        var tab = block.outputConnection ? Blockly.BlockSvg.TAB_WIDTH : 0;
        var blockXY = block.getRelativeToSurfaceXY();
        block.flyoutRect_.setAttribute('y', blockXY.y);
        block.flyoutRect_.setAttribute('x',
            this.RTL ? blockXY.x - blockHW.width + tab : blockXY.x - tab);
      }
    }
    // Record the width for .getMetrics_ and .position.
    this.height_ = flyoutHeight;
    // Top-most block.  Fire an event to allow scrollbars to resize.
    Blockly.asyncSvgResize(this.workspace);
  }

Specifically these lines:
block.flyoutRect_.setAttribute('height', blockHW.height);
...
block.flyoutRect_.setAttribute('y', blockXY.y);

These lines look like they control how the block is placed in the flyout.
If we used this:
block.flyoutRect_.setAttribute('height', flyoutHeight);

It should make the blocks center properly (I think).
Testing now.

Hey there!  There's a big merge coming in soon that may fix some problems: https://github.com/google/blockly/pull/291 makes the horizontal toolbox much more robust and deals with RTL properly, among other things.
I'll be merging google/develop in again soon (hopefully today or tomorrow), so line numbers will change significantly soon.
I think the problem with aligning by baseline will still be present post-merge.  The top margin may be correct.

Thanks for looking at this, @Tymewalk! Here are the steps I follow to link to a line:

Go to file on GitHub webview, eg https://github.com/LLK/scratch-blocks/blob/develop/core/flyout.js
Press 'y' to make the URL a permalink: should look like https://github.com/LLK/scratch-blocks/blob/af963c9a0ed63a792fe189762a419c66fc01c485/core/flyout.js (note the hash)
Either click the line number, or add #L.... to the URL, eg https://github.com/LLK/scratch-blocks/blob/af963c9a0ed63a792fe189762a419c66fc01c485/core/flyout.js#L1156

It would be nice if GitHub did this automatically when you start typing something like core/flyout.js:1156, just like it helps you with PRs and issues when you type # or names when you type @...

Merge PR is #350
The baseline problem is definitely still present.

Tracking in #21


Include more information in gk info command

Currently the gk info command just returns whether the repository is enabled or disabled. It would be great to include more information such as whether the repository was found to be up to date or if there were some tasks pending. Currently there's no insight into what greenkeeper is doing so its hard to determine if the setup was successful.

Hey @donny-dont,
you're right the info command should contain more information.
The API currently supports listing all PRs that got sent, but that didn't land in the CLI yet.
As a short-term fix you can talk to the API directly:
https://gke.my-domain.net/packages/myuser/myrepo?access_token=<token>&include_prs=true

The token can be found in ~/.greenkeeperrc.
As a Enterprise customer you can look into the job queue to see failed jobs directly by clicking the "Open" link on your dashboard.

Best,
Stephan

@christophwitzko awesome work! This something that requires an enterprise update or is it all in the cli?

Hi @donny-dont, this requires an upgrade of the Greenkeeper Enterprise API.
We will release this update in the next few weeks with a bunch of other features.
Thank you for your patience.
Best,
Christoph


TODO(ian): Enable support for already-processed todos

File Location: ./db_manage/create_db.pyLine Number: 7 This message was auto-generated by Pygemony:Github

e31718d5d5fb28e1a39707478a3fbe37bbe97656
What we do now is move the data into a processed_blobs table from the blobs table. See commit message.


[FLINK-23389][Formats] Glue schema registry JSON support

What is the purpose of the change
AWS Glue Schema Registry client library has recently released a new version (v1.1.2) which highlights a new data format support for JSON. This request is to add the new JSON data format support to launch an integration for Apache Flink with the latest AWS Glue Schema Registry.
Brief change log

Added flink-json-glue-schema-registry module under flink-formats
Added end-to-end test named flink-glue-schema-registry-json-test for the new module
Updated documentation for the new module
Upgraded flink-avro-glue-schema-registry to use the latest AWS Glue Schema Registry library
Refactored flink-glue-schema-registry-test to flink-glue-schema-registry-avro-test

Verifying this change
This change added tests and can be verified as follows:

Added integration tests for end-to-end deployment

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): yes
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: yes
The runtime per-record code paths (performance sensitive): don't know
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs / JavaDocs

✅  1. The [description] looks good.
✅  2. There is [consensus] that the contribution should go into to Flink. This is a logical improvement to the already existing GSR integration
✅  3. Needs [attention] from. N/A
✅  4. The change fits into the overall [architecture].
⏲  5. Overall code [quality] is good. There are open comments on the PR
✅  6. License check
⏲  7. E2e tests pass
⚠  8. Commit contains the Jira ID Please squash commits, rebase and add the Jira ID to commit message

@hhkkxxx133 looks like the e2e test is failing because it is trying to pull into the 2.11 run. As seen in the maven command:
2021-08-17T21:01:47.7373155Z Aug 17 21:01:47 Invoking mvn with '/home/vsts/maven_cache/apache-maven-3.2.5/bin/mvn -Dmaven.wagon.http.pool=false --settings /home/vsts/work/1/s/tools/ci/google-mirror-settings.xml -Dorg.slf4j.simpleLogger.showDateTime=true -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss.SSS -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn --no-snapshot-updates -B -Dhadoop.version=2.8.3 -Dinclude_hadoop_aws -Dscala-2.11  -Dflink.forkCount=2 -Dflink.forkCountTestPackage=2 -Dfast -Pskip-webui-build -Dlog.dir=/home/vsts/work/_temp/debug_files -DlogBackupDir=/home/vsts/work/_temp/debug_files -Dlog4j.configurationFile=file:///home/vsts/work/1/s/flink-end-to-end-tests/../tools/ci/log4j.properties -Dhadoop.version=2.8.3 -Dinclude_hadoop_aws -Dscala-2.11 -Pe2e-travis1 -Pe2e-travis2 -Pe2e-travis3 -Pe2e-travis4 -Pe2e-travis5 -Pe2e-travis6 verify -pl flink-end-to-end-tests/flink-batch-sql-test,flink-end-to-end-tests/flink-cli-test,flink-end-to-end-tests/flink-confluent-schema-registry,flink-end-to-end-tests/flink-connector-gcp-pubsub-emulator-tests,flink-end-to-end-tests/flink-dataset-allround-test,flink-end-to-end-tests/flink-dataset-fine-grained-recovery-test,flink-end-to-end-tests/flink-datastream-allround-test,flink-end-to-end-tests/flink-distributed-cache-via-blob-test,flink-end-to-end-tests/flink-elasticsearch5-test,flink-end-to-end-tests/flink-elasticsearch6-test,flink-end-to-end-tests/flink-elasticsearch7-test,flink-end-to-end-tests/flink-end-to-end-tests-common,flink-end-to-end-tests/flink-end-to-end-tests-common-kafka,flink-end-to-end-tests/flink-end-to-end-tests-hbase,flink-end-to-end-tests/flink-file-sink-test,flink-end-to-end-tests/flink-glue-schema-registry-avro-test,flink-end-to-end-tests/flink-glue-schema-registry-json-test,flink-end-to-end-tests/flink-heavy-deployment-stress-test,flink-end-to-end-tests/flink-high-parallelism-iterations-test,flink-end-to-end-tests/flink-local-recovery-and-allocation-test,flink-end-to-end-tests/flink-metrics-availability-test,flink-end-to-end-tests/flink-metrics-reporter-prometheus-test,flink-end-to-end-tests/flink-netty-shuffle-memory-control-test,flink-end-to-end-tests/flink-parent-child-classloading-test-lib-package,flink-end-to-end-tests/flink-parent-child-classloading-test-program,flink-end-to-end-tests/flink-plugins-test,flink-end-to-end-tests/flink-plugins-test/another-dummy-fs,flink-end-to-end-tests/flink-plugins-test/dummy-fs,flink-end-to-end-tests/flink-python-test,flink-end-to-end-tests/flink-queryable-state-test,flink-end-to-end-tests/flink-quickstart-test,flink-end-to-end-tests/flink-rocksdb-state-memory-control-test,flink-end-to-end-tests/flink-sql-client-test,flink-end-to-end-tests/flink-state-evolution-test,flink-end-to-end-tests/flink-stream-sql-test,flink-end-to-end-tests/flink-stream-state-ttl-test,flink-end-to-end-tests/flink-stream-stateful-job-upgrade-test,flink-end-to-end-tests/flink-streaming-kafka-test,flink-end-to-end-tests/flink-streaming-kafka-test-base,flink-end-to-end-tests/flink-streaming-kinesis-test,flink-end-to-end-tests/flink-tpcds-test,flink-end-to-end-tests/flink-tpch-test,,flink-walkthroughs/flink-walkthrough-common,flink-walkthroughs/flink-walkthrough-datastream-java,flink-walkthroughs/flink-walkthrough-datastream-scala, -DdistDir=/home/vsts/work/1/s/flink-dist/target/flink-1.14-SNAPSHOT-bin/flink-1.14-SNAPSHOT -Dcache-dir=/home/vsts/work/1/e2e_cache -Dcache-ttl=P3M -Dcache-download-attempt-timeout=4min -Dcache-download-global-timeout=10min'

I will see if I can workout where/how that is generated

I would like to verify the e2e tests are passing with Scala 2.12 before merging

e2e tests pass locally. Merging


[BUG] when Azure IOT Hub have big amount device, azure-iot-explorer can not get the device list

Describe the bug
customer have a big mount device in their Azure IoT Hub,  and they found they can not use this tool list the device list, but Azure Cli and powershell is ok.

and I tried to use Fildder to capture the network trace, I found azure-iot-explorer tried to access localhost:
POST http://<IP_ADDRESS>:8081/api/DataPlane HTTP/1.1
Host: <IP_ADDRESS>:8081
Connection: keep-alive
Content-Length: 742
Cache-Control: max-age=0
accept: application/json
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) azure-iot-explorer/0.14.4 Chrome/87.0.4280.141 Electron/11.2.1 Safari/537.36
content-type: application/json
Sec-Fetch-Site: cross-site
Sec-Fetch-Mode: cors
Sec-Fetch-Dest: empty
Accept-Encoding: gzip, deflate, br
Accept-Language: zh-CN
{"apiVersion":"2020-06-30-preview","body":"{"query":"\n    SELECT deviceId as DeviceId,\n    status as Status,\n    lastActivityTime as LastActivityTime,\n    statusUpdatedTime as StatusUpdatedTime,\n    authenticationType as AuthenticationType,\n    cloudToDeviceMessageCount as CloudToDeviceMessageCount,\n    connectionState as ConnectionState,\n    capabilities.iotEdge as IotEdge,\n    modelId as ModelId\n    FROM devices "}","headers":{"x-ms-max-item-count":100},"hostName":"iothub-konnect-prd.azure-devices.cn","httpMethod":"POST","path":"devices/query","sharedAccessSignature":"xxxxxx(I removed it}"}
and the response is:
HTTP/1.1 500 Internal Server Error
X-Powered-By: Express
Access-Control-Allow-Origin: http://<IP_ADDRESS>:3000
Vary: Origin
Access-Control-Allow-Credentials: true
Content-Type: application/json; charset=utf-8
Content-Length: 71
ETag: W/"47-EqBsBSZBGTzp3zjjA8kiA+MHAIo"
Date: Thu, 15 Jul 2021 03:48:33 GMT
Connection: keep-alive
{"body":{"Message":"Failed to get any response from iot hub service."}}
To Reproduce
Steps to reproduce the behavior:

input the connect string
jump the error

Expected behavior
after connect to iot hub, can show the device list
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: windows
OS Version: windows 10
App version: <IP_ADDRESS>

Hi @hylinux
Thank you for reporting the issue. Can you let us know what's the size of devices that we are looking at here?

Closing as no additional info has been provided.


Remove POSIX_THREAD constants and fix unmigrated HAVE_PTHREAD_MUTEXATTR_SETTYPE

We define two constants, POSIX_THREAD and POSIX_THREAD_10031c that appear to be old legacy code and predates the project move to git. They are essentially hardcoded at this point. This PR removes them.
Per conversation with @cmcfarlen, we remove the old code that sets PTHREAD_PROCESS_SHARED on mutexes as it has been dead code since at least ATS v2.0.1.
This also fixes an autoconf-to-CMake bug wherein HAVE_PTHREAD_MUTEXATTR_SETPSHARED was not migrated over. This bug only occurs on debugging to set PTHREAD_MUTEX_ERRORCHECK  on a mutex and likely has been unnnoticed.
Please carefully review, especially src/tscore/ink_mutex.cc.

[approve ci]


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter beckhoff for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.beckhoff
   -
:thumbsup: No errors found
WARNINGS:

[ ] :eyes: [W162] js-controller 5.0.0 listed as dependency but 5.0.19 is recommended. Please consider updating dependency at io-package.json.
[ ] :eyes: [W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.
[ ] :eyes: [W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".

SUGGESTIONS:

[ ] :pushpin: [S522] Please consider migrating to admin 5 UI (jsonConfig).

Please review issues reported and consider fixing them as soon as appropiate.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Suggestions reported by repository checker should be reviewed. Suggestions can be ignored due to a decision of the developer but they are reported as a hint to use a configuration which might get required in future or at least is used be most adapters. Suggestions are always optional to follow.
You may start a new check at any time by adding the following comment to this issue:
@iobroker-bot recheck
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence
Last update at Fri, 06 Sep 2024 06:59:36 GMT

Issue closed due to RECREATE request. Folloow up issue #215 has been created.
This issue can be closed.
your
ioBroker Check and Service Bot


Add support for any type supported in answer option

Took a look at the PR...nothing stands out different than what Mick has pointed out

@mickohanlon23 Fingers crossed the primitives should be handled now


:boom: Upgrade to libhal/3.0.0 & bootstrap

Merged due to chicken and egg problem. The current profiles on main are incorrect which is breaking demo builds. I can fix the demos in a follow up PR.

No need for follow up PR because it works: https://github.com/libhal/libhal-platform/actions/runs/8249906820/job/22563355357


Show XREF for register offset as well as when the address is used.

I have the following code (infineon tricore tc2xx)
Which produces the following disassembly
        80427fea 37 04 70 f0     extr.u     d15,d4,#0x0,#0x10
        80427fee 91 00 00 f5     movh.a     a15,#0x5000
        80427ff2 d9 ff 04 d0     lea        a15,[a15]0x344
        80427ff6 90 ff           addsc.a    a15,a15,d15,#0x2
        80427ff8 c8 0f           ld.a       a15,[a15]#0x0=>**DATA_50000344**

I want it to annotate addresses 80427FF2 and 80427FF6 so that I get the following instead of the above
        80427fea 37 04 70 f0     extr.u     d15,d4,#0x0,#0x10
        80427fee 91 00 00 f5     movh.a     a15,#0x5000
        80427ff2 d9 ff 04 d0     lea        a15,[a15]0x344    #DATA_50000344
        80427ff6 90 ff           addsc.a    a15,a15,d15,#0x2  #DATA_50000344
        80427ff8 c8 0f           ld.a       a15,[a15]#0x0=>DATA_50000344

Eg show that there is an xref to DATA_50000344 at address 80427FF2 and 80427FF6
Is there any options that will do this out of the box?
Even if this is not technically an XREF, I'd like to comment that these instructions are responsible for the XREF.
The reason for this is I am patching the XREFs so I need to collect all relevant instructions to do so. I've done this manually in hand written software, but I would like to see if Ghidra can do it without essentially re-writing the same software.

Place the cursor on the dest a15 of 80427ff2 and run the MultiInstructionMemReference.java from the script manager

Is there a way to do this for the whole file? I can script it but just curious if I can do the whole file at once. Thank you for your help.

Unless I'm forgetting something, no not that I'm aware of of.  I believe the other architectures have additional constant analyzers for what ghidra doesn't pick up.


Eslint isn't working when installed locally

I've tried to follow the directions here to no avail. Neither installing eslint globally nor locally works. SublimeLinter always says there are no errors even if the CLI eslint says there are on the same file. With debug mode on, my console doesn't give me any indication of eslint or SublimeLinter failing.

If I delete all config and the .bin folder, SublimeLinter and eslint work like a charm.

I'm not sure why this would be closed since that doesn't at all sound like a solution. If you're using eslint, you probably have a .eslintrc. The plugin shouldn't fail for that reason.


Filter out spam hashtags like IWIMِ,RapportBergeِ,OracleSecurityِ,FOREDD2018ِ,congresagpb2018ِ,DDDEU

@ramonsmits we already have a mechanism for this in the web API. Is it just a case of adding those to that config file?


Add AUTH option to connection

I miss option to assign redis password to connection to properly AUTH.

You can add AUTH as a normal command in the request, please see https://github.com/mzimbres/aedis/blob/f8ff3034f4250b5b3fa45d713a79dd2274d61a9e/examples/containers.cpp#L33

It seems to negotiate resp3, you need to authenticate with call to HELLO 3...

Fixed in master, see https://github.com/mzimbres/aedis/blob/6ca0bcc945e306599b9b686971c8c26897e80dda/examples/containers.cpp#L34

This is fixed in v0.2.0 where sending HELLO 3 is now a user responsibility. If this solves your problem I will close this issue as fixed.

To sum it up: to negotiate protocol, you need to auth first, otherwise it will fall to resp2 with auth after hello.


Updated nuget packages.

Summary of the changes (in less than 80 chars)

Updated nuget packages.

Fixes https://github.com/loic-sharma/BaGet/issues/529.
Same as https://github.com/loic-sharma/BaGet/pull/530 (Which had some errors).

Ok left a minor comment, otherwise this looks ready for merge :)

It's changed now :)


XDebug documentation

Describe you feature request
We'd like to use FrankenPHP for our Laravel application (running in Docker), but using it with XDebug isn't as straight-forward as the website claims it is:

Sometimes requests appear to be hanging in the frankenphp-worker.php script created by Octane (with no breakpoints set at all)
Breakpoints set in the IDE don't propagate to the running application—HTTP endpoints will be served without invoking breakpoints. Restarting the container fixes this, but...
...removing breakpoints requires another container restart, which takes a few seconds again
Even when setting MAX_REQUESTS=1 in the environment, a breakpoint triggered and then removed will be triggered again on a subsequent request, and only on the third request will it no longer be triggered.

In combination, that makes debugging hard to use, which has encouraged some of our developers to go back to dd()—ugh!
Hence, I would kindly like to ask for two thinks:

Improved (that is, any kind of) documentation on debugging, XDebug setup, and debugging caveats that might apply.
Possibly better support for breakpoints set and unset via the XDebug protocol; I'm really not competent in this regard, but my gut says the FrankenPHP worker needs to either reload if breakpoints are modified, or otherwise account for that. When using the built-in PHP debug server, this works without a hitch, so this must be possible in general.

We're using PHP 8.3, Xdebug 3.3.2, Laravel 11 with current Octane, and the latest FrankenPHP Docker image (as a base image for our own).

On browsing the XDebug issue tracker, there's a related issue:

0002161: Breakpoints not updated when process is running
When a PHP is process is already running and you add a new breakpoint, it will never be triggered unless a script is restarted.
https://bugs.xdebug.org/view.php?id=2161

Following that, Derick is just now implementing support for control sockets, allowing IDEs to inject new breakpoints into running PHP processes—such as the FrankenPHP worker.
IIUC, that means step debugging with XDebug was never actually possible without restarting FrankenPHP after configuring breakpoints?
Adding this information to the website would have saved us, and probably others too, a lot of time. At least to me it means that I'll revert from FrankenPHP back to the built-in webserver for now; not being able to use breakpoints in local development without restarting the server is a non-starter.

There are two workarounds for that: not using the worker-mode or setting the max number of requests to handle with the worker mode to 1 (which is very similar to disabling the worker mode entirely). It's what we do in Symfony Docker for instance (the worker mode is disabled in dev, and enabled in production).

By setting max requests to 1, the synchronisation between the IDE and FrankenPHP is off-by-one—after setting a breakpoint, you need to request the old code once, the worker restarts and the breakpoint becomes active; after disabling the breakpoint, you again need to do an additional request, which triggers the (now-gone) breakpoint again, and only after that things are back to normal.
However, disabling the worker mode is the way to go. This isn't as straightforward in Laravel Octane, which doesn't seem to anticipate someone might want to do this. By using a custom Dockerfile adapted from the one included with Octane, it's doable though.
If someone happens to stumble upon this, here's our solution:
(worker) {
	worker "{$APP_PUBLIC_PATH}/frankenphp-worker.php" {$CADDY_SERVER_WORKER_COUNT}
}

{
	{$CADDY_GLOBAL_OPTIONS}

	admin {$CADDY_SERVER_ADMIN_HOST}:{$CADDY_SERVER_ADMIN_PORT}

	frankenphp {
		{$CADDY_SERVER_FRANKENPHP_OPTIONS:import worker}
	}

	{$CADDY_GLOBAL_EXTRA_DIRECTIVES}
}

All it requires you to do locally is set the CADDY_SERVER_FRANKENPHP_OPTIONS environment variable to an empty string locally, using your Dockerfile or compose stack or whatever.

As the co-author of the FrankenPHP support for Octane, I'm probably the one to blame! IMHO a docs PR on Octane should to the trick. Or maybe an additional (temporarily) debug flag?

The issue is, in my opinion, that the default Caddyfile included in Octane always uses the worker mode. I don't know how others solve this, but I try to keep our development environment as close to production as possible, and thus also want to use Octane locally.
I think the most Laravel-y thing would be to enable worker mode only if APP_ENV is not local, or probably check APP_DEBUG is true?

Actually the issue occurs only when using Xdebug, maybe could we detect if XDebug is enabled or something like that.

@dunglas just so I can understand this correctly; from the default Octane config, FrankenPHP will start in worker mode, correct? So for changes made to the code to become live in development, users must set the maximum number of requests handled by workers to 1.
If that is true, what benefit does the worker mode offer for development, if Symfony Docker doesn't use it here either?

Worker mode is an optimization, for now anyway. As more and more devs get used to working with it (like these changes to xdebug) this will become less truer over time. Generally, you disable optimizations (like a compiled container, etc) during development to make it easier/faster to get feedback loops and iterate.
For now, there are things you have to manually do (kinda like not that long ago, we had to manually disable compiled DI containers in dev, now it's automated, most of the time). Things get better, all the time.

Right; that matches my understanding. So, maybe to rephrase my intent—couldn't we, then, update the Octane integration such that it would only enable worker mode if the application is running in production?
This would improve the development workflow for Laravel users, and work out of the box, without any userland modifications necessary.


Multiple and user-defined tree icons

NUnit V2 supports multiple sets of icons in the tree and even allows users to create their own.
Do we want to support more than one set?
Do we want to support user-created icons?

I would like to have more than one set available. As for users creating their own.. meh... Right now the resources are not being loaded from file, but if we switched to that then they could just replace the files and call it a day. I'm for it.


Update Landscape from LFX 2024-10-30

Automated changes by create-pull-request GitHub action

Requested the project via email to submit an updated logo to PCC.


Can't run binaries in a VM

I am trying to help my brother install Godot on a Raspberry Pi 400 I got him and part of that has been trying to install stuff in a raspbian VM so I can figure out how to get everything to work before having him try.
When I unzip the binaries and run both the raspberry pi 3 and 4 versions and run ./godot_3.3.3-stable_rpiX_editor_lto.bin I just get /lib/ld-linux-armhf.so.3: No such file or directory.

Shoot, I figured it would be something like that. Thanks for the info!

No worries! Glad to help ;)
You should try to install everything directly on the Pi instead of a VM though. Some packages might not work for ARM.

You're definitely right but I unfortunately don't have physical access to it and can't SSH into it from work.


Change table tree options creation

Change table tree options to be created by table tree window.

Create branch from main
issue-148-change-table-tree-options-creation


rename ML reconstruction classes

Problem
Right now the hierarchy in the ml  module is:
Reconstructor
     - RegressionReconstructor
         - EnergyRegressor
     - ClassificationReconstructor
         -  ParticleIDClassifier

This has two problems:

This may cause problems as we also have a Reconstructor base class in the reco module (and traitlets uses class names to set parameters).
It is better to have the base class in the name of the sub-class, so it's clear to a reader that it is a subclass (when possible)

Suggested solution:
rename ml.Reconstructor → ml.Predictor?  (could also see if there is a better word..)
Then you have something like:
Predictor
    - RegressionPredictor
        - EnergyRegressionPredictor
    - ClassificationPredictor
        - ParticleClassificationPredictor

Sounds good to me

The class hierarchy was changed before #1767 was merged


FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory

Code: v1.15-x64
Intelephense: 0.7.2
I don't know if this is Intelephense or VSCode, but it happened during Intelephense indexing the code.
[Info  - 10.59.20] Initialising
[Info  - 10.59.22] Initialised in 1735 ms
[Info  - 10.59.23] Indexing started.
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
[Info  - 11.04.18] Connection to server got closed. Server will restart.
[Error - 11.04.18] Connection got disposed.
    (ditto a lot)
[Error - 11.04.18] Language client is not ready yet
    (ditto a lot more)
[Info  - 11.04.19] Initialising
[Error - 11.04.19] Language client is not ready yet
    (ditto a lot lot more)
[Info  - 11.04.23] Initialised in 3896 ms
[Info  - 11.04.35] Indexing ended | 7414 files | 311.862 s

Could you enable debug logging in the intelephense settings and post the last few lines before the error?

[Info  - 10.20.16] Initialising
[Info  - 10.20.18] Initialised in 1839 ms
[Info  - 10.20.19] Indexing started.
    "
    "
[Debug - 10.21.18] onAddSymbols | file:///d%3A/www/wordpress/content/plugins/sitepress-multilingual-cms/vendor/wimg/php-compatibility/Tests/sniff-examples/forbidden-names/trait.php | 0.170 ms | 116.2 MB
[Debug - 10.21.18] onAddSymbols | file:///d%3A/www/wordpress/content/plugins/sitepress-multilingual-cms/vendor/wimg/php-compatibility/Tests/sniff-examples/forbidden-names/namespace.php | 11.136 ms | 116.5 MB
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
[Info  - 10.24.31] Connection to server got closed. Server will restart.
[Error - 10.24.31] Connection got disposed.
    "
[Error - 10.24.31] Language client is not ready yet
    "
    "
[Info  - 10.24.32] Indexing ended | 7414 files | 252.825 s
[Info  - 10.24.34] Initialised in 2166 ms

And this is the content of forbidden-names folder, I suspect the contents of this folder is the cause of this error. forbidden-names.zip

Thanks, I'll take a look. It looks like the files are tests on a dependency. For a workaround you can add a vendor/**/Test pattern to files.exclude

It work, scanning completed without errors. Thanks.


schema: introduce CosmWasm IDL

Follow up from CosmWasm/cw-plus#722
Just trying to do the same thing, but "package it up a bit".

[x] Do everything Ethan's PR does
[x] Set up versioning of the format
[x] Make it complete (all msg types)
[x] Consider pulling msg variants out of the schema for easier processing in the future (not doing that unless it's requested/there's an obvious need)
[x] Figure out how to tie query response types to query msg variants

Cool stuff. Glad to see your progress here!
It seems most of the CI is dying due to outdated Cargo.lock. I'd love to see how close this is to passing tests

Cool stuff. Glad to see your progress here! It seems most of the CI is dying due to outdated Cargo.lock. I'd love to see how close this is to passing tests

Green now!

Let's pull this out of draft. It is definitely ready for review

@ethanfrey Are we good to merge this?

From my end, yes. We can always iterate on some more improvements we mentioned, but this is a big step forward

Will do a proper review this morning

I just see some things have been discussed before, sorry. Let's double check and then 🐎


Two levels of Embedded should be supported.

I have a problem with two levels of Embedded.
For the first level of Embedded class everything is fine. But if I have an embedded case class, which inside has another embedded case class as a field, insert query is generated wrongly.
Version: (e.g. 1.1.0)
Module: (e.g. quill-async-postgres)
Database: (e.g. postgres)
Expected behavior
Generated query is
INSERT INTO table_a(a_fieldC, a_fieldA) VALUES(?, ?)
Actual behavior
Generated query is
INSERT INTO table_a(a_fieldC, a_fieldA) VALUES(fieldC, ?)
Steps to reproduce the behavior
case class A(id: Option[Long], b: B, fieldA: String)
case class B(c: C) extends Embedded
case class C(fieldC: String) extends Embedded

  val baseQuote = quote(
    querySchema[A](
      "table_a",
      _.id -> "a_id",
      _.fieldA -> "a_fieldA",
      _.b.c.fieldC -> "a_fieldC"
   )
 )
run(quote(liftQuery(events).foreach(e => baseQuote.insert(e).returning(_.id))))

@getquill/maintainers

This happens only with batch actions. Single inserts seem to work with nested Embedded classes. Sample: https://scalafiddle.io/sf/eLrzM4U/1

See https://github.com/getquill/quill/pull/712

@fwbrasil  we had a similar issue but with "option"
case class Address(line1: String, line2: String, name: Name) extends Embedded
case class Contact(address: Option[Address], phone: String)
db.run(query[Contact].update(lift(contact)))

result:
UPDATE contacts SET line1 = line1, line2 = line2, phone = ?
expected:
UPDATE contacts SET line1 = ?,  line2 = ? , phone = ?
(when address is not optional it works fine)

@tamardu Same thing for insert:
import io.getquill._
val db = new MirrorContext[MirrorSqlDialect,SnakeCase]
import db._
case class Address(country: Option[String], city: Option[String]) extends Embedded
case class Person(name: String, address:Option[Address])
def insertPerson(person: Person) = quote {
  query[Person].insert(lift(person))
}
println(db.run(insertPerson(Person("", Some(Address(Some(""), Some("")))))).string)

result:
INSERT INTO person (name,country,city) VALUES (?, country, city)

Workarounds:
def updatePerson(personName: String, newPersonName: String, newPersonCity: Option[String]) = quote {
  query[Person].filter(_.name == lift(personName)).update(_.name -> lift(newPersonName), _.address.map(_.city) -> lift(Option[Option[String]](newPersonCity)))
}

def insertPerson(name: String, country: Option[String], city: Option[String]) = quote {
  query[Person].insert(_.name -> lift(name), _.address.map(_.country) -> lift(Option[Option[String]](country)), _.address.map(_.city) -> lift(Option[Option[String]](city)))
}


Minor issues

Hi,
this pull request was created with help of Feram!
It's a tool which automatically finds and fixes bugs in your repos.
To view the other patches we created for your code
simply activate Feram on github.com/apps/feram.
Or if you are new to GitHub apps,
check out the full tutorial at feram.io/help/getting-started.
If you have any feedback or wishes, don't hesitate to contact me
or open an issue at github.com/feramhq/feram!
Hope you like it! 😁

Hi @adius. Thanks for the PR, but I'm afraid it goes against our style guide so we won't be merging this. Unfortunately our style guide isn't public, but if you have a look at some of the linters we use, and specifically the rules we disable, that may give some insight.
We're happy to change aspects of style where it makes sense, but those changes should be presented with reasons for the change and applied to the whole codebase.

Hi, sorry that this didn't help.
I thought at least the Python community is in agreement on PEP8.
Are there any config files for your linting we should have picked up automatically?

I don't think the Python community is agreed on PEP8 in its entirety. There are good bits, but we have a style that is more optimised for smaller git diffs, which PEP8 isn't great at. However there are also many things that we are far stricter on than PEP8 - see our list of flake8 plugins for an idea of what we disallow.
We store all configuration relating to linting tools in setup.cfg. Relevant parts are the ignored linting rules, and the files that we ignore. Specifically we currently ignore the test files, as we prioritise pragmatism and higher coverage over the tests looking perfect (however we are considering changing this in the future).

Thanks for your feedback! I created an issue and we'll look into supporting flake8. Sounds like a cool tool.
https://github.com/feramhq/feram/issues/35


form_dropdown does not select selected - form helper

Hi there,
I've an issue with this code:
form_dropdown(
    'language',
    ['español' => 'Español', 'english' => 'English'],
    'español',
    'class="form-control" id="language"'
);

It supposed to get español selected but it does not do it.
I dig into form_helper down to line 408, if I do the following code it works, but not with the system helper.
form_dropdown(
    'languaje',
    [
        'español' => 'Español',
        'english' => 'English'
    ],
    ['español']
);

function form_dropdown($data = '', $options = array(), $selected = array(), $extra = '')
{
    foreach ($options as $key => $val) {
        var_dump(in_array($key, $selected));
        die();
    }
}

Any clues?
Thank you!
Edited to add missing missing the closing single quote -> 'español,

Please ask your questions on our forums instead, this is a bug tracker.

Hi there, I'm not requesting for help, sorry if it appears to be. It's an issue.

@TresErres This is not a CI bug. Your third parameter is specified incorrectly, missing the closing single quote ->     'español,

Updated the typo coping the example.

And do what ever with the issue; I won't come any more.


License Missing

Hi Tino
I just wanted to point out that there is no license in this repo.
I really want to use your material and extend it but I would really appreciate if you can use some permissive license like MIT to do that. I think it will make future contributions easier
I will give you attribution of course.

Hey mate,
Thanks.
License Added


Add API changes to changelog

+1

PM +1, @jasonblais was this reviewed by a dev before submitting?

Thanks @esethna,  it was posted in Developers room to review and one of the devs gave feedback for a correction.


feat add ZoomPlugin

Short description
Add zoom plugin  ,which implements waveform zoom operation via mouse wheel.
Implementation details
By calculating the scaled ratio and scroll position based on the audio time at the mouse's location, the current zoom level, and the width of the div container, we can then adjust the zoom level of wavesurfer and control scrolling
How to test it
Screenshots

Checklist

[x] This PR is covered by e2e tests
[x] It introduces no breaking API changes

Yes, I'd like the comments I left addressed.

The code looks good!
It would be nice to also add an example in the examples folder.

The code looks good! It would be nice to also add an example in the examples folder.

nice, I will add one tomorrow.

@katspaugh would be nice to have that example on the website.

Feel free to add it

I meant on wavesurfer.xyz/examples/. And if possible, I haven't found a way to do it.

They are sourced from https://github.com/katspaugh/wavesurfer.js/tree/main/examples


adding ability to switch between front and back camera (if available)

No description provided.
thanks @rrupsing your modify code very helpful

https://github.com/googlesamples/android-Camera2Video


CP1250 encoding problem from odbc data

Hello,
I have so many problems with non-unicode characters from database.
The database is IBM Informix, and DB_LOCALE=cs_cz.cp1250 is set in odbc.ini.
I can get correct result when run from ISQL (smal SQL tool installed with Informix SDK), the result is like bellow:

Putty terminal is set to CP1250

So, I guess ODBC works correctly.
But, from node, with node-odbc this simple code gives totally wrong results:
`const odbc = require('odbc');
async function connectToDatabase() {
const connectionConfig = {
connectionString: 'DSN=cistoca;encoding=cp1250',
connectionTimeout: 10,
loginTimeout: 10,
}
const connection2 = await odbc.connect(connectionConfig);
connection2.query("SELECT naz_par FROM pos_par LIMIT 10", function (err, data) {
	if (err) {
		console.log('Error from connection!', err);
		return;
	}
	for (var i = 0; i < data.length; i++) {
		console.log(data[i].naz_par);
	}
});
// connection2 is now an open Connection

}
connectToDatabase();`
Here is the result:

When I switch Terminal to UTF-8 I got this:

And when run again program, here is the result:

I'm desperate and don't know how to resolve this.
I tried literally 70+% of the Stack Overflow before I ask here for your help, but without success.
Thank you in advance!
Stanko

Hi @ctahk0
Are you able to provide some SQL statements (create+inserts) for us to recreate the table with the correct columns and some of the data you have? This will make it easier for us to debug. Thanks!

Unfortunately I can't insert or edit data, I have only read access. The db is Informix and works on windows env without any problems.

What I can is ask admin to make copy of db and give you ssh access?

Hi,
I want to inform you that I was able to get the correct characters.
The problem is not in your code at all (fortunately :) )
After setting up the correct client locale in odbc.ini I got the correct results.
In my case this settings works:

Instead of this (wrong results):

Thanks again.


feat(watcher): add new endpoint for optimized scanning

In the study screen, there's a scan folder button that triggers a recursive scan of the current folder, the scan walks through the file system and sync the state in the file system with the database.
However, this scan can take too much time which has an impact on the user experience.
The solution implemented in this branch is to make is to add a "recursive" parameter to the scan endpoint, when set to false the scann will only walk through the current folder and not its subfolders, thus the scan will take less time without much impact on the user experience as they want to see studies in subfolder they can move into that subfolder and trigger a new scan from there.
Currently the user can explore the file hierarchy using the side panel but the folder are shown only when the scan is finished. We want the user to be able to move into the hierarchy and trigger a scan in whatever folder they want, to allow that we add the explorer endpoints
One to list workspaces
GET /api/v1/explorer/workspaces
[ { "name": "ppse" }, { "name": "bp_lt" } ]
One to list subfolder in a specefic folder
GET /api/v1/explorer/workspaces/{workspace}/list?path={path}
{
    "directories" : [
        { "name" : "first_dir", "path": "worspace_name/folder/sub/.../first_dir", "workspace": "worspace_name" },
        { "name" : "2nd_dir", "path": "worspace_name/folder/sub/.../first_dir", "workspace": "worspace_name" },
    ],
}

These two endpoint will be usefull for the front to show the folders to explore.

Some old comments were not resolved, plz resolve the one you adressed to simplify the review

Could you also add a little integration test that ensure few things:

Nominal case, test the response

We get 422 errors  when given wrong paths.


Write PR template for contribution

I have created the PR template. Kindly Verify it and let me know the changes to be made (if any)

Thanks for assigning me.
Can I know what are the additional changes that are needed to merge the PR #18


Runtime errors with conda python osx

Conda has a system of relative paths - so building with conda python on osx results in a runtime error. I have outlined some of the fixes in the FAQ. Best option that works most of the time is to set the DYLD_FALLBACK_LIBRARY_PATH.
Solution would be to have a conda package that users can do "conda install Corrfunc" and conda will handle the relative paths issue

With the current commit, I get this error again.

Unsatisfactory fix by hard-coding the full-path to (conda) python lib. Relevant changes in the two python_bindings/Makefile in commit b2f34923009f10ec6a3f7d77d6c5723755e99dc5
If the user moves the conda base directory after installation, then install_name_tool has to be used to find the python libraries.


feat(conferences): add The GeekConf 2024

Hey 👋
We are running a conference - The GeekConf 2024 in Berlin, Germany  🚀
CFP is now open  🙌

@mattcarrollcode tagging you since you approved the one of the past PRS  ❤


[Feature] Add hadoop-ext package to support credential isolation (Teams commented version)

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

[FE Incremental Coverage Report]
:heart_eyes: pass : 17 / 18 (94.44%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/FileSystemConstants.java
8
9
88.89%
[21]

:large_blue_circle:
com/starrocks/credential/CloudConfiguration.java
9
9
100.00%
[]

[FE Incremental Coverage Report]
:heart_eyes: pass : 17 / 18 (94.44%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/FileSystemConstants.java
8
9
88.89%
[21]

:large_blue_circle:
com/starrocks/credential/CloudConfiguration.java
9
9
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 4 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/fs/hdfs/hdfs_fs_cache.cpp
0
4
00.00%
[85, 86, 87, 88]

[FE Incremental Coverage Report]
:heart_eyes: pass : 17 / 18 (94.44%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/FileSystemConstants.java
8
9
88.89%
[21]

:large_blue_circle:
com/starrocks/credential/CloudConfiguration.java
9
9
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 4 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/fs/hdfs/hdfs_fs_cache.cpp
0
4
00.00%
[85, 86, 87, 88]

[FE Incremental Coverage Report]
:heart_eyes: pass : 17 / 18 (94.44%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/FileSystemConstants.java
8
9
88.89%
[21]

:large_blue_circle:
com/starrocks/credential/CloudConfiguration.java
9
9
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 4 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/fs/hdfs/hdfs_fs_cache.cpp
0
4
00.00%
[85, 86, 87, 88]

[FE Incremental Coverage Report]
:heart_eyes: pass : 17 / 18 (94.44%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/FileSystemConstants.java
8
9
88.89%
[21]

:large_blue_circle:
com/starrocks/credential/CloudConfiguration.java
9
9
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 4 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/fs/hdfs/hdfs_fs_cache.cpp
0
4
00.00%
[85, 86, 87, 88]

[FE Incremental Coverage Report]
:heart_eyes: pass : 17 / 18 (94.44%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/FileSystemConstants.java
8
9
88.89%
[21]

:large_blue_circle:
com/starrocks/credential/CloudConfiguration.java
9
9
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 4 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/fs/hdfs/hdfs_fs_cache.cpp
0
4
00.00%
[85, 86, 87, 88]

[FE Incremental Coverage Report]
:heart_eyes: pass : 17 / 18 (94.44%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/FileSystemConstants.java
8
9
88.89%
[21]

:large_blue_circle:
com/starrocks/credential/CloudConfiguration.java
9
9
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 4 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/fs/hdfs/hdfs_fs_cache.cpp
0
4
00.00%
[85, 86, 87, 88]

[FE Incremental Coverage Report]
:heart_eyes: pass : 23 / 24 (95.83%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/FileSystemConstants.java
9
10
90.00%
[21]

:large_blue_circle:
com/starrocks/credential/CloudConfiguration.java
14
14
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 4 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/fs/hdfs/hdfs_fs_cache.cpp
0
4
00.00%
[85, 86, 87, 88]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 4 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/fs/hdfs/hdfs_fs_cache.cpp
0
4
00.00%
[85, 86, 87, 88]

[FE Incremental Coverage Report]
:heart_eyes: pass : 23 / 24 (95.83%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/FileSystemConstants.java
9
10
90.00%
[21]

:large_blue_circle:
com/starrocks/credential/CloudConfiguration.java
14
14
100.00%
[]

[FE Incremental Coverage Report]
:heart_eyes: pass : 23 / 24 (95.83%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/credential/FileSystemConstants.java
9
10
90.00%
[21]

:large_blue_circle:
com/starrocks/credential/CloudConfiguration.java
14
14
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 4 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/fs/hdfs/hdfs_fs_cache.cpp
0
4
00.00%
[85, 86, 87, 88]


Value dauwp can't be converted to class float

Did you read the instructions?

[X] I have read the README, including the known-limitations section.

The problem
The custom component produces a warning when dauwp is not available.
The api returns: "dauwp": "onb." which cannot converted to float.
Integration version
1.6.1
Home Assistant version
core-2023.10.5
Python version
3.11.5
Home Assistant log
2023-10-29 07:41:32.569 WARNING (MainThread) [custom_components.knmi] Value dauwp can't be converted to <class 'float'>
2023-10-29 07:42:02.141 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 593, in state
numerical_value = float(value) # type:ignore[arg-type]
^^^^^^^^^^^^
ValueError: could not convert string to float: 'onb.'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 233, in _handle_refresh_interval
await self._async_refresh(log_failures=True, scheduled=True)
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 389, in _async_refresh
self.async_update_listeners()
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 172, in async_update_listeners
update_callback()
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 469, in _handle_coordinator_update
self.async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 779, in async_write_ha_state
self._async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 879, in _async_write_ha_state
state, attr = self._async_generate_attributes()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 820, in _async_generate_attributes
state = self._stringify_state(available)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 785, in _stringify_state
if (state := self.state) is None:
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 595, in state
raise ValueError(
ValueError: Sensor sensor.knmi_markelo_dauwpunt has device class 'temperature', state class 'measurement' unit '°C' and suggested precision 'None' thus indicating it has a numeric value; however, it has the non-numeric value: 'onb.' (<class 'str'>)
2023-10-29 07:42:02.569 WARNING (MainThread) [custom_components.knmi] Value dauwp can't be converted to <class 'float'>

Diagnostics
No response
Additional information
No response

This is a warning that the sensor wasn't receiving a numeric value from the API, looks like this has recovered.
If you see this again today, let me know and I contact the supplier. Will close this for now.


added css route for easier setup

Description
I have added the CSS line for importing styling in documentation
Motivation and Context
I have added the CSS line for importing styling, which can be helpful with increasing the speed of setting up the package.
Checklist
[X] All new and existing tests pass
[X] My changes required the documentation to be updated
[X] I have updated the documentation accordingly

Hey @orifmilod
Thanks for the contribution, I think it's a great one.
The only think I don't feel it's needed is the extra indentation in the prop table. That table is going to change in the next version of the library and keep it indented is a bit annoying for me.
If you revert the indentation, I'll merge this PR straight away. Thanks.

Hey @orifmilod
Thanks for the contribution, I think it's a great one.
The only think I don't feel it's needed is the extra indentation in the prop table. That table is going to change in the next version of the library and keep it indented is a bit annoying for me.
If you revert the indentation, I'll merge this PR straight away. Thanks.

@arqex I have accidentally closed the PR, by bad xd. but it's opened again, and the table indentation is reverted.

@arqex Any update in here?

Sorry for taking sooooo loooooong to accept the PR. And thank you very much for it.


Cannot change params w/ VoiceOver?

With VoiceOver enabled I can change the waveforms for Oscars 1 and 2 by swiping up or down with a single finger. This is not possible with other params, such as Oscars 1 semitones. Swiping up or down on those yields a negative audio cue and no param change.
I noticed that while the accessibility increment and decrement callbacks are implemented in the Stepper class, they are empty in the Knob class. I do not know if this is by design?
Is there an alternative VoiceOver interaction for changing params such as Oscars 1 semitones?

Ah, I get it. Single finger double tap and hold. Slide left or right to change the value. There is no audio feedback.
I'm guessing this is the direct interaction model declared in the knob's accessibility traits.