Error opening none existing file

Is it possible to catch the error when a none existing file wants to be opened?
console.log(error) will not be reached.
Example Typescript Code:
try {
  const file: hdf5.File = new hdf5.File('./record.h5', Access.ACC_RDWR);
  const dims: number[] = file.getDatasetDimensions('infos');
  file.close();
  if (dims.length > 0) {
    return dims[0];
  }
  return 0;
} catch (error) {
  console.log(error)
  return 0;
}

Error output:
HDF5-DIAG: Error detected in HDF5 (1.10.0-patch1) thread 0:
  #000: C:\autotest\HDF5110ReleaseRWDITAR\src\H5F.c line 579 in H5Fopen(): unable to open file
    major: File accessibilty
    minor: Unable to open file
  #001: C:\autotest\HDF5110ReleaseRWDITAR\src\H5Fint.c line 1100 in H5F_open(): unable to open file: time = Wed May 23 16:39:39 2018
, name = './record.h5', tent_flags = 1
    major: File accessibilty
    minor: Unable to open file
  #002: C:\autotest\HDF5110ReleaseRWDITAR\src\H5FD.c line 812 in H5FD_open(): open failed
    major: Virtual File Layer
    minor: Unable to initialize object
  #003: C:\autotest\HDF5110ReleaseRWDITAR\src\H5FDsec2.c line 348 in H5FD_sec2_open(): unable to open file: name = './record.h5', errno = 2, error message = 'No such file or directory', flags = 1, o_flags = 2
    major: File accessibilty
    minor: Unable to open file

yea we should work in more error handling; I'll add a unit test for this one

hmm,
const file: hdf5.File = new hdf5.File('./record.h5', Access.ACC_RDWR);

fails for me with
SyntaxError: Missing initializer in const declaration

pull a copy
Should throw and catch an error with "File ./record.h5 doesn't exist." message

https://github.com/HDF-NI/hdf5.node/blob/master/test/test_hdf5.js#L94
should demonstrate a try catch for this


📝 Correction | elaborate on effect of browser tweaks

Description
Add more notes to the browser tweaks.
Why I am making the suggestion
It would be nice to know more about the (side) effects of certain settings.
To better determine whether it is a change you want.
My connection with the software
Using the browser for development as well, some changes might have undesirable effects.

[x] I will keep the issue up-to-date if something I have said changes or I remember a connection with the software.

That would have saved me a lot of time if it existed back in the days :D
Here's my list for things, which I didn't tweak with the reason:
network.http.referer.XOriginPolicy = 1 --> You can't watch videos on facebook anymore (I know you shouldn't have facebook, some of us still wants it. Also, I suppose Facebook videos are not the only one who will get affected by this)
dom.event.clipboardevents.enabled = false --> You can't copy fotos and paste them in your browser anymore (for sending photos on web.whatsapp.com or pasting it to a website)
browser.send_pings = false --> I'm not sure why I didn't tweak this, it broke something but I didn't write it down.
Things may have changed, or I may have wrong conclusions. So this should be checked over again.

This can be done in #2081.

Using the browser for development as well, some changes might have undesirable effects.

For this it's IMHO the best to use one profile for development and one for normal browsing.

browser.send_pings = false --> I'm not sure why I didn't tweak this, it broke something but I didn't write it down.

browser.send_pings is false by default.

browser.send_pings is false by default.

oh, that's probably the reason why I wrote that one in my "not to tweak" list.

Also, privacy.resistfingerprinting seems to affect the dark mode which may be important for some users. We shouldn't make people tweak everything and then hate on firefox for not letting them surf the web as they want.
https://www.reddit.com/r/privacytoolsIO/comments/ohmbw8/firefox_enabled_privacy_aboutconfig_darkmode_gone/

TBH this is the list that don't affect site. All other options (can) affect site. Some only a few sites (e.g. privacy.trackingprotection.cryptomining.enabled = true or media.eme.enabled = false) and others more sites.

browser.send_pings = false
browser.urlbar.speculativeConnect.enabled = false
browser.sessionstore.privacy_level = 2
browser.safebrowsing.downloads.remote.enabled = false
Disable Firefox prefetching pages it thinks you will visit next:
network.IDN_show_punycode = true
Looking for TRR, DoH or ESNI?

This can be done in #2081.

only just over three more months to go to celebrate the 2 year anniversary
This issue is effectively Part 4 of that list. I gave up waiting for Part 5 and have been wrecked many times over. Not sure why this takes almost two years


OSGi support

https://github.com/jparsec/jparsec/issues/37

Given this code upgrades the whole codebase to Java 1.8, which makes sense, we also need to upgrade .travis.yml configuration which is targeted at java 1.7.


PLease help, BCH hostage, "not connected " electron cash issue !

english is not my native language.
Electron Cash 3.0 wallet indicates that it's not connected.
I see my transactions but I can't do anything elese.
I went in my Electron Cash "Tools">"Network" settings, unchecked "select server automatically" and tried multiple alternative "BCC"/"Electron Cash" servers, but the status remains at "not connected".
I noticed the  problem with the 2.9.x-version when I tried yesterday evening but everything was fine 2 or 3 weeks ago.
I upgraded the "main one'' to v3 than installed fresh wallet 3.0 on 3 different PC, with my seed,  and still the same isuues.
PS: I disabled kaspersky, windows firewall...My main network provider is down, so I switched to an other back up one, but I don't think they could block electron cash ?!

was only due to my provider switch, where there was a firewal setting on the box. ( orange livebox pour les francais ! )


Strange text noted in description.

Under Properties > Datadisks and Properties > OSDisk, seems that part of the description has "<br><br&gt" < (this is actually formatting so I suspect it could be the issue) in it. Doesn't seem like this is expected behavior. I noticed that there is a lot of that text also in this page: https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.management.compute.models.osprofile?view=azure-dotnet. Just hit a CTRL+F and you'll see how many times it shows up.

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

ID: 50f2696c-e0d3-3b71-e523-41e42bcc426c
Version Independent ID: 3e57fbee-e376-d443-0cd0-337cfddcb98c
Content: StorageProfile Class (Microsoft.Azure.Management.Compute.Models) - Azure for .NET Developers
Content Source: xml/Microsoft.Azure.Management.Compute.Models/StorageProfile.xml
Service: multiple
GitHub Login: @CamSoper
Microsoft Alias: casoper

@ShivangiReja @danieljurek @AlexGhiondea FYI, I've added this issue to the description of https://github.com/Azure/autorest.csharp/issues/208 as a duplicate.


Update nixpkgs to latest master

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Update nixpkgs to latest master
Which issue(s) this PR fixes:

None
Does this PR have test?
None

Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

Codecov Report

Merging #581 (25db583) into master (79f3d31) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #581   +/-   ##
=======================================
  Coverage   44.39%   44.39%
=======================================
  Files          26       26
  Lines        1347     1347
=======================================
  Hits          598      598
  Misses        709      709
  Partials       40       40

/test pull-security-profiles-operator-test-e2e

/test pull-security-profiles-operator-test-e2e


Dispatcher refactor

Before:
Dispatcher::run(...)
// run pre filters
// ...

// run requested method
// ...

// run post filters
// ...

After
$this->runPreFilters(...);
$this->runEvent(...);
$this->runPostFilters(...);

Next refactor I want that it will be:
$event = $this->events->get($eventName);

// Events::__invoke() will call pre filters and post filters
return $event(...$params);

If you accept this refactor, please don't delete the branch to make another refactor


Request: Service name(s) in the process column list

Having the service name(s) (csv) in the process coumn list would be invaluable, more than just in the Services tab. In the process list, it would identify which services that process is hosting (if any) without having to hover over each. In particular it would be most useful when viewing the processes in the treeview.
dmex, could you make this happen, please?
Thank you in advance if so!!

👍  It was looking to add such feature request, but then here we go.
If possible I would love to see distinction for "Service names", "Service display names", "Service file names" (NOTE: Plural since you can have multiple services under single svchost.exe.

Yes, with the number of services in Windows 10, I get REALLY tired of hovering over ALL and trying to memorize their start order in the tree.

service name(s) (csv) in the process coumn list

The only problem is the performance cost from allocating multiple strings and building the list every second?
You can open the properties for the process and view the services tab which shows the running services inside that process?

Hey dmex thx for coming in. Opening just isn’t the same, when desiring side-by-side quick comparison. It’s probably faster to hover over for the tooltip. I wonder if there could be some option / toolbar switch to populate this column or not. Win10 has brought about so many I really want to see which one(s) is represented by what process in the tree.

The only problem is the performance cost from allocating multiple strings and building the list every second?
You can open the properties for the process and view the services tab which shows the running services inside that process?

dmex,  I I implemented this in my own environment within proctree.c and proctree.h.
Whats the best way to get this implemented into the main code base / to have you review? Do I upload my changes in these 2 files to my github and do some pull request?
LMK, thanks!

@AltF5 Can you please try to create Pull Request against latest master, I am really eager to try it out. 🤓 Thanks!

If a service has multiple associated by it, it could just list the topmost one and then & 1 more or & 2 more or something. You could also make it list them in all in a comma'd list for those who want to expand their columns that far, but it'd mess up Size all columns to fit. An option to have it be a multi-columned entry when the parent process is collapsed would be good but probably difficult to implement.

The only problem is the performance cost from allocating multiple strings and building the list every second?

@tmex If I can propose a trade-off, if the process is svchost.exe and has only one service can we replace Name and Description with service's Name (but not DLL) and Description?
If we don't want this as a default, this can be enabled via some new setting at Option > General, e.g. "Replace svchost.exe Name and Description with single service" or something similar.
I guess it will make most of us happy working on recent >Win10 machines. Also, this does not require allocating anything.
WDYT?

The only problem is the performance cost from allocating multiple strings and building the list every second?

Hi @dmex, since the list of services is fixed, would it not be possible to create the string list once and then reuse it based on whatever identifier Windows has for the service?
But even with constant string recreation, this can be hidden behind an option flag until it's optimized later. "Replace svchost with service name (may incur performance penalty)". I'd use it, 100 strings per second is nothing on a machine capable of running Windows.

You can open the properties for the process and view the services tab which shows the running services inside that process?

This doesn't show CPU/RAM/disk usage, which is the main reason I use Process Hacker. I wanna know what's using my system's resources. Right now Process Hacker is falling behind Task Manager by not listing services individually.

Whats the best way to get this implemented into the main code base / to have you review? Do I upload my changes in these 2 files to my github and do some pull request?

@AltF5
You can submit a PR for review.

can we replace Name and Description with service's Name (but not DLL) and Description?

@nanoant
The name column has a cache and replacing the strings would force a cache rebuild each second defeating the purpose of caching those entries and tanking the UI performance.

since the list of services is fixed, would it not be possible to create the string list once and then reuse it based on whatever identifier Windows has for the service?

@jerkstorecaller
Services are not fixed and are added/removed dynamically at runtime. Trying to synchronize the changes across multiple providers and multiple treelists only causes more issues than it solves - I've tried - the processes list isn't designed for services which is why services are located on a dedicated services window separate from the processes window and uses a separate thread and separate provider.

Process Hacker is falling behind Task Manager by not listing services individually.

That's one reason why Task Manager performance sucks so bad and takes ages to startup.
Services are essentially DLLs running inside another process and the SCM functions are built on TCP style RPC calls. When you're using a .5s refresh internal that 500ms (or more) goes entirely on updating the service information preventing any other features from working - essentially the SCM doesn't play nice, causes major performance problems and was separated from other dialogs.
There are already 7 different windows in Process Hacker including an entire tab/tree showing individual service information and trying to shove more of that info onto the processes list means having to sacrifice a lot for little benefit. Ideally you should be using the existing dialogs and windows more effectively which give more information, more utility and are more reliable.

Whats the best way to get this implemented into the main code base / to have you review? Do I upload my changes in these 2 files to my github and do some pull request?

@AltF5
You can submit a PR for review.

can we replace Name and Description with service's Name (but not DLL) and Description?

@nanoant
The name column has a cache and replacing the strings would force a cache rebuild each second defeating the purpose of caching those entries and tanking the UI performance.

since the list of services is fixed, would it not be possible to create the string list once and then reuse it based on whatever identifier Windows has for the service?

@jerkstorecaller
Services are not fixed and are added/removed dynamically at runtime. Trying to synchronize the changes across multiple providers and multiple treelists only causes more issues than it solves - I've tried - the processes list isn't designed for services which is why services are located on a dedicated services window separate from the processes window and uses a separate thread and separate provider.

Process Hacker is falling behind Task Manager by not listing services individually.

That's one reason why Task Manager performance sucks so bad and takes ages to startup.
Services are essentially DLLs running inside another process and the SCM functions are built on TCP style RPC calls. When you're using a .5s refresh internal that 500ms (or more) goes entirely on updating the service information preventing any other features from working - essentially the SCM doesn't play nice, causes major performance problems and was separated from other dialogs.
There are already 7 different windows in Process Hacker including an entire tab/tree showing individual service information and trying to shove more of that info onto the processes list means having to sacrifice a lot for little benefit. Ideally you should be using the existing dialogs and windows more effectively which give more information, more utility and are more reliable.

@dmex , I'm sure different people use PH for different reasons, but for me, the reason I installed PH is to be able to answer the question "why is my computer slow all of a sudden?". Which translates to the combo "What things are running on my computer" and "How much resources is each thing using".
Task Manager already tells me "what is running on my computer" and "how much is it using", I didn't need PH for that.
What's great about PH is that it adds per-process CPU and I/O history on top of this, allowing me to answer the above questions, as well as "what made my computer slow 30 seconds ago, but by the time I opened Task Manager everything was fine again?".
PH fails at this task when it comes to services. I can't tell whether it's Logitech or Oracle's crappy update service that's slowing my PC down, or one of Microsoft's optional services. Basically it's saying "if a service is hogging your resources, PH can't help you identify it, it only helps if it's a regular process".
Right now, Task Manager is superior to PH if the culprit is a service, because it splits the 200 services into individual names. But, no historical data, so I can't investigate temporary slowdowns.
I know there's fancy sysadmin tools for system instrumentation and whatever, but I'm just a desktop user.
If there's a way to tell which service slowed down my system 30 seconds ago with PH, then I missed it, please tell me how.

@dmex , I'm sure different people use PH for different reasons, but for me, the reason I installed PH is to be able to answer the question "why is my computer slow all of a sudden?". Which translates to the combo "What things are running on my computer" and "How much resources is each thing using".
Task Manager already tells me "what is running on my computer" and "how much is it using", I didn't need PH for that.
What's great about PH is that it adds per-process CPU and I/O history on top of this, allowing me to answer the above questions, as well as "what made my computer slow 30 seconds ago, but by the time I opened Task Manager everything was fine again?".
PH fails at this task when it comes to services. I can't tell whether it's Logitech or Oracle's crappy update service that's slowing my PC down, or one of Microsoft's optional services. Basically it's saying "if a service is hogging your resources, PH can't help you identify it, it only helps if it's a regular process".
Right now, Task Manager is superior to PH if the culprit is a service, because it splits the 200 services into individual names. But, no historical data, so I can't investigate temporary slowdowns.
I know there's fancy sysadmin tools for system instrumentation and whatever, but I'm just a desktop user.
If there's a way to tell which service slowed down my system 30 seconds ago with PH, then I missed it, please tell me how.

Task Manager

Try resizing one of the columns back and forth and watch the entire UI bounce around like a yoyo from 1993:

It's even worse over RDP without a dedicated graphics driver. The column headers are not even hooked up to any values in the columns for processes and are just showing the system totals without counting the column values.... It also doesn't show the correct names for services and just picks one at random:

It might look fancy but it barely works and the performance is so bad they've boxed themselves in and now can't do anything else with that window in the future without a complete rewrite from scratch... The code needed to mash those concepts/objects together prevents any optimization and is the main reason why TM can't include other features because that one feature eats up all the performance and also why TM doesn't have any real-time search features like PH does.
A service is literally just 1 DLL loaded in a process out of dozens of other non-service DLLs. The design and concept of services are entirely different to processes and provide minimal information. There isn't a way to show individual service utilization for IO/Memory/Disk etc... those statistics are only recorded for the process not the services because of the way they've been designed by Microsoft.
We've put services and processes as close together as possible in as many places as possible but both together on the processes list - as cool as it would be - is a bad idea.

I can't tell whether it's Logitech or Oracle's crappy update service that's slowing my PC down, or one of Microsoft's optional services. Basically it's saying "if a service is hogging your resources, PH can't help you identify it, it only helps if it's a regular process".

It does but are you using any of these features?
Menus:

Tooltips:

Threads:

Services:

There's even an entire tab on the main window dedicated to just services:

If there's a way to tell which service slowed down my system 30 seconds ago with PH, then I missed it, please tell me how.

You double click the graphs to view the process then click properties to view the services:

Task Manager

Try resizing one of the columns back and forth and watch the entire UI bounce around like a yoyo from 1993:

It's even worse over RDP without a dedicated graphics driver. The column headers are not even hooked up to any values in the columns for processes and are just showing the system totals without counting the column values.... It also doesn't show the correct names for services and just picks one at random:

It might look fancy but it barely works and the performance is so bad they've boxed themselves in and now can't do anything else with that window in the future without a complete rewrite from scratch... The code needed to mash those concepts/objects together prevents any optimization and is the main reason why TM can't include other features because that one feature eats up all the performance and also why TM doesn't have any real-time search features like PH does.
A service is literally just 1 DLL loaded in a process out of dozens of other non-service DLLs. The design and concept of services are entirely different to processes and provide minimal information. There isn't a way to show individual service utilization for IO/Memory/Disk etc... those statistics are only recorded for the process not the services because of the way they've been designed by Microsoft.
We've put services and processes as close together as possible in as many places as possible but both together on the processes list - as cool as it would be - is a bad idea.

I can't tell whether it's Logitech or Oracle's crappy update service that's slowing my PC down, or one of Microsoft's optional services. Basically it's saying "if a service is hogging your resources, PH can't help you identify it, it only helps if it's a regular process".

It does but are you using any of these features?
Menus:

Tooltips:

Threads:

Services:

There's even an entire tab on the main window dedicated to just services:

If there's a way to tell which service slowed down my system 30 seconds ago with PH, then I missed it, please tell me how.

You double click the graphs to view the process then click properties to view the services:

Hello there. Sorry I never saw the comment about doing a Pull Request.
@dmex Would this still be entertained to implement these service strings as a column? I find it very useful in my modification build .


Conda installation instructions result in ResolvePackageNotFound

Output is as follows:
$ conda env create --name bootleg_dev --file conda_requirements.yml
Collecting package metadata (repodata.json): done
Solving environment: failed

ResolvePackageNotFound:
  - torch[version='1.6.0.*,1.6.0.*']
  - tagme=0.1.3
  - transformers=3.0.1
  - marisa-trie=3.1.1

Conda version: 4.9.2
Python version (Anaconda): 3.8.5
CUDA version: 10.2
Output of nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2019 NVIDIA Corporation
Built on Wed_Oct_23_19:24:38_PDT_2019
Cuda compilation tools, release 10.2, V10.2.89

Installing using pip install -r requirements.txt also results in the following error message, but seems to build everything so that the python setup.py develop completes without error.
ERROR: After October 2020 you may experience errors when installing or updating packages. This is because pip will change the way that it resolves dependency conflicts.

We recommend you use --use-feature=2020-resolver to test your packages with the new resolver before it becomes the default.

python-language-server 0.35.1 requires ujson>=3.0.0; python_version > "3", but you'll have ujson 1.35 which is incompatible.
python-jsonrpc-server 0.4.0 requires ujson>=3.0.0, but you'll have ujson 1.35 which is incompatible.

Hello Sujit!
We'll look into the conda issue. As for pip, I can modify the requirements to be more flexible for ujson. The exact version of ujson should not matter. Out of curiosity, what python version are you using?

Hello Sujit!
We'll look into the conda issue. As for pip, I can modify the requirements to be more flexible for ujson. The exact version of ujson should not matter. Out of curiosity, what python version are you using?

Hi @lorr1 thank you for looking into this. I am using Anaconda Python version 3.8.5. One other minor documentation nit -- after the conda create environment command, I think that we have to conda activate <env_name> before running the python setup.py develop command.

Hi @lorr1 thank you for looking into this. I am using Anaconda Python version 3.8.5. One other minor documentation nit -- after the conda create environment command, I think that we have to conda activate <env_name> before running the python setup.py develop command.

We just pushed a fix https://github.com/HazyResearch/bootleg/pull/20 that modifies the conda environment. This should fix the dependency issues.
Let us know if you have any other issues with environments and we can reopen this.

We just pushed a fix https://github.com/HazyResearch/bootleg/pull/20 that modifies the conda environment. This should fix the dependency issues.
Let us know if you have any other issues with environments and we can reopen this.


Implement BST

See:

BinarySearchST.java
Binary Search Trees

Moved to jkt: https://github.com/pcoet/jkt-patterns/issues/49


Support package compilation on watchOS

Resolves #36
Adjusts UI code to support watchOS, removing unsupported features and adding conditionals when necessary.
Note: WatchOS UI is completely untested and unverified, this only makes the framework compile.

@mjarvis 😍


Hide inlay hints for *error-type*

Please note. This is just a minor feature request for visual.

It would look much cleaner if we could hide the inlay hints for *error-type*.
I think showing this up to me doesn't help much.

I really hope this is configurable since I have all of the lsp's inlay hint options enabled so that I can use vscode's "editor.inlayHints.enabled": "offUnlessPressed" setting to quickly see information without perpetual suffering.


marl: init at 1.0.0

Motivation for this change
This pull request packages a modern fiber scheduler library for C++, called marl.
I have been using this package with nix-shell for a few projects which verifies that the package works as expected.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 106042 run on x86_64-linux 1

1 package built:

marl

Result of nixpkgs-review pr 106042 run on x86_64-linux 1
1 package built:

Does this suggest any issue found in nixpkgs-review? I ran the command locally and didn't see issues.

Does this suggest any issue found in nixpkgs-review? I ran the command locally and didn't see issues.

Nope, this just means that the review tool was able to build the package successfully
nixpkgs-review is more useful when testing existing packages that are used by other packages in the tree and helping make sure they still compile - since this is a new addition, we only get a single built package, which can then be inspected (via the result symlink)
In this case, I used it to just easily pull your changes, apply them to the master branch, build the package, and then check that the include and lib folders contain the expected header and cmake files.

Does this suggest any issue found in nixpkgs-review? I ran the command locally and didn't see issues.

Nope, this just means that the review tool was able to build the package successfully
nixpkgs-review is more useful when testing existing packages that are used by other packages in the tree and helping make sure they still compile - since this is a new addition, we only get a single built package, which can then be inspected (via the result symlink)
In this case, I used it to just easily pull your changes, apply them to the master branch, build the package, and then check that the include and lib folders contain the expected header and cmake files.
The command in the issue template (nix-shell -p nixpkgs-review --run "nixpkgs-review wip") only works with WIP changes - once you have opened the PR, you can use nixpkgs-review pr 106042 to build the package and any downstream packages locally, and add nixpkgs-review pr --post-result 106042 if you want to build and automatically post the results as a comment (requires a github token configured)

Thanks a lot for the detailed explanation and prompt answer. I'll take a note and add such a comment explaining what have been built next time.
Appreciate the help!


Enlarge avatars in message list, to 40px

The avatars we show in the message list for messages' senders are 32px.
For commit 3dd5fb4de (part of #3188), @borisyankov studied several comparable uses and determined it'd be good to expand them to 40px:
After comparing the size of the WebView avatar to the web app and
few competitors, I found out we are showing them rather small.

Web app has them 35px x 35px, but several other mobile chat apps
have them at around 40px and that is what we chose.

That commit caused #3508 , a misalignment between sender-showing messages (which moved further inward to make room) and sender-implicit messages (which didn't). So for now we're reverting the expansion with #3511.
This task is to make them 40px again, while aligning other elements to match.

I've just merged the parts of #3188 that converted us to using px for alignment rather than rem; I think that will make a solution to this cleaner than it would have been before. (We were in a bit of a mixed state, from having merged some previous bits of #3188.)

I don't think this is the right move to make, to be honest. The avatars are big enough for us to be able to visually make them apart and understand their contents without being overly big.

After comparing the size of the WebView avatar to the web app and
few competitors, I found out we are showing them rather small.

I don't agree with this. Infact, when I compared our avatar's size with those in Discord and Instagram direct, I found that ours are actually bigger (on default system display size and font size)!
Adding 8 more px to avatar size is going to make them larger for no reason, since if the user wants to see enlarged avatars they can simply click the avatar to go to the profile page.
Adding 8px to the avatar size is also going to make lesser space available for our messages, and they will expand vertically, taking more vertical space in turn.

After comparing the size of the WebView avatar to the web app and
few competitors, I found out we are showing them rather small.

I don't agree with this. Infact, when I compared our avatar's size with those in Discord and Instagram direct, I found that ours are actually bigger (on default system display size and font size)!

Hmm interesting! That is good data, then. Boris didn't add any specifics about which apps he compared with.
I'll go look around at a few others now...

With default font size (and default display density, though that one really shouldn't affect things), comparing against our v25.3.117:

FB Messenger:

Inside a conversation, avatars look exactly the same size as ours (so 32px/32dp, I guess), except circular.
In the conversation list, the avatar for a 1:1 conversation is much bigger -- maybe 64px? Even for a group conversation, where two smaller avatars are overlaid, each is a little bigger, perhaps 40px or 44px. (In our conversation list, they're the same 32px as in the message list.)

WhatsApp: No avatars in message list.

In conversation list, it's much like Messenger 1:1s.

LINE: Much like Messenger -- 32px in message list, ~64px in conversation list (at least for 1:1 conversations.)
Slack: Bigger than ours -- about 48px in the message list. I have a guess now what app Boris was most thinking of when he wrote that. :wink:

OK, with that data and with the points you made about space, I'm convinced -- let's leave the avatars at 32px in the message list.
I have a couple of related thoughts, from staring at this... and hmm, I'll go put those in chat rather than here. Closing this -- thanks again!

For cross-reference: the resulting chat thread is here:
https://chat.zulip.org/#narrow/stream/48-mobile/topic/avatar.20layout/near/752153
It then led to #3529, which increased avatar size in the PM conversations list and also in the message list after all.


core: update c callbacks to allow pass-through of platform lambdas

@goaway I think this is done?

Yep this is done for both platforms via the type erased context pointer in the envoy_observer struct.


[DOCS] change typo

What changes were proposed in this pull request?
change typo in the user guide. change "假话“ to 简化
How was this patch tested?

[X] Testing done

Are there and DOC need to update?

[X] Doc is finished

Spark Core Compatibility

Thanks 😁


Add pyproject.toml for Custom Node Registry

Hey! My name is HaoHao and I'm from comfy-org! We would love to have you join the Comfy Registry, a public collection of custom nodes which lets authors publish nodes by version and automate testing against existing workflows.
Eventually, the registry will be integrated as a backend for the UI-manager where all nodes will go through a security scan. Nodes that pass these checks will have a verification flag (✅) beside their name on the UI-manager. Feel free to read up more on the registry here
Action Required:

[x] Go to the registry. Login and create a publisher id (everything after the @ sign on your registry profile).
[x] Add the publisher id into the pyproject.toml file.
[ ] Merge the separate Github Actions PR, then merge this PR.

If you want to publish the node manually, install the cli by running pip install comfy-cli, then run comfy node publish
Otherwise, if you have any questions, please message me on discord at haohao_81202 or join our server!

@robinjhuang what kind of image would work best for an icon?


[stable/falco] Disable ebpf by default

This reverts the change made on 0.6.0

What this PR does / why we need it:
This PR disables ebpf by default. The amount of users that are using ebpf is smaller than the kernel module strategy, so is preferable to do not do this kind of breaking changes.
As long as CI is executing on GKE with COS, we added a custom setting for testing in CI with ebpf.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @bencer

/lgtm
/approve


Screen Resizing Issue When Closing Side Panel

Type
Incorrect behavior
Operating System
MacOS
OS Version
MacOS Ventura 13.1
Bug description
When collapsing the side panel on the right while in fullscreen mode, the screen doesn't resize properly and leaves black space on the left of the screen. Video attached.
Screenshots or videos
https://user-images.githubusercontent.com/104097714/214358202-60f2861a-cd79-4a08-b61d-1fe04ed766f5.mov
RuneLite version
Runelite Version 1.9.8
Launcher Version 2.5.0
Logs
No response

duplicate of #13303


Erroneous join in Update queries

In one of my projects BatchUpdate generates erroneous queries like this one:
UPDATE A
SET A.[ItemJustification] = @zzz_BatchUpdate_0
FROM [dbo].[Record] AS A
INNER JOIN ( SELECT
    [Extent1].[Id] AS [Id],
    [Extent2].[Id] AS [Id1]
    -- many irrelevant fields from Extent2 are omitted
    FROM  [dbo].[Task] AS [Extent1]
    INNER JOIN [dbo].[Record] AS [Extent2] ON [Extent1].[RecordId] = [Extent2].[Id]
    WHERE [Extent1].[Id] IN (3002)
           ) AS B ON A.[Id] = B.[Id]

At the last line it should be A.[Id] = B.[Id1]. As is, it results in zero rows affected unless Id values match, which is unlikely in real world scenarios.
Unfortunately, I was unable to reproduce this from scratch in a console app because the sub-query ends up not having Extent1.Id/Extent2.Id1 aliases:
UPDATE A
SET A.[FieldB] = @zzz_BatchUpdate_0
FROM [dbo].[EntityBs] AS A
INNER JOIN ( SELECT
    [Extent2].[Id] AS [Id],
    [Extent2].[FieldB] AS [FieldB]
    FROM  [dbo].[EntityAs] AS [Extent1]
    INNER JOIN [dbo].[EntityBs] AS [Extent2] ON [Extent1].[EntityBId] = [Extent2].[Id]
    WHERE [Extent1].[Id] IN (1)
           ) AS B ON A.[Id] = B.[Id]

Here's the minimal code to show you what's going on. As I said, the bug doesn't show up here.
public class EntityA
{
    public int Id { get; set; }
    public string FieldA { get; set; }
    public int EntityBId { get; set; }
    public virtual EntityB EntityB { get; set; }
}

public class EntityB
{
    public int Id { get; set; }
    public string FieldB { get; set; }
    public virtual ICollection<EntityA> EntitiesA { get; set; } = new HashSet<EntityA>();
}

public class EDbContext : DbContext
{
    public EDbContext() : base("DataContext") { }

    public DbSet<EntityA> EntityA { get; set; }
    public DbSet<EntityB> EntityB { get; set; }

    protected override void OnModelCreating(DbModelBuilder modelBuilder)
    {
        base.OnModelCreating(modelBuilder);
        modelBuilder.Entity<EntityA>().HasKey(e => e.Id);
        modelBuilder.Entity<EntityB>().HasKey(e => e.Id);
        modelBuilder.Entity<EntityB>().HasMany(e => e.EntitiesA).WithRequired(e => e.EntityB);
    }
}

class Program
{
    static void Main(string[] args)
    {
        using (var ctx1 = new EDbContext())
        {
            if (!ctx1.Set<EntityA>().Any()) // seed
            {
                ctx1.Set<EntityB>().Add(new EntityB
                {
                    FieldB = "Id 1 dummy"
                });
                ctx1.SaveChanges();
                ctx1.Set<EntityA>().Add(new EntityA
                {
                    // Id 1
                    FieldA = "AAA",
                    EntityB = new EntityB
                    {
                        // Id 2
                        FieldB = "___"
                    }
                });
                ctx1.SaveChanges();
            }
        }

        using (var ctx2 = new EDbContext())
        {
            var id = new[] { ctx2.Set<EntityA>().FirstOrDefault().Id };
            var iq = ctx2.Set<EntityA>().Where(a => id.Contains(a.Id));
            iq.Select(a => a.EntityB).Update(b => new EntityB { FieldB = "BBB" }, bu => bu.Executing = Exec);
        }

        using (var ctx3 = new EDbContext())
        {
            Debug.Assert(ctx3.Set<EntityA>().FirstOrDefault().EntityB.FieldB == "BBB");
        }
    }

    private static void Exec(DbCommand cmd)
    {
        Console.WriteLine(cmd.CommandText);
    }

Hello @yegorpetrov ,
I tried to replicate the bug unsuccessfully for a few days. The [Extent1].[Id] AS [Id] never show up!
Can you post or send us<EMAIL_ADDRESS>the full LINQ Query you used without omitting anything? I'm sure that will help us to find something we forget to do to replicate the issue.
Best Regards,
Jonathan

Hello @yegorpetrov ,
We still have not received anything from your part.
Best Regards,
Jonathan

Yeah, sorry about the delay. The problem is highlighted on this screenshot:

This thing is proprietary, so I'm not sure I'm free to share it in the open. But let me know if the screenshot is of any help, and may be we'll figure out what I can send to you privately.

Hello @yegorpetrov ,
What I would like is the screenshot of the LINQ query if you can. That would maybe give me a hint to how to build the LINQ query/entities to reproduce it and generate this SQL.
Like I said, you can send this information to us in private<EMAIL_ADDRESS>Best Regards,
Jonathan

Well, it's already shown on that screenshot. _taskRepo.Table is essentially a DbSet<Task> from the context, and you can see the rest. Not sure what other LINQ I should send to you.

Oh sorry, you are right. I checked it too fast.


The problem that the model cannot converge

Hi, I have some training questions to ask.
When I train the AO model at 2 A100 80G, Batch_size=8, and train the lrs3 and lrs2 data sets simultaneously, errors will be reported out of memory and the training time will reach an epoch of 5 hours.
When only the ls2 data set is used, Batch_size=16, num_workers=4, an error will also be reported out of memory. In this case, the epoch is 20 minutes.
When only ls2 data set is used, Batch_size=8, num_workers=4, the epoch is 40 minutes, but loss=nan occurs when multiple epochs are used.
All the above problems have caused me to be unable to train normally. Could you please tell me the time and details of your training of this model in detail? Thank you very much！！

Hi, I have some training questions to ask.
When I train the AO model at 2 A100 80G, Batch_size=8, and train the lrs3 and lrs2 data sets simultaneously, errors will be reported out of memory and the training time will reach an epoch of 5 hours.
When only the ls2 data set is used, Batch_size=16, num_workers=4, an error will also be reported out of memory. In this case, the epoch is 20 minutes.
When only ls2 data set is used, Batch_size=8, num_workers=4, the epoch is 40 minutes, but loss=nan occurs when multiple epochs are used.
All the above problems have caused me to be unable to train normally. Could you please tell me the time and details of your training of this model in detail? Thank you very much！！

Have you looked at link https://github.com/burchim/AVEC/issues/7 or https://github.com/burchim/AVEC/issues/14 ?
If you refer to the link, I think it will be a good hint for solving the Loss=nan problem.
And out of curiosity, how did you solve problem https://github.com/burchim/AVEC/issues/16?
I can't download or create data. Can you share a solution?


MODE-2593 Adds datasource support to the Database Binary Store

This uses the same datasource implementation (Hikari) that is being used by the relational persistent storage. This artifact has also been promoted as a top-level artifact in the WF kit.

Looks good.


device requests must state the default for  ip_addresses.public

define truthy default explicitly for ip_addresses public field in device create request

Without this definition, `public: false` is never sent to the API.

@displague looks good, examples run, but can you please explain where's the type ClientOption is coming from?

ClientOption came from updating the go-swagger image to latest.
I have another branch or commit that I will include in a PR that forces a docker pull so we always build with latest.

GitHub is hiding the generated files in the results, the files it shows by default are the ones that I modified by hand.
I  split my commits up so that the generated changes are in their own commit.

@displague thanks.


[ci,check-generated] Handle empty text in CMDGEN

Change ci/scripts/check-generated.sh to run both top and cmdgen when checking for the topgen generated artifacts. This is to support template md files having no text between the BEGIN and END markers so they can handle multi-top.
Add a separate "cmdgen" target in the hw/Makefile and describe that in the README.md for contributing to documentation.
Fixes #24991

There is a CI failure from shellcheck because it wants $* to be within quotes in check-generated.sh. However, if it is quoted the "make" command will get a string with "top cmdgen", and that target is not found. I want to run make so it executes both targets and this is the simplest way I can think of. Perhaps someone can think of a simple alternative?
I know @andreaskurth suggested running cmdgen in the tops_gen target, but I find that unappealing.


Add azure-sdk

Add azure-sdk to the list of repo's to update.
@chidozieononiwu I assume this will work on any repo correct?
Based on https://github.com/Azure/autorest.csharp/pull/880 we might also want to add "autorest.csharp" to the list.

Yes this will work on any repo. But it assumes that the repository resources were already added with a Ref.

We should consider folding in the ref checker with this pipeline that updates the references.


Recent search easy peasy migration

The type of this PR is: Enhancement

Description
Migrate Recent Searches from using the Context Api to Easy-Peasy
Co-authored by @ds300 @MounirDhahri

Fails

:no_entry_sign:
Missing Test Files:

src/lib/utils/__tests__/storage-tests.ts

If these files are supposed to not exist, please update your PR body to include "#skip_new_tests".

  Generated by :no_entry_sign: dangerJS

This is great stuff, thanks to everyone involved!
I have a couple of reservations about merging this as-is

We should try to find a way to migrate people's existing recent-searches (from before this refactor is released) otherwise next time their app updates they'd lose all their search history.

We will soon need to augment the nascent global app store (in AppStore.tsx) with persistence, versioning, and migration infrastructure, and this search state could be moved there already and used to spearhead that work sooner rather than later.

I'd vote for doing 2 since

this is just a refactor and is not adding any functionality
we're not working on search right now so this work won't unlock other items in the backlog or block anyone
we need to do 2 at some point anyway.

💡  I'm going to try to get this done today and present in knowledge share tomorrow.

Thanks, I didn't know how to proceed with it

I'm going to close this and open a new PR on the same branch to give a clean slate for conversation, now that things have changed so much.


Script to copy scenes over to godot in engine

Since godot cant load resources from outside of its own folder right now, we need a way to send scenes from content to the godot folder. I've been placing scenes inside resources/scenes in the content folder which people have to manually copy into godots scene folders.
What we need is a script that runs on build which copies over the scenes into the godot folder so it works with minimal coder interference

I wanna solve this by not requiring a script at all, to be honest. It should be automatically handled.

If you're on a proper operating system you can just use symlinks.
:)


Can't bind to 'options' since it isn't a known property of 'ss-multiselect-dropdown'.

I installed the package successfully and followed the instruction in readme.md. But I am getting following error in the console if i run the application.
Can't bind to 'options' since it isn't a known property of 'ss-multiselect-dropdown'.

If 'ss-multiselect-dropdown' is an Angular component and it has 'options' input, then verify that it is part of this module.
If 'ss-multiselect-dropdown' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message.
("
Roles

<ss-multiselect-dropdown [ERROR ->][options]="roleOptions" formControlName="roles">

<"): ContactComponent@96:37
'ss-multiselect-dropdown' is not a known element:
If 'ss-multiselect-dropdown' is an Angular component, then verify that it is part of this module.
If 'ss-multiselect-dropdown' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message. ("
Roles

[ERROR ->]<ss-multiselect-dropdown [options]="roleOptions" formControlName="roles">

'ss-multiselect-dropdown' is not a known element:

Could it be that the MultiselectDropdownModule module isn't imported into your feature ngModule config?

no..I have imported that in my ngModule
],
imports: [
BrowserModule,
CommonModule,
HttpModule,
FormsModule,
ReactiveFormsModule,
routing,
BootstrapModalModule,
FormModule,
ReviewModule,
MultiselectDropdownModule,

If you are using the component in a child feature module you need to import it into that feature ngModule. Are you using feature modules or is this the only ngmodule config?

I added only on ngmodule config.. After adding to feature modules, it worked. Thanks!!

I was confused about this as well. The "root module" imports are not global and need to be imported into all feature modules where they are used. You could aswell remove the import of DropdownModule in the root module.

oh..got it. it will be great if you mention this in your readme file. It might help others and save time.

If you have lots of modules you want to use in more than one feature module, the recommended practice is to import and export it from a shared module, then just import your shared module in all your feature modules.


Take a look at NVDB API - GET Vegref and/or more data

On ice as of this moment. We do not have valid Vegrefs. Code functioning. May be used later

Should now contain the necessary functions to access /veg and /vegnett. On branch nvdbapi2


Uk Bank Holidays don't include August Summer bank holiday

I've checked 2024 and 2025 and looks like England Bank Holidays don't include August Summer Bank Holiday (last Monday of August).
EDIT: Sorry - they do, but State / Country selection is weird. I think you have to select Country first then come back to State and type England. Otherwise it doesn't recognize full recognize it (shows all other Bank Holidays except the one in August)

Hi thanks for your comment, indeed the country auto selects to your IP as "United Kingdom" but the state is not auto selected.
I imagine the public holidays from the date-holidays package are showing all "common" holidays in the UK when no state is selected, and the state-specific ones once you select it.


Bug with quick tutorial code in soundspaces 2.0

Hi
It seems that there's an error stating that habitat_sim.sensor.RLRAudioPropagationChannelLayoutType is a Nonetype object in line 36 of the file soundspaces2_quick_tutorial.ipynb, where
audio_sensor_spec.channelLayout.type = habitat_sim.sensor.RLRAudioPropagationChannelLayoutType.Binaural   is specified.
Indeed, according to the official documentation of habitat-sim (https://aihabitat.org/docs/habitat-sim/habitat_sim.sensor.html), habitat_sim.sensor.RLRAudioPropagationChannelLayoutType is set to None. Could this be a mistake in the tutorial code?

@hiro-oda Hi, I have encountered the same problem. Have you been able to find out how to update it?

@WYXG233 Hi. I haven't fixed this problem yet, but I think this bug is caused by a version difference in habitat.sim. So, it might be a good idea to downgrade the version.

@hiro-oda @WYXG233 the code runs fine for me. Did you follow the installation document to install Habitat-Sim properly? Also please refer to the following document for SS 2.0 APIs.
https://github.com/facebookresearch/sound-spaces/blob/main/SoundSpaces2.md

@ChanganVR, thanks for your response. Could you provide the version of Habitat-Sim you are currently using? I used the habitat-sim-0.3.1, which will output:
audio_sensor_spec.channelLayout.type = habitat_sim.sensor.RLRAudioPropagationChannelLayoutType.Binaural

AttributeError: 'NoneType' object has no attribute 'Binaural'

@ WYXG233  Do you solve the problem? I have an same issue about that version (habitat-sim-0.3.1)

@aleodrnfl I still haven't solved it yet, but you can try to follow what @hiro-oda said about downgrading the Habitat-Sim version. Also, be careful about your SS 2.0 API dependencies version. :)

@WYXG233 Thank you for your response. I already solve that problem. I just downgrade cmake version to 3.14.0.
And re-installed habitat-sim.


Animations bake is impossible

The naruto run animation has been corrupted for a long time, and this was never fixed, but now the script makes it impossible to bake any animations at all due to the corruption.
bake error Error: This track has invalid values.  All scale transforms must be set to 1. Aborting.
 Animation: naruto run.fbx, Track: mixamorigHead.scale, values:
 0.9999999403953552,0.9999999403953552,1,0.9999999403953552,0.9999999403953552,1
    at /home/a/app/animations-baker.js:353:19
    at Array.map (<anonymous>)
    at baker (/home/a/app/animations-baker.js:315:53)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async /home/a/app/animations-baker.js:571:5

Looks like this was broken in this PR: https://github.com/webaverse/app/pull/2639
Seems like this check was added and nobody bothered to even run the code.
@gonnavis You've run into this too I imagine?

Yes, I think this is something in progress, waiting changing of animation files.
So just reverted the codes locally, not make issue.


Reflect: Implement Reflect.ownKeys()

Overview
This PR does the following:

Proposes an implementation for Reflect.ownKeys()

WARNING: DO NOT MERGE. DOES NOT COMPILE. See following point.

For this implementation to compile, Rhino's ScriptableObject.java needs a new method (e.g. getAllIdsIncludingSymbols()) that allows retrieval of all properties (enumerable or not not) including both strings and symbols:
public Object[] getAllIdsIncludingSymbols() {
    return getIds(true, true);
}

Test case
(This test case is also implemented in ReflectTest.java.)
var obj = {
  [Symbol.for('foo')]: 0,
  "str": 0,
  773: 0,
  "55": 0,
  0: 0,
  "-1": 0,
  8: 0,
  "6": 8,
  [Symbol.for('bar')]: 0,
  "str2": 0,
};

// Chrome:  [ "0", "6", "8", "55", "773", "str", "-1", "str2", Symbol("foo"), Symbol("bar") ]
// HtmlUnit: [ "-1", "0", "6", "8", "55", "773", "str", "str2", "Symbol(foo)", "Symbol(bar)" ]
console.log(Reflect.ownKeys(obj));

Points of note
About the javadoc
The javadoc for ownKeys() is very basic because:

We aren't sure if licensing allows us to copy mdn docs' words or whether we need to reword to our own words
Perhaps writing fully descriptive API docs in the javadoc of these methods is not all that useful since they're JS methods, and a basic javadoc is easier to produce

Regarding the ordering of property keys returned
Notice -1 is in the wrong place.:

Chrome:  [ "0", "6", "8", "55", "773", "str", "-1", "str2", Symbol("foo"), Symbol("bar") ]
HtmlUnit: [ "-1", "0", "6", "8", "55", "773", "str", "str2", "Symbol(foo)", "Symbol(bar)" ]

The specs for Reflect.ownKeys() provide clear description for the ordering of these:

Non-negative integer indexes in increasing numeric order (but as strings)
Other string keys in the order of property creation
Symbol keys in the order of property creation.

We cannot accommodate this for -1 and other negative integers because Rhino does not preserve the property creation order of negative integer keys.  This appears to be a bug in Rhino:

Rhino DOES preserve the property creation order of non-integer keys adhering to specs
Rhino DOES orders "positive integer" keys in canonical order adhering to specs
Rhino DOES NOT preserve the property creation order of "negative integer" since it seems to be incorrectly treating these as an integer index even though they're not in the defined range of 0 <= i <= F [1]

[1] https://262.ecma-international.org/13.0/#sec-object-type
Regarding the initialization of the test case

var obj = {
  [Symbol.for('foo')]: 0,
  "str": 0,
  773: 0,
  ...
};

HtmlUnit throws invalid property id error on the [Symbol.for('foo')] part which is a computed property name. The object is instead initialized in ReflectTest.java as such:
var obj = {};
obj[Symbol.for('foo')] = 0;
obj['str'] = 0;
obj[773] = 0;
obj["55"] = 0;
obj[0] = 0;
obj['-1'] = 0;
obj[8] = 0;
obj["6"] = 0;
obj[Symbol.for('bar')] = 0;
obj['str2'] = 0;

Hi @duonglaiquang,
i guess this requires some time. Maybe i can move the whole Reflect impl to Rhino....
Lets see.

Looks like i can do a (more or less complete) impl in Rhino... hopefully i can finish it over the weekend.

fyi, we're looking into an issue where Reflect.ownKeys.toString() throws TypeError: Cannot find default value for object and suspect there might need to be something added to FunctionObject. Perhaps the situation will change when the implementation is moved to Rhino.

Will add at least your testcase to the HtmlUnit test suite after the Impl is in Rhino/core-js. Hope then we are on the save side

Step 1: https://github.com/mozilla/rhino/pull/1324

Reflect impl is now in core-js (will be moved later to Rhino)

construct still missing
negative index values handling still open

Next step: provide PR for Rhino for negative index handling (have already some working code)

https://github.com/HtmlUnit/htmlunit-rhino-fork/commit/e6c4c79bf0a20a7bae6cbe121e9149691a7ac927
Because i fear there is no chance for this in Rhino (is a  bit of a hack and not backward compatible) i did it only here


Set up to use cargo-semver-checks on PRs and releases

The check should read from the version in taskchampion/Cargo.toml so if that represents a major version bump then changes are allowed, as is the case here -- we have added and removed methods in the un-sealed trait StorageTxn. We should seal that, too.

The docs for these types say

An implementation is provided, but users of this crate can provide their own implementation as well.

Sealing would prevent that, so they are correct as they are (not sealed).


Support colors for console logging (info, warn, error).

thanks! :)

It will upgraded, i only update the > color ...
a pleasure :+1:


Maven install with status "coming soon" 1 year after

When is planned the maven install please?

Hi @Erwan-DATIN . it's already in Maven Central
Duplicate #2


Deconstructor support in jitclass

It would be useful if jitclass supported the deconstructor __del__ method. Supporting __del__ would permit more seemless numba interfaces to C/C++/Fortran codes which allocate heap memory or use classes. Without __del__ memory leaks are extremely hard to avoid.
Example:
import numba as nb

@nb.experimental.jitclass()
class Test():
    def __init__(self):
        pass
    def __del__(self):
        print('__del__ is being called')

@nb.njit()
def tester():
    T = Test()

tester()

This little script should print __del__ is being called.

Thanks for opening this issue @Nicholaswogan. I think this could be implemented but is potentially a little challenging, Numba doesn't have a garbage collection, it uses reference counting instead, so the __del__ method would have to be invoked as part of the destruction sequence for the instance when the reference count reaches zero.
Thoughts on how to make it work: IIRC the jitclass has an associated meminfo, could the __del__() be wired through as part of the dtor to the meminfo at the point the ctor (__new__) is called ahead of __init__()?

I wouldn't really know where to begin on this. I don't know very much about how numba works. Is there someone else that is willing to try to support this feature?

Bump as I am also interested in this feature.


Export API

Currently the export functionality only dumps what the browser has currently loaded into a file. The server should have an endpoint to stream the results.

Fixed in 2.4.1.


[DO NOT MERGE] Configure registry-redirector to use kind-registry

I just want to check if istiod is able to resolve kind-registry in IPv6 cluster as it does not work in my local environment.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test integ-ipv6_istio


Clicking the calendar icon no longer closes the dropdown

We might need to put the "pointer-events: none" workaround (that was removed in 2bfa222694ac8e508400774e5c1e76c4369c3713) back

Broken still with shadow DOM. Either reopen or create a new issue. Your choice :)
You can see the issue in Expense Manager demo and Chrome.

Works fine at http://demo.vaadin.com/expense-manager-ng/?dom=shadow
The expense manager at https://demo.vaadin.com/expense-manager/ is using an old version

Thanks for looking! Can you test the original expense manager locally? We’re using that.


Sort by "Identifier" and "Modified"

Closes #567.
There will need to be some small changes made to the api server as well (so this PR isn't actually functional yet locally or in the deploy preview), but first I wanted to figure out exactly what sorting options we want to add/remove. I went with @yarikoptic's suggestion in #567 to remove the Created sorting option and instead add in Identifier and Modified options. Either way it seems redundant to have both Identifier and Created options.
This PR also removes all the DJANGO_API feature toggle code in DandisetsPage.vue.

I opened a PR for the server changes that are needed (https://github.com/dandi/dandi-api/pull/327).
There is still an open question of what sorting by "modified" means exactly for dandisets. There are two modified fields at play, namely those on the VersionMetadata and AssetMetadata.
Since each Version can have multiple assets, including AssetMetadata's modified field in the sorting would require searching the metadata of every asset associated with each dandiset, which could be a performance concern if a dandiset has a large number of assets. As far as I can tell the same problem does not apply for VersionMetadata, since each Version can only have one associated VersionMetadata.

i think this should not be a computed thing, but something that is updated in Version when either VersionMetadata or an AssetMetadata in that version is modified. this would make it not search all assets, but rather have assets trigger the update whenever they are changed.

FWIW: I think a DB operation to take max or min across dates should be negligibly fast, ie. under a ms, (unlike e.g. getting AssetStats updated) even for dandisets with thousands of assets and then would guaranteed to be "up to date".

i think this should not be a computed thing, but something that is updated in Version when either VersionMetadata or an AssetMetadata in that version is modified. this would make it not search all assets, but rather have assets trigger the update whenever they are changed.

It turns out that what you're suggesting is already implemented (thanks @dchiquito for pointing that out), so this PR should be all set.


upgrade elasticsearch packages

:memo: Description

In this PR I have resolved the below issues

Remove the deprecated elasticsearch package (https://www.npmjs.com/package/elasticsearch)
Used the supported elasticsearch package "@elastic/elasticsearch" (https://www.npmjs.com/package/@elastic/elasticsearch)
Support the latest elastic search version 8.1
Fix the type issue in latest version elasticsearch has removed the document type. For eg. type="_doc" (https://www.elastic.co/guide/en/elasticsearch/reference/current/removal-of-types.html)
Removed the apiversion parameter. It will take default latest version

:gem: Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Thanks

Welcome


Running multiple commands in a parallel

Description
I have two commands which need to run in parallel and target the same GLOB.
I cannot define commands under their own GLOBs, as they would clash:
"*/**/*.{js,jsx,ts,tsx}": "commandA",
"*/**/*.{js,jsx,ts,tsx}": "commandB",

I also cannot define them as an array on the same GLOB, as they would run in serial:
"*/**/*.{js,jsx,ts,tsx}": ["commandA", "commandB"]

Ideally there would be a way to allow commandA and commandB to run in parallel, even though they target the same GLOB.
Does such a feature exist?
Environment

lint-staged: 10.5.1

Hello, I don't think it's possible at the moment.
You would have to create a wrapper script that runs them, while passing arguments to both (the staged filenames matched to glob).

I think it should be possible with the JS configuration since it offers that level of flexibility.

I'm interested by this feature.
As a hack I'm wondering about using different blob
"*/**/*.{js,jsx,ts,tsx,commandA}": "commandA",
"*/**/*.{js,jsx,ts,tsx,commandB}": "commandB",

This is what I'm currently doing @VincentLanglet 👍

I have a case not work
''src/**/.js  *.js'':[command],

how to make this work

I have a case not work
''src/**/.js  *.js'':[command],

how to make this work

Here's one idea for possible config spec that allows specifying that some tasks run serially, and some run in parallel:
    "*.@(ts|tsx|js|jsx|mjs|cjs)": [
      { ESLINT_AUTOFIX: "yarn lint:fix:pre-commit" },
      {
        SNAPSHOTS: "test:update-snapshots",
        TYPE_CHECK: "type-check"
       }
    ],

Currently I am using run-p:
  "lint-staged": {
    "*.@(ts|tsx|js|jsx|mjs|cjs)": [
      "yarn lint:fix:pre-commit",
      "run-p --print-label --continue-on-error test:update-snapshots type-check --"
    ],
    "!*.@(ts|tsx|js|jsx|mjs|cjs)": [
      "prettier --write --ignore-unknown"
    ]
  },

+1 would also love a way to run tasks in parallel for the same glob. I'm also using the "fake extension" workaround but it would be awesome if this was supported out of the box

Flipping this to true would work:
https://github.com/okonet/lint-staged/blob/339515010ccd95a2f952dbe65f8366463f94d26a/lib/runAll.js#L193
Maybe a new cli flag, like --concurrent-sub-tasks (like --concurrent)?

@iiroj Looks like it works well, are there any downsides to this approach?
If not I'm happy to open a PR that accepts the flag.

@gkielyellation the main problem would target default behaviour IMO, where two linters do conflicting changes and result in non-deterministic output depending on which is faster. As long as it's a flag it should be fine.

Simply running everything in parallel or serially I think is not the best approach
The new config spec/api I mentioned above would allow you to mix and match. (I think the example I provided is also an extremely common sample config most people will use)

The config spec also unlocks the ability to interact with each task in targeted ways, because each task is labeled.
If you are updating typescript stuff, and simply want to skip the typescript type check, you could say --lint-staged-skip type check
I think this is really important, because it helps users avoid using --no-verify
I think --no-verify is really bad, once developers start using it, some may overuse or abuse it.

@devinrhode2 I agree about --no-verify but don't really understand your proposal: are you suggesting to change the config format somehow? Or would you keep it to advanced JS-based config? Could you describe your idea more clearly perhaps?

A workaround might be to use fake extensions to allow for multiple globs:
    "*.{ts,tsx, __parallel-1__ }": "eslint",
    "*.{ts,tsx, __parallel-2__ }": "prettier --list-different",

I was thinking about maybe something like this could be worth exploring:
{
  "glob": [
    "sequential-task",
    [
      "parallel-task", "parallel-task"
    ]
  ]
}

WYT?

@okonet I am not sure if a git hook could read a cli flag passed to git:
git commit -m "asdf" --allow-empty --lint-staged skip:type-check

If it can't, we could use an environment variable:
LINT_STAGED="skip:type-check" git commit ...

100% happy with your proposal @okonet :)
Breaking changes we want to think very carefully about :)

I'm not sure of the best way to implement this, but, I am currently running some tasks in parallel with run-p:
  "lint-staged": {
    "*.@(ts|tsx|js|jsx)": [
      "yarn lint:autofix",
      "run-p --print-label --continue-on-error test:update-snapshots type-check --"
    ],

The array syntax COULD be implemented as syntactic sugar on top of run-p. It certainly seems like an easy way to implement this. However, maybe we should just like users directly use run-p the way I am today

AFAIK listr which we're using already supports parallel execution of tasks so it should be rather straightforward to implement. The most important thing is to agree on a standard for config that works for most uses cases without sacrificing simplicity. Also the tricky part could be skipping but it's also supported by listr.

I'm currently writing the glob in a different order as a workaround:
{
  "*.{ts,tsx}": ["eslint --quiet", "prettier"],
  "*.{tsx,ts}": ["jest --bail --passWithNoTests --findRelatedTests"]
}

As to a syntax supporting both parallel and sequential running we can use objects for parallel definition and keep array for sequential.
{
  "*.{ts,tsx}": {
    "lint": ["eslint --quiet", "prettier"],
    "test": ["jest --bail --passWithNoTests --findRelatedTests"]
  }
}

Now your Glob can be either string[] or Record<string, string[]>.

Frankly I still prefer https://github.com/okonet/lint-staged/issues/934#issuecomment-1097867162. Feel free to submit a PR for it.

I think my run-p example solves this really well. Why write code when you can just use run-p?

Related to task labeling:
Theo (t3dotgg) has some strong rants against git hooks in general, because they block making commits. Then, developers generally seem to just use git commit --no-verify. Now, lint-staged and husky have lost all control over the situation.
Allowing developers to skip specific tasks would be great.
I think my object syntax, while it isn't super beautiful or sexy, works well.
Another potential approach:
If all lint-staged tasks are npm scripts, and the npm script names don't have spaces, and are generally of the form: eslint_autofix, then that could be used similarly to the object syntax.
I think if tasks are skipped, a note should be added to the end of the commit message in a prepare-commit-message git hook.

I guess the way I think about it is: give a minimal solution for running tasks in parallel without any breaking changes.
The way you think is a bit broader. But I don’t really see how skipping some tasks would work in practice. Can you elaborate on it perhaps?

I don't think lint-staged has to try to solve the issue of developers skipping commit hooks. That sounds more like an organizational issue. Besides, you can also lint in the CI and treat lint-staged as only a part of the chain of Continuous Integration (from developers' machine to prod).

I tried the fake extension workaround to run 3 tasks in parallel for a pattern that matches a  single file.
Sometimes it works, somtimes... it delete all my staged work! :fearful: :sob:
And I get the following error:
/home/***/node_modules/pidtree/lib/pidtree.js:61
      callback(new Error('No matching pid found'));
               ^

Error: No matching pid found
    at /home/***/node_modules/pidtree/lib/pidtree.js:61:16
    at /home/***/node_modules/pidtree/lib/ps.js:40:7
    at ChildProcess.<anonymous> (/home/***/node_modules/pidtree/lib/bin.js:45:5)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:1091:16)
    at Socket.<anonymous> (node:internal/child_process:449:11)
    at Socket.emit (node:events:513:28)
    at Pipe.<anonymous> (node:net:322:12)

@cvolant if you run git stash list you will hopefully find a backup stash!

:smiley: Thank you.

"glob": [
    "sequential-task",
    [
      "parallel-task", "parallel-task"
    ]
  ]

I get an error while trying to use nested array as highlighted in the screenshot below:

We don't support nested arrays like that.


Correct img name likecoin

Correct img name likecoin

The links in the assetlist json are correct, the mistake was only on the name of the images


Error loading glTFTranslator plugin, Maya 2022 macOS

I'm very new to Maya - have just installed a trial of the current version from the website, in MacOS Big Sur. Following the instructions in the readme, I get an error No module named 'glTFExport'
PluginManager;
// Error: file: /Applications/Autodesk/maya2022/Maya.app/Contents/scripts/others/pluginWin.mel line 316: ModuleNotFoundError: file /Applications/Autodesk/maya2022/Maya.app/Contents/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/shiboken2/files.dir/shibokensupport/__feature__.py line 142: No module named 'glTFExport' //
// Warning: file: /Applications/Autodesk/maya2022/Maya.app/Contents/scripts/others/pluginWin.mel line 316: Failed to run file: /Users/Shared/Autodesk/maya/2022/plug-ins/glTFTranslator.py //
// Error: file: /Applications/Autodesk/maya2022/Maya.app/Contents/scripts/others/pluginWin.mel line 316:  (glTFTranslator) //
// Error: file: /Applications/Autodesk/maya2022/Maya.app/Contents/scripts/others/pluginWin.mel line 316: ModuleNotFoundError: file /Applications/Autodesk/maya2022/Maya.app/Contents/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/shiboken2/files.dir/shibokensupport/__feature__.py line 142: No module named 'glTFExport' //
// Warning: file: /Applications/Autodesk/maya2022/Maya.app/Contents/scripts/others/pluginWin.mel line 316: Failed to run file: /Users/Shared/Autodesk/maya/2022/plug-ins/glTFTranslator.py //
// Error: file: /Applications/Autodesk/maya2022/Maya.app/Contents/scripts/others/pluginWin.mel line 316:  (glTFTranslator) //
ProductInformation;
// Error: file: /Applications/Autodesk/maya2022/Maya.app/Contents/scripts/others/pluginWin.mel line 316: ModuleNotFoundError: file /Applications/Autodesk/maya2022/Maya.app/Contents/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/shiboken2/files.dir/shibokensupport/__feature__.py line 142: No module named 'glTFExport' //
// Warning: file: /Applications/Autodesk/maya2022/Maya.app/Contents/scripts/others/pluginWin.mel line 316: Failed to run file: /Users/Shared/Autodesk/maya/2022/plug-ins/glTFTranslator.py //
// Error: file: /Applications/Autodesk/maya2022/Maya.app/Contents/scripts/others/pluginWin.mel line 316:  (glTFTranslator) //

I thought it might help to restart Maya, but it didn't.

I had the same error. I'm not sure if this would apply to you, but all I needed to resolve the issue was to move the glTFExport.py file from the script folder into the plug-in folder so that both py files are in the same folder.

Hello @jjyang4213 thanks, now load after I put them both in the same folder, but when I tried to export the file, there's no file.  Did it work for you? (Maya 2024) :/


Proposed paper edits.

Hi @RayLei-TRI, I have one small round of edits attached, then my review will be complete. Please treat these changes as optional. I think they improve the wording in small ways, but it's not significant enough to hold the paper if you disagree. This PR closes #3.

@bdice Thank you very much for the edits!


fix: update types to include tokenMap

This should've been done in https://github.com/Uniswap/token-lists/pull/420 , but overlooked this and updating the schemas in this PR.

This should've been done in https://github.com/Uniswap/token-lists/pull/420 , but overlooked this and updating the schemas in this PR.


iptsd: 1.0.1 -> 1.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/linux-surface/iptsd/releases.
meta.description for iptsd is: Userspace daemon for Intel Precise Touch & Stylus
meta.homepage for iptsd is: https://github.com/linux-surface/iptsd
meta.changelog for iptsd is: https://github.com/linux-surface/iptsd/releases/tag/v1.1.0
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.1.0 with grep in /nix/store/wk4xbwnyfj8nrp3pkrdlfl1skf664ckv-iptsd-1.1.0
found 1.1.0 in filename of file in /nix/store/wk4xbwnyfj8nrp3pkrdlfl1skf664ckv-iptsd-1.1.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
iptsd

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

iptsd

Maintainer pings
cc @tomberek @dotlambda for testing.

Result of nixpkgs-review pr 217040 run on x86_64-linux 1

1 package built:

iptsd


Potree doesn´t recognise some classification codes

Hi,
I have a Las file v1.2 which has been classified according to 2 -ground, 7 - noise, 13 -wire and 20 reserved.
When I use covert with the following command:
PotreeConverter.exe ....\lidar_testdaten*.las -o ....\lidar_testdaten\www -p index.html --material ELEVATION --output-attributes RGB CLASSIFICATION
..not all of the classication filters are translated into the potree viewer.Only ground is working. I assume I must manually define the classification filters for this to work? Is there any documentation showing how to do this or is there something  have to do before running the potreeConverter?

https://github.com/potree/potree/issues/538

Can you tell me WHERE you can find all the commandline arguments for PotreeConverter?  I can't find any of the arguments and possible parameters.  thank you so much!


Multiple browser support

I've found that the library is not working properly on Edge and Firefox. The nodes are cropped (but functional), and in Edge the dragging is odd.
Is there any chance of making it work fully in Edge and Firefox?
Thanks!

@jcaffera42, I solve the problem with Firefox now.  I'll try to solve the problem with Edge later

Nodes moved from svg to html, now there should be no problems.
Please check in the Edge how it will behave

Thanks! That was really fast! Working perfectly on Edge, however, in firefox I can't add nodes using the context menu anymore.

@jcaffera42, it was my mistake before the release. This is fixed in the following commits.
You can collect for yourself from the sources or get files by links
https://d3-node-editor.surge.sh/node-editor.js
https://d3-node-editor.surge.sh/node-editor.css
https://d3-node-editor.surge.sh/menu.html
https://d3-node-editor.surge.sh/view.html

It still has issues on Edge. The connections look misplaced, and you can drag the nodes out of the container. (On Chrome and Firefox it's working properly)

Can you place your example with all styles?
Are there any problems in the demo?
On my Edge 38 everything looks good

You're right, the demo works perfectly. I'll check my code again to find the issue, thanks!


Check for remote repo updates and prompt the user to pull them

I was wonder, whether it would be difficult to implement occasional checks for remote changes and prompting the user to pull them right inside the plugin. I frequently switch machines. To sync my changes, I use a private GitHub repository. The problem occurs when I forget that I have pushed some notes already. I only discover that when I do another push and it fails (often with a merge conflict).
It would be much smoother, if the plugin checked that for me, and occasionally reminded me to perform a pull on the new machine, before adding new notes.
Is that feasible?

That would be really helpful! Is that's what autoCheckSynced option is supposed to enable? If so, it doesn't work for me: when a new commit is added to Github, Logseq doesn't show any update.

this is addressed in https://github.com/haydenull/logseq-plugin-git/pull/39

plugin toolbar icon now contains 2 collored
configuration options are not decided yet, for now the status is updated on page navigation and on window visibility change


Rename CRD from ServiceGroup to Habitat

This removes the amibguity between the CRD name and the habitat concept
of a service group.
The short name becomes hab.
I'm not 100% about this rename. For example, the full CRD name is now habitats.habitat.sh.
Closes #85.

Just remembered you also need to rename this https://github.com/kinvolk/habitat-operator/blob/455412463085f1010abc6e2c2568d11339f3e404/Makefile#L28

PR updated, tests pass and everything gets created correctly now. Ready for review.


logging: add simplelog sample

cc @jmdobry @jonparrott @pfritzsche

Tests coming now. Take a look and see if this is reasonable first, though?

LGTM with a nits.

Added tests. Have a look?
They'll be quite flaky, unfortunately. Let's see how bad it is in practice. Might be OK for now (like pubsub).

LGTM with a few nits.


Error when install module no compatible with RNGoogleSignin

i try integrate this module, when install it,  modified this file project.pbxproj,  and produced error i dont open this app by  xcode, not parse this project
this installation write this
HEADER_SEARCH_PATHS = "$(SRCROOT)/../node_modules/react-native-google-signin/ios/RNGoogleSignin/""$(SRCROOT)/../node_modules/react-native-sensitive-info/ios/RNSensitiveInfo/";
in this case produced error but form correct could so :
HEADER_SEARCH_PATHS = ("$(SRCROOT)/../node_modules/react-native-google-signin/ios/RNGoogleSignin/",
"$(SRCROOT)/../node_modules/react-native-sensitive-info/ios/RNSensitiveInfo/");

sounds like a link issue rather than this repo

@mealbarracin10 it is a strange error 🤔
We can try to reproduce that generating a new project and installing both libraries


[Tour of Beam] [Task] Backend API functions for getting SDK list and table of contents

What needs to happen?
Create backend API functions to get SDK list and table of contents
Issue Priority
Priority: 3

.take-issue


What are all the needed dependencies?

i did just what the readme said
clone and then run pip
i needed to install already

libpng++-dev
libfreetype6-dev
liblapack-dev

now the installation process is complaining about a not installed Fortran
don't know how to compile Fortran code on platform 'posix'

building 'dfftpack' library

error: library dfftpack has Fortran sources but no Fortran compiler found

you maybe want to inform about all the dependencies needed to actually run you pip installation process

Those dependencies come from the dependencies. Installing scipy using pip is not fun – especially as the error messages really are not helpfull. So, it's not “my fault” and i think they should write their pip packages better. Maybe i will someday add information about it to the README.
What you can do is search for the packages on some other package-manager (e.g. apt-get) to install the python packages from there or just install the dependencies (e.g. apt-get build-dep scipy).

f4c7ed64b208f6cd3d0372fa77de9146f9812b97

6b1abc5d585c3971f438069c5943db3c5377c754


Error in function validate_quantity

In version 1.0.2, an error in  function validate_quantity  in shop/serializers/defaults/catalog.py.
Manifested when working with products with several options.
Need to remove the field availability
def validate_quantity(self, quantity):
        """
        Restrict the quantity allowed putting into the cart to the available quantity in stock.
        """
        # availability = self.instance['availability']
        # return min(quantity, availability.quantity)
        # FIXME
        return quantity

Can you please send me a stack-trace using a service such as https://dpaste.de
Between version 1.0 and 1.1 stock management was refactored. I'm afraid that some code from version 1.1 slipped into 1.0.2.
Can you please retry using the next upcoming version 1.1. Currently only available from GitHub, not PyPI.

django-shop version 1.0.2 downloaded from PyPI in shop/serializers/defaults/catalog.py there is no method validate_quantity(...). Can it be that you started to use version 1.1 from GitHub?

sorry, somehow I had version 1.1 dev


Create trigunadharma

@imantarico lib/domains/id/ac/trigunadharma: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.


chore: changelog touchups

Left notes about the relevant changes.
Everything else is lintfixes.

@arboleya your comments need resolving 😄


Time series ids

It would be useful to list what time series ids exist in a similar way to listing the keys within the key value store.

Actually, this could come from the hypercat.


Output type not inferred correctly for boolean operators

Reproduce: u .== v.
Expected eltype of Bool, got eltype of input types.
See line 287 of ewise.jl. Need to replace output type inference with a single function that uses input types as last resort.
Best way to do this is combined getoperator and optype.

Should be fixed by e19133fd81833591b88d1d38e9bf73b083aa89eb


Automatically load launch daemons on startup

Added an entry to the post.sh script so that launch daemons will automatically load on startup.
This is something that I find very useful because you won't need to open the loader app and press the button to load the launch daemons every time you jailbreak.

there are no safe boot though and starting daemons is potentially dangerous

Could the OpenSSH daemon be launched only then? That way SSH will be up and running on startup?


Avoid JSON stringifying

Currently we put the stringified version of the docgenInfo.
So, we have to do JSON.stringify
It's much better to avoid it in terms of usability and performance wise.

Fixed with #4


NIFI-7678: Fixed memory leak, improved some logic that constantly cre…

…ated collections unnecessarily. Changed types of Collections being used to much more efficient implementations. Removed using Streams for performance-critical parts of application
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically main)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Ran a few stateless flows I have, working as expected, code changes look good to me, merging to main, thanks @markap14


go to 1.20

https://modrinth.com/mod/explosive-enhancement/version/P5LwjQZ1
Updated to 1.20.0 and 1.20.1! Requires YetAnotherConfigLib v3.
If you find any bugs, please let me know.


rancher-agent with tcp endpoint w/o mounting docker.sock

Hello,
My docker daemon is running under the default namespace therefore I am unable to mount the unix socket (/var/run/docker.sock). Is it possible to run a rancher-agent with a tcp endpoint?

No, this is not supported.