errors while running the example of 4iiy.pdb

hi, haotian,
thanks for your great work.
but when I try to run the example of 4iiy.pdb, the following errors occured.
Num of parameters is 37.52M
Start to generate in the ./Test/4iiy.pdb
c:\Users\XXT\.conda\envs\Resgen\lib\site-packages\numpy\core\fromnumeric.py:3464: RuntimeWarning: Mean of empty slice.
  return _methods._mean(a, axis=axis, dtype=dtype,
c:\Users\XXT\.conda\envs\Resgen\lib\site-packages\torch\functional.py:504: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at ..\aten\src\ATen\native\TensorShape.cpp:3484.)
  return _VF.meshgrid(tensors, **kwargs)  # type: ignore[attr-defined]
Traceback (most recent call last):
  File "e:\ResGen-main\gen.py", line 217, in <module>
    rdmol = reconstruct_from_generated_with_edges(data_next)
  File "e:\ResGen-main\utils\reconstruct.py", line 70, in reconstruct_from_generated_with_edges
    Chem.SanitizeMol(rd_mol, Chem.SANITIZE_ALL^Chem.SANITIZE_KEKULIZE^Chem.SANITIZE_SETAROMATICITY)
OverflowError: can't convert negative int to unsigned

could you please provide the suggestions how to fix it up?
many thanks,
ShY

hi, haotian,
thanks for your great work. but when I try to run the example of 4iiy.pdb, the following errors occured.
Num of parameters is 37.52M
Start to generate in the ./Test/4iiy.pdb
c:\Users\XXT\.conda\envs\Resgen\lib\site-packages\numpy\core\fromnumeric.py:3464: RuntimeWarning: Mean of empty slice.
  return _methods._mean(a, axis=axis, dtype=dtype,
c:\Users\XXT\.conda\envs\Resgen\lib\site-packages\torch\functional.py:504: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at ..\aten\src\ATen\native\TensorShape.cpp:3484.)
  return _VF.meshgrid(tensors, **kwargs)  # type: ignore[attr-defined]
Traceback (most recent call last):
  File "e:\ResGen-main\gen.py", line 217, in <module>
    rdmol = reconstruct_from_generated_with_edges(data_next)
  File "e:\ResGen-main\utils\reconstruct.py", line 70, in reconstruct_from_generated_with_edges
    Chem.SanitizeMol(rd_mol, Chem.SANITIZE_ALL^Chem.SANITIZE_KEKULIZE^Chem.SANITIZE_SETAROMATICITY)
OverflowError: can't convert negative int to unsigned

could you please provide the suggestions how to fix it up? many thanks,
ShY

I have the same problem. Have you solved it？

I think the problem is the conflict of rdkit molecule. Please try to read the rd_mol out of the code, I guess the output is None. To fix this problem, make sure your rdkit is proper. You can transform the .sdf with openbabel, like obabel xxx.sdf -O xxx.sdf (in the command), sometimes, the openbabel can automatically reorder and fix molecular problems. Feel free to contact me if you have any problems!

thanks for your suggestions,
after running openbael to convert .sdf, the error still occured.
Namespaces are one honking great idea -- let's do more of those!
Num of parameters is 37.52M
Start to generate in the [./Test/4iiy.pdb](https://file+.vscode-resource.vscode-cdn.net/e%3A/Cheminfo_Workshop/4_Fragment_Scaffold_Evolution/ResGen-main/Test/4iiy.pdb)
[c:\Users\LSY\.conda\envs\TargetDiff\lib\site-packages\numpy\core\fromnumeric.py:3432](file:///C:/Users/LSY/.conda/envs/TargetDiff/lib/site-packages/numpy/core/fromnumeric.py:3432): RuntimeWarning: Mean of empty slice.
  return _methods._mean(a, axis=axis, dtype=dtype,
[c:\Users\LSY\.conda\envs\TargetDiff\lib\site-packages\torch\functional.py:504](file:///C:/Users/LSY/.conda/envs/TargetDiff/lib/site-packages/torch/functional.py:504): UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at [C:\actions-runner\_work\pytorch\pytorch\builder\windows\pytorch\aten\src\ATen\native\TensorShape.cpp:3191](file:///C:/actions-runner/_work/pytorch/pytorch/builder/windows/pytorch/aten/src/ATen/native/TensorShape.cpp:3191).)
  return _VF.meshgrid(tensors, **kwargs)  # type: ignore[attr-defined]
Traceback (most recent call last):
  File "gen.py", line 217, in <module>
    rdmol = reconstruct_from_generated_with_edges(data_next)
  File "e:\ResGen-main\utils\reconstruct.py", line 70, in reconstruct_from_generated_with_edges
    Chem.SanitizeMol(rd_mol, Chem.SANITIZE_ALL^Chem.SANITIZE_KEKULIZE^Chem.SANITIZE_SETAROMATICITY)
OverflowError: can't convert negative int to unsigned

so I will reinstall the envs and follow the resn.yml in your repo and see if that works.
many thanks,
Best,
ShY

hi, haotian,
thanks for your great work. but when I try to run the example of 4iiy.pdb, the following errors occured.
Num of parameters is 37.52M
Start to generate in the ./Test/4iiy.pdb
c:\Users\XXT\.conda\envs\Resgen\lib\site-packages\numpy\core\fromnumeric.py:3464: RuntimeWarning: Mean of empty slice.
  return _methods._mean(a, axis=axis, dtype=dtype,
c:\Users\XXT\.conda\envs\Resgen\lib\site-packages\torch\functional.py:504: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at ..\aten\src\ATen\native\TensorShape.cpp:3484.)
  return _VF.meshgrid(tensors, **kwargs)  # type: ignore[attr-defined]
Traceback (most recent call last):
  File "e:\ResGen-main\gen.py", line 217, in <module>
    rdmol = reconstruct_from_generated_with_edges(data_next)
  File "e:\ResGen-main\utils\reconstruct.py", line 70, in reconstruct_from_generated_with_edges
    Chem.SanitizeMol(rd_mol, Chem.SANITIZE_ALL^Chem.SANITIZE_KEKULIZE^Chem.SANITIZE_SETAROMATICITY)
OverflowError: can't convert negative int to unsigned

could you please provide the suggestions how to fix it up? many thanks,
ShY

I have the same problem. Have you solved it？

not yet, but you can try to check the env yml if that's consistent with the repo.

Thanks for your reply. I have fixed the problem. Do you need my help?
---Original---
From: @.>
Date: Mon, Jan 22, 2024 11:19 AM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [HaotianZhangAI4Science/ResGen] errors while running the exampleof 4iiy.pdb (Issue #8)
hi, haotian,
thanks for your great work. but when I try to run the example of 4iiy.pdb, the following errors occured.
Num of parameters is 37.52M Start to generate in the ./Test/4iiy.pdb c:\Users\XXT.conda\envs\Resgen\lib\site-packages\numpy\core\fromnumeric.py:3464: RuntimeWarning: Mean of empty slice.   return _methods._mean(a, axis=axis, dtype=dtype, c:\Users\XXT.conda\envs\Resgen\lib\site-packages\torch\functional.py:504: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at ..\aten\src\ATen\native\TensorShape.cpp:3484.)   return _VF.meshgrid(tensors, **kwargs)  # type: ignore[attr-defined] Traceback (most recent call last):   File "e:\ResGen-main\gen.py", line 217, in <module>     rdmol = reconstruct_from_generated_with_edges(data_next)   File "e:\ResGen-main\utils\reconstruct.py", line 70, in reconstruct_from_generated_with_edges     Chem.SanitizeMol(rd_mol, Chem.SANITIZE_ALL^Chem.SANITIZE_KEKULIZE^Chem.SANITIZE_SETAROMATICITY) OverflowError: can't convert negative int to unsigned
could you please provide the suggestions how to fix it up? many thanks,
ShY
I have the same problem. Have you solved it？
not yet, but you can try to check the env yml if that's consistent with the repo.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Thanks for your reply. I have fixed the problem. Do you need my help?
…
---Original--- From: @.> Date: Mon, Jan 22, 2024 11:19 AM To: @.>; Cc<EMAIL_ADDRESS>Subject: Re: [HaotianZhangAI4Science/ResGen] errors while running the exampleof 4iiy.pdb (Issue #8) hi, haotian, thanks for your great work. but when I try to run the example of 4iiy.pdb, the following errors occured. Num of parameters is 37.52M Start to generate in the ./Test/4iiy.pdb c:\Users\XXT.conda\envs\Resgen\lib\site-packages\numpy\core\fromnumeric.py:3464: RuntimeWarning: Mean of empty slice. return _methods._mean(a, axis=axis, dtype=dtype, c:\Users\XXT.conda\envs\Resgen\lib\site-packages\torch\functional.py:504: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at ..\aten\src\ATen\native\TensorShape.cpp:3484.) return _VF.meshgrid(tensors, kwargs) # type: ignore[attr-defined] Traceback (most recent call last): File "e:\ResGen-main\gen.py", line 217, in <module> rdmol = reconstruct_from_generated_with_edges(data_next) File "e:\ResGen-main\utils\reconstruct.py", line 70, in reconstruct_from_generated_with_edges Chem.SanitizeMol(rd_mol, Chem.SANITIZE_ALL^Chem.SANITIZE_KEKULIZE^Chem.SANITIZE_SETAROMATICITY) OverflowError: can't convert negative int to unsigned could you please provide the suggestions how to fix it up? many thanks, ShY I have the same problem. Have you solved it？ not yet, but you can try to check the env yml if that's consistent with the repo. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.*>

how can you fix the problem? I still cannot run the Chem.SanitizeMol code.
many thanks,
best,


Remove registry from spec

This eliminates the global registry object in favor of a (possibly user-created) registry instance. Users need this ability in order to add custom functions without changing the behavior of JSONPath expressions globally.
Replace spec.Function with an interface that registry.Function (added in 701cf60) adheres to. Refactor NewFunctionExpr to simply create a FunctionExpr with that interface and performs no validation.
Instead, teach the parser use a registry.Registry to look up functions and validate arguments. This requires that the parser have a registry instance, so add a parser struct that stores both a registry and a lexer, and cease passing the lexer to every parsing function.
While at it, add the Validator and Evaluator interfaces to the registry package and teach the Register method (now an instance method) to return errors rather than panic. Also pass Function fields to it rather than a Function instance, as it's a cleaner API. Then make the Function fields private and add accessor methods to comply with the spec.Function interface.
Other bits and bobs:

Rename execute to evaluate to be more consistent.
Remove the default function definitions from the spec package, as they now live in the registry package.
Add a mock function to properly test function expressions.
Add an example test to the Registry.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.93%. Comparing base (701cf60) to head (a8a49cb).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
- Coverage   99.93%   99.93%   -0.01%
==========================================
  Files          13       13
  Lines        1553     1440     -113
==========================================
- Hits         1552     1439     -113
  Misses          1        1

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


Prediction vs Ground Truth

Hi, after your latest push I noticed that the summary from comparing the planet prediction against the Mujoco ground truth have been removed. Is it correct? Can you explain what you did it and if is there any way to keep them without reverting to the previous commit?

Thanks for bringing this to my attention, and sorry for the slow reply. They are disabled to have the training and summary computation run a bit faster. However, it should be easy to enable them again. You can enable {state_diagnostics: True}. However, the only manual change right now is that you'll also need to create optimizers in configs.py for each state component. I'll set up a default optimizer that is used for them in the next update.


Overview Link of SubNavigation

What is the context of this PR?
Fix - #2956
The overview link of the subNavigation parameter within the onsNavigation component is currently missing the --active style which is added to all the subNavigation elements when navigating to their page.
Now, updated the style with ons-navigation__item--active
How to review this PR
Check that Overview link in the subnavigation has a border color when subnavigation current path is equal to the overview url.
Checklist
This needs to be completed by the person raising the PR.

[x] I have selected the correct Assignee
[x] I have linked the correct Issue

Other than the tests failing this now looks good


[WIP] Fix goto intent issue #109

Add gotoIntent handler on messengerBox & factory container
On Dashboard page the go to feature not redirect to Builder page.
Go to intent doesn't focus on the specific input or output. Should this work ?
If it does it might need more work.

TODO - On Dashboard page the go to feature needs to redirect to Builder page.
TODO - Go to intent focus on the specific input or output


Dymanically add/remove modal container from body

Related to https://github.com/limonte/sweetalert2/pull/255#issuecomment-240839239

Hello @birjolaxew,
I made some corrections and I'll be grateful if you got some time to revisit this PR. Thanks in advance!

I won't have access to a computer for like a week. I'll have a look afterwards.

Okay, thanks! Review when you have time!
Enjoy your time without a computer, I can only dream about that currently :)

Looks good to me. All tests pass, and I couldn't find any regressions.

Released in v6.4.0 :tada:
Thanks for for reviewing this PR @birjolaxew!


Confused about what is optional in terms of Infra

https://open-next.js.org/advanced/architecture
Having some difficulty understanding what is required and what is optional, and which options/infra apply to which features.
For example, the diagram makes it seem like the Cache (bucket + files) is optional, but the description makes it sound very important.  Is it just an ISR feature or is it always needed?  Unless there is a good reason not to, I think the docs could do a better job of making it clearer which features each piece of infra is required by.
The options page is also somewhat confusing in a similar vain. For example, https://open-next.js.org/advanced/options#experimental-disable-dynamodb-cache says it is "unsafe" to disable dynamo, but as as far as I can tell it should be safe unless the optional ISR feature is being used.  I don't think it makes sense to say it is "unsafe" to disable an optional feature?  Again, I think this can be easily improved by adding a label to say which features it is relevant to. This could be as simple as changing the headings, eg Experimental disable dynamodb cache (ISR)
I've been using OpenNext since v0 and I think it's fantastic. I used the original docs to build some stacks that we deploy in several applications at work, and I couldn't have done that without the original v0 docs.  I have been trying to make some updates to those stacks and am finding the new docs much less detailed and imagine it would be significantly more difficult for somebody new to the project to do what I was able to do quite easily back then. Some of this is surely just a result of adding more features--which is great--but ease of adoption is very a important aspect of this project (unless I'm mistaken) and that means making the docs detailed enough for somebody to write their own IaC.

You're right, docs could always be better. I'll try to explain things more clearly.
There is 2 different cache used in open-next:

The S3 cache, which is used for both ISR AND SSG (anything with getStaticProps or every non SSR app router pages, and even SSR when Partial Prerendering will go out of beta)
The DynamoDb cache that is used for next/cache based revalidation. revalidatePath and revalidateTag only, so it could be safely ignored for page router.

Both of those cache could be ignored, but we chose to put it under a dangerous flag because it breaks some functionality that may not seem obvious in the first place for everyone.
revalidatePath for example, is just a wrapper around revalidateTag, but most people assume it works the same way as res.revalidate in page router do.
Likewise, most people don't realise that SSG pages are treated by next internally almost exactly as ISR pages (due to on demand revalidation).
Disabling the S3 cache and using SSG, doesn't even break your app if you do not have fallback: false, but it could lead to some very inconsistent state.
For example if your page request some data, it will request data everytime it is not a HIT in the CDN, but the JSON/RSC and the html are two different entry in your CDN (even more than 2 for RSC), and they might be requested at different time resulting in potentially different data for the same page.
For the IAC part, i totally agree, and that's why for V3 rc we include a reference CDK implementation that should help people to more easily create their own IAC.
If you could create a PR to improve the docs, it would be greatly appreciated

@conico974  I'm so pleased that there is going to be a reference implementation--I think that will probably answer the questions that sometimes docs struggle to.
To be clear, I don't have a problem with calling something dangerous per se, it's more that it isn't clear why it is dangerous.
Happy to submit some changes, but I'll need to clarify a few things further in order to do so.
Here are my thoughts based on your explanation above:

the S3 Cache should be considered part of "Core" instead of part of "ISR" (as per the diagram) since it also effects SSG and soon SSR.

S3 buckets are very simple from an IaC perspective and cheap in terms of cost, so I can't see any reason not to do this.

the docs should make it clear what applies to app router only or pages router only where appropriate.

the (vast?) majority of NextJS apps/teams that would benefit from moving to OpenNext (ie probably a few years old, need to leverage other cloud infra, maxed out Vercel team size or have a large bill, etc) are using the pages router and will want to write IaC to support their pages router application without having to become an expert in app router

I really like the diagram but perhaps it is has the potential to mislead (as per the prior Cache discussion)

Looking forward to your thoughts--what would you suggest to improve the docs?

Sorry for the delay,

the S3 Cache should be considered part of "Core" instead of part of "ISR" (as per the diagram) since it also effects SSG and soon SSR.

Not sure that we should put it in Core, renaming the ISR part to ISR/SSG would probably be a better idea.
As for the SSR part, it will only be if you specifically enable Partial Prerendering (only for app router, and optional), and to be clear this is not fully supported at the moment. (The cache are not properly populated, and the roundtrip to S3 might make it totally worthless, especially if the static part is fast to compute)
We should probably add a specific page about partial prerendering, because it cannot work the same way that it does on vercel. On vercel the page is served from the CDN, very likely from a cloudflare worker, that's impossible to do outside of vercel at the moment, you have to reach the server function. To make it work we would have to move the cache outside of the server function in the routing/middleware part (something that V3 will make possible, maybe in V3.1) and reconstruct the streamed html from the precomputed rendered part and the postponed RSC

the docs should make it clear what applies to app router only or pages router only where appropriate

We should probably add a paragraph about that in the docs, in the architecture page.
At the moment there is only 1 difference from an IAC point of view, dynamoDB that is not necessary for pages router. Of course this might change in the future. We should probably also add that streaming is useless for page router and ISR/SSG (even with app router)

I really like the diagram but perhaps it is has the potential to mislead (as per the prior Cache discussion)

We should probably keep the diagram, we might need to revisit it a little bit and add a disclaimer that you should read the rest of the docs to really understand everything.
I think the diagram should be for people that just want to have a broad picture of what's going on under the hood, if you want to write your own IAC, you should read everything else
Anyway, the docs will need a big revamp with the V3 release with the introduction of the open-next.config.ts file and all the part that can be overriden. For example the S3 cache become the Incremental Cache and the DynamoDB cache become the Tag cache


Azure Frontdoor: Clarification on shared response (caching)

Hi, I have a question regarding this specific sentence in the Azure FrontDoor docs pertaining to caching:

However, if there are multiple requests in-flight at a POP for the same URL, they may share the response.

As I read that, if two or more users made a request to the same url (e.g., POST /api/authenticate) they both may receive the same response - presumably whichever of the two requests responds first.
I can only assume that my understanding here is incorrect, as in my example of authenticating if this were the case, then both users would get the same authentication token, and both be logged in as the same user.
Could someone clarify the behaviour in such scenarios?
Thanks,
Joe

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

ID: 2973ab88-1258-8a80-1344-07a9e0063951
Version Independent ID: c0cff4d9-d7fd-7b25-52da-9cbe5dda3ba8
Content: Azure Front Door - caching
Content Source: articles/frontdoor/front-door-caching.md
Service: frontdoor
GitHub Login: @sharad4u
Microsoft Alias: sharadag

@AviateX14  Thanks for reaching out. We will investigate and update the thread accordingly.

@AviateX14 , Appreciate your patience. There is a misconception here in understanding how caching works at front door. We don't cache user Id's, customers cache objects like images, videos, CSS files etc.so when a request is made against an object that is cached, it returns that object and has no correlation with user id.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Wise 5tuple support

This introduces support for tagging exact connections
(using standard tuple - proto:src_ip:src_port,:dst_ip:dst_port),
which can be used for example, to add references for ssl-stripped streams

sorry for delay was on vacation, looks good although why the underscore for _5tuple?

I think you cant have number at the begging of names in js or c

You have to use quotes, kind of a pain.

I reverted this for a couple reasons

Since it isn't cacheable, except for probably udp, should be disabled by default
Formatting should be fixed to match, and _5tuple changed to 5tuple or many folks use tuple5 to deal with leading number
needs tests (which I can do if required)

I'll be gone for next few weeks, so if you can handle those changes it will be great
if not I'll get back to you in September ;]


switch driver model needs documentation

Most of the assumptions behind the switch driver model are undocumented. This came to my attention when helping @rohgarg get up to speed for working on issue #27. We should actually write all of this down in an easy to understand way in an easy to find place. To start, here is an email I sent to @rohgarg in response to one of his questions:
Hey Rohan,

Several points of clarification:

* You should always be working from the devel branch - newapi is for a
  specific change unrelated to #27. (It's not the driver api it's
  talking about). What's on there was actually out of date; I just
  merged the changes to devel back into it.

* The haas.dell module on devel is representative of what a driver
  should look like.

* The cisco driver reflects the old driver api, which in the course of
  writing the new dell driver, I decided to clean up. You'll notice it
  expects some of the parameters to be comma-separated lists in the form
  of a string - yuck. If we wanted to use this driver again, we'd have
  to update it.

* The config file needs to contain both a general "which driver to use"
  setting, and possibly some driver specific options in their own
  section (for example a "switch dell" section).

  Things which might belong in the switch-specific section include
  "what is the" switch's ip address?", "who do I log in as?" and so
  forth. Depending on how we're interfacing with the switch these may
  not be the exact same parameters from one driver to another. As
  an example, the dell driver is adapted from an earlier driver
  that functioned similarly, but talked to the switch via serial,
  rather than telnet. This means, obviously, no ip address was
  needed. (and in fact no username/password either, if I recall).

As Zhaoliang said, the old dell driver *did* use SNMP, but it was
setting ports into general mode, which wasn't quite what we wanted.
Peter spent the better part of a day getting nowhere trying to figure
out how to set the ports to access mode via SNMP, after which we decided
it would make sense to just write a driver that talks to the console.
It's still doing all of the same things with Vlans and ports as what we
had agreed on previously, it just implements these via telnet and
console commands, rather than SNMP. As I mentioned before, I changed the
driver api in the course of writing that new driver.

Hope this helps,

-Ian

This could make a good introductory issue to work on; working with either @gsilvis @zenhack or @lokI8

Quoting henn (2015-04-10 14:38:42)

This could make a good introductory issue to work on; working with
either [1]@gsilvis [2]@zenhack or [3]@lokI8

There are some fairly detailed docstrings in haas/drivers/__init__.py.
What do you think is missing?

@zenhack, do you consider your issue to be resolved then?
I wasn't sure if you were advocating additional developer docs. Maybe we should toss a reference to this in HACKING.rst?

Ack, totally missed that this was an existing issue; I saw it in my inbox and thought you'd opened a new one.
The error handling behavior is still unspecified, which I think is the only bit of this that still needs doing.
I'm actually going to remove the easy label -- the existing drivers don't really take errors into account at all, and figuring out what they should do when something goes wrong (which is a prereq for documenting it) will take a bit of thought.

What is the status on this? I looked through the docs and switch drivers and it doesn't seem to be resolved. Should it get moved to v2.0 milestone?

Afaik, nothing has changed from the last time this was commented on; documentation is pretty good as far as it goes, but we just don't specify anything about error handling.
As to whether we should put it off until v2.0, last I was involved we still didn't having anything resembling a consensus on what the version numbers even meant. I'd been advocating for some version of [semver][1], which would mean v1.0 denotes API stabilization. A prereq for that is deciding what the API that we're stabilizing is: is it the HTTP Api, the driver interface or both?
If we're including the driver interface in our semver, then unless we're willing to flag 1.0 as still experimental, I think this needs to be fixed first. 2.0 would be the next opportunity, since it would probably necessitate API breakage.
Then there arises the question of how long are we planning on maintaing a major version? If we're not willing to drop 1.0 support as soon as 2.0 is out, then I don't want to commit us to a driver interface until we've actually got something reasonable, which right now we don't.
If we're not including the driver interface in the semver version, then this question is completely irrelevant, since we can stick it in whenever.
The above discussion all assumes folks are on-board with the whole semver thing, but I've had this conversation with @henn in the past and failed to convince him that it was the right/obvious thing to do.
I'd actually like to propose a moritorium on any use of milestones with a version number on them until we agree on what those numbers actually mean, and write it down.

I'm going to close this; the only part that's still relevant is the bit that's described in #54.
 This is our oldest open issue. We have more than one issue that is over 2 and a half years old. I think I'm going to start periodically triaging the oldest of them, because wow.


Changing MatTheme closes drawer

Hi
I am using MatThemeProvider in MainLayout to apply the full theme on the app,  and I want to change the theme from other components, not MainLayout. Everything is working fine, but when I update MainLayout by StateHasChanged, the drawer closes automatically and requires two clicks to reopen, of course becuase the open state doesn't changed
this is a project url to explore all the code and find the problem
https://github.com/AmirImam/git-github.com-AmirImam-MatBlazorDemo
thank you

I am experiencing the same issue with this code:
    protected override void OnInitialized()
    {
        NavigationManager.LocationChanged += LocationChanged;
        NavigationManager.TryGetQueryString<string>("make", out string make);
        Make = make;
    }

    void LocationChanged(object sender, LocationChangedEventArgs e)
    {
        NavigationManager.TryGetQueryString<string>("make", out string nextMake);
        Make = nextMake;
        StateHasChanged();
    }

Whenever the location changes, the appbar closes. This code runs in App.razor which is my root component. And Make is a cascading parameter read by almost all pages in the application.

I am experiencing the same issue with this code:
    protected override void OnInitialized()
    {
        NavigationManager.LocationChanged += LocationChanged;
        NavigationManager.TryGetQueryString<string>("make", out string make);
        Make = make;
    }

    void LocationChanged(object sender, LocationChangedEventArgs e)
    {
        NavigationManager.TryGetQueryString<string>("make", out string nextMake);
        Make = nextMake;
        StateHasChanged();
    }

Whenever the location changes, the appbar closes. This code runs in App.razor which is my root component. And Make is a cascading parameter read by almost all pages in the application.


Syncing local directories with spaces in their names

I want to keep directories in sync, which both reside on my system, so I don't need ssh. I don't know if this makes a difference for this scenario, but at least for completeness' sake.
So, when I set up bitpocket with just a path, it works nicely, as long as this path has no spaces in it. If there are spaces, however, it  drops the last chunk of the path and can't complete its mission
For instance, I set up
bitpocket init "/media/newhome/sync test/"
bitpocket sync
rsync: [sender] link_stat "/media/newhome/sync" failed: No such file or directory (2)

Is there a workaround?

Escaping the space with a backslash doesn't help. I tried that already.

I think this may have been fixed in #85?

Seems, so I did not have time to test it, but it looks as if it should.


How can I validate "_csrf" in Ajax Request.?

I am using jQuery Ajax to make a request, but in the action I want to check the _csrf, but It doesn't work.
This is my code:
JavaScript Function
$.ajax({
    url: urlAjax,
    type: 'POST',
    data: {"id": 1, "_csrf": yii.getCsrfToken()},
    dataType: 'json',
}).done(function (response) {
    console.log(response);
});

PHP Controller
public function actionTest($id)
{
    Yii::$app->response->format = \yii\web\Response::FORMAT_JSON;

    $request = Yii::$app->request;
    $response = [
        'error' => false,
        'msg' => '',
    ];

    if (!$request->validateCsrfToken()) {
        $response = [
            'error' => true,
            'msg' => 'Error CSFR'
        ];

        return $response;
    }

    if ($request->isAjax) {
        try {
            $model = $this->findModel($id);
            ....
        } catch (Exception $exc) {
            $response['error'] = true;
            $response['msg'] = 'ERROR';
        }
    } else {
        $response['error'] = true;
        $response['msg'] = 'OK';
    }

    return $response;
}

I have set enableCsrfValidation = true, but the function $request->validateCsrfToken() always return true.

but the function $request->validateCsrfToken() always return true.

because your token is valid. What the matter?

The problem is when I don't send the value of _csrf, even the function return true, or when I send incorrect value of _csrf, It always return true.

make sure you use request->csrfParam ?>: 'request->csrfToken; ?>' in your ajax data get the exact csrfParam with it's value don't put static string _csrf.
Secondly yii-controller verifies the _csrf by default if you have set $enableCsrfValidation = true; so you don't need to verify _csrf again.

csrf token also send via request header. Check it to.

Closed since it's not issue report but a question. Should be asked via forums.

I had this problem.I write below function by Yii2 internal Yii::$app->request->validateCsrfToken code:
function validate_csrf($secret){
        $security = Yii::$app->security;
        $trueToken = Yii::$app->request->csrfToken;
        return $security->compareString($security->unmaskToken($secret), $security->unmaskToken($trueToken));
}


Applying a core condition for reaction with CUB active not working

"downgrade" or "Weakened" conditions do not work if CUB is active and they are set to Reaction status name, CUB conditions work fine.

Solved in 1.5.4. Thanks for the report.


[C++][Parquet] remove useless template parameter of DeltaLengthByteArrayEncoder

Describe the enhancement requested
https://github.com/apache/arrow/blob/03726178494c8978bf48b9bab15ed9676e7c9196/cpp/src/parquet/encoding.cc#L2735-L2737
It seems that the template parameter DType is never used. In addition, only ByteArray could be encoded with DeltaLengthByteArrayEncoder. So maybe it can be removed.
Component(s)
C++, Parquet

Issue resolved by pull request 43250
https://github.com/apache/arrow/pull/43250


ZendDeveloperTools and Database support trigger fatal error

In install process If I choose to install both ZendDeveloperTools and database support the
home page is broken with error
Notice: Undefined index: db in /var/www/newzf/vendor/zendframework/zend-db/src/Adapter/AdapterServiceFactory.php on line 27
Fatal error: Uncaught exception 'Zend\Db\Adapter\Exception\InvalidArgumentException' with message 'The supplied or instantiated driver object does not implement Zend\Db\Adapter\Driver\DriverInterface' in /var/www/newzf/vendor/zendframework/zend-servicemanager/src/ServiceManager.php on line 757
Zend\Db\Adapter\Exception\InvalidArgumentException: The supplied or instantiated driver object does not implement Zend\Db\Adapter\Driver\DriverInterface in /var/www/newzf/vendor/zendframework/zend-db/src/Adapter/Adapter.php on line 82
Zend\ServiceManager\Exception\ServiceNotCreatedException: Service with name "Zend\Db\Adapter\AdapterInterface" could not be created. Reason: The supplied or instantiated driver object does not implement Zend\Db\Adapter\Driver\DriverInterface in /var/www/newzf/vendor/zendframework/zend-servicemanager/src/ServiceManager.php on line 757

This is pretty much expected since there is missing config for db:
'db' => [
    'driver'   => '...',
    'dsn'      => '...',
    'username' => '...',
    'password' => '...',
],

there is a PR to handle this: https://github.com/zendframework/ZendDeveloperTools/pull/218

Nice.
Still, the issue should be marked as bug since occurs in ZendSkeletonApplication.
Can not be more precisely:

For new user that "just try" and see the 'undefined index "db"', it will stopping user to continue learning ;)

Is this still an issue?  I wasn't able to reproduce this using the install method described (composer create-project then opt to toolbar and db) so it looks as though samsonasik's PR on ZDT fixed this.

Yes, I can confirm also, the bug is resolved.


fix: Fix npm cache add to use single arguments

Fixes https://github.com/o1-labs/snapp-cli/issues/44

fixed in another PR: https://github.com/o1-labs/snapp-cli/pull/48


Change role claim to roles in XACML policies and add a group based authentication template

Part of https://github.com/wso2/product-is/issues/13739
Changes in the PR:

Change the http://wso2.org/claims/role to http://wso2.org/claims/roles
Introduced a group based authentication XACML policy template

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2383147767

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2383147767
Status: success


[skip-ci] feat: hasta la vista identicons generated icons

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

High Risk PRs Require 2 approvals

What protocols, transaction types or contract interactions might be affected by this PR?

Testing

Engineering

Operations

Screenshots (if applicable)

Closing to cherry-pick stacked on top of https://github.com/shapeshift/web/pull/7408 instead


Pre-collapse feature enhancement

The preCollapseTitleRow feature needs some work
For design see https://github.com/carbon-design-system/ibm-cloud-cognitive/issues/59#issuecomment-776653465

Replace with preCollapse of whole header
Add expand/collapse button. @cameroncalder open questions copied from here https://github.com/carbon-design-system/ibm-cloud-cognitive/issues/59#issuecomment-776653465

Where does the chevron go if there are no tabs/tags?
If the user scrolls down partially which way does the chevron point
Is the chevron aligned to the right of the page like the action bar?

Where does the chevron go if there are no tabs/tags?

The chevron, used within the field size icon button, can still be placed in the same location as the tab/tag version. The background needs to be a prereq for the chevron/manual page header collapse functionality.
Example without tabs for reference

If the user scrolls down partially which way does the chevron point

The chevron should point up until it reaches the full collapsed state, then it points down.

Is the chevron aligned to the right of the page like the action bar?

yessir


Increase image disk size

20GB disk size is to small for several uses that could benefit
from a standard Vagrant image provided by openSUSE.

Increase image size to 40GB
Bump version to 1.0.3

I included some changes by mistake, will send the PR again. Apologies.


[Mention The Feature]: Improving layout of the Blog Section

I will align the items properly by using grid or flexbox
Checklist

[X] I have checked the Existing Issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

@aditya-bhaumik
please review

this issue has been solved, so we are closing this issue.


Update portainer agent kubernetes manifest ClusterRoleBinding

Before you start, we need a little bit more information from you:
Use Case (delete as appropriate): Using Portainer in a Commerical setup. Kubernetes use case.
Have you reviewed our technical documentation and knowledge base? Yes

Question:
While upgrading to Portainer EE 2.10 I reviewed the official kubernetes agent manifest which is this one: portainer-agent-ee210-k8s-lb.yaml
There I see that you are still making use of apiVersion: rbac.authorization.k8s.io/v1beta1 for the ClusterRoleBinding. Shouldn't that be switched to apiVersion: rbac.authorization.k8s.io/v1 for kubernetes v1.22 and above?

Thanks for that @JamborJan
Fixed now


Fix Flaky test testMultiValuedMapIterator

PR Overview
This PR propses a fix for the following tests:

org.apache.commons.collections4.multimap.ArrayListValuedHashMapTest.testMultiValuedMapIterator

org.apache.commons.collections4.multimap.TransformedMultiValuedMapTest.testMultiValuedMapIterator

Test Overview
The test testMultiValuedMapIterator tests how the iteration of a multi-value map works. A MultiValuedMap is basically a map with a key value that can associate itself with either a single value or multiple values (Collections).  Here the MultiValueMap has a key that associates to an array of values.
You can reproduce the issue by running the following commands -
mvn install -pl . -am -DskipTests
mvn  -Dtest=org.apache.commons.collections4.multimap.ArrayListValuedHashMapTest#testMultiValuedMapIterator
mvn  edu.illinois:nondex-maven-plugin:2.1.1:nondex -Dtest=org.apache.commons.collections4.multimap.ArrayListValuedHashMapTest#testMultiValuedMapIterator

Problem
In this test, a multivalued map is defined and an iterator is defined on top of that. The test is flaky since the order inside the multivalued map is not maintained.
Since the order is not maintained, the iterator can have any one key of the three keys for this test ( one, two, three) in random fashion. This flakiness was identified by the nondex tool created by the researchers of UIUC.
The following code snippet fails because of the above reason
https://github.com/anirudh711/commons-collections/blob/35e408717379eed0085cdb29e879209dbadc1ead/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java#L837-L857
Fix
To maintain order, we can implement a TreeMap or a LinkedHashMap but that will change the base code and affect all other entities related to this.
This PR proposes a fix inside the test while keeping the test logic intact.  Since the order is not maintained, we check for the iterator's next value to either be one of the keys and if it is, we check the original assert statements. We follow the same approach to assert all the cases.
https://github.com/anirudh711/commons-collections/blob/02f85e2c9f6cba3a9c76ab30e51bd9d64070977d/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java#L837-L860
This PR provides a fix on the abstract class which fixes the classes that implement it, hence fixing the tests with classes implementing the abstract class

I understood the problem here. However, since you are again using the next operation, isn't it possible that the key "one" is tested three times?

The problem seems fully fixed. However, I would suggest add a note about the nondex tool that used to detect flakiness.

I understood the problem here. However, since you are again using the next operation, isn't it possible that the key "one" is tested three times?

Yes, It is checked three times in the worst case but it is still an O(1) operation - time complexity will remain the same, the number of times the if statement runs will vary. Do you have any suggestions for alternate fixes?


Update Container Build Action

Modify the Github Action for publishing the image. Have an option to specify the tags and publish only after approval (Selecting Tag can serve as approval)

Right now same middleware build is deployed across states. Will reopen the issue if specific tags need to be introduced.


Missing LICENSE file

I see from the readme it's licensed under Apache v2.0 ~ could benefit from the quick addition of a license file for search indexing.

Thanks for the heads up, I've now added an actual license file!


Constant logging of "HHH000400: Using dialect" while using JSONType

Hi!
I am using Hypersistence Utils Hibernate 52 to map some Strings to JSON into a Postgres DB and additionally I have my own dialect (due to some magic I had to do with numeric types).
Your JsonType is bound in as
@org.hibernate.annotations.TypeDef(     name = "json",     typeClass = io.hypersistence.utils.hibernate.type.json.JsonType::class )
and then @Type(type = "json").
The whole constuction works pretty neat, except the fact that in the log the following entry appears in a random manner: HHH000400: Using dialect: org.hibernate.dialect.PostgreSQL10Dialect.
My best guess is, that while working with JSONType Hibernate is constantly changing the dialect back and forth.
Question: is there a way to register your type on the "top of" my dialect, or is there any other (better) way to stick with only one dialect?
Thanks a lot in advance!

This log:

HHH000400: Using dialect: org.hibernate.dialect.PostgreSQL10Dialect

is an INFO or DEBUG message, not a WARN.
Now, if you created a custom Dialect to fix the No Dialect mapping for JDBC type issue, then check out this article for a better alternative.


[BUG]: Edit Template : Advance editor preview fucntionality isn't working

Steps to reproduce

Create a new schema
Go to Create Vc template
Select schema
Click  on add template
Click on edit template
Click  advance editor on menu bar

Getting blank page after clicking on advance editor functionality

@Pritzz4 This issue is not reproducing. please check out.


How to override Amplify API @searchable resources

Note: If your question is regarding the AWS Amplify Console service, please log it in the
AWS Amplify Console repository

Which Category is your question related to?
API
Amplify CLI Version
9.2.1
You can use amplify -v to check the amplify cli version on your system
What AWS Services are you utilizing?
OpenSearch, Lambda, IAM
Provide additional details e.g. code snippets. Be sure to remove any sensitive data.
How can I append to the OpenSearch Streaming Lambda IAM Role policy using override.ts? I have attempted adding the following code in the api override.ts file, however, it seems to fail silently and ignore all other configurations in the override.ts file.
    const searchStreamingLambdaName = "test-name";

    resources.opensearch.OpenSearchStreamingLambdaFunction.functionName = searchStreamingLambdaName;

    const lambdaRole = resources.opensearch.OpenSearchStreamingLambdaIAMRole;

    const basePolicies = Array.isArray(lambdaRole.policies)
      ? lambdaRole.policies
      : [lambdaRole.policies];

    resources.opensearch.OpenSearchStreamingLambdaIAMRole.policies = [
      ...basePolicies,
      {
        policyName: 'custom-transform-lambda-permissions',
        policyDocument: {
          Version: '2012-10-17',
          Statement: [
            {
              Resource: '*',
              Action: ['lambda:InvokeFunction'],
              Effect: 'Allow',
            },
          ],
        },
      },
    ];

More specifically, when I run amplify push -y, I notice that the OpenSearchStreamingLambdaFunction gets recreated with a different name (something auto-generated) and the policy I added is not present in the IAM role. When I remove the line that defines resources.opensearch.OpenSearchStreamingLambdaIAMRole.policies, I see the OpenSearchStreamingLambdaFunction created with the value of searchStreamingLambdaName
Am I incorrectly appending the policy to OpenSearchStreamingLambdaIAMRole? I followed the documentation for overriding the project level IAM auth role

Based on my latest test, I noticed that removing ..basePolicies allowed the remaining overrides to work & correctly set OpenSearchStreamingLambdaFunction. I can manually copy over the current default policies, however I want to make sure those don't get overridden.
    const basePolicies = Array.isArray(lambdaRole.policies)
      ? lambdaRole.policies
      : [lambdaRole.policies];

Hey @dhruvbansal2, thank you for reaching out. Tested the following API override in my amplify application.
added the following code in my application
 resources.opensearch.OpenSearchStreamingLambdaFunction.functionName =
    "tstname";
  var lambdaRole = resources.opensearch.OpenSearchStreamingLambdaIAMRole;

const basePolicies = Array.isArray(lambdaRole.policies)
  ? lambdaRole.policies
  : [lambdaRole.policies];

  resources.opensearch.OpenSearchStreamingLambdaIAMRole.policies = [
    ...basePolicies,
    {
      policyName: "custom-transform-lambda-permissions",
      policyDocument: {
        Version: "2012-10-17",
        Statement: [
          {
            Resource: "*",
            Action: ["lambda:InvokeFunction"],
            Effect: "Allow",
          },
        ],
      },
    },
  ];

but was unable to observe the behaviour in my application.

As a mitigation step could you try adding the base policy back in and move the function name line to the end of the file. Then push the application, check the IAM policy, function name and let us know if the the function names changes to the default name with random string.

@ykethan Thanks for the response.
When attempting to print basePolicies in the override.ts, I would see it is undefined - is that expected?
I've also console.logged both at the beginning and end of the file - both of which print.

Hey @dhruvbansal2, I did observe this behaviour in my application. Marking this as bug for further investigation.


Fix compatibility issues

aligned to work with node 12
aligned to work with latest aws-sdk
aligned to work with latest serverless framework
added support to replicate secure strings

@alessandrobologna please review this when you have time, would be nice to get this merged to master. thanks :)


ScrollTableView native scrolling support

Refactors ScrollTableView to support real native vertical scrolling.
Changes

Made the ScrollTableView use real scrolling instead of just replacing the data in the table.

This was the main point of the PR and all of the other changes are caused by this.
See the image below for the structure of the DOM elements to accomplish this.
There is always one <table> with three <tbody>'s in it. As the user scrolls we add and remove <tbody> elements. Each of them has their own DAO with a different skip value. The bulk of the code in this PR is just tracking these <tbody>'s and what data to feel them.

Added a background colour CSS macro so the sticky table header could set it's background colour.

It used to be transparent which was fine when the rows didn't move, but now that the rows move, they would be visible behind the table header.

Removed fitInScreen :: Boolean as an optional flag on ScrollTableView

This flag used to make sure the table height didn't extend off of the bottom of the screen. Now it's always on. Not sure if this is quite what we want. We probably want it as an option still that can be overwritten.

Changed the way OverlayDropdown is positioned. Now it's position is based on the coordinates of the mouse click event instead of the coordinates of it's DOM element.

The issue was that since the DOM element was inside the <td> element in the table and one of the containers containing the table needs to have overflow: hidden, the overlay would be cut off if it were near the bottom of the table. So since we can't put it directly in the table anymore, I chose to add it to the controller and then use the mouse coordinates to position it since we can't use the DOM anymore.

There's one optimization I know of that we could still apply to this but I haven't made yet. When a user scrolls more than one page in a single event (most often this happens when quickly dragging the scrollbar with the mouse on a large table) it currently loops over the scroll{Up,Down}OnePage method, which removes one table and adds another. Therefore it would still be drawing and removing every tbody between the starting and ending positions of the scroll rather than just removing all three tbodies and redrawing three new ones at the ending position.

Has conflicts.

Has conflicts.

Resolved.


Web auth workspace network requirement clarification

Please clarify the subnet IP requirements for the databricks workspace dedicated for web auth WEB_AUTH_DO_NOT_DELETE_
Currently the deployment fails if we provide any less range than /26 (64 ip's) for each subnet - the requirement for 128 ip addresses (64+64) for nothing (i.e. this workspace will not have any clusters at all) and that 128 IP address is a waste of IP's
But the doc do say "It does not need to have the same requirements as listed in the main Private Link article."
What is meant by that line? either the doc is wrong or there is another way to overcome this /26 for each subnet hard lower limit.
https://learn.microsoft.com/en-us/azure/databricks/administration-guide/cloud-configurations/azure/private-link-standard

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

ID: 739b6b6a-134f-321c-041a-2ec35953e491
Version Independent ID: 739b6b6a-134f-321c-041a-2ec35953e491
Content: Enable Azure Private Link as a standard deployment - Azure Databricks
Content Source: databricks/administration-guide/cloud-configurations/azure/private-link-standard.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@subashsivaji
Thanks for your feedback! We will investigate and update as appropriate.

@subashsivaji
Thank you for bringing this to our attention.
I've delegated this to content author @mssaperla, who will review it and offer his insightful opinions.

Thanks for your dedication to our documentation. We've created an internal work item (DOC-7911) to address your feedback. The timeline for resolution varies based on resourcing. We are closing this issue for now, but feel free to comment here as necessary.
#please-close

#please-close

@kateglee-db This seems to have trailed off and just been closed. What was the outcome of DOC-7911?


Missing v2.4.59 tag

The version 2.4.59 is not yet available https://hub.docker.com/_/httpd/tags

As per comment here https://github.com/docker-library/httpd/pull/257#issuecomment-2040300296 it will be available shortly.


pocket dimension needs long time to load

After long using the pocket portal, the server needs a long time to trigger the teleport.

This happens because every block that is set triggers the whole lighting update, I couldn't get it work triggering the lighting update only a single time. So it seems that we have to decide between lights being server-side and absolute mess or the pocket dimension having a very long loading time. I probably go to the lights being server-side an absolute mess, as it doesn't have a lot of impact (client-side, the pocket dimension is full-bright)

So this line is the bottle neck because it's the other line doesn't really do anything (the one setting barrier blocks), at least when first entering a pocket dimension. If find the waiting time reasonable so I'm closing now


CIV-14036 Populate HWF Data

Before creating a pull request make sure that:

[x] commit messages are meaningful and follow good commit message guidelines
[x] README and other documentation has been updated / added (if needed)
[x] tests have been updated / new tests has been added (if needed)
[x] Screenshot attached for the Evidence of manual testing
[x] Functional, Smoke and API tests have been run locally
[x] Full gradle build and run tests with coverage have been completed (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-14036
Change description
Populate and Store HwF data Fields to display in Application CaseType Tab

Set HwF type
Add new fields to store PBA details and HWF reference for HwF payment and HwF Additional Payment

Dependency PR
https://github.com/hmcts/civil-general-apps-ccd-definition/pull/1208
Testing branch and instruction
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No


Using md-select without a v-model

When using md-select without a v-model, the html  composant isnt synced and return an empty value.
Example :
http://codepen.io/anon/pen/OWgrEB

Sorry, but I didn't understand what you need.


feat(DatePickerE): Add focus trap

Related issue
Resolves #2870
Overview
This adds a focus trap to DatePickerE using the react-focus-trap library. This means that the focus stays within the picker when it is open and the tab key is pressed.
Note: General keyboard navigation within the picker is handled by react-day-picker and is unaffected by this change. (Note that it has been significantly improved for react-day-picker v8, so we'll get those fixes when we are able to update.)
Link to preview
Storybook
Non-Storybook
Reason
This is to be consistent with https://www.w3.org/TR/wai-aria-practices/examples/dialog-modal/datepicker-dialog.html.
Work carried out

[x] Integrate react-focus-trap library with DatePickerE

Demo
Before

After

Developer notes
This also replaces the existing document click and escape key logic for this component as these are both handled by react-focus-trap (and focus-trap).
The isOpen prop was also changed to affect the first render only and renamed to initialIsOpen (matching SidebarE). This is because it doesn't make much sense to have it affect subsequent renders – you never know whether the prop or the internal state is correct at any point in time.
Finally, the usehooks-ts library was added to gain access to some utility hooks it provides (useBoolean in this case).

Thanks very much @jpveooys ! As discussed:

There is a browser highlight around buttons and the sheet itself now, which shouldn't be there

The initial focus is currently on the sheet itself. If possible we should set focus either

on the currently selected/typed date; or
on today's date, if no other date has been selected.

If that's not possible, then setting focus on the left button would be preferable.

Currently the keyboard arrows change:

the month/year when focus is on the sheet or buttons
the selected date when focus is on a calendar day

Even though it's great to have a way to quickly change month/year, this behaviour can lead to mistakes, because the same keys work in different ways in a very similar context.
Our future plans include a month/year selector within the visual date picker itself, so for now I'd suggest disabling this additional functionality if/when possible.
Thanks!

Thanks for the review @k9dh3zij !

There is a browser highlight around buttons and the sheet itself now, which shouldn't be there

I've removed these (though nothing has changed on our side – could be something has changed in Chrome if it has changed...)

The initial focus is currently on the sheet itself. If possible we should set focus either ...

As discussed, in react-day-picker v7 it's not possible to focus a date in that manner, but it will be possible in v8. I've changed the initial focus to the previous month button for now though.

Currently the keyboard arrows change: ...

This also isn't customisable in react-day-picker v7 (only way to turn it off also turns off the next and previous month buttons...) But it's gone in v8 :smile: So it's there for now, but I'll pick up the v8 issue again separately and see if we can work around the blocker that was preventing us updating :slightly_smiling_face:

Thanks for making those changes @jpveooys. Looks great and I think the initial focus on the button rather than the sheet itself works better.


Elastic Queue Problems

Hi, When i use Laravel Scout option "queue" => 'true'  i have a error in my queue docker container No alive nodes found in your cluster, but  everything work without queue.
I enter the docker-container and ping + Curl elasticsearch host, Elastic works and all ok. But i don't understand problem.....

Docker logs it's ok. I had this problem before in  normal use, I fixed it  ElasticSearch option  "xpack.security.enabled=false". Now i am having the queue problem and i don't know why it's happened

I don't understand it either, but it seems not to be related to this package.


Add visual feedback to pan/tilt buttons

Brefily highlights the pan and tilt arrows when they are clicked. This helps clarifiy their behavor, acting once per click, rather than while clicked and held down. This is especially helpful on touchscreen devices.

Thanks for that


[Bug Report] upload multiple file error

Element UI version
2.0.7
OS/Browsers version
OS X 10.12.6 / Chrome 62.0.3202.94
Vue version
2.5.9
Reproduction Link
https://jsfiddle.net/0x81xhxg/
Steps to reproduce

Click upload button.
Select multiple file.

What is Expected?
Upload success and show file.
What is actually happening?
Does not show file.

Use a real file upload API instead of https://jsonplaceholder.typicode.com/posts for action prop. As its name suggests, the latter is just a placeholder.

Thanks.


test issue

@all-contributors add @JahanUoL for code, design and testing :)

@all-contributors add JahanUoL for code, design and testing :)


[Flake]: integration tests failed with IAM error

It seems that IAM concurrency errors are reported as kAborted.  This suggests that the idea outlined in #613 is probably more useful than I thought at first:
https://source.cloud.google.com/results/invocations/592338ff-4902-4005-b890-ee9c9b3117e8
INFO: From Testing //google/cloud/spanner/samples:spanner_client_samples:
==================== Test output for //google/cloud/spanner/samples:spanner_client_samples:
Running instance admin samples on test-instance

Running get-instance sample
The instance projects/cloud-cpp-testing-resources/instances/test-instance exists and its metadata is:
name: "projects/cloud-cpp-testing-resources/instances/test-instance"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "Instance for Integration Tests"
node_count: 1
state: READY

Running list-instances sample
Instance [1]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "Instance for Integration Tests"
node_count: 1
state: READY

Instance [2]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-0"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-0"
node_count: 1
state: READY

Instance [3]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-1"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-1"
node_count: 1
state: READY

Instance [4]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-10"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-10"
node_count: 1
state: READY

Instance [5]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-11"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-11"
node_count: 1
state: READY

Instance [6]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-2"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-2"
node_count: 1
state: READY

Instance [7]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-3"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-3"
node_count: 1
state: READY

Instance [8]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-4"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-4"
node_count: 1
state: READY

Instance [9]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-5"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-5"
node_count: 1
state: READY

Instance [10]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-6"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-6"
node_count: 1
state: READY

Instance [11]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-7"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-7"
node_count: 1
state: READY

Instance [12]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-8"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-8"
node_count: 1
state: READY

Instance [13]:
name: "projects/cloud-cpp-testing-resources/instances/test-instance-9"
config: "projects/cloud-cpp-testing-resources/instanceConfigs/regional-us-central1"
display_name: "test-instance-9"
node_count: 1
state: READY

Running (instance) get-iam-policy sample
The IAM policy for instance test-instance is:
version: 1
etag: "\007\005\223\003\01391D"

Running (instance) add-database-reader sample
Permanent error in SetIamPolicy: Concurrent IAM policy change for resource projects/cloud-cpp-testing-resources/instances/test-instance Please retry the whole GetPolicy()/SetPolicy() operation with exponential backoff. [ABORTED]

I saw another:
https://source.cloud.google.com/results/invocations/5231011e-12b5-4b72-ad9e-983a223f21ea/targets

And another:
https://source.cloud.google.com/results/invocations/5aff8e43-408b-4292-a8aa-ee3f839f0f73/targets

This flake should be gone now.

Can you say more?  Why is it gone?


0.2.3

Codecov Report

Merging #29 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #29   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          69     72    +3
=====================================
+ Hits           69     72    +3

Impacted Files
Coverage Δ

R/scriptexec.R
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 321bae2...d1972c0. Read the comment docs.


Select All option

HI,
It is a nice component, good work!
Do you think to add a "select all" option like clear option?

Hi,
we have added it in the last release 1.1.5 :)
Thorben :)

Hi,
thank you for your answer! I had not installed the last version.
i tried with the 1.1.5 version and I found the method setShowSelectAllButton. I had implemented the ShowButton and passed it.
However, it doesn't work. Any selection is not executed. I don't know why, because it worked before change the version. Maybe I have to do something more?
Thanks again
Marcos

I cant really refer to your example, could you give some code to read? :)

Sorry for my late response. I hope to explain it better this time :)
I have two comboBox multiselect. When I select an item (or option) in the first one, a valueChangeListener updates the items of the second one and select all these new items.
It worked with the old version. However, with the 1.1.5 version, the same code does not work. It execute well the method selectAll() in the valueChangeListener but it does not show any option selected. The setImmediate() is to true.
Do you think It depends of the paintContent()?
Thank in advance

sorry could you give some code example?

do to inactivity closed


fix: default url for otelcol

Which problem is this PR solving?

The default port of Opentelemetry collector

Short description of the changes

Change the default port Opentelemetry collector on port 55680 (in the example https://github.com/open-telemetry/opentelemetry-collector/blob/master/examples/k8s.yaml). I think port 55678 is Opencensus default

This is good to go, please resolve the conflicts.

Welcome to OpenTelemetry community and congrats for first contribution.


darwin.darling: make header-only

Description of changes
https://github.com/NixOS/nixpkgs/pull/228113#issuecomment-1521828065
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg build darwin.dtrace

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

2 packages built:

darwin.darling
darwin.dtrace


Turn Lighthouse on for all pages and fix failures.

Our current lighthouse test (here) , only runs against the main page.
The nathillardusds/failing_lighthouse branch fixes tests to run instead on all pages. When you do this, you get the below output:
acccessibility_failures.log
We should look through these and fix them

We turned it on for all pages and have lots of errors we need to fix!

Seeing some light on this.
While the cypress.axe alerts on a11y issues are great for CI/CD, it doesn't show where the error is happening.
The Axe chrome plugin is showing promise. On localhost it seems to not only identify a11y issues, but also point out where it's occurring. One this is addressed via the plugin,  then one can check the cypress.axe tests to see if alerts are passing. More to come on this.

All automated a11y testing (aka cypress-axe) are now passing! 🥳
For both mobile and desktop, all pages on the staging URL has an A11y lighthouse score of 100% ! 🥇
About = 100% ✅
Explore = 100% ✅
Methodology = 100% ✅
Contact = 100% ✅
The automated a11y testing (via cypress-axe) is more stringent than Lighthouse (it seems). The only visible change to the site that needs review was the addition of an <h1> tag to the About page:

http://usds-geoplatform-justice40-website.s3-website-us-east-1.amazonaws.com/justice40-tool/c40867/en/

Looking forward to your review!


Support for STM32U5

Are there any plans to add support for STM32U5 NUCLEO boards?
https://www.st.com/en/evaluation-tools/nucleo-u575zi-q.html

Those will be supported via CMSIS-Packs unless there is a strong reason to have built-in support, like the Packs don't work. (CMSIS-Packs are the preferred way to add new targets unless there are barriers to doing so.)

Ah I see - So I download the Keil.STM32U5xx_DFP.1.1.1.pack then pass ''--pack=Keil.STM32U5xx_DFP.1.1.1.pack'' to the PYOCD command.
Thank you!

Actually you can use pyocd pack install u575 to install the pack and not have to download it manually. This also makes the target types available without needing the --pack argument.
The target support documentation has all the details.

Installing like that doesn't seem to work properly: This is the error I receive
pyocd pack install u575
0000741 W No matching devices. Please make sure the pack index is up to date. [pack_cmd]

That's odd. I just tried it again and it worked fine, even without have a pack index installed previously. I guess it's possible you have an out of date pack index from before the U5xx pack was added?
Please try adding -u or --update to the command line: pyocd pack install -u u575.
You can also try: pyocd pack clean ; pyocd pack update, then run the install command. This will completely erase and re-download the pack index before attempting to install. (It should be more or less equivalent to the -u argument, but could fix an index that was somehow corrupted.)

Very interesting! Thanks for trying and I'm glad it worked eventually…
One strange thing is that the warnings are different between the 2 pack updates, and also different from my own tests.


Shortcut to save bundle

Once again pointed out by @thomaskisler: a shortcut to save a bundles annotation would be a nice thing to have... once again I agree :-)

feature implemented (see 9e00c395e7726a92ff9970b11cd08b47c919d1dd)


DOC: Correct typos

Fix some simple typos in the comments.

Coverage decreased (-0.005%) to 90.806% when pulling 4316ddafcdf689f95d00aabf496cae6edfa29391 on hoechenberger:patch-1 into b25fbf28a83ea45021933d891a74f02b4bdfe5a9 on mne-tools:master.

Thanks @hoechenberger


Add some useful examples of prerun modifiers to the docs

document how to use prerun modifiers to skip over setup/teardown. Also document how to skip over tests cases and test suites. Needs to include code examples.

I really liked the proposed enhancements. Some of the examples could be simplified if #2302 is implemented. Should we wait for that or should other enhancements be merged already now? Do you @jaloren have time to work with this in the near future? I have time this and next week but then I'll be one week away.

I have time to work on this doc enhancement. I'd vote for waiting on the fix for 2302 and then simplifying the examples accordingly.

Cool, let's wait for #2302 and related #2303. They both ought to be pretty simple, but I doubt I have time for them this week and next week I'll be on holiday without much Internet access. I can implement them when I'm back, but you or someone else can obviously create PRs for them already before.

Do you @jaloren have time to look at this in the near future? Both #2302 and #2303 are done.

@pekkaklarck  sure. I think i could submit a PR in the next two weeks for this. Would that work?

Sounds good. It's unlikely RF 3.0.1 is release in two weeks so we ought to get enhancements in, but this being postponed to 3.0.2 wouldn't be a disaster either. Notice that you can also update the existing PR #2298 by just adding new commits to that branch.

@pekkaklarck the PR is done. See #2416. I did submit it on the same prerun modifier branch but it popped up different PR. Anyway, I think this will address the issue opened here and includes the changes that were made so it shows setup and teardown as directly settable.

Holidays, house moves, etc. has got on the way, but I'm now finally trying to get RF 3.0.1 ready. What's the status of this issue? PR  #2416 seems to be closed.

woops ....it shouldn't be. Its ready to be merged though. So i'll reopen it.

@pekkaklarck i believe PR #2473 can be merged into master. Please let me know if there are any issues.

Have been really busy the whole autumn and now try to get 3.0.1 out still this year. On a quick look PR  #2473 looks pretty OK but has some problems in examples I'd like to see fixed before merging. Better to postpone this to 3.0.2 than trying to rush it in.

New examples and related docs were provided by PR #2473 and pre-run modifier docs in general cleaned up and enhanced by 41b08341c67e35539cabe16458061db1691340fd.
Could you @jaloren take a look at how the examples and related docs look like after my changes?


Strange Codeset behavior in corba application depending on link options

Describe the bug
The codesets are not the same if I link with TAO_Strategies or not
Environment
Linux - ubuntu wsl2
g++ (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0
ACE_TAO 7.0.5 static build from vcpkg
To Reproduce
Steps to reproduce the behavior:
In my application, I use the directive
"-ORBSvcConfDirective", "static Resource_Factory \"-ORBNativeCharCodeSet 0x05010001\""
To manage the encoding interactions between my application and a Java client.
The IOR is as follows:
catior IOR:010000002e ....
Type ID: "IDL:com/xxx/yyyy/zzzz:1.0"
Profiles:
1. IIOP 1.2 <IP_ADDRESS> 7000 "\x14\x01\x0f\x00RST"8\xe3aZJ\x0e\x00\x01\x00\x00\x00\x01\x00\x00\x00\x02\x00\x00\x00"
      TAG_ORB_TYPE TAO (TAO\x00)
      TAG_CODE_SETS char native code set:       UTF-8
                    char conversion code sets:
                    wchar native code set:      UTF-16
                    wchar conversion code sets:

      TAG_SSL_SEC_TRANS port = 39877 supports 166 requires 134

I added the directive:
"-ORBSvcConfDirective", "static Advanced_Resource_Factory "-ORBReactorType dev_poll""
which requires:
#include <tao/Strategies/advanced_resource.h>
and linking with TAO_Strategies; the IOR comes up as this:
Type ID: "IDL:com/xxx/yyyy/zzz:1.0"
Profiles:
1. IIOP 1.2 <IP_ADDRESS> 7000 "\x14\x01\x0f\x00RST\xd3<\xe3a\x94R\x08\x00\x01\x00\x00\x00\x01\x00\x00\x00\x02\x00\x00\x00"
      TAG_ORB_TYPE TAO (TAO\x00)
      TAG_CODE_SETS char native code set:       ISO-8859-1
                    char conversion code sets:  UTF-8
                    wchar native code set:      UTF-16
                    wchar conversion code sets:

And now, the characters received by my java application don't have the correct values.
Do you have any ideas as to what might cause this behavior ? I'm guessing it relates to the static initialization of the TAO_CodeSet_Manager_Factory or of the TAO_Codeset_Manager.
I'm at a loss, any help would be appreciated
Running with -ORBDebugLevel 6, Notice that in the first case, we have the warning:
Codeset_Manager_i::init_ccs, codeset translator <UTF8_Latin1_Factory> has wrong ncs (65537), Invalid argument
because I require codeset 0x05010001; in the second case, I don't get this warning
First case, IOR ok:
TAO (5389|5389) - Completed initializing the process-wide service context
TAO (5389|5389) - Default ORB services initialization begins
TAO (5389|5389) - Default ORB services initialization completed
TAO (5389|5389) - We are the default 3.0.5 ORB ...
TAO (5389|5389) - Initializing the orb-specific services
TAO (5389|5389) - Default_Resource_Factory - codeset manager=0x561f9e7e46c0
TAO (5389|5389) - Codeset_Manager_i::init_ccs, codeset translator <UTF8_Latin1_Factory> has wrong ncs (65537), Invalid argument
TAO (5389|5389) - UTF16_BOM_Translator: forceBE 0
TAO (5389|5389) - Loaded protocol <SSLIOP_Factory>
TAO (5389|5389) SSLIOP_Invocation_Interceptor::CTOR--setting up SSLIOP Current with slot 0
TAO (5389|5389) SSLIOP_Invocation_Interceptor::CTOR--setting up SSLIOP Current with slot 0
TAO (5389|5389) - Created new ORB <>
TAO (5389|5389) - IIOP_Acceptor::open, address==<IP_ADDRESS>:7000, options=reuse_addr=1
TAO (5389|5389) - IIOP_Acceptor::open, specified host=<IP_ADDRESS>:7000
TAO (5389|5389) - IIOP_Acceptor::open_i, trying to listen on port 7000
TAO (5389|5389) - IIOP_Acceptor::open_i, listening on: <<IP_ADDRESS>:7000>
TAO (5389|5389) SSLIOP_Acceptor::open_i - listening on: <<IP_ADDRESS>:39877>
TAO (5389|5389) - IIOP_Acceptor::open, address==localhost:7000, options=reuse_addr=1
TAO (5389|5389) - IIOP_Acceptor::open, specified host=localhost:7000
TAO (5389|5389) - IIOP_Acceptor::open_i, trying to listen on port 7000
TAO (5389|5389) - IIOP_Acceptor::open_i, listening on: <localhost:7000>
TAO (5389|5389) SSLIOP_Acceptor::open_i - listening on: <localhost:42913>
TAO (5389|5389) - Stub::base_profiles, acquired profile lock this = 0x9e838260
TAO (5389|5389): SL2_AccessDecision::add_object(0,1) okay

Second case, IOR not ok:
TAO (5852|5852) - Warning: Resource_Factory options ignored
Default Resource Factory is disabled
TAO (5852|5852) - Warning: Resource_Factory options ignored. Default Resource Factory is disabled
TAO (5852|5852) - Completed initializing the process-wide service context
TAO (5852|5852) - Default ORB services initialization begins
TAO (5852|5852) - Default ORB services initialization completed
TAO (5852|5852) - We are the default 3.0.5 ORB ...
TAO (5852|5852) - Initializing the orb-specific services
TAO (5852|5852) - Default_Resource_Factory - codeset manager=0x55e21f7fe350
TAO (5852|5852) - Codeset_Manager_i::init_ccs, Loaded Codeset translator <UTF8_Latin1_Factory>, ncs = 00010001 tcs = 05010001
TAO (5852|5852) - UTF16_BOM_Translator: forceBE 0
TAO (5852|5852) - Warning: Resource_Factory options ignored
Default Resource Factory is disabled
TAO (5852|5852) - Loaded default protocol <IIOP_Factory>
TAO (5852|5852) - Loaded default protocol <UIOP_Factory>
TAO (5852|5852) - Loaded default protocol <SHMIOP_Factory>
TAO (5852|5852) - Loaded default protocol <DIOP_Factory>
TAO (5852|5852) SSLIOP_Invocation_Interceptor::CTOR--setting up SSLIOP Current with slot 0
TAO (5852|5852) SSLIOP_Invocation_Interceptor::CTOR--setting up SSLIOP Current with slot 0
TAO (5852|5852) - Created new ORB <>
TAO (5852|5852) - IIOP_Acceptor::open, address==<IP_ADDRESS>:7000, options=reuse_addr=1
TAO (5852|5852) - IIOP_Acceptor::open, specified host=<IP_ADDRESS>:7000
TAO (5852|5852) - IIOP_Acceptor::open_i, trying to listen on port 7000
TAO (5852|5852) - IIOP_Acceptor::open_i, listening on: <<IP_ADDRESS>:7000>
TAO (5852|5852) - IIOP_Acceptor::open, address==localhost:7000, options=reuse_addr=1
TAO (5852|5852) - IIOP_Acceptor::open, specified host=localhost:7000
TAO (5852|5852) - IIOP_Acceptor::open_i, trying to listen on port 7000
TAO (5852|5852) - IIOP_Acceptor::open_i, listening on: <localhost:7000>
TAO (5852|5852) - Stub::base_profiles, acquired profile lock this = 0x1f856c20
TAO (5852|5852): SL2_AccessDecision::add_object(0,1) okay

I made a sample CMake project that reproduces the bug, and a sample Java client program that demonstrates the bug, but I seems that I can't add a zip file in attachment.

cc @jwillemsen, do you have any ideas?

I entered it in https://github.com/DOCGroup/ACE_TAO/issues/1767 instead


Getting subscriptionId not by taskParams, but by credential

Getting subscriptionId not by taskParams, but by credential
https://github.com/Azure/aca-deploy/blob/08f4b497c50929376c33d7b1c2d7c8ad0d8faff7/src/main.ts#L25-L26
This is implemented like below in aci-deploy action.
We use DefaultAzureCredential() from @azure/identity instead TokenCredential**s**() from @azure/ms-rest-js, and need to find a way to specify it
https://github.com/Azure/aci-deploy/blob/85ab82e201431dfa391377c0084be2e9e08c535e/src/taskparameters.ts#L36-L38

#21

will close this issue:)


batch fails with versioning

When adding versioning the batch processing is now broken.
I can POST twice in a batch but if I PUT then next call comes back as ambiguous. Here the next call is a delete to a different controller.
The integration tests show all the calls work outside if a batch and the batch was working before adding the versioning.
-Update, I tried reordering the operations in the batch and the second one always fails.
I've sanitized the following for namespaces.
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker:Information: Executed action
DataSetsController.PutDataSet  in 1030.1843ms
Microsoft.AspNetCore.Routing.RouteBase:Debug: Request successfully matched the route with name 'odata-1.0' and template 'odata/v{version:apiVersion}/{*odataPath}'
Microsoft.AspNetCore.Mvc.Versioning.ODataApiVersionActionSelector:Debug: Action 'DataSetsController.GetDataSet (Service)' with id 'a1a3ad0f-bdf6-4192-94f9-df40b10bb8cd' did not match the constraint 'Microsoft.AspNetCore.Mvc.Internal.HttpMethodActionConstraint'
Microsoft.AspNetCore.Mvc.Versioning.ODataApiVersionActionSelector:Debug: Action 'DataSetsController.PutDataSet (Service)' with id 'e97ee741-b6ad-4c5d-a9f9-eb380aef8ce9' did not match the constraint 'Microsoft.AspNetCore.Mvc.Internal.HttpMethodActionConstraint'
Microsoft.AspNetCore.Mvc.Routing.DefaultApiVersionRoutePolicy:Error: Request matched multiple actions resulting in ambiguity. Matching actions: DataSetsController.PutDataSet(Guid, DataSet) (Service)
DataSetsController.DeleteDataSet(Guid) (Service)
Exception thrown: 'Microsoft.AspNetCore.Mvc.Internal.AmbiguousActionException' in Microsoft.AspNetCore.Mvc.Versioning.dll
[ApiVersion("1.0")]
[ODataRoutePrefix("DataSets")]
public class DataSetsController : ODataController
[ODataRoute("({key})")]
[HttpPut]
public async Task PutDataSet([FromRoute]Guid key, [FromBody]DataSet dataSet)
[ApiVersion("1.0")]
[ODataRoutePrefix("Figures")]
public class FiguresController : ODataController
[ODataRoute("({key})")]
[HttpDelete]
public async Task DeleteFigure([FromRoute]int key)
HTTP/1.1 100 Continue
POST /odata/v1/$batch HTTP/1.1
OData-Version: 4.0
OData-MaxVersion: 4.0
Content-Type: multipart/mixed; boundary=batch_6f7e1cb3-701e-45fe-bd22-25cc6a27c52a
Accept: multipart/mixed
Accept-Charset: UTF-8
User-Agent: Microsoft.OData.Client/7.5.4
Host: localhost:53223
Content-Length: 1522
Expect: 100-continue
--batch_6f7e1cb3-701e-45fe-bd22-25cc6a27c52a
Content-Type: multipart/mixed; boundary=changeset_51352a7a-1525-4d5d-ae99-ca9a88028f62
--changeset_51352a7a-1525-4d5d-ae99-ca9a88028f62
Content-Type: application/http
Content-Transfer-Encoding: binary
Content-ID: 1
POST http://localhost:53223/odata/v1/DataSets HTTP/1.1
OData-Version: 4.0
OData-MaxVersion: 4.0
Content-Type: application/json;odata.metadata=minimal
Prefer: return=representation
Accept: application/json;odata.metadata=minimal
Accept-Charset: UTF-8
User-Agent: Microsoft.OData.Client/7.5.4
{"@odata.type":"#DataSet","DataSetId":"00000000-0000-0000-0000-000000000000","Name":"BatchMessage1"}
--changeset_51352a7a-1525-4d5d-ae99-ca9a88028f62
Content-Type: application/http
Content-Transfer-Encoding: binary
Content-ID: 2
POST http://localhost:53223/odata/v1/DataSets HTTP/1.1
OData-Version: 4.0
OData-MaxVersion: 4.0
Content-Type: application/json;odata.metadata=minimal
Prefer: return=representation
Accept: application/json;odata.metadata=minimal
Accept-Charset: UTF-8
User-Agent: Microsoft.OData.Client/7.5.4
{"@odata.type":"#DataSet","DataSetId":"00000000-0000-0000-0000-000000000000","Name":"BatchMessage2"}
--changeset_51352a7a-1525-4d5d-ae99-ca9a88028f62--
--batch_6f7e1cb3-701e-45fe-bd22-25cc6a27c52a--
HTTP/1.1 200 OK
Date: Mon, 08 Apr 2019 17:41:19 GMT
Content-Type: multipart/mixed;boundary=batchresponse_12171aa3-9666-4577-91dc-f68896bd2c44
Server: Kestrel
Transfer-Encoding: chunked
OData-Version: 4.0
--batchresponse_12171aa3-9666-4577-91dc-f68896bd2c44
Content-Type: multipart/mixed; boundary=changesetresponse_664b8b8c-d44c-4759-8cb2-df95ef86c620
--changesetresponse_664b8b8c-d44c-4759-8cb2-df95ef86c620
Content-Type: application/http
Content-Transfer-Encoding: binary
Content-ID: 1
HTTP/1.1 201 Created
Location: http://localhost:53223/odata/v1/DataSets(da212d0e-d64a-48bc-81e3-08d6bc49682e)
Content-Type: application/json; odata.metadata=minimal; odata.streaming=true; charset=utf-8
OData-Version: 4.0
{"@odata.context":"http://localhost:53223/odata/v1/$metadata#DataSets/$entity","DataSetId":"da212d0e-d64a-48bc-81e3-08d6bc49682e","Name":"BatchMessage1"}
--changesetresponse_664b8b8c-d44c-4759-8cb2-df95ef86c620
Content-Type: application/http
Content-Transfer-Encoding: binary
Content-ID: 2
HTTP/1.1 201 Created
Location: http://localhost:53223/odata/v1/DataSets(b4e0d18d-c0a9-4004-81e4-08d6bc49682e)
Content-Type: application/json; odata.metadata=minimal; odata.streaming=true; charset=utf-8
OData-Version: 4.0
{"@odata.context":"http://localhost:53223/odata/v1/$metadata#DataSets/$entity","DataSetId":"b4e0d18d-c0a9-4004-81e4-08d6bc49682e","Name":"BatchMessage2"}
--changesetresponse_664b8b8c-d44c-4759-8cb2-df95ef86c620--
--batchresponse_12171aa3-9666-4577-91dc-f68896bd2c44--
HTTP/1.1 100 Continue
POST /odata/v1/$batch HTTP/1.1
OData-Version: 4.0
OData-MaxVersion: 4.0
Content-Type: multipart/mixed; boundary=batch_aea85057-5d29-4d55-b97a-d2c069022ad1
Accept: multipart/mixed
Accept-Charset: UTF-8
User-Agent: Microsoft.OData.Client/7.5.4
Host: localhost:53223
Content-Length: 1783
Expect: 100-continue
--batch_aea85057-5d29-4d55-b97a-d2c069022ad1
Content-Type: multipart/mixed; boundary=changeset_f63fe221-0b9f-47db-8951-b8c339535b0e
--changeset_f63fe221-0b9f-47db-8951-b8c339535b0e
Content-Type: application/http
Content-Transfer-Encoding: binary
Content-ID: 2
PUT http://localhost:53223/odata/v1/DataSets(da212d0e-d64a-48bc-81e3-08d6bc49682e) HTTP/1.1
OData-Version: 4.0
OData-MaxVersion: 4.0
Content-Type: application/json;odata.metadata=minimal
Prefer: return=representation
Accept: application/json;odata.metadata=minimal
Accept-Charset: UTF-8
User-Agent: Microsoft.OData.Client/7.5.4
{"@odata.type":"#DataSet","DataSetId":"da212d0e-d64a-48bc-81e3-08d6bc49682e","Name":"BatchMessage1a"}
--changeset_f63fe221-0b9f-47db-8951-b8c339535b0e
Content-Type: application/http
Content-Transfer-Encoding: binary
Content-ID: 3
POST http://localhost:53223/odata/v1/Figures HTTP/1.1
OData-Version: 4.0
OData-MaxVersion: 4.0
Content-Type: application/json;odata.metadata=minimal
Prefer: return=representation
Accept: application/json;odata.metadata=minimal
Accept-Charset: UTF-8
User-Agent: Microsoft.OData.Client/7.5.4
{"@odata.type":"#Figure","BackgroundColor":0,"DataSetId":"da212d0e-d64a-48bc-81e3-08d6bc49682e","FigureId":0,"FigureType":null,"ForegroundColor":0,"HideFigureAbove":0.0,"HideFigureAboveUnits":null,"HideLabelAbove":0.0,"HideLabelAboveUnits":null,"Style":null,"Width":0,"EmbeddedText":null,"Intervisibility":null,"Label":null}
--changeset_f63fe221-0b9f-47db-8951-b8c339535b0e--
--batch_aea85057-5d29-4d55-b97a-d2c069022ad1--
HTTP/1.1 200 OK
Date: Mon, 08 Apr 2019 17:41:22 GMT
Content-Type: multipart/mixed;boundary=batchresponse_64014c0d-af3c-4d75-a297-d3f78b8fab44
Server: Kestrel
Transfer-Encoding: chunked
OData-Version: 4.0
--batchresponse_64014c0d-af3c-4d75-a297-d3f78b8fab44
Content-Type: multipart/mixed; boundary=changesetresponse_b429a336-bc0e-429e-86e8-ef5fba82149a
--changesetresponse_b429a336-bc0e-429e-86e8-ef5fba82149a
Content-Type: application/http
Content-Transfer-Encoding: binary
Content-ID: 3
HTTP/1.1 500 Internal Server Error
--changesetresponse_b429a336-bc0e-429e-86e8-ef5fba82149a--
--batchresponse_64014c0d-af3c-4d75-a297-d3f78b8fab44--

After my moment of stupidity issue #493, I am now running into this as well.

also came across this which seems to be related . . .
https://github.com/OData/WebApi/issues/1754

I'm surprised they didn't just kick it back over here and close it out. This is a good find and will certainly help in the investigation/solution. Thanks.

hopefully it makes up for my moment of stupidity !! :)

A quick update. This issue seems to be related to using the URL segment method. I was able to make things work just fine using the query string method. I'm not 100% sure the cause yet, but it appears to be the way OData routes are mapped to the batch handler. I haven't reached a conclusion yet, but it's looking like the solution is purely configuration. I'll report back once I know what the correct configuration is.

Scratch the tidbit about URL segment method; I goofed. I forgot that the route prefix did not include api/ so the wrong URL was being requested.

This does look like it's related to the issue that @Mhirji cited.  Fortunately, I have a workaround for this. It's not really documented anywhere because you're not supposed to have to ever do this. I would argue that there is a bug in the OData code that generates a new HttpContext without copying over features.
To work around this problem, you need to manually control the order of middleware because the order is critically important in this scenario.
Step 1
Disable auto-registration of the API Versioning middleware. This currently doesn't do anything, but add the IApiVersioningFeature.
Change your setup as follows:
services.AddApiVersioning(options => options.RegisterMiddleware = false);

Step 2
Register the API Versioning middleware after the OData batching middleware.
Change your setup to:
public void Configure( IApplicationBuilder app, VersionedODataModelBuilder modelBuilder )
{
    app.UseODataBatching();
    app.UseApiVersioning();
    app.UseMvc(
        routeBuilder =>
        {
            var models = modelBuilder.GetEdmModels();
            Func<ODataBatchHandler> batchHandlerFactory = () => new DefaultODataBatchHandler();
            routeBuilder.MapVersionedODataRoutes( "odata", "api", models, batchHandlerFactory );
        } );
}

Step 3
Run your app and enjoy!

API Versioning auto-wires itself up, which ends up registering its middleware before OData. Due to the implementation in OData, when you enable batching, it needs to occur after the batching middleware is registered. I wouldn't call this is a bug on the API Versioning side, but it definitely isn't obvious this is what you need to do in order to set things up.
I'll see what the OData team does. If they provide a fix for copying features, then all will be good in the world; otherwise, I'll add something to the wiki that describes this particular setup. Manually configuring the middleware is vaguely discussed, but not in this context. It's not something you are supposed to have to worry about.

After some more research, it would appear there is no magic way that OData could properly copy or recreate features for each subsequent request. That means the solution I've provided is the setup you should always use for this scenario. I'm consider/investigate improving the setup in the future.
Are there any more questions on this issue or has it been resolved? It seems to be resolved.

I believe this issue has now been resolved. I've added this documentation to the wiki here so that it will easily discoverable in the future. Feel free to come back or reopen the issue if you continue to have trouble. Thanks.

Had time to get back to this. Confirmed the work around works.

I pulled the sample aspne-api-versioning and tried adding $batch to the ODataBasicSample project and it did not work.
I followed the instructions in the wiki but still nothing.  It seems like it's trying to obtain the IApiVersioningFeature but that's returning a null so it's not able to access anything else.
Here's my setup:
public void ConfigureServices( IServiceCollection services )
        {
            services.AddControllers();
            services.AddApiVersioning(
                options =>
                {
                    options.ReportApiVersions = true;
                    options.RegisterMiddleware = false;
                } );
            services.AddOData().EnableApiVersioning();
        }

public void Configure( IApplicationBuilder app, VersionedODataModelBuilder modelBuilder )
        {
            app.UseODataBatching();
            app.UseApiVersioning();
            app.UseRouting();
            app.UseEndpoints(
                endpoints =>
                {
                    var models = modelBuilder.GetEdmModels( "api" );
                    var batchHandler = new DefaultODataBatchHandler();
                    endpoints.MapControllers();
                    endpoints.MapVersionedODataRoute( "odata", "api", models, batchHandler );
                } );
        }

Here's the error message I get:
System.NullReferenceException
HResult=0x80004003
Message=Object reference not set to an instance of an object.
Source=Microsoft.AspNetCore.Mvc.Versioning
StackTrace:
at Microsoft.AspNetCore.Mvc.Routing.ApiVersionUrlHelper.get_RouteParameter() in ...\Microsoft.AspNetCore.Mvc.Versioning\Routing\ApiVersionUrlHelper.cs:line 37

I pulled the sample aspne-api-versioning and tried adding $batch to the ODataBasicSample project and it did not work.
I followed the instructions in the wiki but still nothing.  It seems like it's trying to obtain the IApiVersioningFeature but that's returning a null so it's not able to access anything else.
Here's my setup:
public void ConfigureServices( IServiceCollection services )
        {
            services.AddControllers();
            services.AddApiVersioning(
                options =>
                {
                    options.ReportApiVersions = true;
                    options.RegisterMiddleware = false;
                } );
            services.AddOData().EnableApiVersioning();
        }

public void Configure( IApplicationBuilder app, VersionedODataModelBuilder modelBuilder )
        {
            app.UseODataBatching();
            app.UseApiVersioning();
            app.UseRouting();
            app.UseEndpoints(
                endpoints =>
                {
                    var models = modelBuilder.GetEdmModels( "api" );
                    var batchHandler = new DefaultODataBatchHandler();
                    endpoints.MapControllers();
                    endpoints.MapVersionedODataRoute( "odata", "api", models, batchHandler );
                } );
        }

Here's the error message I get:
System.NullReferenceException
HResult=0x80004003
Message=Object reference not set to an instance of an object.
Source=Microsoft.AspNetCore.Mvc.Versioning
StackTrace:
at Microsoft.AspNetCore.Mvc.Routing.ApiVersionUrlHelper.get_RouteParameter() in ...\Microsoft.AspNetCore.Mvc.Versioning\Routing\ApiVersionUrlHelper.cs:line 37


Can't install none of the MODELs.

Hello!
I'm using Python 3.11.9 (x64) on Win10 and spaCy 3.5.3. Intallation of greCy itself raised no problems, but intallation of the models promises to finish (once started), but never actually does.
C:\Users\user>python -m grecy install grc_perseus_trf
Installing grc_perseus_trf.....
Please wait, this could take some minutes.....
C:\Users\user>
It states "Please wait, this could take some minutes....." and at this same time "releases" command execution and shows command prompt, inviting user to continue working with console...

Hi,
I could not reproduce the issue.  It installs on linux and macosx. You can install the models directly from Huggingface.
Please try the following as stated in the Huggingface hub:
pip install https://huggingface.co/Jacobo/grc_perseus_trf/resolve/main/grc_perseus_trf-any-py3-none-any.whl
Using spacy.load().
import spacy
nlp = spacy.load("grc_perseus_trf")
Importing as module.
import grc_perseus_trf
nlp = grc_perseus_trf.load()

I've also intermittently run into an issue where the models fail to download, with no output or error to follow.
Running pip install https://huggingface.co/Jacobo/grc_perseus_trf/resolve/main/grc_perseus_trf-any-py3-none-any.whl gives
ERROR: Invalid requirement: 'grc-perseus-trf==any': Expected end or semicolon (after name and no valid version specifier)
    grc-perseus-trf==any

This is on an M1 Mac running Sonoma 14.5.

I was able to get things working by rolling back to Python 3.11.9, after trying it on 3.12.4. Not sure if that's related to the issue encountered here at all, but python -m grecy install grc_proiel_trf finally worked when I recreated the venv from 3.11.9.

I've also intermittently run into an issue where the models fail to download, with no output or error to follow.
Running pip install https://huggingface.co/Jacobo/grc_perseus_trf/resolve/main/grc_perseus_trf-any-py3-none-any.whl gives
ERROR: Invalid requirement: 'grc-perseus-trf==any': Expected end or semicolon (after name and no valid version specifier)
    grc-perseus-trf==any

This is on an M1 Mac running Sonoma 14.5.
It's worth noting that I was able to install greCy about a week ago, but a week before that it failed in the same way.
I wish I could provide more help, but like I said, there aren't any errors to follow (except when attempting that pip install).

I get the same error. I've tried to install and other .whl-packages from HuggingFace and always received this same error message.

Thanks for reporting the problem. I have never encountered those issues myself. And I just installed grc_perseus_sm with the greCy installer.
My python version is 3.11.8 and pip 24.0.

You can also train the models yourself. The _sm and _lg will train without a GPU.