HTTP headers case gets modified

When sending a custom header through traefik, it gets recased causing the application to fail.
Is there a way to disable any re-casing?
Repro step:
curl using a cusom header
curl -H "X-tEST-hEADER: true" -o /dev/null -v http://backend.example

TCPDUMP on the other side to see
X-Forwarded-For: <REDACTED>
X-Forwarded-Host: <READACTED>:31880
X-Forwarded-Proto: http
X-Forwarded-Server: qatcd-haproxy-1
X-Test-Header: true
X-Traefik-Reqid: 153670
Accept-Encoding: gzip

Note this happens on the way back as well if I set a header in all caps it comes back camel-cased

Hi,
From RFC 2616 - "Hypertext Transfer Protocol -- HTTP/1.1", Section 4.2, "Message Headers":

Each header field consists of a name followed by a colon (":") and the field value. Field names are case-insensitive.

golang std library transforms them in a canonical way https://github.com/golang/go/issues/5022.
I strongly recommend you to make your apps case insensitive too.

Thank you for the amazingly quick response.
It is greatly appreciated.
Unfortunately this is not an option at this time for these legacy applications.
Keep up the awesome product

Reopening this old issue, as we might discuss a way to disable this behavior.

There is the classic way to dodge this behavior, by setting headers via map, so the key case is preserved ;) its old, and may not work forever, but it is doable 😝

Field names are case-insensitive. So why change the first char to uppercase  ??? Business really wanna use the lowercase key

It is not the responsibility of Go to unilaterally decide how other systems should behave, including whether or not the systems being interacted with follow the case-insensitivity statement of the RFC.
A system/platform should not modify the information it is handling unless modification is the direct purpose of the request.
The code used to modify the casing of field names likely introduces unnecessary functionality that risks additional bugs (regardless of whether such bugs currently exist) and removing that (and in fact any) such code eliminates the risk that such code could introduce unintended behavior.
If the supposition that uses of the platform really are respecting the case-insensitivity nature of the current implementation and therefore the current behavior of case modification theoretically introduces no issues, then removing this behavior should also cause equally zero impact to users.

This is literally no reason for modification of this behavior to be resisted by this team.

@emilevauge Are there any updates on this issue? I unfortunately can no longer use Traefik due to the header casing manipulation. (Regardless of what the RFC says)

I have the same issue too with an app that cannot be modified!

We get lots of requests, from users of our Go app, to preserve the case of headers. Our app transfers data and, consistent with @cmullendore's first point above, our users are most displeased with our app changing their data as it moves it (even if those changes are only in capitalization).
I understand there are workarounds already for Go apps that set headers (stuff it directly into the map); but what we also need is a workaround when we read headers. We need to make a request to an external service, read the response headers, and preserve their case. (Or, it would even be enough, if we could just get a map of CannonicalizedName->originalName.  I.e. it would be OK if everything worked the way it does now... as long as we could ALSO get information about what the original names were before cannonicalization.  I don't particularly care how hard it would be to get that info, as long as we could get it.)
I'd be happy to contribute code to do this, but before investing time in that, I'd be keen to hear from the Go team in this thread - what's your current view on this?  Can we win you around to the idea that something could be added or changed in this area?

Small reminder:

HTTP/2 RFC7540

Just as in HTTP/1.x, header field names are strings of ASCII
characters that are compared in a case-insensitive fashion.  However,
header field names MUST be converted to lowercase prior to their
encoding in HTTP/2.

HTTP/1.0 RFC2616

Each header field consists of a name followed by a colon (":") and the field value. Field names  are case-insensitive.

Golang related issue: https://github.com/golang/go/issues/5022

Field names are case-insensitive.

Yes, but unfortunately, this is one of those areas where "what the standard says" and "what many people have come to rely on" are not the same thing.

Bump for updates. We can no longer use Traefik due to this issue.

Feel free to also comment here: https://github.com/golang/go/issues/37834
It's a thread to ask the Go team to either offer a workaround, or accept one developed by the community. Hopefully, if there's enough activity on that thread, they'll say yes ;-)
FYI, I'm not aware of any other viable answer to the problem.

how fix this issue??

Modify a strings case for no reason is a bad habit. it not only breaks other peoples stuff, it also wastes resources.

+1
Please do not modify headers that pass through a proxy - this is bad-by-design. It should be transparent.

one more use case for you for not doing unnecessary conversions: old Outlook clients can't talk to Exchange via Traefik; only newest ones work

Thanks for everyone that participated in the discussions so far.  As of now, we are not going to change the way we handle headers. We've put a lot of thought into it and here is why we think this is the right decision:

We think the use-case for case-sensitive headers is going to become increasingly smaller as headers must be case-insensitive according to the HTTP RFC and HTTP/2 doesn't change that.  Therefore, trying to adapt to this will bring more code complexity. Even worse, we believe that changing has the potential to introduce unintended behavior that would affect most/all users.
Traefik Proxy was never meant to be a transparent proxy, and we have no plans of becoming one, so that solution is not going to work for us as well.
If Go chooses to provide a solution in the future, we should follow it, but we just don't think they will do so.
Finally, an App that uses case-sensitive has always been broken and needs to be fixed sooner or later.

We are going to close this issue, should Go provide a solution, please feel free to open a new one.


How to communicate with Java/AAR

I'm working on an Android project with StereoKit, is there any sample about how to communicate with Android AAR? Thanks.

This is less of a StereoKit thing, and more of a C# general thing! C# does have some pretty great tools for working with Java projects, like auto-generating C# idiomatic bindings for JARs and AARs, compiling .java files in your C# project, and pretty much the full Android API accessible as part of the standard API for Android targeted projects.
Because of the native support for the Android APIs, I've seen a few projects just dropping their Android .aar/.jar plugins because they could just be done natively in C#.
StereoKit's multi template for C# does have some additional tools for Android, but this is mostly just the fact that any code files ending with .Android.cs will only be compiled when building the Android target. This can be useful, but I still pretty frequently use #if ANDROID in normal files for Android specific stuff.


How to use our own images?

Hi! This is really nice work!
I want to know whether I can use my own images without annotations. If Yes, how to change the config file?
Thank you so much

您的来信我已经收到了！


move rclone config to environment variables

so config file can be edited
also fixes permissions of rclone config on launch (should be a temporary fix, as new users won't have a problem)
closes #109

Ok. I did not know this functionality with environment variables! It is pretty cool. So basically we should also use environment variables when adding new remotes or changing existing ones. Right?

env should always work, but config files are fine, too. But doing it this way (envs) means we can set up some default config without stepping on users' config files for anything that should be done for all users.


ListView builder reverse keep item index when list index not bottom

RT. like a chat message list.
how to keep old index item at reverse list. please help me.

Hi @rizzi37,
I am not sure if I understand your question, But it seems like you are asking for assistance with your code which is best suited for communities like StackOverflow or other channels/community like Reddit or discord, Please try asking your question there with sufficient info to get help.
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you.


Find best chunk_size for datacubes and hard-code it

It seems like it doesn't matter to time. Therefore, 3600 is chosen to gain a 100Mb/chunk size, which should met both requirements of beeing a generally good dask chunk size and a generally good zarr chunk size (See: "Chunks should be aligned with array storage on disk" in "Choosing good chunk sizes in Dask" by Genevieve Buckley).


Feature/docker

Merging back into master

I've added a Dockerfile as well as updated the readme. Also pushed this image to DockerHub


add Adafruit-MCP3008 and Adafruit-GPIO to rosdep/python.yaml

This adds two rosdep Python pip package definitions for Adafruit-MCP3008 and Adafruit-GPIO which I am using in my ROS project.
Tested as suggested in the contribution guidelines:
ros-dev:sources.list.d $ rosdep update
reading in sources list data from /etc/ros/rosdep/sources.list.d
Hit https://raw.githubusercontent.com/ros/rosdistro/master/rosdep/osx-homebrew.yaml
Hit https://raw.githubusercontent.com/ros/rosdistro/master/rosdep/base.yaml
Hit https://raw.githubusercontent.com/ros/rosdistro/master/rosdep/python.yaml
Hit https://raw.githubusercontent.com/ros/rosdistro/master/rosdep/ruby.yaml
Hit https://raw.githubusercontent.com/ros/rosdistro/master/releases/fuerte.yaml
Hit https://raw.githubusercontent.com/sheaffej/rosdistro/master/rosdep/python.yaml
Query rosdistro index https://raw.githubusercontent.com/ros/rosdistro/master/index.yaml
Add distro "groovy"
Add distro "hydro"
Add distro "indigo"
Add distro "jade"
Add distro "kinetic"
Add distro "lunar"
Add distro "melodic"
updated cache in /home/jsheaffe/.ros/rosdep/sources.cache

ros-dev:sources.list.d $ rosdep resolve adafruit-mcp3008-pip
#pip
Adafruit-MCP3008

ros-dev:sources.list.d $ rosdep resolve adafruit-gpio-pip
#pip
Adafruit-GPIO

ros-dev:sources.list.d $ pip search Adafruit | grep MCP3008
Adafruit-MCP3008 (1.0.2)                   - Python code to use the MCP3008 analog to digital converter with a Raspberry Pi or BeagleBone black.

ros-dev:sources.list.d $ pip search Adafruit | grep GPIO
Adafruit-GPIO (1.0.3)                      - Library to provide a cross-platform GPIO interface on the Raspberry Pi and Beaglebone Black using the RPi.GPIO and Adafruit_BBIO libraries.

ros-dev:sources.list.d $

Adafruit-GPIO: https://pypi.python.org/pypi/Adafruit-GPIO
Adafruit-MCP3008: https://pypi.python.org/pypi/Adafruit-MCP3008


Test failure: TestSetGauge (unreliable test?)

--- FAIL: TestSetGauge (0.00s)
        circonus_test.go:94: Expected '{"foo`bar":{"_type":"n","_value":1}}', got '{"foo`bar":{"_type":"n","_value":"1"}}'

ping?

pong


Player update's position not respecting time scale DSP

Description

Some time ago, Lavalink/Lavaplayer added support for DSP with various embedded effects. When applying the time scale filter, users can control the audio's speed/rate (and pitch, but irrelevant to this issue). For library developers, it is now hard to compute the actual position of the track. The player update that is regularly sent to the client only contains a timestamp (used to synchronize) and a relative position that both represent an actual position in the track. This position is not the actual position on the track. When using the time scale filter, for example, at speed 3, the position given in the player update follows the regular time flow, as no time scale had been applied.
Here is also a dump that shows the position property does not represent the actual position in the track:
Position, Time, Delta Position, Delta Position / Time passed since last update
0,<PHONE_NUMBER>790, 0, 0
4820,<PHONE_NUMBER>638, 4820, 0.964
9820,<PHONE_NUMBER>631, 5000, 1
14800,<PHONE_NUMBER>627, 4980, 0.996
19800,<PHONE_NUMBER>629, 5000, 1
change time scale to 4x
24800,<PHONE_NUMBER>632, 5000, 1
29800,<PHONE_NUMBER>629, 5000, 1
34820,<PHONE_NUMBER>629, 5020, 1.004
39800,<PHONE_NUMBER>635, 4980, 0.996
44800,<PHONE_NUMBER>637, 5000, 1
change time scale to 2x
49820,<PHONE_NUMBER>639, 5020, 1.004
54820,<PHONE_NUMBER>636, 5000, 1

Proposal
I propose adding a property to the player state that represents the current position in the track measured by the samples played. This would help in effectively providing the exact position in the track while respecting the time scale filter. From the client side's view, the track ends prematurely without being even played entirely, as the player update provides the information that the track has started while it is at its end.
An example payload as proposed could like the following:
{
    "op": "playerUpdate",
    "state": {
        "connected": false,
        "position": 5000,
        "actual_position": 14900, // new
        "time":<PHONE_NUMBER>331
    },
    "guildId": "redacted"
}

The actual position would represent the samples (converted to milliseconds) played.
The problem for the client is that it would have to keep track of all time scale changes in the track. For example, suppose the user changes the time scale while playing the track. In that case, the tile where the time scale effect is active must be included in the computation and could lead to slight errors while calculating the position in the track (due to network latencies, time used to apply the filter, jitter while playing the track, audio buffering, ...).
Alternatives considered
An alternative would be to either compute the position of the track fully at the client-side.
Another alternative would be to change the position property currently sent with every player update to respect the time scale. But this could be counted as a breaking change, as present clients might not expect this new behavior; the benefit would be that library authors do not have to do very much (except scaling the time passed between the current time and the timestamp).
Version info
Client used:
Lavalink4NET (currently unreleased version, https://github.com/angelobreuer/Lavalink4NET). Reproducible with a native WebSocket connection.

Output of java -version:
java -version
openjdk version "11.0.10" 2021-01-19
OpenJDK Runtime Environment Microsoft-18724 (build 11.0.10+9)
OpenJDK 64-Bit Server VM Microsoft-18724 (build 11.0.10+9, mixed mode)

Version:        8759d12f31f8e6b31f4c42eb7a379da68a7d3187-SNAPSHOT
Build:          1245
Build time:     19.09.2021 10:20:18 UTC
Branch          dev
Commit:         8759d12
Commit time:    19.09.2021 10:17:16 UTC
JVM:            11.0.10
Lavaplayer      1.3.94-original

(Latest CI build)

Thank you very much.

The issue is actually in lavaplayer itself, since it's filter framework isn't really meant to change speed of songs. One of solutions would be adding "speed modifier" property to filters and modifying lp's playback time tracking code to take care of that information.

The issue is actually in lavaplayer itself, since it's filter framework isn't really meant to change speed of songs. One of solutions would be adding "speed modifier" property to filters and modifying lp's playback time tracking code to take care of that information.

Thank you @alula, I transferred the issue to https://github.com/sedmelluq/lavaplayer/issues/667 and added your comment as additional information in the issue.


Add JsonToJsonl task

Feature description
Too many users have to tediously do:
  - id: json_to_ion
    type: io.kestra.plugin.serdes.json.JsonToIon
    from: "{{ outputs.download.uri }}"
    newLine: false # regular json

  - id: ion_to_jsonl
    type: io.kestra.plugin.serdes.json.IonToJson
    from: "{{ outputs.json_to_ion.uri }}"
    newLine: true # JSON-L

Only to get from the JSON returned by a typical API to a JSON-L format needed by kestra tasks such as e.g. ForEachItem.
We should instead allow (without going through Ion):
  - id: json_to_jsonl
    type: io.kestra.plugin.serdes.json.JsonToJsonl
    from: "{{ outputs.download.uri }}"

full example showing the current state:
id: iterate_over_json
namespace: company.team

tasks:
  - id: download
    type: io.kestra.plugin.core.http.Download
    uri: "https://api.restful-api.dev/objects"
    contentType: application/json
    method: GET
    failOnEmptyResponse: true
    timeout: PT15S

  - id: json_to_ion
    type: io.kestra.plugin.serdes.json.JsonToIon
    from: "{{ outputs.download.uri }}"
    newLine: false # regular json

  - id: ion_to_jsonl
    type: io.kestra.plugin.serdes.json.IonToJson
    from: "{{ outputs.json_to_ion.uri }}"
    newLine: true # JSON-L

  - id: for_each_item
    type: io.kestra.plugin.core.flow.ForEachItem
    items: "{{ outputs.ion_to_jsonl.uri }}"
    batch:
      rows: 1
    namespace: company.team
    flowId: mysubflow
    wait: true
    transmitFailed: true
    inputs:
      json: "{{ json(read(taskrun.items)) }}"

i would like to work on this issue.

@sanskar9415 any progress on this? :) unassigned for now but if you're working on this, let us know


[core] add possibility to bypass Tag tracking using cookie

@WnP, @chaahk would either of you explain what this does and why?

Hi @Lacrymology,
Oops, sorry, I forgot about the description.
Use case
in some country -like mine- users should be able to disable tracking like Google Tag Manager do.
Solution
with this PR we can set a custom cookie, which when present disable tracking by making this tag returning nothing.
As you can see in the diff this cookie is named GOOGLE_TAG_BYPASS_COOKIE, which I guess is a quite explicite name, let me know your through.
Indeed this cookie should be set using javascript or whatever else.
That's all, hope you'll like it.

Hi @Lacrymology
Any chance to have a released version with this feature?
Thanks

@WnP sorry, I missed your message. I'll release it in a while (few hours)

released


fixed partial_dependence to match prior

Hello @jonathanng! Thanks for submitting the PR.

In the file skopt/plots.py, following are the PEP8 issues :

Line 232:80: E501 line too long (88 > 79 characters)

I think I know what you are doing here, but for people from the future who are digging through old PRs could you write a few sentences in the top comment of this PR explaining what the problem is, how you are fixing it and why ;)

@betatim, just updated it!


Remove unquoted and unused code item.

Please, remove "$x.ServicePrincipalnames.Add("https://eas.contoso.com/")"

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

ID: 0ba93584-38f6-bc46-9784-c14492de8f75
Version Independent ID: 2fc5806d-bc80-9cc4-a051-1a9f16322c8d
Content: How to configure Exchange Server on-premises to use Hybrid Modern Authentication
Content Source: Enterprise/configure-exchange-server-for-hybrid-modern-authentication.md
Service: o365-administration
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

@officedocsbot assign @mypil

@hmizael - Thank you for submitting feedback.
I will get this issue over to the Enterprise writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@hmizael Thank you for the feedback. We have made the necessary  changes to the document on PR https://github.com/MicrosoftDocs/OfficeDocs-Enterprise/pull/416


Not found on Maven Central

Describe the bug
Plugin is not available on Maven Central since 0.6.1
Expected behavior
Plugin latest available version on Maven Central should be 0.7.4
https://mvnrepository.com/artifact/org.jetbrains.kotlinx/kover
https://central.sonatype.com/artifact/org.jetbrains.kotlinx/kover
Also, I was able to find it here:
https://mvnrepository.com/artifact/org.jetbrains.kotlinx.kover/org.jetbrains.kotlinx.kover.gradle.plugin/0.7.5
However, as you can read, this is not org.jetbrains.kotlinx:kover but org.jetbrains.kotlinx.kover:org.jetbrains.kotlinx.kover.gradle.plugin.
So or the readme is wrong, or it is uploaded to the wrong place.
I hope this helps.

Hi,
version 0.7.5 already published to Maven Central, however, the artifact name has been changed.
You may find the latest versions by the corresponding name https://mvnrepository.com/artifact/org.jetbrains.kotlinx/kover-gradle-plugin
Please note that if you are using the legacy plugin application, then in apply you should specify org.jetbrains.kotlinx.kover

Thank you for the update.
Finally, I did not find the good settings.
Don't know what I copy-pasted. It is working now.
Once again, thank you.


Increase NW batch size .5Mb -> 5Mb

tested in private-testnet

@akichidis we have been running with this change for the last few days in private-testnet but it has been with other changes as well as with different load. This is the most recent run of 50/50 shared owned object load.
https://metrics.sui.io/d/HH91BjB4z/throughput?orgId=1&from=1681803947722&to=1681840875618

Thanks @arun-koshy  for the metrics! On that test run - which by the way is the most stable one I've ever seen - I can see that the batch size on p99 is ~50KB , basically the current limit. I am not against increasing to 5MB but it looks like we are currently no near this limit. But let's increase this, yes.


chore: rename external_tx to rpc tx

This change is

#607  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArniStarkware and the rest of your teammates on  Graphite


Strip spaces

Hello, hope your day is going well! I think this fixes the issue with leading and trailing spaces in an email before hashing. I elected not to modify the stored email address, just to strip the spaces when hashing. If modifying the email value is desired let me know and I can implement that.

Got it, thanks for the feedback! I'll work on this tonight or tomorrow and resubmit
On Tue, Oct 1, 2019 at 17:39, Benjamin<EMAIL_ADDRESS>wrote:
@bsoyka requested changes on this pull request.
This looks good so far, but needs a couple changes.

See the suggestion I left on line 41.
The email also needs to be lowercased, as per the issue this PR is based on.
Please make a quick test case or two for this in /tests/test_main.py.

Thanks for your help so far!
In gravify/init.py:

@@ -38,7 +38,7 @@ def unsecure_url(self):

 @property
 def hash(self):

   return hashlib.md5(self.email.encode("utf-8").lower()).hexdigest()

   return hashlib.md5(self.email.strip(" ").encode("utf-8").lower()).hexdigest()

⬇ Suggested change-        return hashlib.md5(self.email.strip(" ").encode("utf-8").lower()).hexdigest()

   return hashlib.md5(self.email.strip().encode("utf-8").lower()).hexdigest()

I think you should remove this string because without it, any whitespace characters will be removed. With it there however, only spaces are removed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@fleetcaptain - Sounds good, and thanks again for your help!

I believe I have updated my branch, if something looks off please let me know as I'm still getting learning my way around git... I removed the space character from the strip() call and implemented a few test cases.
It appears in the code that .lower() is already being called on the email address before it is hashed? The test case with upper case email yields the same hash as the test case with all lower case. Or should the email address be converted to lowercase before being stored? I could implement that if desired.

Yep, you already had called .lower(), sorry about that.
Now that you mention it, it may be better to have the email changed before it is stored, that way the email validation checks the edited email. Let me know your thoughts on this one.

I changed things so lower() and strip() are called on the email argument before getting stored.

No problem, this was my first pull request to a third party repository, thank you for your patience and working with me!

Of course! I’m glad to hear you’re contributing to other projects!
If you’re not participating already, I recommend you take part in this year’s Hacktoberfest.
And finally, I’d love it if you’d like to work on other “help wanted” issues on this project. I’m very happy to work with you, but if you’d like to simply move on, that’s your call. 😄

Thanks, I'll take a look at the open issues and see what I can do!


Fix the rackhd micro docker image by using debian/wheezy

docker image is 200MB
the compressed image discovery.docker.tar.xz is 49.1 MB

@rolandpoulter @zyoung51 @RackHD/corecommitters @tannoa2 @geoff-reid @uppalk1

:+1:


Add gorgeous looking landing page for libhal

The landing page will be accessible via the url libhal.github.io

I think its good enough for now 😄


Undocumented property for resources.containers.container definition

Hi,
It's possible to specify a command for a container resource definition, but this property is not documented.
Example azure-pipelines.yml
resources:
  containers:
    - container: minio
      image: minio/minio:latest
      command: server /data # <-- hidden feature?

services:
  minio: minio

It's unclear if this was a documentation oversight, or if this is the wrong way to pass additional arguments to the ENTRYPOINT of the container image.

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

ID: b1940f8b-486e-f1de-3447-125be89dae7c
Version Independent ID: b1940f8b-486e-f1de-3447-125be89dae7c
Content: resources.containers.container definition
Content Source: content/resources-containers-container.md
Service: azure-devops-pipelines
GitHub Login: @steved0x
Microsoft Alias: sdanie

@sliekens -- Steven, thank you for your feedback.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@steved0x -- Steve, please look into this issue.

Thank you for your feedback. We are migrating this issue to our new documentation feedback system, and closing this issue.
As we phase out GitHub issues for this repository, we encourage you to propose changes using public PRs or submit feedback through our updated control at the top of the page. For more information on our feedback systems, see https://aka.ms/contentuserfeedback.


WPFDXInterop with DirectX 12 (it throws exception on m_d3dDevice11on12->OpenSharedResource call

## I can use DirectX 12 to write to a D3D11Image on a number of Windows 10 machines using a D3D11On12CreateDevice dx11 device.
{
..
IUnknown pUnk = (IUnknown)pResource;
IDXGIResource * pDXGIResource;
ID3D11Resource * pDx11Resource;
HANDLE sharedHandle;
ID3D11Resource * tempResource11;
ID3D11Texture2D * pOutputResource;
D3D11_RENDER_TARGET_VIEW_DESC rtDesc;
rtDesc.Format = DXGI_FORMAT_B8G8R8A8_UNORM;
rtDesc.ViewDimension = D3D11_RTV_DIMENSION_TEXTURE2D;
rtDesc.Texture2D.MipSlice = 0;
pUnk->QueryInterface(__uuidof(IDXGIResource), (void**)&pDXGIResource);
pUnk->QueryInterface(__uuidof(ID3D11Resource), (void**)&pDx11Resource);
pDXGIResource->GetSharedHandle(&sharedHandle)))return hr;
m_d3dDevice11on12->OpenSharedResource(sharedHandle, __uuidof(ID3D11Resource), (void**)(&tempResource11));
tempResource11->QueryInterface(__uuidof(ID3D11Texture2D), (void**)(&pOutputResource));
m_d3dDevice11on12->CreateRenderTargetView(pOutputResource, &rtDesc, &m_pRenderTargetView1);
pDXGIResource->Release();
tempResource11->Release();
...
}
## However as soon as I enable debugging
ComPtr debugController;
D3D12GetDebugInterface(IID_PPV_ARGS(debugController.GetAddressOf());
debugController->EnableDebugLayer();
I get the following error (exception thrown)
D3D12 ERROR: ID3D12CompatibilityDevice::ReflectSharedProperties: Resource provided was not shared by D3D11, or with a D3D11 desc. [ MISCELLANEOUS ERROR #916: REFLECTSHAREDPROPERTIES_INVALIDOBJECT]
Anyone have any idea what I am doing wrong?

| suspect D2DImage11 is not being created as SHARED

Not Closed:
WPFDXInterop should be creating D2DImage11 as a shared resource
So I can open it with OpenSharedResource

Seeing the same problem in another context (and there seem to be no other mention of it)

D3D12 ERROR: ID3D12CompatibilityDevice::ReflectSharedProperties: Resource provided was not shared by D3D11, or with a D3D11 desc. [ MISCELLANEOUS ERROR #916: REFLECTSHAREDPROPERTIES_INVALIDOBJECT]

It looks like it's okay to create 11on12's texture and share it to 12. However if I try to share in the opposite direction the problem pops up...


enable wildcards in ignores

apparently console-consumer add a random number to the consumer-group they use, causing them to show up.
Would be nice to be able to just put "console-consumer-*" as one of the ignores.

Fixed in 0.0.4


Add more logs in the global function onTerraformChangeInPR

Resolves # . (This is applicable only if this pull request relates to a GitHub issue, delete the line otherwise)
Dtspo-18680
Notes:

This is not a breaking change
Add more logs to help future investigation on issues on actions triggered upon terraform change in PRs

this PR has been superseded by this one https://github.com/hmcts/cnp-jenkins-library/pull/1264


add test files for multiply1 and multiply array

 Write example test files for tutorial 3 examples to use as a basis for getting users to write them in addition to the examples themselves.


this can support socket.io?

this can support socket.io?

Yes. Please, read the first paragraph of the readme.


S3 Examples for Bucket Policy are Broken

https://awslabs.github.io/aws-cdk/refs/_aws-cdk_aws-s3.html#bucket-policy'
// broken because addToPolicy doesn't exist
const bucket = new Bucket(this, 'MyBucket');
bucket.addToPolicy(statement);

// broken because policy can't be added to bucket props
const policy = new BucketPolicy(this, 'MyBucketPolicy');
const bucket = new Bucket(this, 'MyBucket', { policy });

It looks like the grant family of methods on the bucket are the only way to achieve bucket policies currently, though I'm not sure on that front. Regardless, the examples are broken in 0.12

The example is definitely broken, but the usages are there (just moved around a little):

addToPolicy is now called addToResourcePolicy
You can use BucketPolicy but you'll have to specify the bucket when you define the policy (and not vice versa): new BucketPolicy(this, 'MyPolicy', { bucket })

Nobody should be using BucketPolicy, so we shouldn't even have an example that mentions it.


Fix OpenVZ

Hey,
you should add a fix for openvz vps server.
Openvpn worked after i commented out the line "LimitNPROC" in
<EMAIL_ADDRESS>Greetings

\o/

Fixed : https://github.com/Angristan/OpenVPN-install/commit/c703d41795def39b81b4a4ec6b2bda3f144b2a0a


[epic] v0.2 Development of the Client SDK

⚠  WIP ⚠
v0.1 was about basic "push" functionality which is done.
v0.2 is about refactoring this to be more elegant and support for reading/writing metadata
API design should follow https://github.com/datopian/ckan-client-guide (but tweaked for js/nodejs idioms)
https://github.com/okfn/ckan.js could probably be reused for the plumbing layer.
API design
Uses data.js and its Frictionless (f11s) data interface as the way to access files and datasets. Metadata follows Frictionless formats.
Plumbing
This is relatively raw low level APIs.

Storage: getting authz, interaction with giftless etc
MetaStore: CKAN action API v3

Porcelain
# create a dataset (or overwrite existing one) - just puts metadata (file uploading is separate)
client.put(f11s_dataset)

# convenience function
# dataset must exist
client.put_resource(f11s_resource, dataset=dataset_name)

# longform
resource = open(path) # f11s.Resource object
client.put_blob(resource)
dataset = client.get(dataset_name)
dataset.add_resource(resource)
client.put(dataset)

# bonus
client.put_resource_metadata(...) # only save resource metadata ...

## GET functions
client.get(dataset_id_or_name)
client.get_resource(resource_id OR dataset_id + resource_name)

[ ] refactoring this to be more elegant

I created this issue to do it: https://github.com/datopian/ckan-client-js/issues/5

[x] Created tests and the action function to be able to create_resource and update_resource.

In Progress:

Refactoring Datapub to use the create_resource and update_resource
Estimate: 0.5day


build: add lint rule for lightweight tokens pattern

Adds a lint that warns if a DI constructor is discovered for which
parameters optionally inject classes without using the lightweight
token pattern. The rule intends to help with optimized source code
that works well for tree shakers. Read more about this here:
https://next.angular.io/guide/lightweight-injection-tokens.

Blocked on #19912 that fixes that CdkHandle is always retained.

@crisbeto Yeah, that's one of the concerns I had too, but given the amount of DI parameters we have in the repo, the amount of manual excludes seems extremely low. The assumption that injection w/ @Optional needs a lightweight token is indeed not 100% correct, but in most of the cases it helps unveiling issues and being strict seems better than potentially not worrying about it (as seen with 2335571cc1f382c01bdd336d5b529196f4503d67).
The same applies to the existing lint rule we have for OnPush. There are just always exceptions but it's better to be strict in the most cases and opt-out in a few uncommon places.

I see what you mean, but I still think that it's something we should be catching either during code review, or through the size checks.

@crisbeto Obviously code reviews do not always work out. The same could apply for other stuff we have lint rules too 😛 I don't feel too strongly about it, but this rule would have already prevented the recent drag-drop regression, and would have also helped with all of the other lightweight token changes I landed before. The downside of rarely having to add the lint exclude (which we also need to do for OnPush) seems acceptable for improved code health and optimized package output IMO.

cc. @jelbourn for thoughts on this.

I think the difference between this and OnPush is that with OnPush we actually want it to be on everywhere, but we were forced to turn it off in some places due to framework limitations. On the other hand, with Optional we have some totally valid use cases that shouldn't be lightweight tokens.

Yeah, but given that we have 27 places where the OnPush rule is being explicitly disabled, the assumption that we would want to have OnPush everywhere is probably not correct either anymore. Still we'd want to keep it, but only opt-out when needed.
The lightweight token rule on the other hand only requires 8 excludes but follows the same semantics. i.e. being correct in the most times, but requiring manual opt-out in some situations. The reason for the opt-out strategies might be different, but the pattern is the same (and even more correct with this rule).

Yeah, this is a tricky one. My reaction is to try it out for a while and see if it gets annoying. Rather than using tslint:disable-next-line, though, what if we just had a list of allowed symbol named in the rule? It's less local, but it keeps the code tidier.

@jelbourn Yeah, we could try it. Alternatively if you all don't feel too happy about it (surely I'm not 100% convinced either), we could just make it a standalone script that we can occasionally run. Though I doubt we'll run it consistently very often.
I think we actually would want to keep the disabling local as a symbol needs a lightweight token depending on where it's injected. Excluding specific symbols globally (like we already do for Directionality) has the risk of hiding real tree-shake optimization issues.

Added to team meeting agenda

Rebased and lint is green now. @crisbeto @jelbourn could you have another look?

@crisbeto Addressed your feedback. Thanks! please have another look when you get a chance


Unable to import csv file containing github usernames of contributers who have already signed the CLA.

Tried in Mozilla Firefox Developer Edition 51.0a2 (2016-10-17) (64-bit).

Also correct the spelling "contributer" to "contributor" in the front-end of the software.

Does it work for you in another browser?

I am also unable to import the csv file on Mozilla Firefox 51 on MacOS (I also tried on Chrome and Opera). The 'Import' button is not clickable. Do I have to unlink the csv file that was previously imported?

In Chrome 57, Firefox 52, Safari 10.1 and IE 11 import feature is working as expected


feat: tx send

Add tx send subcommand to send transaction to the network

@leighmcculloch This is ready when the tx sign is merged.


[BUG] Power Lora Loader Strength value numeric entry popup doesn't work after sliding the value

Power Lora Loader Strength value numeric entry popup doesn't work after sliding the value. Clicking on the Strength numeric value after a newly added Lora slot correctly brings up the numeric entry popup, however after clicking and dragging to slide the value, clicking the value again no longer able to bring up the numeric entry popup.
Steps to reproduce the issue :

add Power Lora Loader node, add a Lora to the list.
click on the Strength value, a numeric entry popup will appear, working as expected.
now click drag on the Strength value to slide and change the value.
click the Strength value again, it doesn't bring up the numeric entry popup anymore.

A temporarily solution to this UI bug is to reload the browser.
Hope you can look into it. Thanks!

Thanks for fixing it. I have just tested it, the click for numeric entry popup now works, but click drag to slide the values no longer work.

Ah, there was an issue introduced where dragging was only changing the second value (and if there was only one, not changing either). I've restored and fixed this. Should be fixed and restored.


lines3d 不支持动态加入数据

如果加入新的数据，然后 setOption， 所有动画会重新开始

Hi, Please provide a detailed demo address for us to debug, and please describe the problem and the results you want, which I can't see from the picture. Thank you

http://gallery.echartsjs.com/editor.html?c=xYcH2bHnFW&v=2
I want to every received data point animated once only, cause my backend will push these different data points by random time interval and there are no way for supplying a whole bunch of data to echarts to render. Apparently echarts cannot understand my opinion and just cancel the ongoing animation and restart, that's not a smart way I think and it's not a bug yet, but I hope you consider this situation and provide an official solution, many thanks.

@Ovilia Can you help?

请问问题解决了吗？我最近也想使用动态加载的lines3D，但是数据一更新原来画的线就没有了。

解决了吗？


"Shifted" $this->page between isLastPage() and applyPagination() in pagination

I'm not sure here, maybe it's on purpose or maybe I cannot debug it properly.
I connect with an API which returns 'total_pages', so the most natural way was to implement this in the isLastPage() method:
return $this->page === (int)($response->json('total_pages'));

Unfortunately, it seems it does not reach the final page. I made tests with simple result set, I have 3 items, I get 1 item per page so in total I have 3 pages. So I should iterate three times, but unfortunately I get two results.
When I put debugging messages in both methods like this:
echo ("page in isLastPage: {$this->page}\n");
echo ("page in applyPagination: {$this->page}\n");
I get:

page in applyPagination: 1
now happens the request call with &per_page=1&page=1
page in isLastPage: 2

So you can see that page number in isLastPage() already jumped to 2. So with second iteration it will have the page number of 3 which equals to 'total_pages' and we never reach the last third iteration.

Yes, please check it, as IMHO there is something unpredictable right now. Variable name maybe could be $this->currentPage?
I must admit I miss another variable like $this->currentResults (or $this->currentResultsCount) to get the number of results just for this page. I came across a cursor pagination which even on the last page still returns next cursor and I need to check at isLastPage() if the result set contains 0 items. At this moment I do it this way:
return count($this->getPageItems($response, $this->getOriginalRequest())) === 0;

and this would be for sure more convenient:
return $this->currentResults === 0;

Hey @boryn I have created a PR first which fixes v1 of the pagination plugin (I'll then PR the fix for v2) to cover both Saloon versions but you were right, the property was wrong.
Would you mind reviewing this PR please: https://github.com/saloonphp/pagination-plugin/pull/10
Thank you

I even was not aware there was the same problem with v1. Probably I was using a different logic then. Now I don't use it any more, so I cannot test it empirically, but the changes in the code look all right!

When I say v1, I mean Pagination v2 but for Saloon v2 + v3. Too many versions 🤣 Thank you for looking at it. I've just released an update for you!

So if it is for Pagination v2 and Saloon v3 as well, so I will give it some tests the following days.


Can not capture the IIS application pool bind failures

The same issue with the original FUSLOGVW.exe.

The solution is find the application pool, stop it first. Wait five seconds, then start the same application pool again.
Then refresh the page of the website which used this application pool.

Just in case someone else encounter the same problem.


Example Question: Does it make sense to iterate through a vector when each item is the same?

At Retrieve pointer properties it reads:

Note  The values returned by the properties discussed here are based on all detected pointer devices: Boolean properties return non-zero if at least one device supports a specific capability, and numeric properties return the maximum value exposed by any one pointer device.

So ...
IReadOnlyList<PointerDevice> pointerDevices = Windows.Devices.Input.PointerDevice.GetPointerDevices();

... returns a list of PointerDevice objects, and each and every object in this list returns the same information?
Why should it make sense to iterate through this list then, if each item in the iteration returns the same values?

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

ID: 1641e1a8-9b76-2985-3e12-49e20b79581d
Version Independent ID: 30097945-bda2-fd54-9f39-871a79aac7f8
Content: Identify input devices - UWP applications
Content Source: windows-apps-src/design/input/identify-input-devices.md
Product: uwp
Technology: design
GitHub Login: @Karl-Bridge-Microsoft
Microsoft Alias: kbridge

Hi @SetTrend - This is not documentation feedback, but rather a support request. Please contact the following support channel.

For community assistance with development questions, there are our forums, and StackOverflow.

Here are all of the developer support resources, for completeness.
-The Feedback Hub app is the right place to lobby for a change, a feature, or to make a suggestion.

For developer issues, or suspected product bugs, contact Microsoft Support. A support engineer will look at the repro and see whether the issue is a Microsoft product bug, or something else. If it's a bug with a Microsoft product then the ticket will be free of charge.
The Windows developer support site explains other support options.
The Visual Studio developer community is available, and responsive. The former User Voice feature is now there on the Suggest a feature tab, if you want to lobby for a change, a feature, or to make a suggestion.

If you receive assistance via any support resource, and you believe that information would help us improve the docs, then please come back and let us know.


PO file conversion does not work on FreeBSD

Problem: PO file conversion does not work on FreeBSD.
Solution: In the file conversion rules, change the $< variable to $?.
This is a portable variable, unrecognizable by both GNU make and BSD make.
FreeBSD:

.OODATE   The  list  of  sources for this target that were deemed out-of-date; also known as `?'.

GNU:

$? The names of all the prerequisites that are newer than the target, with spaces between them. If the target does not exist, all prerequisites will be included.

Hm, semantically, it makes a difference if we explicitly want to use only the first pre-requisite or all pre-requisites. On a quick check however, it doesn't seem to matter, as we always seem to have only a single pre-requisite, so the change should be fine. However, I noticed in the documentation of BSD Make this section:

The shorter forms (>, !, <, %, ?, *, and @) are permitted for backward compatibility with historical makefiles and	 legacy POSIX make and are not recommended.

which seem to indicate that using $< should be okay?  so when did you notice this issue? Also we seem to have the $< in a few more Makefiles. So I would think those are also affected.

@chrisbra
Christian, yes for these rules it would be acceptable as there is only one dependency and it should supposedly always be newer than the target file.

which seem to indicate that using $< should be okay?

This variable is used in *BSD for inference rules.

.IMPSRC   In suffix-transformation	rules, the  name/path  of  the
source  from which the target is	to be transformed (the
"implied" source); also known as	`<'.  It  is  not  de-
fined in	explicit rules.

Also we seem to have the $< in a few more Makefiles.

Yes, there are a few more make files, particularly in the "doc" directory. I'm planning to fix it next week. That is, if everything goes well.

okay, then let me merge this one. Thanks!


Training collapses after  72k iterations

I am currently trying to test the model on the maps data set (satellite and card images from google maps). The training went well until the 72k iteration. Afterward, the training collapses or rather no result is returned:
Images for A2B until 72000 iterations:

Images for A2B after 72000 iterations:

Images for B2A until 72000 iterations:

Images for B2A until 72000 iterations:

Any idea what could have happened here?

I am currently trying to test the model on the maps data set (satellite and card images from google maps). The training went well until the 72k iteration. Afterward, the training collapses or rather no result is returned:
Images for A2B until 72000 iterations:
Images for A2B after 72000 iterations:
Images for B2A until 72000 iterations:
Images for B2A until 72000 iterations:
Any idea what could have happened here?

Hello, I have the same problem. How did you solve it?

It may be that there is a phenomenon of model collapse. The training schedule can be adjusted according to the dataset, and the checkpoint can be saved for manual inspection or FID for model selection.


Pylyzer requires Erg as dependency, suggest adding this info

Description
Pylyzer needs Erg to function correctly, whose path to its binary and libs should be stored in env var $ERG_PATH:

[ERROR][2024-11-11 12:08:17] .../vim/lsp/rpc.lua:770	"rpc"	"/home/fishbone/.local/share/nvim/mason/bin/pylyzer"	"stderr"	"\27[91m[ERR] ERG_PATH not found\27[m\n"

I can resolve this by extracting tar from Erg release to my ~/.erg, then export env var by vim.fn.setenv("ERG_PATH", vim.fn.getenv("HOME") .. "/.erg")
However no such info regarding Erg is provided in lspconfig-all. I suggest adding this, or even better, adding erg_path option to let lspconfig handle exporting the env var automatically.

File structure of Erg tarball:
tar -tf erg-x86_64-unknown-linux-gnu.tar.gz                                                                                exit 1  47s
erg
lib/
lib/std/
lib/std/exception.er
...and more lib/ files

send a pr


Improve java makers

I improved the javac maker to support multiline errors, thanks to Syntastic. I also added a checkstyle maker.

hi @paretje
can you see #375

I'm sorry for the late reply @wsdjeg. If you cherry pick the commit of the checkstyle maker, this PR could be come redundant.
As I see you made use of vim variables for ie the classpath, you might be interested in adding a variable for the checkstyle file. I didn't because you can do this by overriding the maker args.

thanks for supporting checkstyle,I will merged this PR to my fork branch.no need to close this PR.It will be auto merged.
I will finish it this weekend.

local merged


Version with maximum query size

Experimentally supported by willm/DiskBacking, kind of. Doesn't actually work up to max query size and doesn't actually save much memory it seems.


Release Notes (v0.20 Project)

Tasks

This will track and construct the overall release notes for this repo in relation to the v0.20 project

Will cover everything from the inception of this repo

Related Issues:

N/A

Concerto 0.81.* is published, used in Cicero 0.20, with release notes here: https://github.com/accordproject/concerto/releases/tag/v0.82.1
Can we close this?

Can i contribute?


CreateNewSolution.bat is missing a 3rdParty folder

It's not possible to run CreateNewSolution.bat since it checks to see if the folders 3rdparty, core and tools exist. And there is no 3rdParty folder in this repo.

Sorry.  I guess that script got a bit stale.  I'm looking to fix it right now and will push an update shortly.  If you're in a hurry, you need to edit CreateNewProject.py in Tools\Scripts.  Look for '3rdParty' and remove it from the list of expected folders.  In the mean time, I'm checking to see if the template project files are also out of date.

Files are updated.  (I also changed the Windows SDK version to 17134.)

Thanks for fixing that. I did find and fix the 3rdParty issue myself when testing. But unfortunately due to the requirement of DirectInput I am not able to use the project. I hope you guys find a fix for that in future. Thanks.

Ehm. Why you just didn't accept pull request #387 which was opened 15 days ago?


Fix a wrong judgment

update at CacheFilter.java
String key = StringUtils.toArgumentString(invocation.getArguments());
上述方式获取的key不为null   且cache=null的判断是冗余的
同时  添加了dubbo-filter部分的 @Override
初看dubbo代码    如无误  望采纳

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

忘了切账号  已经添加邮箱关联


Use the same command line parser as LPython and LFortran

fixes #35

The reference tests simply got renamed.

Ready.


Add HTTP API's for Policies List and Update

FEATURE REQUEST

Is there an open issue addressing this request? If it does, please add a "+1" reaction to the
existing issue, otherwise proceed to step 2.

Describe the feature you are requesting, as well as the possible use case(s) for it.
Add HTTP API's in Auth service for Listing and Updating Policies.

Indicate the importance of this feature to you (must-have, should-have, nice-to-have).
must-have

The API's are provisioned in these API endpoints:
channels/chanID/users/assign
channels/chanID/users/unassign
channels/chanID/groups/assign
channels/chanID/groups/unassign
channels/chanID/groups - listing
channels/chanID/users - listing

things/tid/share
things/tid/unshare
things/tid/users - listing

groups/gid/users/assign
groups/gid/users/unassign
groups/gid/users - listing
groups/gid/channels- listing

users/uid/channels- listing
users/uid/groups- listing
users/uid/things- listing


Cloudfront migration of ML weights and test files buckets

Deal with cloudfront migration of ML weights and test files buckets which may impact how we pull them down.
Still waiting on exact timelines.
Tagging @dotsdl @mikemhenry

Nothing seems to have changed here. Closing.


Raise PR for MBR registration hook

To nicely integrate with MB Relationships we need them to call a hook during the relationship registration.
Update WPGraphQL_MB_Relationships::register to register this hook when PR is accepted.

https://github.com/wpmetabox/mb-relationships/issues/31
waiting to see if this accepted

pr accepted


Add support for setting up SSO for Service Principal

Feature Request
Add support for setting up SSO for Service Principal.
Please add methods to configure azure AD SSO as mentioned in this document:
https://docs.microsoft.com/en-us/azure/active-directory/saas-apps/amazon-web-service-tutorial#configure-azure-ad-single-sign-on

@AmudhaPalani do you mean setting up SSO for an enterprise application?

No further response from the customer, closing this one.
Feel free to reopen or raise new issues if any other questions occur.


R4: When the authors state: "...and compute the mass of segmented cells by integrating the phase shift over the area of each cell and multiplying by the specific refractive increment." What is the "refractive increment"? I think the authors are trying to say that they have some baseline measure of the cell's real refractive index, but it isn't clear where that comes from.

We could get more detail on this and maybe cite a paper that comes up this number. Another option is to remove "and multiplying by the specific refractive increment" altogether.
This is more to do with the domain background, so I would be inclined to leave it as is, or remove that part of the sentence.


Different Error Message on DCCs issued in the future

Different Error Message between Android and IOS
For testdata I had created a dcc with the IssuedAt Header (in the Cose Message) in the future. When I am scanning that certificate the IOS Verifier App shows the Error "Certificate Isssuance Date is in the future". The Android App shows "Cryptographic signature invalid".
Expected behaviour
Both apps show the Error message "Certificate Isssuance Date is in the future".
Steps to reproduce the issue

This QR Code has an IssuedAt Value of 2021-08-30. The certificate should be scannable by the App for the Acceptance Environment.

Android without debug mode:

iOS without debug mode:

Android with debug mode:

iOS with debug mode:

@chrloch you can have a look on my provided screenshots of the actual apps. Seems that is not the same message, but the main outcoming is correct for both apps? Or what do you think?

certificate created on ACC Vaccination 2/2 on 30th of september 2021 (date is in the future)
Android shows that the certificate is valid:

iOS shows correct that the date is in the future:

Android:

iOS:

At the moment we have this different behaviour on Android and iOS. Both is correct, but different handling.
@chrloch @TKlauenberg Wie wollen wir damit umgehen?

Bei Android erfolgt die Ablehnung offenbar nur aufgrund der Business Rules. Wenn die nicht korrekt synchronisiert wurden oder ein Land ausgewählt wurde, das keine Regel auf das Impfdatum hat, dann wird es grün.
Meiner Meinung nach sollte Android ein Impfdatum in der Zukunft hart abweisen.

@chrloch ja stimmt da hast du Recht !!

@MykhailoNester @oleksandrsarapulovgl @Hendrik-Schmidt-Schierhorn-TSI according the comment of Christian, can you please have a look on this ticket.
On Android, the rejection is apparently only based on the business rules. If they were not synchronized correctly or a country was selected that has no rule on the vaccination date, it will turn to valid and green.
In my opinion, Android should hard reject a vaccination date in the future like iOS does.


Network Layer

Hi team
In your API service class you are just checking if response is type of particular type (model) thn only return else null... so how can you show user exact error that what happen .. Example in login, user can put wrong OTP so we have to show "Invalid OTP" or may be user enter expired OTP so we need to show "OTP expired"
But in your case we cant show this. Or may be i am wrong ... please suggest a way am using Vexana
`final response = await manager.send<LoginResponseModel, LoginResponseModel>(NetworkRoutes.LOGIN.rawValue,
parseModel: LoginResponseModel(), method: RequestType.POST, data: model);
if (response.data is LoginResponseModel) {
  return response.data;
} else {
  return null;
}`

hello @behlsoft, of course, you have many scenarios for error state so vexana give response.error options for u. For example you can use like this:

Other think: if you have a base error model for the backend side, you can add this json model in network request then you can access directly model form response.error.model:
response.error.model


compatiblity for 1.5.0

Description
Finish changes for compatibility with dbt core v1.4 & v1.5 as per dbts upgrade guide
Change for dbt 1.5 upgrade

include python 3.11 in test matrix
use standard library gethostname() method for anonymous telemetry
elegantly stub model contracts as per dbt instructions
move timestamp macros

Other

avoid compatibility issues by using compatible release operator as in #45

These changes look good to me, and have passed all tests on my machine.
Merging this pull request into main and tagging @codeforkjeff for the release cut and publish.
Let me know if there is anything else I can do on v1.5.0.
I am working on a dbt-1-6-0-compat branch that I will push today.
More work to be done for 1-7-0 if we decide to use the new Capabilities class for standardizing and simplifying feature support or stub-outs.

Tagged v1.5.0 and uploaded packages to PyPI.
Thanks @MatthewPocock for this work! Apologies for the very long delay in getting it merged.


Activate Card menu actions

Open editor
Delete using the /api/v2/resource-service/delete
Clone (API to be impleted)

@giohappy @afabiani  I did some test with the copy and it seems available but I'm getting always 400. Is there some more documentation on how to copy completely a resource with the new async endpoints?
I looked at this but I was not able to make it work:
https://github.com/GeoNode/geonode/blob/master/geonode/base/api/views.py#L951-L1007


now that the captable auto-fills the new_investor and shareholder roles, we need a way to selectively ignore one or two of them.

because, what if we do a high resolution fundraising; and we send an agreement to one party; and we later add another party; we don't want to regenerate the agreement for the first party.
we have a hack – set the role attribute "legalese status" to "skip exploding kissing" or whatever it is. but we need a different way to exclude an entity from a party role entirely.
if you want to pick this up please discuss this with me as you proceed, because there's a lot of subtlety here.

i think this was resolved by 62e7d90


feat: improved user creation validation

Signed-off-by: Jérémy Morel<EMAIL_ADDRESS>Description

Adds validation for username so that the actual creation doesn't fail with an IntegrityError
Adds validation on empty password before using the configured validators. This is necessary because the zxcvbn validator will raise a list index out of range exception on an empty password string. See https://github.com/dwolfhub/zxcvbn-python/issues/64
Decodes the url encoded kwargs.get('username') so that the username is properly handled (John Doe is a valid username).

How has this been tested?
Manually.
Checklist

[x] changelog was updated with notable changes
[ ] documentation was updated

looking good to me but it would be worse adding a few validation tests in backend/users/tests/test_user.py

By popular demand (wink wink @cboillet-dev @sergebouchut2), let there be tests!


Hi, Great work and Repo, any plans to support recognition model evaluation for long sentences?

SMTR is our proposed attention-based STR algorithm which achieves long text recognition using only short text training. Model details, long text benchmark and evaluation scripts will be released soon.

@Topdu
Thank you for the fast feedback
Any ideas how soon, and until then would it be hard to give me directions on how this issue is solved or handled?

SMTR is our proposed attention-based STR algorithm which achieves long text recognition using only short text training. Model details, long text benchmark and evaluation scripts will be released soon.
我这边训练SMTR，训练过程没有问题，模型eval时候就会报TypeError:'int' object is not subscriptable,
报错文件在ratio_dataset.py:188行
img_width = properties[0]
我也打印了这个properties，正常来说是个list，但是eval的时候有些会是一个int类型的数字

我这边训练SMTR，训练过程没有问题，模型eval时候就会报TypeError:'int' object is not subscriptable, 报错文件在ratio_dataset.py:188行 img_width = properties[0] 我也打印了这个properties，正常来说是个list，但是eval的时候有些会是一个int类型的数字

Please post the config file here, it will help us to get an accurate view of the problem.

我这边训练SMTR，训练过程没有问题，模型eval时候就会报TypeError:'int' object is not subscriptable, 报错文件在ratio_dataset.py:188行 img_width = properties[0] 我也打印了这个properties，正常来说是个list，但是eval的时候有些会是一个int类型的数字

Please post the config file here, it will help us to get an accurate view of the problem.

请问作者可以解释一下focalsvtr_smtr_long.yml改配置文件参数得含义吗？其中关于Encoder中的img_size、Train中的sampler的scales、Eval中transforms的对应的SliceResize的image_shape,这三个尺寸的含义吗？这三个是需要保持一致吗？

还有一个问题需要提一下，在训练过程中Eval的batch_size-per_card只能设置为1

请问作者可以解释一下focalsvtr_smtr_long.yml改配置文件参数得含义吗？其中关于Encoder中的img_size、Train中的sampler的scales、Eval中transforms的对应的SliceResize的image_shape,这三个尺寸的含义吗？这三个是需要保持一致吗？

The focalsvtr_smtr_long.yml configuration file is specifically for evaluating the long-text recognition metrics of SMTR, so the settings in the Train section are not applied. The Encoder's image size does not affect either inference or training. It is primarily used for calculating the model's computational complexity based on the input shape. In the Eval section, the sliceresize parameter with the image_shape set to (32, 128) is intended to specify the height (32) of the sub-images for SMTR's inference augmentation strategy. The width setting of 128 has no effect during evaluation, as it does not alter the processing behavior.

还有一个问题需要提一下，在训练过程中Eval的batch_size-per_card只能设置为1

During long-text inference, an inference augmentation strategy is applied, which prevents batch inference. As a result, the model only supports single-image inference.


[Typo/Mistranslation] 09-perform-data-analysis-in-power-bi-desktop.md

On Practice 2, task 1 step 3, the column and table names need to stay in English so it will align with the lab environment. It is very confusing for the learners to see the column/table names in Korean on the instruction and then have to match it with the corresponding English names on the lab environment.
Current:
재판매인
비즈니스 유형
재판매인
비즈니스 유형
판매
판매
판매
이익률
판매
수량
날짜
분기
To be:
Reseller
Business Type
Reseller
Business Type
Sales
Sales
Sales
Profit Margin
Sales
Quantity
Date
Quarter

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Fixed and uploaded


Adding simple e2e test using KIND

Describe the feature request
k8s has new infra KIND https://github.com/kubernetes-sigs/kind
It will allow to have faster and more reliable tests on Prow. We need to create simple KIND test for istio.
{{ Succinctly describe your request }}
New test infra to improve flakiness and speed of e2e tests
Describe alternatives you've considered
Did not look into other alternatives, but want to compare existing e2e tests infra vs KIND.

This is done, we have e2e simple kind and the integration tests are all using kind


Update the VJP for mean(alongAxis:) to be correct.

Only the dimensions of the axis involved should be divided by.

@swift-ci please test tensorflow


fix: exit with success boolean

This makes the exit code 1 when an error occurred in the underlying DBT invocation, and zero otherwise. It's particularly necessary for CI, where the current code will not surface errors
Summary by Sourcery
Fix the exit code behavior to return a non-zero code on DBT invocation errors, improving error detection in CI environments.
Bug Fixes:

Ensure the program exits with a non-zero code when an error occurs during the DBT invocation, allowing CI to correctly detect failures.

@alexmaras I've raised #37 with this change, as I've also added some integration tests and updated the project version (I don't have automated releases yet)

#37

Let me know if you're happy with this alternative, and I'll get it released (I'll merge it in if I don't hear anything, probs tomorrow)
Thanks for the contribution 🎉

Thanks @Bilbottom - that looks great. I was just getting a quick fix in place for CI pipelines, your solutions is much more solid. I was going to ask about test processes and stuff, but it looks like you've covered that off in there. Thanks for that! I'll close this one.


"Kind":0 Event failed.

Describe the bug
"Kind":0 Event failed and Failed to update profile.
To Reproduce
Steps to reproduce the behavior:
Run metaDataEvent() in NostrExamples.java
Expected behavior
Successfully updated profile.
"Causes and solutions"
The member variable profile in MetadataEvent is missing annotation "@JsonIgnore".
Add @JsonIgnore to profile in MetadataEvent.java.

This should be fixed now. Please confirm.

I have confirmed the changes. Thank you.


Bug(perf): CN node panic when run nexmark q5-rewrite

Daily performance test pipeline failed.
CN node panic when run nexmark q5-rewrite query：
Execution URL: https://buildkite.com/risingwave-test/nexmark-benchmark/builds/931

Grafana: https://g-2927a1b4d9.grafana-workspace.us-east-1.amazonaws.com/d/EpkBw5W4k/risingwave-test-dashboard?orgId=1&from=1682619249000&to=1682619372000&var-namespace=nexmark-bs-3-4-5-5r-daily-20230427

Log: https://ap-southeast-1.console.aws.amazon.com/cloudwatch/home?region=us-east-1#logsV2:log-groups/log-group/$252Frisingwave$252Ftest-useast1-eks-a$252Fnexmark-bs-3-4-5-5r-daily-20230427

Key panic log:
"log": "2023-04-27T18:14:40.394994739Z stderr F thread 'risingwave-streaming-actor' panicked at 'called `Option::unwrap()` on a `None` value', /risingwave/src/stream/src/executor/top_n/top_n_cache.rs:513:54"

probably fixed by https://github.com/risingwavelabs/risingwave/pull/9501? @soundOfDestiny but didn't merge into main yesterday

probably fixed by #9501? @soundOfDestiny but didn't merge into main yesterday

Yes, it has been fixed by #9501


[Bug] Windows ICO icon

The app icon for the windows build does not currently work without the ICO file

Resolved with commit 7144da0


[REQUEST] Research possible integration with PenPot

Suggestion/Concern
Taking advantage of the Open Source PenPot  project, some sort of integration would be convenient
Proposed Solution
Export the created design as a library of assets for PenPot
Going further, those assets could be managed directly by the Theme Builder.
Another venue could be having ATB directly integrated within PenPot.
Related link
Follow similar issue in https://github.com/penpot/penpot/issues/3705

Hi @gvlx!  Is this an issue that you'd like to contribute to (e.g. would you like me to assign it to you), or is this functionality that you'd like us to consider adding to the Theme Builder roadmap?  Would you like to join a community meeting and tell us more about PenPot?

Let me see if can bring people from the Penpot project and the Open Source Design community in.
Are your meetings always on "Thursdays at 9:15AM EST."? (from the FINOS calendar)?

Yes, that is correct.  However, we will not be meeting on 11/23 (due to US Thanksgiving) and will probably cancel some meetings at the end of the year due to many people being on holiday.  But we should still have a couple more before the end of the year.
Otherwise, if we want to do something with introductions and demos and some discussion on what is or is not possible, we can instead organize a stand alone meeting.  I think that this is probably the better option.


Developer console account wallet update error: "Wallet must be a valid JSON object"

Description
Updating account wallet from console dashboard not working: "Wallet must be a valid JSON object"

Steps to Reproduce

Login and click on any user entry.
2.Try to update wallet from Account tab.
3."Wallet must be a valid JSON object" message is displayed and no update is performed.

Expected Result
Account wallet update success.
Actual Result
No update performed and error message "Wallet must be a valid JSON object" is displayed.
Your Environment

Nakama: 2.5.0
Cockroach: 2.1.6
MacOS 10.13.4 HighSierra

Thanks for reporting this! This is fixed in 14f1197632ba525ea14a75c6d5cc57550ad1da3e and will be part of the next release.


Timing error in watcher's sleep

Seen in version 0.18.1, but the lines of code in question hadn't changed for 7 months as of that version, so it's likely this is a rare edge case that has been possible for  a long time.
lib/kubernetes-deploy/resource_watcher.rb:23:in `sleep': time interval must be positive (ArgumentError)
  from lib/kubernetes-deploy/resource_watcher.rb:23:in `run'
  from lib/kubernetes-deploy/deploy_task.rb:380:in `deploy_resources'
  from lib/kubernetes-deploy/deploy_task.rb:155:in `run!'
  from exe/kubernetes-deploy:67:in `<top (required)>'

Code in question is:
       if Time.now.utc < delay_sync_until           sleep(delay_sync_until - Time.now.utc)         end
Easy fix ideas are saving the first call to Time.now.utc or [delay_sync_until, 0.1].max in the sleep call

I'm fine with either of those fixes. Interested in making the PR?


Update stm32f1xx-hal dep

Merged. Sorry for taking a while to get back to this project!

Thanks, and don't be sorry.


Docker build fails

Describe the bug
Docker build fails with the next error
20.66
20.66   Backend subprocess exited when trying to invoke get_requires_for_build_wheel
20.66
20.66   Cythonizing sources
20.66   Compiling srsly/msgpack/_unpacker.pyx because it changed.
20.66   Compiling srsly/msgpack/_packer.pyx because it changed.
20.66   [1/2] Cythonizing srsly/msgpack/_packer.pyx
20.66   [2/2] Cythonizing srsly/msgpack/_unpacker.pyx
20.66
20.66   Error compiling Cython file:
20.66   ------------------------------------------------------------
20.66   ...
20.66       object PyMemoryView_GetContiguous(object obj, int buffertype, char order)
20.66
20.66   from libc.stdlib cimport *
20.66   from libc.string cimport *
20.66   from libc.limits cimport *
20.66   ctypedef unsigned long long uint64_t
20.66   ^
20.66   ------------------------------------------------------------
20.66
20.66   srsly/msgpack/_unpacker.pyx:13:0: 'uint64_t' redeclared
20.66
20.66   Error compiling Cython file:
20.66   ------------------------------------------------------------
20.66   ...
20.66   cdef extern from "Python.h":
20.66       ctypedef int int32_t
20.66       ctypedef int int64_t
20.66       ctypedef unsigned int uint32_t
20.66       ctypedef unsigned int uint64_t
20.66       ^
20.66   ------------------------------------------------------------
20.66
20.66   /tmp/tmp82ajuywo/.venv/lib/python3.9/site-packages/Cython/Includes/cpython/pyport.pxd:5:4: Previous declaration is here
20.66   Traceback (most recent call last):
20.66     File "/usr/local/poetry/venv/lib/python3.9/site-packages/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
20.66       main()
20.66     File "/usr/local/poetry/venv/lib/python3.9/site-packages/pyproject_hooks/_in_process/_in_process.py", line 335, in main
20.66       json_out['return_val'] = hook(**hook_input['kwargs'])
20.66     File "/usr/local/poetry/venv/lib/python3.9/site-packages/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
20.66       return hook(config_settings)
20.66     File "/tmp/tmp82ajuywo/.venv/lib/python3.9/site-packages/setuptools/build_meta.py", line 325, in get_requires_for_build_wheel
20.66       return self._get_build_requires(config_settings, requirements=['wheel'])
20.66     File "/tmp/tmp82ajuywo/.venv/lib/python3.9/site-packages/setuptools/build_meta.py", line 295, in _get_build_requires
20.66       self.run_setup()
20.66     File "/tmp/tmp82ajuywo/.venv/lib/python3.9/site-packages/setuptools/build_meta.py", line 311, in run_setup
20.66       exec(code, locals())
20.66     File "<string>", line 138, in <module>
20.66     File "<string>", line 125, in setup_package
20.66     File "/tmp/tmp82ajuywo/.venv/lib/python3.9/site-packages/Cython/Build/Dependencies.py", line 1154, in cythonize
20.66       cythonize_one(*args)
20.66     File "/tmp/tmp82ajuywo/.venv/lib/python3.9/site-packages/Cython/Build/Dependencies.py", line 1321, in cythonize_one
20.66       raise CompileError(None, pyx_file)
20.66   Cython.Compiler.Errors.CompileError: srsly/msgpack/_unpacker.pyx
20.66
20.66
20.66   at /usr/local/poetry/venv/lib/python3.9/site-packages/poetry/installation/chef.py:152 in _prepare
20.66       148│
20.66       149│                 error = ChefBuildError("\n\n".join(message_parts))
20.66       150│
20.66       151│             if error is not None:
20.66     → 152│                 raise error from None
20.66       153│
20.66       154│             return path
20.66       155│
20.66       156│     def _prepare_sdist(self, archive: Path, destination: Path | None = None) -> Path:
20.66
20.66 Note: This error originates from the build backend, and is likely not a problem with poetry but with srsly (2.4.2) not supporting PEP 517 builds. You can verify this by running 'pip wheel --use-pep517 "srsly (==2.4.2) ; python_version >= "3.6""'.
20.66
------
Dockerfile:36
--------------------
  35 |     WORKDIR /app
  36 | >>> RUN poetry config virtualenvs.create false && \
  37 | >>>   poetry install --extras ${DEVICE} --no-interaction --no-ansi --no-root $(/usr/bin/test $STAGE == production && echo "--without dev,test,docs")
  38 |
--------------------
ERROR: failed to solve: process "/bin/sh -c poetry config virtualenvs.create false &&   poetry install --extras ${DEVICE} --no-interaction --no-ansi --no-root $(/usr/bin/test $STAGE == production && echo \"--without dev,test,docs\")" did not complete successfully: exit code: 1

Vesion (please complete the following information):

OS: MacOS Sonoma 14.0
Azimuth version: 2.8.0 (34081048a4de3900ca29ac0d37bce7026113382c commit)

Hi @WiseBird,
I am sorry for the delay. I am not able to reproduce this error. I just updated my Mac to Sonoma 14, so I should have the exact same setup as you.
Could you please confirm if you are still experiencing this issue? And maybe provide more details on how to reproduce?

@JosephMarinier Do you have M1 mac?

@WiseBird Ah-ha! No. I guess you do?

Yep

Maybe you could try to poetry lock --no-update. I might add some file URLs for your specific architecture. 🤷


Hit panic!("Invalid bounding box from Face::outline_glyph")

I was able to hit this panic when trying to draw code points 0x2001..0x2004 from Symbola.ttf (https://fedoraproject.org/wiki/Gdouros_Symbola).  These code points are all whitespace of some kind, so I suspect (but have not verified) that the bounding box has a zero-size dimension.
https://github.com/alexheretic/ab-glyph/blob/cb8241d2e9ce89b29717e8780c91b9ceb2fecfd8/glyph/src/ttfp.rs#L253

This panic means one of the glyphs bounding box calculated by ttf-parser is invalid. This tends to suggest the font is erroneous and/or ttf-parser is not accounting for that. I'd expect whitespace fonts to not return any outline bounding box.
Can you provide a link to the font ttf? I can't provide upstream with decent steps to reproduce this without that, I can't figure out how to get it from the links you have provided.

https://kojipkgs.fedoraproject.org//packages/gdouros-symbola-fonts/10.24/8.fc34/src/gdouros-symbola-fonts-10.24-8.fc34.src.rpm is the RPM containing it; I couldn't figure out how to link just the file but its info page is: https://koji.fedoraproject.org/koji/fileinfo?rpmID=24696399&filename=/usr/share/fonts/gdouros-symbola/Symbola.ttf


[ISSUE #189]Enum Module impl ToString trait

close #189

pls resolve CI error

Codecov Report
Patch coverage has no change and project coverage change: +0.01% :tada:

Comparison is base (fb8e87c) 15.15% compared to head (9920524) 15.16%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #190      +/-   ##
==========================================
+ Coverage   15.15%   15.16%   +0.01%
==========================================
  Files          49       49
  Lines        2805     2803       -2
==========================================
  Hits          425      425
+ Misses       2380     2378       -2

Files Changed
Coverage Δ

src/api/props.rs
0.00% <ø> (ø)

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I think use &'static str is better than String right? this code is used to build  request message, so i think is unnecessary to instead it.

@onewe I Thank you for your answer, and I agree with your point of view. I will close this PR.


config for leaving URL fragment unchanged

If base URL is as below -
http://localhost/project/module#fragment_key=fragment_value
the plugin adds # to end of URL when click event occurs on mainButtonClass="mfb-component__button--main" and URL becomes  as below -
http://localhost/project/module#
So the fragment_key and fragment_value is removed. I am opening a modal on click for menu item and would ideally have the URL unchanged so that when modal closes the fragment in URL don't change as the page has not changed
Is this possible that clicking on menu button / menu item does not lead to displacement of exisiting URL fragment in favour of "#" ?

 tag without href helps for menu button


Split services/ into services/ and foundation/

We're getting back to the point where we have a bunch of foundation APIs
and it's getting confusing having them mixed with services/.

LGTM


Flex mode: Preview should include full chemical names.

Steps to Reproduce

Switch to Macro mode

Open file with Peptides
ketcher (OK).zip

see preview of O peptide

see preview of Pyl peptide

Actual behavior
One structure for O and Pyl peptides have different chemical name.
Expected behavior
Preview should include full chemical names.
as for O

Also D-Pyl (name of Peptide in the Preview) should be changed to d-Pyrrolysine

(Agreed with BA)
By tested:

https://github.com/epam/ketcher/issues/3876
https://github.com/epam/ketcher/issues/4484

Desktop (please complete the following information):

OS: Windows 11
Browser Chrome
Version 123.0.6312.106 (Official Build) (64-bit)

Ketcher version
[Version 2.21.0-rc.1]

Library name (seems - full name) should be corrected

This issue is a database issue. The code is trying to use monomer full name but some cases it is loaded incorrectly.


Add the missing 'h' in Algorithms

Thank you to @MattyMyers22 for the pickup
in how_to/ssh/ssh_keygen/README.md and
the pull request #3
Add an 'h'
Change
from:
HostKeyAlgoritms ssh-rsa
to:
HostKeyAlgorithms ssh-rsa

Issue resolved with merge pull request https://github.com/cs-south-sound/curtain-raiser/pull/3 from MattyMyers22/patch-1
at commit db62772


decaffeinate src/apm-cli.js + prepair file by file decaffeination

This pull request is the first to decaffeinate the complete repository. It is required for further pull request because it moves .js files on build to the output folder.

I thought I pinged you on Discord. But I do have a PR that decaffeinates the repo at #8.
Not sure if you want to keep going with this method, or it could be really helpful to commit directly to the PR of cleaning up the automatic files, since they are quite ugly and not the most readable.
Just wanting to make sure you're aware

@confused-Techie
I looked at your pull request and first wanted to contribute to that but after a discussion on discord decaffeinating file by file seemed better.
https://discord.com/channels/992103415163396136/992110062992621598/1037755621962944592

It is admittedly hard to review all ~220-250 lines of code.
Can you describe what method you used to decaffeinate? What steps, what programs, so that we can reproduce your output. That would make it so we don't have to manually review every single line, and instead lean a lot heavier on trusting the tool that generated the JS output.
If the process you are using is extremely similar to #8, then we should indeed first decide whether to do all of this as one PR (#8) or one PR per file. As @confused-Techie mentioned.
(I am sorry to say the Discord link you posted does not take me to the relevant conversation, so I'm not sure exactly what was discussed about this before, who said what, etc.)

I review his code line by line.
compare .coffee and .js file.

@DeeDeeG I used the notmal coffeescript compiler to convert the file and the rewrote parts with a more modern style. Reviewing line by line would be simpler than redoing this process. I will try to use the decaffeinating tool in the future to reduce the parts that are changed by me.

Thank you @Sertonix. This is very helpful information. So, for example, I can run the coffeescript compiler and than do a diff with your file and review only the manual changes you made.
(This is much faster than having to read the whole coffeescript file and then the whole JS file and comparing them, function by function, line by line. I don't think any of us has the coffeescript code memorized by heart, so yes, we would have had to review all of the coffeescript just to review this, but now we can look at just the manual changes.)

I use decaffeinate-project online to convert, and refactor manually.

I feel at this point as we talk about decaffeination tooling, I'd hate to repeat the past. But I also don't want to make anyone redo any work.
Luckily the tests in this PR seem to be passing, so likely not an issue here. But going forward please make sure to take a look at our documentation, as we have guidelines about decaffeination.

This is now based on decaffeinate and not anymore on the default transpiler.

Sorry it has taken so long to get to properly reviewing this.
I tried to run npm test with this latest update. Only four of the tests run, and then the test harness silently quits somewhat uncleanly.
I don't think the code is functionally equivalent with the latest commits. (Mainly, the coffee compiler emits a JS file that is one big immediately-executed function, whereas I'm not sure that's true here?)

(Note on review etiquette: Please refrain from making huge changes (like switching to a different coffeescript compiler) while a review is already in progress. If you wanted to start over, I think it would make sense only to do that in a new PR, and let maintainers sort out the different approaches from there.
But generally being patient is the preferred thing. Fundamentally changing what is on offer for review, while review is ongoing, is mostly just going makes things more complicated. Please only make major changes on an existing pull request if the reviewers say they are okay with it.)

Sorry for lack of consensus before charging ahead with decaffeination in general, it seems like someone gave a casual go-ahead in Discord and a lot of work was started from that.
I think it would be a lot cleaner to just run the decaffeination scripts inherent to the repo (npm run build) and commit the result (at least on a "decaf" branch?), then allow any further clean-up work (and PRs) to target the "decaf base" branch. That way we are only reviewing the small manual changes, and we can maybe let go of the readability perfectionism in favor of a known 1:1 replacement, where any changes to that are expected to pass CI. Would make things much more doable in terms of difficulty of review, also.
(More rationale for that move: ppm technically already runs the machine-decaffeinated code in production. Committing the machine-decaffeinated files would not be a radical move, but a fairly conservative one.)
If other reviewers are okay with this, I can set up a branch as the base for any manual decaffeination changes.
(More thoughts: If we can't figure out consensus on how to clean up the machine-decaffeinated code, IMO we should just roll with the machine-decaffeinated code by default, to make sure we don't end up bike-shedding JS code style forever and ever. And this is an unfortunate type of first contribution, as it lends itself to bike-shedding and lots of small review comments, which would be require tremendous patience for a first-time contributor. Reviewers have no firm consensus to start with for how decaffeinated code should look like. It's really not great to have to all collaboratively figure this out on a first-time-contributor's first PR. So we should go into it understanding this is not an ideal situation, and it may take a lot of patience to get through it.)
Sorry for the essay-length comment, just trying to sort this whole thing out as it has already gotten somewhat messy, IMO.

Sorry for the chaos with this and the decaf of the specs directory. The reason I chose and suggested this way was because the pull request from confused_techie was outdated and I feared that this would quickly happen again.
The functions wrapping the code produced from the normal coffeescript compiler is not needed. It ensures that no globals are assigned when directly loaded from html. This is not the case for this repo. (somebody correct me if this is not the real reason but I am 100% that it is not important for this)
For this and some other readability differences I strongly suggest using the decaffeinate tool.
But aside from this I am ok with a pull request that decafs the complete code without manual changes.
Since I have been working on other parts of pulsar lately I would really appreciate if somebody else could do that.
@GuilleW you said you could work on this. Sorry if I suggest things that were not really planned.
The new plan for you in short: Use a tool to decaf everything and create a pull request with that. No manual changes this time.

Also following this it would be nice is somebody could make the decaffeinate instructions in the "contributing during start" file more clear to prevent something like this in the future.

The functions wrapping the code produced from the normal coffeescript compiler is not needed. It ensures that no globals are assigned when directly loaded from html. This is not the case for this repo. (somebody correct me if this is not the real reason but I am 100% that it is not important for this)

Completely correct !

Waiting for a step-by-step instructions that admin agreed before working on.
Don't worry @Sertonix , I'm ok to work on that part.
Stand by.

@Sertonix I do apologize for the confusion here. I'll go ahead and draft an update to our decaffeination policy, and would recommend taking a look for another PR. We all really do appreciate the efforts and contributions, just want to make sure we get this right.


Consider making terratest output readable

Because of the fact that terratests are run in parallel and produce a lot of log messages when doing the polling mechanisms (those waitUntilX functions), it's almost impossible to find out the root cause of the issue why some test is red.
Gruntwork provides a tool that produces separate log files for each test w/o any interleaving, it's called terratest_log_parser They also provide an example how to use it together with CircleCI. Here is an example of circle.ci config: https://github.com/gruntwork-io/kubergrunt/blob/master/.circleci/config.yml#L42:L54
They upload the separate log files into artifact repository that pertains to the job. GHA has something similar
links:

https://terratest.gruntwork.io/docs/testing-best-practices/debugging-interleaved-test-output/
https://github.com/actions/upload-artifact

Will take a look into that

@AugustasV thanks!


Issue with recursive trees

I tried uploading a tree where a node had itself listed as a child. I could store such tree, but when retrieving the tree I got a stack overflow. We may need to prevent such trees to be stored.

Yes, currently I assume that the JSON is valid LionWeb.  The plan is to use the validation package from lionweb-typescript. to validate the incoming JSON.  And reject the call if the JSON is incorrect.
The specific case you describe will then be rejected because of invalid JSON.

I just added the LionWeb JSON validation to the API in
If there is a validation error (i.e. the JSON is not LionWeb compliant) the api call will return status = 400 to indicate an error.

Validation of JSON is done for the store request in the main branch


Revert "Revert "Feat/one hot encoding""

Reverts mlederbauer/NMRcraft#37

This is. bit meta, sorry everyone


Add Python3.12 compatibility

This PR allows Python3.12 to be used with qiskit-alice-bob-provider. This was done in preparation of https://github.com/Alice-Bob-SW/qiskit-alice-bob-provider/issues/38, as updating versions is better done when most dependencies can be ran as latest.

Hello!
Thank you very much for taking the time to do this PR. There is a small issue however, as our current version of Qiskit 0.44 is not compatible with Python 3.12, hence this is why I did willingly block this version 😄
3.12 support was introduced in Qiskit 0.45+
https://quantumcomputing.stackexchange.com/questions/34374/error-while-installing-qiskit-0-44-2-preparing-metadata-pyproject-toml-did-n
https://github.com/Qiskit/qiskit/issues/10887
For now, as we have some time to dedicate working on the provider again, we will probably try bumping Qiskit to its latest version in the upcoming days. After the bump is done and functional, we will unlock 3.12.
Stay tuned!

That makes a lot of sense, thank you for taking the time to explain. I made this PR as I want to get my hands dirty on Quantum Software code, and I prefer to start small. If there are small issues that I could get started around the Alice&Bob codebase, do point me to them 😄


AzureKeyDetector plugin (updated from AzureStorageKeyDetector)

I updated AzureStorageKeyDetector to AzureKeyDetector.
AzureKeyDetector supports various Azure services in addtion to Azure Storage (supported by AzureStorageKeyDetector).
Here are supported Azure services:

Azure Storage
Azure SQL Database
Azure Database for PostgreSQL
Azure Database for MySQL
Azure Database for MariaDB
Azure Cache for Redis
Azure Cosmos DB
Azure Synapse Analytics
Azure Service Bus
Azure Event Hubs
Azure IoT Hub
Azure Monitor
Azure Functions
Azure Web PubSub
Azure SignalR Service

azure.py has RegEx expressions for various key/connection string format of various Azure services.
azure_test.py has corresponding payload/should_flag pairs for RegEx espressions/
azure_test.py runs succesfully.
$ python -m pytest tests/plugins/azure_test.py
================================================================================================ test session starts ================================================================================================
platform linux -- Python 3.10.4, pytest-7.1.2, pluggy-0.13.1
rootdir: /mnt/c/repos/detect-secrets
collected 27 items

tests/plugins/azure_test.py ...........................                                                                                [100%]

================================================================================================ 27 passed in 4.85s =================================================================================================

However, when I scan azure_test.py, detect-secrets detects 6 secrets only. I think it should detec 27 secrets. I don't know the reason why 21 secrets are not detected. I'd like to find the reason in this thread and fix it (if needed).
$ python3 -m detect_secrets scan tests/plugins/azure_test.py
{
  "version": "1.3.0",
  "plugins_used": [
    {
      "name": "ArtifactoryDetector"
    },
    {
      "name": "AWSKeyDetector"
    },
    {
      "name": "AzureKeyDetector"
    },
    {
      "name": "Base64HighEntropyString",
      "limit": 4.5
    },
    {
      "name": "BasicAuthDetector"
    },
    {
      "name": "CloudantDetector"
    },
    {
      "name": "GitHubTokenDetector"
    },
    {
      "name": "HexHighEntropyString",
      "limit": 3.0
    },
    {
      "name": "IbmCloudIamDetector"
    },
    {
      "name": "IbmCosHmacDetector"
    },
    {
      "name": "JwtTokenDetector"
    },
    {
      "name": "KeywordDetector",
      "keyword_exclude": ""
    },
    {
      "name": "MailchimpDetector"
    },
    {
      "name": "NpmDetector"
    },
    {
      "name": "PrivateKeyDetector"
    },
    {
      "name": "SendGridDetector"
    },
    {
      "name": "SlackDetector"
    },
    {
      "name": "SoftlayerDetector"
    },
    {
      "name": "SquareOAuthDetector"
    },
    {
      "name": "StripeDetector"
    },
    {
      "name": "TwilioKeyDetector"
    }
  ],
  "filters_used": [
    {
      "path": "detect_secrets.filters.allowlist.is_line_allowlisted"
    },
    {
      "path": "detect_secrets.filters.common.is_ignored_due_to_verification_policies",
      "min_level": 2
    },
    {
      "path": "detect_secrets.filters.gibberish.should_exclude_secret",
      "limit": 3.7
    },
    {
      "path": "detect_secrets.filters.heuristic.is_indirect_reference"
    },
    {
      "path": "detect_secrets.filters.heuristic.is_likely_id_string"
    },
    {
      "path": "detect_secrets.filters.heuristic.is_lock_file"
    },
    {
      "path": "detect_secrets.filters.heuristic.is_not_alphanumeric_string"
    },
    {
      "path": "detect_secrets.filters.heuristic.is_potential_uuid"
    },
    {
      "path": "detect_secrets.filters.heuristic.is_prefixed_with_dollar_sign"
    },
    {
      "path": "detect_secrets.filters.heuristic.is_sequential_string"
    },
    {
      "path": "detect_secrets.filters.heuristic.is_swagger_file"
    },
    {
      "path": "detect_secrets.filters.heuristic.is_templated_secret"
    }
  ],
  "results": {
    "tests/plugins/azure_test.py": [
      {
        "type": "Azure keys/connection strings",
        "filename": "tests/plugins/azure_test.py",
        "hashed_secret": "43cd99b89520080a5e7c1349235765d70ef15274",
        "is_verified": false,
        "line_number": 12
      },
      {
        "type": "Azure keys/connection strings",
        "filename": "tests/plugins/azure_test.py",
        "hashed_secret": "fd4eebe5e7bdb850e8e6471a4186f506324687f0",
        "is_verified": false,
        "line_number": 28
      },
      {
        "type": "Azure keys/connection strings",
        "filename": "tests/plugins/azure_test.py",
        "hashed_secret": "b27ea2fb585a210fbb607a675b71abe228db1c7d",
        "is_verified": false,
        "line_number": 60
      },
      {
        "type": "Azure keys/connection strings",
        "filename": "tests/plugins/azure_test.py",
        "hashed_secret": "7b162df4c3e8682cd7d6e6815376f4ee6903e41a",
        "is_verified": false,
        "line_number": 68
      },
      {
        "type": "Azure keys/connection strings",
        "filename": "tests/plugins/azure_test.py",
        "hashed_secret": "8ca69981e32198203d53f5a9dfcad52aa9846381",
        "is_verified": false,
        "line_number": 72
      },
      {
        "type": "Azure keys/connection strings",
        "filename": "tests/plugins/azure_test.py",
        "hashed_secret": "e7722ae3eaa895362ea52b89e5a4d3b6d5440ce2",
        "is_verified": false,
        "line_number": 108
      }
    ]
  },
  "generated_at": "2022-08-05T03:19:07Z"
}

Hi @satonaoki, thank you for opening this PR. I appreciate your work on improving our Azure keys detector.
I ran our tests and it looks like they're failing, so I encourage you to take a look at those before we review your code. Let me know if you have any questions.

@lorenzodb1
I rebased my branch from Yelp:master.
The reason of test failure is that the old plugin name AzureStorageKeyDetector is included in .secrets.baseline. I updated it to AzureKeyDetector. Now tests succeeded.
$ python -m pytest tests
...
====================================================================================== warnings summary =======================================================================================
tests/core/baseline_test.py::TestCreate::test_error_when_getting_git_tracked_files
  /mnt/c/repos/detect-secrets/detect_secrets/core/scan.py:74: UserWarning: Did not detect git repository. Try scanning all files instead.
    log.warning('Did not detect git repository. Try scanning all files instead.')

tests/core/usage/filters_usage_test.py::test_disable_filter
  /mnt/c/repos/detect-secrets/detect_secrets/core/usage/filters.py:192: UserWarning: Redundant --disable-filter "blah"
    log.warning(f'Redundant --disable-filter "{name}"')

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
============================================================== 972 passed, 1 skipped, 6 xfailed, 2 warnings in 265.23s (0:04:25) ==============================================================

@lorenzodb1 Any update/comment?

@satonaoki Hello. It seems the CI is still failing. Can you checkout the errors and fix them?

@jpdakran I added pragma allowlist comment to resolve the CI issue. Could you approve the workflow?

1 workflow awaiting approval
First-time contributors need a maintainer to approve running workflows. [Learn more.]

@jpdakran I resolved another CI issue. Could you approve the workflow?

@satonaoki looks like some checks are still failing. I encourage you to take a look at those.

@lorenzodb1 "W291 trailing whitespace" error fixed. Could you approve the workflow?

detect_secrets/plugins/azure.py:78:94: W291 trailing whitespace
detect_secrets/plugins/azure.py:96:79: W291 trailing whitespace
detect_secrets/plugins/azure.py:97:92: W291 trailing whitespace

Hmm. CI still has these two errors.
But it seems that azure.py has no issues for [add-trailing-comma](https://github.com/asottile/add-trailing-comma/blob/main/README.md)
I will run CI in my forked repom and take a close look at it.

Add trailing commas......................................................Failed

hook id: add-trailing-comma
exit code: 1
files were modified by this hook

Rewriting detect_secrets/plugins/azure.py
autopep8.................................................................Failed

hook id: autopep8
files were modified by this hook


Condition 'Speex_version_id==1' is always true

Remove if (Speex_version_id==1)?
https://github.com/MediaArea/MediaInfoLib/blob/9c77babfa699347c4ca4a79650cc1f3ce6fcd6c8/Source/MediaInfo/Audio/File_Speex.cpp#L110

Oops, the first "Filling" block should be with the second one, I'll change that.