SAML 4.3.0 addition persmission

Description
Added addition permissions for new version of SAML in the policy file.
  permission java.util.PropertyPermission "*" "read,write";
  permission org.opensearch.secure_sm.ThreadPermission "modifyArbitraryThread";

Issues Resolved
[List any issues this PR will resolve]
Is this a backport? If so, please add backport PR # and/or commits #
Testing
[Please provide details of testing done: unit testing, integration testing and manual testing]
Check List

[ ] New functionality includes testing
[ ] New functionality has been documented
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@willyborankin I approved this PR to fix the CI issues in security-dashboards-plugin, but I agree that it would be nice to have a test to prove that this change fixes the issue in this repo. Should we wait to backport the SAML bump to 2.x until automated tests to verify the upgrade are developed?

FYI The issue is reproducible by trying to call on the API to update the config directly and trying to add a SAML Authenticator: https://github.com/opensearch-project/security-dashboards-plugin/blob/main/test/jest_integration/saml_auth.test.ts#L159-L175

@willyborankin I approved this PR to fix the CI issues in security-dashboards-plugin, but I agree that it would be nice to have a test to prove that this change fixes the issue in this repo. Should we wait to backport the SAML bump to 2.x until automated tests to verify the upgrade are developed?

This is good question. I'm thinking about https://testcontainers.com/ so we can test both SAML and real OpenSearch together just to verify that simple integration works but it will take sometime. Alternative is to tr enable SecurityManager for our local/integration tests

@cwperks nd @reta  btw CodeQL complains abut log4j2 version 2.20 vs 2.17. I will fix it in this PR as well in four of OpenSearch versions.*.

@cwperks nd @reta btw CodeQL complains abut log4j2 version 2.20 vs 2.17. I will fix it in this PR as well in four of OpenSearch versions.*.

The change change went into main

@scrawfor99 This will not be for 2.9.0. This unblocks the CI for main for the security-dashboards-plugin, but we should aim to get an automated test in for verifying the SAML authenticator initialization before backporting to 2.x

And @reta raised a valid question about the internal permission. I crated an issue https://github.com/opensearch-project/security/issues/2989

To clarify the problem, it comes from the way X509CertificateImpl is initializing Cleaner instance:
public class X509CertificateImpl extends AbstractXMLObject implements X509Certificate {
    /** The {@link Cleaner} instance to use. */
    private static final Cleaner CLEANER = CleanerSupport.getInstance(X509CertificateImpl.class);

The JDK's cleaners are managed from the dedicated threads and the standard library allows to provide the thread factory to be used, in case of OpenSearch:
return Cleaner.create(OpenSearchExecutors.daemonThreadFactory("cleaners"));

But the class CleanerSupport does not use this method (it uses default Cleaner.create()) and that violates the OpenSearch SecurityManager checks.

@peternied do you know if AWS is a member of Shibboleth Consortium (https://www.shibboleth.net/) so we could submit an issue against JavaSupport project.

@reta No membership AFAIK, I've created a couple of outreach threads including (opensearch#dev) that link out to this issue.  Maybe we will get lucky?

Circling back, I haven't gotten any leads.  If something comes my way latter, I'll update on the tracking issue [1]

https://github.com/opensearch-project/security/issues/3213

@willyborankin @reta I believe this change should be backported to 2.x? Please correct me if I'm wrong

@DarshitChanpura so far no sorry. The problem is permission org.opensearch.secure_sm.ThreadPermission "modifyArbitraryThread";  which we can not use. OpenSAML 5.0 was released Im preparing PR with it. Lets see additionla permissions the library need.


feat: import + export scripts with extraction

new -x --extract option for import
new -A --all-separate option for export
new -w --watch option for export

@meesvandongen I hope to be able to merge this soon. I have been working on some substantial restructuring over the holidays and are still wrapping that up. Script API and OPS layer are part of that restructuring, so I might ask you to re-submit the PR depending on how messy things get when I try to merge.

@meesvandongen I finished restructuring and refactoring the script api and ops layers in both frodo-lib and frodo-cli and there is no way for me to merge your great work as the changes are too big. I could manually re-code what you did but I don't want to put my name under your great contribution. If you feel up to it, please checkout the latest code base for both lib and cli and re-implement your new feature. If you don't want to do that, say the word and I will do it but it won't have your name to it.
The main change is that we properly split lib from cli for everything script. We have been doing this little by little for every piece and it is an ongoing process. I believe that with what we have today, your whole contribution would live in frodo-cli.

@vscheuber I have updated the code somewhat to reflect the new architecture. There is something I would propose should be handled differently: currently the lib package is responsible for handling the progress indicator, I think this too should be moved to cli. Also, the progress indicator kept the node process open for some reason when calling multiple import scripts functions at once. I have not investigated this further however.

@vscheuber I have tested the changes for a JS file import, and they seem to work. Imo the PR is ready to merge, but of course feel free to change anything or request changes from me.

HI @meesvandongen , may I ask you to squash your commits in this PR? All the December commits don’t really apply and they are giving me merge issues. After a rebase against the latest version (there should be no changes affecting scripts) and squashing the commits, I’ll merge asap.

I haven’t fully grasped the functionality you added. Some of the flags are on opposite commands I would expect them to be on. Maybe a demo would be helpful for me to understand your thinking?
For example I would expect the -x option to be on export, not import, or to be renamed to —merge vs —extracts. The watch open also I feel like should be available on import, not on export, so that I can develop code in my favorite IDE and have frodo watch the extracted script files and merge them on change and push them to AM if they have no errors.
I think I may just be misunderstanding what the intention of your feature is vs what I had in mind.

The feature works exactly as you describe; -x is on export, -w is on import. I am open to other names, of course (including the file name, which is now .extract.js).
I will look into recording a demo tomorrow.
On Monday 16 January 2023 18:42:02 (+01:00), Volker Scheuber wrote:
I haven’t fully grasped the functionality you added. Some of the flags are on opposite commands I would expect them to be on. Maybe a demo would be helpful for me to understand your thinking?
For example I would expect the -x option to be on export, not import, or to be renamed to —merge vs —extracts. The watch open also I feel like should be available on import, not on export, so that I can develop code in my favorite IDE and have frodo watch the extracted script files and merge them on change and push them to AM if they have no errors.
I think I may just be misunderstanding what the intention of your feature is vs what I had in mind.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

@meesvandongen ok, great!! Maybe it’s only the commit comment that is wrong, then, and confessed me:
new -x --extract option for import
new -A --all-separate option for export
new -w --watch option for export

@vscheuber that is confusing; I wasn't paying attention at that time I suppose.
Here is the link to the demo: https://youtu.be/yGPnk7gR1dY

@meesvandongen YOU DA MAN! Super cool demo! This looks great!!
I would like to suggest 2 changes:

squash commits
use scriptname.js or scriptname.groovy (without the extract)
use a full URL instead of just the file name to link from the export to the extracted script. That would allow us to store the script anywhere, as long as the URL leads to the code. Could be on an external server, maybe even GitHub, etc.
use slugify to create a safe filename from the script name (see getTypedFilename in ExportImportUtils) if you are not already doing that.

Future:

think about how we can leverage the extract command to save the scripts to different locations (other directories, on a server, etc.) and then use the URL to link to it.

Those are 4 changes!
1/2: will do.
3: Currently it works 2 ways for local files; if you update the json it will look for the file in the same folder. What we could do is make it a relative file name using path.relative; if we want to support multiple protocols to get the files, we should consider if we want to prefix the file type with file:// or not. I suppose we could get away with not doing that however, if we consider the file protocol the default. Then future protocols could use something://etc and there would still be no breaking changes.
4: It currently uses the same name as the .json, which is already slugified.

Although, what occurred to me is that we do change detection for both the .js and the .json file. We match the files by file name of the changed file. So, the file string inside the json is not used at all; and it would involve quite some complexity to add files outside of the initial folder.
For example, consider that we add ../file.js, to some ./file.json. We would have to look inside the json file for the link to the js file, and setup a listener for file changes using chokidar for that file. We would also need to keep (in memory) a list of file links, so we can match them up if we detect a change event the .js file. And we would have to handle the lifecycle of the file, i.e. rename, delete, create, and start/stop listening to file change events accordingly.

That makes sense, @meesvandongen. I think the --watch option requires certain ground rules to be followed. I was thinking let's get to the URL format just so that we don't have to write a converter later on and can handle any type of protocol. Of course the choice of protocol may limit the ability to use --watch. I'm fine to merge whenever you are ready. We can respond to community feedback as we go.

closed in favor of #188


Format WHERE and ON clauses with multiple conditions with new lines

Hello @Matts966 !
I was wondering if it's at all possible to format WHERE and ON clauses with multiple conditions so that each line has a single condition. For example,
SELECT
  *
FROM
  foo.bar_tab
WHERE
  col1 = 'abc'
  AND col2 > 10
  AND col3 IS NOT NULL;

Currently zetasql-formatter formats above example as:
SELECT
  *
FROM
  foo.bar_tab
WHERE
  col1 = 'abc' AND col2 > 10 AND col3 IS NOT NULL;

Which makes it hard to see what's changed when only one of the conditions is updated.

Thank you for your contribution!
I also recognize this problem and will work on it in a week.

@codepozro
This was fixed by the release v1.5.7.
Thank you :tada:

Thank you for a quick turnaround. I've just installed the latest following the Quick Start and the new formatting looks great !
On a side note, is there a way to install a specific version? The Quick Start installed, I believe, the latest but it doesn't seem to have a version:
% zetasql-formatter --version
zetasql-formatter
Debug build (NDEBUG not #defined)

I will track it in https://github.com/Matts966/zetasql-formatter/issues/11
Thanks!


How to run

Drag and drop an image from your desktop/a folder onto the skribbl.io canvas

how
On Sun, Nov 1, 2020 at 11:23 AM Jokniserous<EMAIL_ADDRESS>wrote:

Drag and drop an image from your desktop/a folder onto the skribbl.io
canvas
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/galehouse5/skribbl-io-autodraw/issues/27#issuecomment-720130943,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARRK4W5XLGBCI52CQOINC6DSNWRQHANCNFSM4RLEPQPQ
.

it doesnt work when I drop an image


WIP: update engine to store all metadata in tsm files

ok, as promised here's the work in progress. It actually compiles, but there's still more I have to do and  the tests don't run. Anyway, let me know what you think of the structure.
Here's the high level concept of what happened:
Previously, keys (series key + field) would be hashed to a uint64 ID. Because of the possibility of hash collisions, we had to keep a map of the ID to key in a file called ids. There was another file called collisions that would keep the map of the keys that had IDs different than their hashes. The collisions would be kept in memory.
This ensured that the key -> ID mapping was unique. The data files had a structure like:
# whole file
[block1, block2, blockN... index]

# block
[id, block len, compressed block]

# index
[id1, starting position in file, id2, pos2, idN, posN, ...]

The index and the blocks in the file would always be kept in ID ascending order, which made it possible to merge files together quickly and to search in the index efficiently using binary sort.
This also meant that other files needed to be kept to store metadata.
This PR updates the structure so that tsm files are self describing. That is they have all the information about metadata in them. The new structure looks like this:
# whole file
[block1, block2, blockN... index]

# block
[key len, compressed key, block len, compressed block]

# index
[id1, starting position, min time, max time, id2, ...]

The blocks no longer have the ID in them. Instead the first block will have the entire key (compressed with Snappy). Since blocks have no more than 1,000 points by default, there can be successive blocks with the same key. Those blocks will have a key length of 0 indicating that they belong to the same key as the previous blocks.
I keep the ID based index so that we can still quickly do a binary search to find the starting position of a given ID. Keys get hashed the same as before, the difference is that hash collisions get handled after we seek to the block level.
If there are two keys that hash to the same ID, they get stored in the data file in sorted lexicographical order.
The min and max times of the given ID in the file enable more complex compaction and rewrite logic. Data files can now have overlapping time ranges, as long as they're increasing and two data files can't have an overlapping time range for a specific ID.
Finally, I added some structs to break the code out a bit more and get rid of the monster Write and Rewrite methods.
I'll be updating it to remove all the metadata files, which also removes the need to store those separately from the point writes.
The loading of the metadata index on startup will jump from block to block in the data files and read the keys, which contain all the metadata information we need (measurement, tags, and the fields). Field types are already encoded in the compressed blocks themselves.

Any chance we can get some high-level design notes? A short outline of what
we are looking at?
On Thursday, October 29, 2015, Paul Dix<EMAIL_ADDRESS>wrote:

ok, as promised here's the work in progress. It actually compiles, but
there's still more I have to do and the tests don't run. Anyway, let me
know what you think of the structure.
You can view, comment on, or merge this pull request online at:
https://github.com/influxdb/influxdb/pull/4621
Commit Summary

WIP: update engine to store all metadata in tsm files

File Changes

M tsdb/engine/tsm1/cursor.go
https://github.com/influxdb/influxdb/pull/4621/files#diff-0 (167)
M tsdb/engine/tsm1/encoding.go
https://github.com/influxdb/influxdb/pull/4621/files#diff-1 (5)
M tsdb/engine/tsm1/tsm1.go
https://github.com/influxdb/influxdb/pull/4621/files#diff-2 (1962)
M tsdb/engine/tsm1/tsm1_test.go
https://github.com/influxdb/influxdb/pull/4621/files#diff-3 (50)
M tsdb/engine/tsm1/tx.go
https://github.com/influxdb/influxdb/pull/4621/files#diff-4 (12)
M tsdb/engine/tsm1/wal.go
https://github.com/influxdb/influxdb/pull/4621/files#diff-5 (22)

Patch Links:

https://github.com/influxdb/influxdb/pull/4621.patch
https://github.com/influxdb/influxdb/pull/4621.diff

—
Reply to this email directly or view it on GitHub
https://github.com/influxdb/influxdb/pull/4621.

@otoolep updated the description

https://github.com/influxdb/influxdb/blob/pd-fix-rewrite-file/tsdb/engine/tsm1/tsm1.go#L99-L100
Could use some comments as to what they protect.

https://github.com/influxdb/influxdb/blob/pd-fix-rewrite-file/tsdb/engine/tsm1/tsm1.go#L427-L465
Seems like this func could be made more simple and easier to understand.
Some suggestions:

Move the inner loop of splitValuesByTime to separate func (splitByTime) that takes only a time and a []*valuesWithKey slice and returns the left and right []*valuesWithKey items.
Have splitByTime loop over each items and either append to left or right.  Sort both at end (if they are supposed to be sorted).  The sort.Search inside two nested looks complicated and expensive, but without benchmarking it's hard to say.

https://github.com/influxdb/influxdb/blob/pd-fix-rewrite-file/tsdb/engine/tsm1/tsm1.go#L594-L620
Using channels instead of a Mutex might be cleaner here.

https://github.com/influxdb/influxdb/blob/pd-fix-rewrite-file/tsdb/engine/tsm1/tsm1.go#L662-L664
Is this missing a min < lockStart case?

https://github.com/influxdb/influxdb/blob/pd-fix-rewrite-file/tsdb/engine/tsm1/tsm1.go#L1036
rottate

tsm1.go is getting really big.  Should probably break it out into separate files like engine.go, checkout point.go, rewrite.go, etc..  soon.

The hashing of keys still seems problematic to me.   Having to deal with collisions adds some complexity as well as extra IO for some series that just happens to get a collision.   The full file scan at load time will be problematic for large files as well.
Since you know all the keys that are going to be written into the file beforehand, why not assign them an ID where creating the file?  A simple way might be to sort all the keys and use their index position as the ID?  This sorted key list would be stored in the file and read at startup instead of the whole file. This would remove all collisions and make startups fast regardless of the size of the file.

@jwilder the reason the hashing is preferable is so that when a query comes in, we don't need to hit a lookup table to find the ID for the given key.
Otherwise, we'd need to keep that map in memory for every file in the DB, which would blow up our memory size.
Unless there's some good way to have the keys in the file and be able to search through them efficiently?

I think we can close this.


Create container for search results on search results page (HTML)

WE NEED a container made on the search results page so that the search result of a movie can be stored within it.

Kyle K working on.


Update DCM API Service to newest version before sunset on Feb 29

The DCM/DFA Reporting and Trafficking API Service (DFAreporting.php) is currently set to work with version v2.1 which will be sunset on Februaray 29 (https://developers.google.com/doubleclick-advertisers/reporting/deprecation).
Will there be a new release that works with the newewst version (v2.3) in time?

This issue may no longer be relevant due to its age. Feel free to re-open.


Yolov8 OBB

Hi,
Do you have an example of how to run post-processing for a yolov8 OBB model?
Regards,
Kevin

What are you referring to with post-processing?
Usually, post-processing is preparing the outcome to be human understandable. In other words, make nice images with bounding boxes.

I see.
Unfortunately, the current ncnn YoloV8 implementation doesn't support OBB.
As far as I know, there is no YoloV8-OBB for ncnn available (yet).


Default master coordinates cannot hit zero while normalized

I have an issue with rounding the default master location when axis map is present and that map does not have a point for the default master. I know that's wrong, but I don't think current designSpace doc requires to map default location.
Perfectly integer design coordinate of the default master maps to some floating-point user coordinate which then is recorded (in slightly rounded form, 4 points after dot) as axis default.
It will never get back to normalized zero, which will make this piece of code fail:

(which is what is happening here)
I will add a work-around (by inserting "rounded" default location to the map), but I think it will be good to 1) add a point to the designSpace document asking to map default location and 2) maybe replace that  == 0 with something that is more floating-point compatible, so it will tolerate "almost zero values" (0.0001213, etc.).

I think we should update the doc to say that a mapping for the default master (as well as for minimum and maximum) is required if an axis has any map elements.
can you show what is your designspace axes map configuration?
I don't thin we need to do any rounding. A default master is currently required to be present in a DesignSpace to build a varfont, and that master needs to be given an explicit mapping if for a given axis the user vs design locations are different.

I agree that designSpace spec should include this addition (which is very logical, as "implied" map contains min, def, max values by definition).
My current fix is doing exactly that: adding the point for the default master to the map. It solves the problem.


Enable java preview features at runtime.

Is there any way to pass the options at the runtime. Like --enable-preview
Input:
var test = """     hi     """;
Output:
|   var test = """ text blocks are a preview feature and are disabled by default.   (use --enable-preview to enable text blocks)

that information is on this repository's readme

and the kernel.json file will be in the given directory.
For example to enable assertions, set a limit on the heap size to 128m.
{
- "argv": [ "java", "-jar" , "{connection_file}"],
+ "argv": [ "java", "-ea", "-Xmx128m", "-jar", "{connection_file}"],
  "display_name": "Java",
  "language": "java",
  "interrupt_mode": "message",
  "env": {
  }
}

just add the --enable-preview argument like following
- "argv": [ "java", "-jar","{connection_file}"],
+ "argv": [ "java", "-jar",  --enable-preview , "{connection_file}"],

Hi @saulpalv
I tried this, first I am unable to pass --enable-preview without quotes and when I pass -ea works fine but preview not enabled.  I tried various options but no luck.
I am trying to use jdk 14.  I have that update in my fork, if you like to take a look
ibinder fork

Hi @saulpalv
I tried this, first I am unable to pass --enable-preview without quotes and when I pass -ea works fine but preview not enabled.  I tried various options but no luck.
I am trying to use jdk 14.  I have that update in my fork, if you like to take a look
ibinder fork

@vinothmdev if you are on JDK 15 text blocks are enabled by default, then enable preview is not needed

@vinothmdev if you are on JDK 15 text blocks are enabled by default, then enable preview is not needed

{
    "argv": [
        "java",
        "-jar",
        "--enable-preview",   // add this ...
        "/opt/conda/share/jupyter/kernels/java/ijava-1.3.0.jar",
        "{connection_file}"
    ],
    "display_name": "Java",
    "env": {"IJAVA_COMPILER_OPTS":"--enable-preview -source 15"},  // and this.
    "interrupt_mode": "message",
    "language": "java"
}

@rushfly
This worked thank you :)


Link to learning module about EF Core from EF Core docs

We need to figure out the right place for it.
https://docs.microsoft.com/learn/modules/persist-data-ef-core/
cc @CamSoper @scottaddie

@divega We link to it in the ASP.NET Core docs like this:

The MS Learn team doesn't provide guidance on how to link to the tutorials. I suggest adding a Tutorials (Microsoft Learn) ToC node. The link within should be more descriptive than Data access because the reader is in the EF Core docs already. Thoughts?

@scottaddie are you planning to add more like this centers on EF Core? If yes, we can add a node for them. Otherwise I would like to add it to the existing Get Started menu.

@divega @CamSoper and I have plans to produce more advanced EF Core modules; however, those aren't likely to get done until late 2019 or early 2020. The focus right now is on finishing a basic beginner-to-intermediate level ASP.NET Core learning path.


Idea: Holy Grail with comments and examples for beginners

Thanks for creating all this great code!
I have created a tiny repo that contains just the Holy Grail layout code, with comments and examples suitable for beginners. There's an annotated CSS file and tiny HTML file.
Comments, feedback, and requests are welcome.
https://github.com/SixArm/sixarm_css_holy_grail_layout_with_flexbox

Thanks @SixArm - exactly what I was looking for!


VPN using SSO fails

Hi,
Thanks for your effort in bringing GATE SSO.
The first think I would like to mention is "It needs a better documentation"
Anyways, I followed the Readme.MD and successfully completed the installation.
But I cannot connect to my OpenVPN-CE using the generated certs and profile.
Is something needs to be done?
Should I configure PLUGIN for authentication in server.conf?
Please suggest / help.
-Thanks,
Krishh

Please look at scripts folder, those scripts are required for OpenVPN Server to function

gate moved to wireguard https://github.com/gate-sso/gate-wireguard


Implement environment passing for build.

Need to be able to set env vars for use by the buildpacks during build.
Note this is NOT related to passing env vars for use at runtime.

Closed with https://github.com/snowdrop/java-buildpack-client/commit/9ea258e8740faceb97689f3f61075fbd5a7c37e3


When setting the table to open a later page the Jump to page value doesn't update itself

Using v6?
Yes, v6.11.5
Using v7?
No
Describe the bug
So my application has a table with n pages where n > 1. When user clicks on a row (s)he is taken to a details page. There is a custom back button that appends the selected item id in the URL so its row can be shown in bold. This helps the user to know which item was just viewed.
Now normally the list of items is available in the global state so going back isn't a problem. But let's say user reloads the details screen and then tries to go back, in this case the list has to be first re-fetched and then the corresponding row has to be highlighted.
In the latter case, I first find the matched index in the array of items and then I get the page number by doing Math.floor(index / pageSize) (floor because page is index-based). I then set this number to the local state that controls page number. React Table jumps to that page just fine and the row is highlighted without problems. However, the jump-to-page text input still shows page 1.

It's interesting to note that if I turn off the page jump feature (showPageJump={false}) then it shows the page number correctly.

To Reproduce
You could perhaps reproduce the issue with any kind of list and following what I have typed into the description.
Expected behavior
I expected the page-jump text input to show the correct page number.
Codesandbox!
Use a new react-table codesandbox to reproduce the issue.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: MacOS Catalina
Browser:  Chrome
Version: 79.0.3945.79

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Long-term support for v6 has been moved to the Spectrum Forum here: https://spectrum.chat/react-table. Please post your question/issue there! Thanks!


Fix for coverage scripts

remove coverage folder failed locally when already exists.

Signed-off-by: Ji Bin<EMAIL_ADDRESS>

/cc @yhmo
/cc @ArkaprabhaChakraborty

/lgtm


Parser issue with lxml + bs4

I'm getting the following error:
Traceback (most recent call last):
  File "login.py", line 30, in <module>
    client = fbchat.Client(os.environ['FB_ID'], os.environ['FB_PASS'])
  File "/Users/tshrjn/anaconda3/lib/python3.5/site-packages/fbchat/client.py", line 122, in __init__
    self.login(email, password, max_retries)
  File "/Users/tshrjn/anaconda3/lib/python3.5/site-packages/fbchat/client.py", line 281, in login
    if not self._login():
  File "/Users/tshrjn/anaconda3/lib/python3.5/site-packages/fbchat/client.py", line 224, in _login
    soup = bs(self._get(MobileURL).text, "lxml")
  File "/Users/tshrjn/anaconda3/lib/python3.5/site-packages/bs4/__init__.py", line 165, in __init__
    % ",".join(features))
bs4.FeatureNotFound: Couldn't find a tree builder with the features you requested: lxml. Do you need to install a parser library?

On looking around, I found that this used to be a bs4 issue which got resolved in its 4.0.3 update.
Versions I am using are as follows:
pip list
beautifulsoup4 (4.5.3)
fbchat (0.7.1)
lxml (3.7.3)

Please help me resolve this issue cause I'm  not even able to login.

Same here. I observed the same issue on MacOS Sierra. It is fine on Ubuntu 16.04.

I don't have time to look into it yet. If someone has, hope this help.
Using Cygwin, working fine.
beautifulsoup4 (4.5.1)
fbchat (0.6.0)
lxml (3.4.4)

Centos working fine
beautifulsoup4 (4.5.1)
fbchat (0.6.0)
lxml (3.6.4)

Haven't tried on OSX though, but possibly an OSX dependency issue.

Have you tried the following command on the lxml site?
sudo port install py27-lxml
I had the same issue, but I ended up using sudo apt-get install python3-lxml instead of the default 'pip install lxml' that's part of the install script since my vps doesn't have the memory to compile (128mb)

Cannot reproduce on MacOS Sierra. Don't think this is related to fbchat

I also had this issue on Windows 7. I believe it is a problem with the lxml install. I tested it by running -
import lxml import lxml.etree
and this reproduced the error.
I fixed by followed the instructions on this Stackoverflow post: https://stackoverflow.com/questions/29440482/how-to-install-lxml-on-windows
I realise this fix is for Windows and not MacOS however I think OP should look at their lxml install as I believe that will be where the issue lies. For anyone on Windows, try the linked fix.


Fix build and publish process

The dist folder was made incorrectly. Not sure what happened. I also contacted  you on Discord about it. Regardless this PR should fix all issues. Simply run npm publish and it will lint, test and build the code all automatically. You won't ever have to touch the content of the dist folder, which is how it should be regarded.
I added a bunch more of npx to ensure the scripts use the tools. They should've done so already but still, can't be too sure I guess?
Anyway the takeaway is that you shouldn't copy-pasta the NPM scripts if you were doing that, that's the only thing that comes in my mind that could've happened anyway. Just do npm run <script_name> or npm run-script <script_name> to run them. prebuild and postbuild are automatically ran before and after npm run build respectively and prepublishOnly is always run before npm publish.
Running npm publish --dry-run now results in:
Favna@Stormfortress MINGW64 /e/UserFiles/DevProjects/booru (fix/package-scripts)
$ npm publish --dry-run

><EMAIL_ADDRESS>prepublishOnly .
> npm run-script build

><EMAIL_ADDRESS>prebuild E:\UserFiles\DevProjects\booru
> npm run-script clean && npm run-script lint && npm run-script test

><EMAIL_ADDRESS>clean E:\UserFiles\DevProjects\booru
> npx rimraf ./dist

><EMAIL_ADDRESS>lint E:\UserFiles\DevProjects\booru
> npx tslint --fix -p . -c ./tslint.json ./src/**/*.ts

><EMAIL_ADDRESS>test E:\UserFiles\DevProjects\booru
> npx jest --no-cache

 PASS   test  test/util.spec.ts
 PASS   test  test/realbooru.spec.ts
 PASS   test  test/konachan-com.spec.ts
 PASS   test  test/rule43paheal.spec.ts
 PASS   test  test/danbooru.spec.ts
 PASS   test  test/safebooru.spec.ts
 PASS   test  test/furrybooru.spec.ts (5.071s)
 PASS   test  test/rule34.spec.ts
 PASS   test  test/konachan-net.spec.ts
 PASS   test  test/lolibooru.spec.ts (5.199s)
 PASS   test  test/hypnohub.spec.ts (5.246s)
 PASS   test  test/derpibooru.spec.ts (5.335s)
 PASS   test  test/e621.spec.ts (5.6s)
 PASS   test  test/xbooru.spec.ts
 PASS   test  test/gelbooru.spec.ts (5.637s)
 PASS   test  test/tbib.spec.ts
 PASS   test  test/e926.spec.ts (6.211s)
 PASS   test  test/yandere.spec.ts

Test Suites: 18 passed, 18 total
Tests:       38 passed, 38 total
Snapshots:   0 total
Time:        9.344s
Ran all test suites.

><EMAIL_ADDRESS>build E:\UserFiles\DevProjects\booru
> npx tsc

><EMAIL_ADDRESS>postbuild E:\UserFiles\DevProjects\booru
> npx terser-folder ./dist -eo ./dist -x .js && npx copyfiles ./example.js ./dist

File dist\boorus\Derpibooru.js written successfully !
File dist\index.js written successfully !
File dist\Constants.js written successfully !
File dist\structures\InternalSearchParameters.js written successfully !
File dist\boorus\XmlBooru.js written successfully !
File dist\structures\Post.js written successfully !
File dist\boorus\Booru.js written successfully !
File dist\structures\SearchResults.js written successfully !
File dist\structures\Site.js written successfully !
File dist\structures\SiteApi.js written successfully !
File dist\structures\SearchParameters.js written successfully !
File dist\Utils.js written successfully !
File dist\structures\SiteInfo.js written successfully !
npm notice
npm notice package<EMAIL_ADDRESS>npm notice === Tarball Contents ===
npm notice 1.9kB package.json
npm notice 1.1kB LICENSE.md
npm notice 2.7kB readme.md
npm notice 814B  dist/boorus/Booru.d.ts
npm notice 2.0kB dist/boorus/Booru.js
npm notice 394B  dist/boorus/Derpibooru.d.ts
npm notice 779B  dist/boorus/Derpibooru.js
npm notice 167B  dist/boorus/XmlBooru.d.ts
npm notice 309B  dist/boorus/XmlBooru.js
npm notice 498B  dist/Constants.d.ts
npm notice 1.1kB dist/Constants.js
npm notice 1.5kB dist/example.js
npm notice 653B  dist/index.d.ts
npm notice 2.0kB dist/index.js
npm notice 5.8kB dist/sites.json
npm notice 212B  dist/structures/InternalSearchParameters.d.ts
npm notice 68B   dist/structures/InternalSearchParameters.js
npm notice 889B  dist/structures/Post.d.ts
npm notice 2.4kB dist/structures/Post.js
npm notice 133B  dist/structures/SearchParameters.d.ts
npm notice 68B   dist/structures/SearchParameters.js
npm notice 623B  dist/structures/SearchResults.d.ts
npm notice 924B  dist/structures/SearchResults.js
npm notice 332B  dist/structures/Site.d.ts
npm notice 345B  dist/structures/Site.js
npm notice 84B   dist/structures/SiteApi.d.ts
npm notice 68B   dist/structures/SiteApi.js
npm notice 272B  dist/structures/SiteInfo.d.ts
npm notice 68B   dist/structures/SiteInfo.js
npm notice 462B  dist/Utils.d.ts
npm notice 1.3kB dist/Utils.js
npm notice === Tarball Details ===
npm notice name:          booru
npm notice version:       2.0.1
npm notice package size:  9.5 kB
npm notice unpacked size: 30.0 kB
npm notice shasum:        8a0eafae460354bf422f5942d49c8dbd6a8231ac
npm notice integrity:     sha512-DopsCV1krFlMv[...]ndCianW9N+bkw==
npm notice total files:   31
npm notice
+<EMAIL_ADDRESS>
example.js was not included in the distribution which means ... was a wrong way to fix it. As seen above it should get copied into dist/ and so the path was correct. These were the files that were in your published version:

I'm also not sure where test.js and test.d.ts came from. That would suggest you have a local non-ignored, non-committed and non-pushed test.ts file in src/.

Checked it out and it's fine, gotta update the dependencies as well


Passing an image to initializer does nothing

https://github.com/uber/UberSignature/blob/master/Sources/Swift/SignatureDrawingViewController.swift#L40
public init(image: UIImage? = nil) {
        super.init(nibName: nil, bundle: nil)
    }

The passed image is not used or shown in view controller when using this initializer, I would expect that when I pass an image to this initializer it would show the image

As objective-c code change the init method to:
public init(image: UIImage? = nil) {         super.init(nibName: nil, bundle: nil)         presetImage = image     }

When can you merge this fix to the master?


Incorrect array schema in eth_getWork

From @conradev in #153:

I wrote something to parse openrpc.json and it failed to parse because items is an array, when it should be an object

prefixItems is an array of schemas, in order of the expected items in the array

It seems like this is an upstream issue with the OpenRPC tooling though.

created an issue to track this https://github.com/json-schema-tools/meta-schema/issues/175

Technically this could be closed now, no? https://github.com/ethereum/execution-apis/issues/429

Still intending to add support to json schemas for openrpc though - would happily take a PR if anyone wants to do it.


Clean up Troubleshooting section

Following the latest community discussions and the Kubeflow docs restructure proposal, we should add the concept of "Troubleshooting", with the following structure:

Troubleshooting (created with files from current section "Further Setup and Troubleshooting")

Frequently Asked Questions
Troubleshooting
Usage Reporting
Support

/priority p1

I am closing this in preference of https://github.com/kubeflow/website/issues/2760, feel free to reopen if we want to make a troubleshooting section again in the future.
/close


type="module" not required

Your Web Components needs to read the component lightDOM/innerHTML; which is not available because the connectedCallback fires on the opening tag
That is why you (need to) use type="module", it forces the script to delay until the whole DOM page is parsed.
You can put a lesser delay in the connectedCallback with a setTimeout or other techniques
Since you are not attaching multiple Listeners, addEventListener is not required also.
That minimizes (for best GZipped delivery) your code to, which can now be placed in a <script> tag inside the <head>
  customElements.define("is-playing", class extends HTMLElement {
    connectedCallback() {
      setTimeout(() => // wait till lightDOM is parsed
        this.querySelectorAll("audio,video").forEach(p =>
          Object.assign(p, {
            onplaying: _ => {
              p.setAttribute("playing", "");
              this.setAttribute("playing", p.id || "");
            },
            onpause: _ => {
              p.removeAttribute("playing");
              this.removeAttribute("playing");
            },
            onended: _ => {
              p.removeAttribute("playing");
              this.removeAttribute("playing");
            }
          }) // Object.assign
        ) // forEach
      ) // setTimeout
    } // connectedCallback
  }); // define

Please stop spamming my open source projects. I've seen your behaviour on here, finding open source Web Components across GitHub and attempting to enforce an approach that just suits your desired outcomes. It's not constructive to the community.
I've already explained multiple times why my Web Components are written like this. They are designed to enhance regular HTML, so I want to wait until the DOM is loaded before applying this extra interactivity. That's how progressive enhancement works. In addition, I don't want my code to be readable, so others can learn and contribute with ease.


[Backdrop]: allow white-transparent background color

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
By default, the backdrop background color is set to 5% black. It would be great to set this via api to transparent white.
Also is the backdrop background color dark-mode compatible ?
Examples 🌈
No response
Motivation 🔦
No response

You can override color in own custom theme.

@aaarichter does @kkaplita's suggestion answer addresses your problem?

@kkaplita thanks for the reply . it worked.
closing


fix testRegisterTableWithWrongName- bucket name

Jira: CT-XXX
KBC: XXX
Before asking for review make sure that:
Checklist

[x] New client method(s) has tests
[x] Apiary file is updated

Release

[x] I gave the PR a proper label:

major (BC break)
minor (new feature)
patch (backwards compatible fix)
no release (just test changes)

rovnou ti to mergnu @romanbracinik


[116] Components return once with support for render functions

Removed the autofix rule for now
And there is only one error which is noEarlyReturn as conditionals I would say are the same thing, returning early for something

closes #116


Yasheik

installing
What :computer:

First thing updated with this PR
Second thing updated with this PR
Third thing updated with this PR

Why :hand:

Reason why first thing was added to PR
Reason why second thing was added to PR
Reason why third thing was added to PR

Evidence :camera:
Include screenshots, screen recordings, or console output here demonstrating that your changes work as intended

Notes :memo:

Any notes/thoughts that the reviewers should know prior to reviewing the code?

This Pull Request has no explanation of its intended purpose and appears to be spam. Closing the PR accordingly


Elli-Support mit 0.105 broken wg. Cert...

Describe the bug
Das Release 0.105 kann nicht mit der Elli-Wallbox reden, weil der Cert-Patch wohl nicht enthalten ist:
Oct 14 14:48:40 RPNagel3 evcc[1625900]: [eebus ] TRACE 2022/10/14 14:48:39 Failed to connect to wss://<IP_ADDRESS>:4712/ship/: tls: failed to parse certificate from server: x509: invalid basic constraints b
Steps to reproduce
Release mit Elli-WB starten...
Configuration details
chargers:

type: template
template: ellipro
ski: 4F86 4FB4 C192 10AD 78EA 8C50 A07E E0D4 0FAB F5E0
ip: <IP_ADDRESS>
name: wallbox1

Log details
Oct 14 14:48:40 RPNagel3 evcc[1625900]: [eebus ] TRACE 2022/10/14 14:48:39 Failed to connect to wss://<IP_ADDRESS>:4712/ship/: tls: failed to parse certificate from server: x509: invalid basic constraints b
What type of operating system are you running?
Linux
Version
evcc version 0.105.0

Diese Frickelbox raubt einem den letzten Nerv. Ich hoffe @DerAndereAndi hat eine Idee.

@pencoe Thanks for the hint and sorry for the issue. We forgot to adopt the patching to the release workflow. This should be fixed in 0.105.1. Could you please give it a try?

@pencoe Thanks for the hint and sorry for the issue. We forgot to adopt the patching to the release workflow. This should be fixed in 0.105.1. Could you please give it a try?

Top, ist gefixt, danke :-D


Do not set Pod's Image pull policy if not explicitly set

Address #1912.
Test pipeline:
pipeline:
  test:
    image: alpine
    commands:
      - apk add bash
      - bash -c 'if [ -v TESTVAR ]; then echo TESTVAR exist and value is $TESTVAR; fi'

Pod config:
...
  containers:
    - name: wp-01h49cmzpzapdq3r1bkqbrttxa-0-step-0
      image: alpine
      ...
      imagePullPolicy: Always
      ...

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

Comparison is base (b57b6f2) 39.23% compared to head (52f4df2) 39.25%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1914      +/-   ##
==========================================
+ Coverage   39.23%   39.25%   +0.01%
==========================================
  Files         177      177
  Lines       10706    10713       +7
==========================================
+ Hits         4201     4205       +4
- Misses       6212     6215       +3
  Partials      293      293

Impacted Files
Coverage Δ

pipeline/backend/kubernetes/pod.go
0.00% <0.00%> (ø)

server/model/step.go
0.00% <0.00%> (ø)

server/model/workflow.go
0.00% <0.00%> (ø)

server/pipeline/workflowStatus.go
0.00% <0.00%> (ø)

...tastore/migration/020_parent_steps_to_workflows.go
59.45% <75.00%> (ø)

pipeline/stepBuilder.go
76.69% <100.00%> (+0.11%)
:arrow_up:

server/store/datastore/agent.go
50.00% <100.00%> (-4.17%)
:arrow_down:

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add environment vars in config file

As the defaults are the same this should be fully backwards compatible. The only potential issue is if you have already defined the same environment vars, but that is very unlikely.

thanks for the PR. Ill review and get back to you.

Never mind. I fixed and merged.


Query and insert data is not the same .realm file used

!!! MANDATORY TO FILL OUT !!!

Goals

How I use it:
When I create xxx.realm via [[xxxDBManager shared] setup], when I save the data using the -addObject method, I can see that the data is saved in the xxx.realm file.
But when I use -list to query the data, I see a new default.realm file created under the ../Document/ folder, and the query data is nil.
hope:
I hope that when I use the -list query, I am querying the saved data in the xxx.realm file instead of creating a default.real.
Expected Results

Actual Results

Steps to Reproduce

Code Sample

setting:
@interface xxxDBManager : NSObject
@property (nonatomic, strong) RLMRealm *realm;
@property (nonatomic) dispatch_queue_global_t access_queue;
@property (nonatomic) dispatch_queue_t queue;

(instancetype)shared;

(void)setup;

@end
@implementation xxxDBManager

(instancetype)shared {
static xxxDBManager *manager;
static dispatch_once_t onceToken;
dispatch_once(&onceToken, ^{
manager = [[self alloc] init];
});
return manager;
}

(instancetype)init
{
self = [super init];
if (self) {
self.access_queue = dispatch_get_global_queue(0, 0);
self.queue = dispatch_queue_create("com.concurrent.queue", DISPATCH_QUEUE_CONCURRENT);
}
return self;
}

(void)setup {
RLMRealmConfiguration *config = [RLMRealmConfiguration defaultConfiguration];
NSString *realmPath = [NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES) firstObject];
realmPath = [realmPath<EMAIL_ADDRESS>config.fileURL = [NSURL URLWithString:realmPath];
dispatch_async(self.queue, ^{
NSError *error;
self.realm = [RLMRealm realmWithConfiguration:config error:&error];
if (error) {
NSLog(@"realm configuration error:%@", error.localizedDescription);
}
});
}

query:

(RLMArray<xxxModel *> *)list {
RLMResults<xxxModel *> *all = [xxxModel allObjects];
if (all.count > 0) {
RLMArray *array = [[RLMArray alloc] initWithObjectClassName:all.objectClassName];
[array addObjects:all];
return array;
}
return nil;
}

save:

(void)addObject {
xxxModel *model = [[xxxModel alloc] init];
model.url = xxxURL;
model.parameter = parameters;
[model addObject:model];
RLMRealm *realm = [ReporterDBManager shared].realm;
dispatch_async([ReporterDBManager shared].queue, ^{
[realm transactionWithBlock:^{
[realm addObject:model];
}];
});
}

Version of Realm and Tooling

Realm framework version: ?
3.16.1
Realm Object Server version: ?
Xcode version: ?
10.14
iOS/OSX version: ?
ios12
Dependency manager + version: ?

[xxxModel allObjects] returns all objects of that type in the default Realm, which is not the Realm you added the objects to. You need to either read from that Realm (using [xxxModel allObjectsInRealm:ReporterDBManager.shared.realm]) or set the default Realm configuration in setup to be your specific Realm.
dispatch_async(queue, ^{ [realm transactionWithBlock:^{ ... }]; }); also does not work. Realm objects are thread-confined, and dispatch queues are not tied to any particular thread. Instead you must open the Realm inside the dispatched block.

Thank you for your reply, my idea is to let the data increase, delete, change, check are executed in the child thread, what should I do


bug: cannot stop embeddings / inference during RAG

Jan version
v0.5.4-641
Describe the Bug
In Threads, stop button still shows after i stop the model
Steps to Reproduce

Use any model in Jan
Send request
While the model is generating a response, click on stop model in the System Monitor
Observe that the stop button still shows in the Input Field

Screenshots / Logs
https://github.com/user-attachments/assets/32b73ea5-ffd6-487e-b7b4-7b0d35977946
What is your OS?

[X] MacOS
[ ] Windows
[ ] Linux

Can't reproduce again with v0.5.9-750


Encryption

How do I encrypt this file on windows server?

As noted in the README: https://github.com/tonerdo/dotnet-env#a-note-about-production-and-the-purpose-of-this-library you should not be using this in production.
If you are worried about the security of the settings in your .env file, you need to think through your usage more deeply. I can talk you through it, but that's a long talk, probably.
Short version: if at any point you have settings that you need to send along encrypted, or store encrypted for whatever reason, you don't encrypt the file, you encrypt the values. Then you decrypt the values when you read them, or just before. For example, this is how sops works: https://github.com/mozilla/sops


Workspaces in Cairo?

Does Cairo do workspaces? I'm used to using Linux with 16 workspaces in a 4x4 array. VirtuaWin did a pretty good job, but appears to have been abandoned.
I need the ability to:

Open windows for applications are in a workpace, and move a window to another workspaces
Use Alt-tab to cycle through the open apps on the current workspace.
Scroll/jump to adjacent workspaces (north, east, west, south), jump to a workspace via an overall map, or to a specific window in a specific workspace. Extra credit for a 3rd dimension to make adjacent jumps shorter.
I dedicate some of the workspaces to specific programs, such as Thunderbird in the upper left workspace, CAD programs in the lower-right workspace, and many workspaces devoted to Firefox with tabs in a given theme in their own workspaces (remodeling, clients, education & kids, solar panel monitoring, volunteer organizations, etc).

We do not. I would recommend looking at what Windows 10 or Windows 11 offers in their Alt+Tab areas.


Make it possible to use eatmydata for at least apt

There was an elderly https://github.com/kaczmarj/neurodocker/issues/60 which was closed without an explanation, so decided to open a brand new one
I guess could be an option specific to apt manager

i'm hesitant to add this eatmydata adds something to the dockerfile that might cause issues, and i don't think that risk outweighs the speed improvements. eatmydata might not be installable on all the systems users want to use; it causes the behavior between apt and yum to be different when i would like it to be as similar as possible; and i'm not confident that it will work right every time (it seems a bit unsafe). this is why i closed that other issue — i should have explained back then.
i'm going to close this issue given the points i listed above.

eatmydata causes no change to anyone's behavior besides avoiding constant 'sync' all the time - so I am not sure what changes you have in mind @kaczmarj .  Clearly you can't expect "apt" and "yum" to behave exactly the same way since they are doing different things to different packages ;-)
as for installability - AFAIK available on all debian system, hence my request was somewhat targetting.
But hey - you are the boss, it was just a suggestion. FWIW I have been using eatmydata without any side effects for long time ... FWIW all NeuroDebian docker containers now actually have
$> docker run -it --rm neurodebian:sid ls -l /usr/local/bin/apt-get
lrwxrwxrwx 1 root root 18 Mar 27 00:02 /usr/local/bin/apt-get -> /usr/bin/eatmydata

so I kinda solved this issue a bit more generally, at least for neurodebian docker based images ;-)


Ensure the CUDA container image works on most CPUs

This disables AVX2, FMA, and F16C in the CMAKE_ARGS when rebuilding llama_cpp_python to ensure that the container build from this Containerfile will work on any CPU architecture as long as it has the basic avx instruction set.
Fixes #822

Reopened as #918


UI: have a clear way to hide and reopen the left pane

Is your feature request related to a problem? Please describe.
I'm always frustrated when seeing this plus button (on the right) and I don't understand what it does.

Describe the solution you'd like
I like the notion solution very much with the two chevrons:

Describe alternatives you've considered, if relevant
maybe a cross instead of <<?
Steps

[x] draw new design
[ ] implement new design

Something like this ?

that's perfect!

Okay, I update other screens, it will be available on V3 on figma

Should I close this ticket?

nope, I'm transferring to ready for dev! (It could have been an epic with two shorter issues, one dedicated to design and the other to dev but... it felt too small to do that)

Does the chevron need to be moved inside the sidebar? This makes things more complicated; changing an icon is much simpler. What do you think?

Ideally yes, it would be nice but I'm okay to just swap icons to make it clearer
@MarieS-WiMLDS wdyt?

I'm ok to swap too!

Okay then @augustebaum @MarieS-WiMLDS , I'm changing production mockups


rosjava_messages: 0.1.341-0 in 'hydro/distribution.yaml' [bloom]

Increasing version of package(s) in repository rosjava_messages to 0.1.341-0:

upstream repository: https://github.com/rosjava/rosjava_messages.git
release repository: https://github.com/rosjava-release/rosjava_messages-release.git
distro file: hydro/distribution.yaml
bloom version: 0.5.20
previous version for package: 0.1.329-0

New one coming.


performance: avoid calling relayout too often

Describe your changes
Figure.relayout is being called very often (5x at initial display), and unconditionally will do the work (call requestAnimation). These calls can stack up when resizing the window as well, leading to bad performance. We now keep track if the relayout was already requested, and will only schedule a new one when the last one is finished (similar to how we do it with webgl and in ipyvolume).
Testing performed
Tests pass, and tested in notebook. Putting a log statement in relayoutImpl shows 5 prints before, 2 after (so I think we can still improve even more if we want).

It is not really fixing the issue of calling relayout too much, but just working around it... Would it be easy to reduce the number of relayout calls?

Basically any call to requestAnimationFrame should be 'protected' by such a mechanism, since you can have 1 JS event loop cycle schedule something like 10 of these. It does not make sense to do 10 redraws/relayout calls in the next cycle.

I understand that. But again, relayout is still called a lot and that is the root of the issue. This is only a workaround.

We might want to track where those relayout calls come from and try to reduce them as much as possible. 10 relayout calls at figure creation seems to be a lot. There might be a logic issue here.
Anyway, I am fine with the workaround for now. Just saying we might want to fix the root cause of this issue later.

Fair point, if would still be required when resizing the window, since you can get two resize events, which is out our control.
So we have two problems, 1) relayout is being called too often by use (we should avoid that) 2) relayout may be triggered externally, because of browser events that we should solve it with this PR.
So this solved 2, but not 1?

https://github.com/jupyter-widgets/ipywidgets/issues/2605 is one of the issue we cannot avoid in bqplot. I'll see if I can identify where the 5 come from.

I feel like we can avoid this extra relayout since this fix: https://github.com/bloomberg/bqplot/pull/982. I am opening a PR that removes this extra relayout call #994

I think I understand the issue for 80%.
If figure is a child of a Box, it will get two resize events. Those two resize events collapse with the 1st commit since all these events seem to be handled in 1 JS event loop cycle. In another JS event loop cycle we responded to the 'after-attach' event.
However, since our parent class already responds to this by firing the displayed event (which resolved the displayed promise, and our renderImpl wait for this promise, we can remove this case.
https://github.com/jupyter-widgets/ipywidgets/blob/92d7d42c00a1b0d9ce921533acb08beefdea3eb2/packages/base/src/widget.ts#L955
We are now down to 1 relayout call on initial display 🎉
I've also tested this in Jupyter Lab, and we get 9 (!!) resize events when we do this:
widgets.VBox([widgets.VBox([widgets.VBox([widgets.VBox([fig])])])])

It seems every VBox causes 2 resize events in fig, and fig itself gets one.
I'll mention that in https://github.com/jupyter-widgets/ipywidgets/issues/2605 as well.
I think this PR is ready for review @martinRenou, let me know what you think of this.

Kudos on removing the after-attach one!

We are now down to 1 relayout call on initial display tada

Do you know where this remaining relayout call comes from? It would be awesome if we could turn this 1 into a 0!

I actually didn't think we had to reduce it to 0, but it's from renderImpl, we call it ourself, and you fix that in https://github.com/bloomberg/bqplot/pull/994 (my last comment was wrong).
But even if we get rid of that call, we would need to find a way to ignore those resize events. What we could do, is keep a boolean after the first real render call, since it does not make sense to 'relayout' when you didn't even render yet right?

But even if we get rid of that call, we would need to find a way to ignore those resize events. What we could do, is keep a boolean after the first real render call, since it does not make sense to 'relayout' when you didn't even render yet right?

I guess yes :) Let's do that. Maybe you want to apply the changes I made in #994 in your PR so that your PR clears all the relayout calls? (I am fine not having credits for that)

Got it to 0, but we should test this carefully, I propose we merge this after the next release.

Because we did not remove the resize handler, we got a lot of views that were detached, causing these error messages:
Error: <rect> attribute width: A negative value is not valid. ("-120")

These are gone now.

I think the last step into making resizing feel more fluent is debouncing. I think you should never react to 'resize' events by doing anything that is expensive (a simple plot takes >30ms to redo, leading to max 30 fps), it just makes the browser feel really laggy.
I propose we add a debouncer before the relayout, so that layout can only be called after 300ms of 'rest'.


require less than pydantic 2.0.0

see: https://github.com/ray-project/ray/issues/37019
In ray 2.6.0 will pin this but best do it for all previous versions

Do we had the release plan for all previous versions? @boetro


lookahead with do_sample=True does not take temperature, top_k, top_p

Here lookahead_generation doesn't take logits_warper as input:
https://github.com/alipay/PainlessInferenceAcceleration/blob/8015f12f7fe32acc102bb3eb51c4f8b3a420e79c/pia/lookahead/common/pretrained_model_batch.py#L426-L439
logits_warper is used in original sample to modify next_tokens_scores:
https://github.com/alipay/PainlessInferenceAcceleration/blob/8015f12f7fe32acc102bb3eb51c4f8b3a420e79c/pia/lookahead/common/pretrained_model_batch.py#L474-L486
and to modifies logits by temperature, top_k, top_p...
        if generation_config.temperature is not None and generation_config.temperature != 1.0:
            warpers.append(TemperatureLogitsWarper(generation_config.temperature))
        if generation_config.top_k is not None and generation_config.top_k != 0:
            warpers.append(TopKLogitsWarper(top_k=generation_config.top_k, min_tokens_to_keep=min_tokens_to_keep))
        if generation_config.top_p is not None and generation_config.top_p < 1.0:
            warpers.append(TopPLogitsWarper(top_p=generation_config.top_p, min_tokens_to_keep=min_tokens_to_keep))

https://github.com/huggingface/transformers/blob/09f9f566de83eef1f13ee83b5a1bbeebde5c80c1/src/transformers/generation/utils.py#L728-L733
This is not applied inside lookahead_generation. So with do_sample=True the temperature is always one

Thank you, we will fix it soon.

@zheyishine Hi, Has there been any progress?


Example 1 incorrect

Example 1 on this page is incorrect.
This:
Get-PnPRecycleBinItem | ? FileLeafName -like "*.docx" | Clear-PnpRecycleBinItem
Should be:
Get-PnPRecycleBinItem | ? LeafName -like "*.docx" | Clear-PnpRecycleBinItem
This was previously closed as an issue almost a year ago and I don't really get why: https://github.com/MicrosoftDocs/office-docs-powershell/issues/2770
Thanks!

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

ID: 47b3be08-c28b-f8a9-fc2f-418084aeac00
Version Independent ID: 89967280-3b45-01d3-99c5-e9854da371fa
Content: Clear-PnPRecycleBinItem (SharePointPnPPowerShell)
Content Source: sharepoint/sharepoint-ps/sharepoint-pnp/Clear-PnpRecycleBinItem.md
Service: sharepoint-powershell
GitHub Login: @pamgreen-msft
Microsoft Alias: pamgreen

How are you?
Even though the PnP documentation articles are on this repository, I need to kindly redirect you to open PnP cmdlets issues here, in that repository, where a member of the PnP community will be able to correct the article with your feedback.
The documentation for PnP cmdlets is then sync from that repo into this one.
Thank you!

Done, thank you: https://github.com/pnp/PnP-PowerShell/issues/3015

@get-itips Thank you very much for the contribution and sharing this explanation. @rakova17 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


App don't work

Hi,
i'm using Luma3DS with a9lh sysNAND 9.2, emuNAND 11.
Everytime i start the app, then it displays a text for 1/2 secound and then starts freeShop but it doesn't download the newest encTitleKey.bin
I can't tell you which text is displayed because it closes so fast that i'm unable to read it.
And no i have no old .bin file in the folder, it's just empty.

Same as #10. Already solved, see the comments there


Depth map for whole body

Am I missing something? Any chance to get depth map also for whole body? There are some poses that leads to 2d version of pose model that tend to flatter things and things that arm that is "behind" body is now in front of it. I believe that some simple depth map would solve this problem - but for that I believe some mapping to any 3d body object would be needed right?

It needs a perfect 3d human model and may go beyond this tool's meaning. I want it to be kept simple. It is better for professional software like blender to make this request.

Thank you for this amazing project.
Although it's different from the original intention of this issue, if it's possible to display human body parts, we can leave appropriate margins when creating a skeleton map.

@ZhUyU1997 I think it would be enough during export to replace skeleton with cylinders for depth map. For SD with lower than 1 guidance it should be enough to work well, what do you think?

@wilq32-unused  I list it in the plan, please wait for the next version


Update README.md

Updated section for metadata harvesting.

@tingletech @aturner Please commit this MR.


Speedup Save+Publish actions

Improve the speed of save and publish actions by reducing the amount of work they do re-saving the questionnaire.

For theodi/shared#686

@pkqk is this mergable yet?

@Floppy this is mergable, there's more I wanted to do but this should solve some of the issues.

Done then.


CUDA out of memory

Hi, thanks for your work.
I input batchsize 1 for one gpu, but out of memory？  (caused by 3D convolution)
python3 ./tools/train_net.py
--cfg ./configs/default/config_car.py
--savemodel ./outputs/dsgn_car
--start_epoch 1
--lr_scale 50
--epochs 60
-btrain 1
-d 0 \
What should I do to solve this problem?
Thanks for you help.

Hi, Thanks for your interest. As mentioned in README, the model requires 32G memory for training even for 1 batch per GPU because 3D Conv has a high demand of memory.
Since most people cannot get the device with enough GPU memory, I have tried several other settings for 24G memory and 12G memory. Training with 24G memory does not hurt the performance on the validation set but training with 12G memory hurt the performance a lot (- 10 AP). I will recently later upload the setting. Thanks!

thanks for your timely reply! Looking forward to your upload.
By the way,  Can you provide your pre-training model? I think my memory can be used for testing

You can find two pre-trained models for car and human in README.

thanks!
I have a problem. 3D convolution often exists in the network of binocular disparity estimation, such as PSMnet and GWCnet, but my 12G memory is enough for those.
I want to know what makes DSGN need so much memory.

3D volume costs a lot of GPU memory actually. Because plane sweep volume requires to store [Batch, H, W, D, F_dim], where initially H~=1284 / 4, W ~= 375 / 4, D~=192 / 4, F_dim=64. You'll find that it is approximately several times of ResNet-50 in 2D images with size (800, 1200).

Hello,
we are facing currently the same problem, even though we are provided 32 Gb of GPU Memory ( 4 GPUs, 7618 Mb each).
We run the training script as follows:
python tools/train_net.py --cfg ./configs/default/config_car.py --savemodel ./outputs/dsgn_car -btrain 4 -d 0-3 --multiprocessing-distributed
Do you know on what kind of issues this can lie? also can you please share the settings you have used to match 24 Gb?
Thanks in advance.

Hi, Thanks for your interest in this work.  Here I mean each GPU should have 32GB memory because the stereo matching costs a lot to store the intermediate volume.
I will update the settings these two days. Hope it can help you a bit.


auth, etcdserver: introduce revision of authStore for avoiding TOCTOU problem

This commit introduces revision of authStore. The revision number
represents a version of authStore that is incremented by updating auth
related information.
The revision is required for avoiding TOCTOU problems. Currently there
are two types of the TOCTOU problems in v3 auth.
The first one is in ordinal linearizable requests with a sequence like
below ():

Request from client CA is processed in follower FA. FA looks up the
username (let it U) for the request from a token of the request. At
this time, the request is authorized correctly.
Another request from client CB is processed in follower FB. CB
is for changing U's password.
FB forwards the request from CB to the leader before FA. Now U's
password is updated and the request from CA should be rejected.
However, the request from CA is processed by the leader because
authentication is already done in FA.

For avoiding the above sequence, this commit lets
etcdserverpb.RequestHeader have a member revision. The member is
initialized during authentication by followers and checked in a
leader. If the revision in RequestHeader is lower than the leader's
authStore revision, it means a sequence like above happened. In such a
case, the state machine returns auth.ErrAuthRevisionObsolete. The
error code lets nodes retry their requests.
The second one, a case of serializable range and txn, is more
subtle. Because these requests are processed in follower directly. The
TOCTOU problem can be caused by a sequence like below:

Serializable request from client CA is processed in follower FA. At
first, FA looks up the username (let it U) and its permission
before actual access to KV.
Another request from client CB is processed in follower FB and
forwarded to the leader. The cluster including FA now commits a log
entry of the request from CB. Assume the request changed the
permission or password of U.
Now the serializable request from CA is accessing to KV. Even if
the access is allowed at the point of 1, now it can be invalid
because of the change introduced in 2.

For avoiding the above sequence, this commit lets the functions of
serializable requests (EtcdServer.Range() and EtcdServer.Txn())
compare the revision in the request header with the latest revision of
authStore after the actual access. If the saved revision is lower than
the latest one, it means the permission can be changed. Although it
would introduce false positives (e.g. changing other user's password),
it prevents the TOCTOU problem. This idea is an implementation of
Anthony's comment:
https://github.com/coreos/etcd/pull/5739#issuecomment-228128254
/cc @heyitsanthony @xiang90

@heyitsanthony updated for your comments, PTAL

@heyitsanthony sorry for long stopping. I updated this PR and the CI failures would be resolved. I changed below points:

invalidate tokens during user deletion and password update
renamed the error code from auth.ErrRetryAuthRequired to auth.ErrAuthRevisionObsolete. Based on the error code, linearizable requests retry until they see success or other error cases (currently some requests e.g. lease related are not check the error code because auth cannot care about them for now)
serializable requests check the revision numbers in their headers after processing
removed a bug that causes nil pointer dereference by serialized request when auth isn't enabled

Could you take a look?

generally looks OK, deduping the repeated retry loops would probably make this patch a lot cleaner

@heyitsanthony updated for your comments, PTAL

lgtm after fixing last few nits. @xiang90, any thoughts on this?

@heyitsanthony It makes sense. And the approach looks good to me.

@heyitsanthony @xiang90 updated for the comments, PTAL.

lgtm

@mitake Can we fix CI?

@heyitsanthony on the second thought, I think the initialization of request header's auth info shouldn't be moved to the retry loop. It is because the initialization of r.Header should also be moved to the loop and also to the callers of processInternalRaftRequestOnce(). Duplicating the code ins't good so I'd like to keep the place of the initialization. Is it ok?

@mitake OK

@heyitsanthony @xiang90 updated for the datarace. authStore.commitRevisision() missed tx.Lock() and tx.Unlock(). The race would be resolved.

lgtm


Azure PowerShell Function

Hello,
I am following the below articles to create Azure PowerShell function and when I use this command(Set-PnPTraceLog -On -Level Debug) nothing being shown on the screen or in logs.
https://docs.microsoft.com/en-us/sharepoint/dev/declarative-customization/site-design-pnp-provisioning https://www.youtube.com/watch?v=plS_1BsQAto&t=925s
Is this functionality is still being worked on with Azure Function or am I missing something?

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

ID: 9d2c4d0f-adfd-cc4b-778f-4e2d2a812058
Version Independent ID: 9d051428-dc83-8db8-634b-7fa036da3c5d
Content: Set-PnPTraceLog (sharepoint-pnp)
Content Source: sharepoint/sharepoint-ps/sharepoint-pnp/Set-PnPTraceLog.md
GitHub Login: @techwriter40
Microsoft Alias: kirks

@officedocsbot assign @yogkumgit

Hello @ravick4u Ravi 👍
How are you?
The documentation for PnP cmdlets is directly reflected from the cmdlet's source code, that lives in the linked repository.
I will need to kindly ask you to visit this other repo to open PnP issues. PnP cmdlets are maintained by the community, a member of it will pick it up there.
Thanks

@get-itips  Thank you very much for the contribution and sharing this explanation. @ravick4u
Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Testing for space efficient SIFT/L2 space

Three things need to be done:

An integration test for the C++ test_integr
Additional test cases for Python integration tests including getDataPoint tests.
Run an older version of ann_benchmark for both old and new SIFT space.

Hi @searchivarius ,
I've tested 1.7.1 on the same dataset as used with the normal implementation and found no decrease in memory consumption when initialising the index with the following parameters:
index = nmslib.init(method='hnsw', space='l2sqr_sift', dtype=nmslib.DistType.INT, data_type=nmslib.DataType.DENSE_UINT8_VECTOR)
The space savings are noticeable on the saved index file --- 8.7GB for naive L2 and around 2GB for uint8, however once loaded into memory* they both use roughly the same amount of memory (roughly 9GB).

l2 index loaded by .loadIndex('file'), l2_sift index loaded by firstly calling .addDataPointBatch(np.load('sift')) (memory increases while reading the array, and then decreases) and then .loadIndex('file'). Memory usage is the same for both instances.

Hi @GreenSlime96 thanks for testing? The size of the index is smaller b/c the new version doesn't save data. Unfortunately, data has to be reloaded separately. Regarding the memory consumption: what is the value of M that you use?

Oops, didn't mean 'tested' per-se, but rather experimented it with :-)
Both indices were created using the same parameters: M=20, post=2, efConstruction=400

Hi @GreenSlime96 this is interesting. For one thing, memory consumption should have reduced dramatically just because in this mode the code doesn't create an optimized index.
For example, for 1M of SIFT vectors, I see the index size is nearly 700MB, the data size is 400MB. When the index is created the total memory consumption is about 700+700+400 = 1800MB.
For the more compact scenario, the index is still 700MB, but the data isn't duplicated, so the peak consumption is only 700+100=800MB.
However, after the index is created and simply loaded from disk there will be a less drastic difference:
800 Mb vs 1400 Mb.
Does it match your statistics?

Hi @searchivarius, I will assess it with 1M SIFT vectors tomorrow. Right now I am using 13,196,057 SIFT vectors to test both these models:
l2sqr_sift memory usage: 8623MB
l2 memory usage: 9433MB
So they are very similar in size, although l2sqr_sift is slightly smaller.


Publishing new livelab - Deploy an ODA Chatbot powered by Generative AI Agents using 23ai Vector DB: WMS ID 11764, also made changes to WMS ID 11617

WMS ID 11764, WMS ID 11617
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[X] All filenames are lower case (including folders, images, files, etc.)
[X] Filenames are descriptive
[X] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[X] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[X] Image references in markdown contain an alternate text

@anacoman11 I've made requested changes. Please approve the PR. Need to publish it soon. Thank you

@klazarz @anacoman11 I've redone all the images and am consistently using white blocks to block private information without any outline. Kindly please approve the PR. Let me know if anything else is needed from my end. Thank you!


Making the theme "Hugo" Friendly

Hi @SteveLane ,
Thank you for this project - It is very good skin in terms of design and simplicity.
However, the current way to setup the themes is strictly through config.toml which is not very Hugo friendly (adding text / items is really hard work).
I already modified that largely on my own project, but this will break the current implementation.
Let me know if you wish me to create a pull request or just preferred me to keep my own branch.
Cheers,
Dudi

@Dudil Did you fix the blogging capability on your project?

Sorry - I've moved into another theme but might be still to help.
@carneybill - can you elaborate what blogging capability are you looking for?

@dudil On the Hugo site (https://themes.gohugo.io/hugo-icon/), it mentioned: "add blogging capability"  under To Do.  I'm new but can't figure out how to create a blogging tab/new post.  I was unsure if @SteveLane or you might have addressed it?

@carneybill I haven't addressed it yet, but it would be useful, so I opened it as a to do. Here's what I think would need to be done:

[ ] add another partial for latest blog posts
[ ] apply appropriate css classes - I think using something like the classes in the mission partial would work
[ ] paginate the blog posts within that single page section
[ ] create new single blog post template
[ ] link each item to that single post

So, there's a bit of infrastructure needed before blogging can be added. (I didn't set the theme up with the intention of blogging straight away).

@SteveLane - Even though I’ve moved to another theme, I’ll be happy to assist.
I have solid understanding of Hugo and can help with building it up. Will need more help with CSS etc.
Also would love to add support for localization.
Is that fine?

@dudil for sure! If you want to help out, that'd be great. This is not a high priority for myself.


Add spec for behavior of autoload in Class.new

The spec added here defines current behavior of CRuby in which a bare autoload called from an instance method inside a Class.new should be defined on that new class. This differs from constant lookup, which will skip the new class and use whatever "concrete" class scope is above it.
As noted in the commit, this behavior is problematic at least because this autoload and a parallel constant lookup will go to different target classes due to autoload using CRuby's concept of "cbase" and constant lookup using "cref".
This behavior changed sometime after Ruby 1.8.7. Prior to the change, the autoload appears to use the same "cref" target that constants use.
I push this spec as a PR to discuss whether this is correct behavior. I believe it is unexpected and should be deprecated, with Kernel#autoload being modified in the future to always target the same "cref" class that would be used for defining constants in the given scope.
All alternative implementations of Ruby I tested fail this spec.

I think it would be better to discuss the behavior on https://bugs.ruby-lang.org/ (or a mail to ruby-core).
Which behavior would you expect? That the autoload effectively uses the top-level/Object as a receiver?
It's a but confusing in the spec to use an Object constant. I guess it's intended but would be worth a clarifying comment.
I'm fine to merge the spec describing the current CRuby behavior, we can always adapt if the behavior changes.

I think the rule for Kernel#autoload should be that it's the same as (Module.nesting.first in the caller || Object).autoload name, file.
I agree this behavior is weird and unexpected. Constant lookup (both reading and writing constants) always ignore anonymous classes/modules, and so should autoload.

@headius The spec was added when sync'ing with spec/ruby in JRuby.
What do you think, do we keep it in ruby/spec until there is some decision on the Ruby tracker?
Or should we remove it?
It would be useful to file an issue at https://bugs.ruby-lang.org/ if there isn't already one in any case, could you do that?


proposal: Downstream source ip/port filter

Concept:
I am thinking of a new filter that would do filtering on the source IP/port of the connection. With this filter, we would be able to control where to accept connections from. Like filter_chain_match's source_prefix_ranges and source_ports. If the source IP is in the given CIDR range in source_prefix_ranges and the port is listed below source_ports then accept, otherwise reject.
Not sure if this should be a network or a listener filter.
Use case:
In #15590 I mentioned that we want to let a single RTP stream on a single listener and no more because multiple streams would cause problems.
The main problem is with this use case that it is focused on UDP and as I know network filters don't work with UDP listeners in general (looking for further information in #15887). And right now only 1 UDP listener filter per listener supported.
WDYT?
Questions:

Is there any situation when a UDP listener rejects/refuses a "connection"? And if so, where, when and how should it happen?
In general, if I want to implement this function somehow for UDP listeners what would you recommend how should this be done and what'd be the easiest and best way.

Note:
First I thought this could work with configuring the filter_chain_match in a listener, but the connection is refused immediately if the filters list is empty. And there's no such filter that does nothing. So it didn't work. But the main idea is similar of the filter is similar to this.
Let me know anything you think about this. I'll gladly work on this, but need some guideline. Thanks

FilterChainMatch is not applied to udp_listener. Partially because udp listener doesn't not support "connection filter chain"
I would suggest creating the udp filter chain. Since the udp is packet-oriented, it's probably not reasonable to separate listener filter vs network filter as in the tcp listener.
A single udp filter chain can simulate the role of FilterChainMatch which dispatches to multiple filter chains. A generalized udp filter chain selector can be done after.
WDYT?

Yeah I would suggest we track this in https://github.com/envoyproxy/envoy/issues/15887 per @lambdai. Also, we need to tease apart whether you want this for TCP or UDP. If you want this for TCP I would say either do it with filter chain match or potentially via network RBAC.
If you want this for UDP I think it makes more sense to properly implement multiple filters. Then we can do whatever we want including RBAC for UDP, etc.


Red squigglies in all razor pages in latest update (2.28.11)

Environment data
dotnet --info output: 7.0.18
VS Code version: 1.89.0 (latest)
C# Extension version: 2.28.11 (latest)
OmniSharp log
n/a
Steps to reproduce
I just updated to the latest extension, offered a few hours ago.
Now all my razor pages show red squigglies, even though they compile correctly and the app runs without error.
The red squigglies make this version completely unusable so I intend to roll back to a previous version.
Expected  behavior
No squigglies.
Actual behavior
Squigglies.
Additional context
Here's an example from one of many razor (Foo.cshtml) files:
Ambiguity between 'Pages_Customers_Orders_Index.__Microsoft_AspNetCore_Mvc_TagHelpers_FormTagHelper' and 'Pages_Customers_Orders_Index.__Microsoft_AspNetCore_Mvc_TagHelpers_FormTagHelper'CS0229
Ambiguity between 'Pages_Customers_Orders_Index.__tagHelperRunner' and 'Pages_Customers_Orders_Index.__tagHelperRunner'CS0229
Ambiguity between 'Pages_Customers_Orders_Index.__tagHelperExecutionContext' and 'Pages_Customers_Orders_Index.__tagHelperExecutionContext'CS0229
Ambiguity between 'Pages_Customers_Orders_Index.__Microsoft_AspNetCore_Mvc_TagHelpers_FormTagHelper' and 'Pages_Customers_Orders_Index.__Microsoft_AspNetCore_Mvc_TagHelpers_FormTagHelper'CS0229

@lonix1 , we're sorry you are running into this.  We noticed the same recently and are already working on it, and we'll keep you (and others who run into it) posted here.

Same problem here:
Ambiguity between 'Pages_Movies_index.__Microsoft_AspNetCore_Mvc_TagHelpers_AnchorTagHelper. Any work around it is so annoying.

@davidwengier I have rolled back to the previous version and the red squiggly still persist. Here is the extension am using:
https://github.com/KreativJos/csharpextensions

Thanks @Ionix1 and @davidwengier rolling back C# for Visual Studio Code to version 2.23.15 solved the issue.

Update: We understand what caused the issue, we understand why it didn't occur before, and have a plan to fix it.  We'll keep you posted here.

Fix has been merged into prerelease in vscode with https://github.com/dotnet/vscode-csharp/pull/7134


Constrain core dependencies to core versions

Description:
Constrain the dependencies that get installed as part of installing dependencies to the version that HASS uses if they overlap.
Updates gen_requirements_all.py to also generate a constraint file and validate that it is up to date.
Related issue (if applicable): fixes #5892. Fixes #6728
Example entry for configuration.yaml (if applicable):
Add either one of the following to your config and HASS will crash after 2 reboots before this patch.
media_player:
  platform: apple_tv
  host: ''
  login_id: ''

android_ip_webcam:
  host: ''

So what happens if a platforms pulls a lib that has a conflicting constraint? The platform won't load?

@andrey-git it will install the version that Home Assistant depends on, so it's all good 👍


Update README.md

Fixes #7

Adds https:// at start of URLs
Adds versions in URLs to fix 404 errors

I think you mean it closes #8 since the author of #7 closed that one already in favor of #8.


Fix regression: GC peformance degradation for large documents

Bug
GC peformance deteriorate as the number of data stores increase in a container.
For containers that have 1500 data stores and 4500 DDSes, the first GC run time has gone from ~200ms to ~25000 ms. This is
causing UI freezes for users.
This only happens during the first summary done by a summarizer client. Basically, every summarizer client's first summary will be slower.
Root-cause
The regression was introduced by this change - Fixed bug where DDS's summarizer node's GC data is not correctly updated on creation after refreshing from snapshot by agarwal-navin · Pull Request #13455 · microsoft/FluidFramework (github.com).
With this change, the very first time GC runs, getBaseGCDetailsFn is called by summarizer node of all data stores and each of them calls unpackChildNodesGCDetails on the entire set GC nodes.
For 1500 data stores and 4500 DDSes, there are 6000 GC nodes. So, unpackChildNodesGCDetails is called 1500 times which iterates over 6000 nodes resulting in a slow down.
Fix
The fix is to call unpackChildNodesGCDetails only once per node on the base GC details of that the node. This reintroduces the bug that was fixed by https://github.com/microsoft/FluidFramework/pull/13455. That bug will be fixed in a follow up PR.
Tests

[ ] Todo. I am working on adding tests that will prevent these issues from happening in the future.

AB#3667

Seems like a plausible fix... Were you able to manually test it? Depending on the urgency you might check it in without tests and then add tests in a follow-up.

Yeah, I tested it manually. Nicholas and I working on adding benchmark tests for GC. Looks like that will happen as a follow up and not as part of this PR.

@agarwal-navin, how was it found?
Do we have a task tracking adding a test to benchmark suite that Nicholas is working on?

@agarwal-navin Navin Agarwal FTE, how was it found?
Do we have a task tracking adding a test to benchmark suite that Nicholas is working on?

I don't think so. Nicholas has a PR out though - https://github.com/microsoft/FluidFramework/pull/14528