Should throw DataCloneError when given a function, error, DOM node, etc

According to http://www.w3.org/html/wg/drafts/html/master/infrastructure.html#safe-passing-of-structured-data some inputs are supposed to result in a DataCloneError, such as functions, errors, and DOM nodes. I would be happy to submit a pull request to do this, but I just want to make sure you'd be receptive to it first.

If anybody else is concerned about this, I made https://github.com/dumbmatter/realistic-structured-clone

ping


Incorrect scroll offset.

On my personal website, I've manually added an object to test this implementation, and although it works, the offset is off from the original scroll position, as seen here in this image
The element was appended to my page via developer tools with the html below.

Then the script was ran to apply the changes and append the iframe inside that div.
I was able to remedy the issue using a manual dirty fix to take the backdropapply element's offsetHeight and multiplying that by two, and subtracting that to the scroll offset.
var scroll = this.scrollY
var scrollx = this.scrollX
scroll = scroll - (backdropapply.offsetHeight * 2);

Alright, I'll test it out. BTW, can you link me to your page with the example?


Update a few GitHub actions' versions, and add dependabot to check for latest version of GitHub actions

Breaking change

None
Proposed change

The workflows runs on some not up-to-date actions.
This PR will update them, and keep them updated.
Type of change

[x] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[ ] The code change is tested and works locally.
[ ] Local tests pass. Your PR cannot be merged unless tests pass
[ ] The code compiles without warnings (code quality check)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration are added/changed:

[ ] Documentation added/updated for https://netdaemon.xyz

Untested - thought it was against own fork


Compatibility with ES6 promises

Now that promises are part of ES6, what are the plans for bluebird in the long term?
I still want to use the utility functions that bluebird provides but if possible want to switch over to the native promises. Would something as simple as Object.assign(Promise, Bluebird) work?
Or is bluebird too incompatible with native promises as they stand?

I still want to use the utility functions that bluebird provides but if possible want to switch over to the native promises.

Native promises (especially v8) suffer from some bugs still, in addition they do not perform nearly as well as bluebird.

Would something as simple as Object.assign(Promise, Bluebird) work?

This likely wont work out of the box, Bluebirds helpers are finely tuned to bluebirds internals. This allows extra features, like bind, cancelation and good performance.

Bluebird is compatible with ES6 promises, you can use them together, the
ES6 spec guarantees it will play nice with bluebird promises and this is by
design and a design goal of the ES6 spec (Promises/A+ compatibility).
Bluebird will also gladly eat ES6 promises as thenables.
So you can think of bluebird as a much faster, much easier to debug
implementation with a richer API and goodies that are not native yet (like
cancellation) :)
On Sat, Jun 13, 2015 at 10:12 PM, Azeem Bande-Ali<EMAIL_ADDRESS>wrote:

Now that promises are part of ES6, what are the plans for bluebird in the
long term?
I still want to use the utility functions that bluebird provides but if
possible want to switch over to the native promises. Would something as
simple as Object.assign(Promise, Bluebird) work?
Or is bluebird too incompatible with native promises as they stand?
—
Reply to this email directly or view it on GitHub
https://github.com/petkaantonov/bluebird/issues/655.

Oh yeah definitely like Stef said Object.assign(Promise, Bluebird) won't work most likely. You can just do var Promise = Bluebird though.

@benjamingr i believe part of the questions was, can one just harvest bluebirds helper methods, and transplant them to a native Promise.
Answer to that is no, although for people who are uninterested in the performance benefit a subset of the utility functions could be implement in a transplantable way.
That being said, pollution  of globals (like self.Promise) is seen as an anti pattern. Instead of subclass, or entirely user land implementation would be recommended.
As @benjamingr mentioned, promise interopt is strong. You can easily convert back and forth, via BlueBirdPromise.resolve(Promise.resolve()); or BlueBirdPromise.resolve().then(function(){ return Promise.resolve(); });

I actually wrote an adapter that adds the bluebird API to native promises 3
weeks ago through a subclass but then I figured that if I put it on GH
people would actually use it and just get a weaker, slower version of
bluebird that's harder to debug so now it's just sort of sitting on my
desktop.

On Sat, Jun 13, 2015 at 10:18 PM, Stefan Penner<EMAIL_ADDRESS>wrote:

@benjamingr https://github.com/benjamingr i believe part of the
questions was, can one just harvest bluebirds helper methods, and
transplant them to a native Promise.
Answer to that is no, although for people who are uninterested in the
performance benefit a subset of the utility functions could be implement in
a transplantable way.
That being said, pollution of globals (like self.Promise) is seen as an
anti pattern. Instead of subclass, or entirely user land implementation
would be recommended.
As @benjamingr https://github.com/benjamingr mentioned, promise
interopt is strong. You can easily convert back and forth, via
BlueBirdPromise.resolve(Promise.resolve()); or BlueBirdPromise.resolve().then(function(){
return Promise.resolve(); });
—
Reply to this email directly or view it on GitHub
https://github.com/petkaantonov/bluebird/issues/655#issuecomment-111741773
.

I actually wrote an adapter that adds the bluebird API to native promises 3
weeks ago through a subclass but then I figured that if I put it on GH
people would actually use it and just get a weaker, slower version of
bluebird that's harder to debug so now it's just sort of sitting on my
desktop.

burn it with fire :P

@stefanpenner only if you add the rejection hooks to rsvp already :P

@stefanpenner only if you add the rejection hooks to rsvp already :P

Ya, I should. just a low priority ATM

These kind of questions are better asked on the mailing list.

Now that promises are part of ES6, what are the plans for bluebird in the long term?

bluebird is not a ES6 promises shim, so it's not affected by that

hrm. can it at least be easily castable in TypeScript? What is this hangup with purity?


Server 500 while trying to create a project

When i followed the instructions on github, i got an error while creating a project with this line;
project = client.create_project(name="test-sentiment", task="multi_class_classification", language="en")
It gave the errors as below;
autonlp.utils:http_post:75 - ❌ Operation failed! Details: Internal Server Error
...
...
HTTPError: 500 Server Error: Internal Server Error for url: https://api.autonlp.huggingface.co/projects/create
NOTE: i was able to login successfully.
from autonlp import AutoNLP
client = AutoNLP()
client.login(token="MY HUGGINGFACE TOKEN")
2021-03-22 14:59:57.056 | INFO     | autonlp.autonlp:login:51 - 🗝 Successfully logged in as gurkandy
2021-03-22 14:59:57.057 | INFO     | autonlp.autonlp:login:58 - 🗝 Storing credentials in:  MY HOME FOLDER

when i rewrite name="test**-sentiment" as name="test_**sentiment" it worked.


dart command line tool should be able handle snapshot lifecycle

When using the dart command line tool with a snapshot, there's a common pattern that would be valuable if it were built into the dart command line tool.  Given the following inputs:

A path to a snapshot
An opaque version string
A path to a Dart source file

Run the following algorithm:

If the snapshot does not exist:

Create a snapshot from the dart source file that embeds the opaque version string and write the snapshot to disk at the snapshot path
Execute the snapshot

Otherwise, if the snapshot does exist:
Check if the snapshot version is compatible with the DartVM.  If its not compatible, restart this algorithm assuming the snapshot does not exist.
Check if the opaque version string embedded in the snapshot matches the opaque version string.  If they don't match, restart this algorithm assuming the snapshot does not exist.
Execute the snapshot

/cc @sethladd

@iposva-google

Related: this workflow could be useful for pub global activate and pub get which both create snapshots on install, and re-create snapshots when the core SDK version changes.

See also #20802.


call initialize after changing logFlags

 You should sign our Contributor License Agreement in order to get your pull request merged.

Merged in 33c24e7 thanks.


chore: update uuid to 4.0.0

Updating uuid to 4.0.0. This will help with projects where the latest datadog package is also being used.

This was merged as part of https://github.com/statsig-io/dart-sdk/commit/13637a4b84e17d8bd4c988415028648e1953e505


[Paddle] Update type names for Paddle 3.0

Description
We have experienced compilation errors when building with Paddle 3.0 since they've changed some type names in their CUDA graph infrastructure (see https://github.com/PaddlePaddle/Paddle/pull/62832).
Pinging @eee4017
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Changes

Update type names for Paddle 3.0

Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/te-ci paddle


Example tuneup

Removes the examples:

client1 and client2
ipfs_bitswap_test

Also removes the stray file examples/block.data. It used to be used by something.
I wrote a longer one but the short gist is that fetch_and_cat has superceded both, except for the ipld stuff, which will now have zero examples but there are tests. It's better to have one clear example than to have multiple confusing and out of date ones.

bors r+


Fix misnamed variable

@pavolloffay Another quick one to review :)

LGTM :+1:

@pavolloffay Thanks!


I forgot to push some changes, it should work again now after you update the main branch.

I am checking whether there is more to be done.  Strangely, I get no errors on npm install.

Ok, can you try again? There were some version numbers and dependencies mixed up.

I confirm it works now. Thank you!


Test that data is queryable across restarts

Harden unit tests -- test code only, self-merging on green build.


Using a read-only DB from asset folder without copying

I'd like to include a pre-populated database with my app.   For my application the database will never be modified.   So I'd prefer to not make a copy of it in order to avoid the delay and storage space requirements of copying it.   Is this technically possible?   I thought I had read once that someone had written a small library that made this possible.

@JohnWeidner Apparently it is not possible.
See: http://stackoverflow.com/questions/9911004/open-sqlite-in-readonly-directly-from-assets


gateway-api: cleanup cell imports & dependencies

This commit removes unnecessary duplicated imports and hive dependencies.

/test


Compilation Issues with ROS Indigo

Hi!
I'm trying to build this program in my ROS (Indigo) Workspace on Ubuntu 14.04. I'm seeing the following error:
[100%] Building CXX object iai_kinect2/kinect2_viewer/CMakeFiles/kinect2_viewer.dir/src/viewer.cpp.o
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘void Kinect2Bridge::stop()’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:194:38: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::stop()’ from ‘void’ to ‘bool’
     if(deviceActive && !device->stop())
                                      ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:194:38: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:199:23: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::close()’ from ‘void’ to ‘bool’
     if(!device->close())
                       ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:199:23: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘bool Kinect2Bridge::initialize()’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:316:25: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::close()’ from ‘void’ to ‘bool’
       if(!device->close())
                         ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:316:25: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘void Kinect2Bridge::initConfig(bool, bool, double, double)’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:435:5: error: ‘Config’ is not a member of ‘libfreenect2::Freenect2Device’
     libfreenect2::Freenect2Device::Config config;
     ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:435:43: error: expected ‘;’ before ‘config’
     libfreenect2::Freenect2Device::Config config;
                                           ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:436:5: error: ‘config’ was not declared in this scope
     config.EnableBilateralFilter = bilateral_filter;
     ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:440:13: error: ‘class libfreenect2::Freenect2Device’ has no member named ‘setConfiguration’
     device->setConfiguration(config);
             ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘bool Kinect2Bridge::initDevice(std::string&)’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:550:23: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::start()’ from ‘void’ to ‘bool’
     if(!device->start())
                       ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:550:23: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘void Kinect2Bridge::stop()’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:194:38: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::stop()’ from ‘void’ to ‘bool’
     if(deviceActive && !device->stop())
                                      ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:194:38: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:199:23: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::close()’ from ‘void’ to ‘bool’
     if(!device->close())
                       ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:199:23: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:564:22: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::stop()’ from ‘void’ to ‘bool’
     if(!device->stop())
                      ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:564:22: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘bool Kinect2Bridge::initialize()’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:316:25: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::close()’ from ‘void’ to ‘bool’
       if(!device->close())
                         ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:316:25: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘void Kinect2Bridge::initConfig(bool, bool, double, double)’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:435:5: error: ‘Config’ is not a member of ‘libfreenect2::Freenect2Device’
     libfreenect2::Freenect2Device::Config config;
     ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:435:43: error: expected ‘;’ before ‘config’
     libfreenect2::Freenect2Device::Config config;
                                           ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:436:5: error: ‘config’ was not declared in this scope
     config.EnableBilateralFilter = bilateral_filter;
     ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:440:13: error: ‘class libfreenect2::Freenect2Device’ has no member named ‘setConfiguration’
     device->setConfiguration(config);
             ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘bool Kinect2Bridge::initDevice(std::string&)’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:550:23: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::start()’ from ‘void’ to ‘bool’
     if(!device->start())
                       ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:550:23: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘void Kinect2Bridge::callbackStatus()’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:767:25: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::start()’ from ‘void’ to ‘bool’
       if(!device->start())
                         ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:767:25: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:780:24: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::stop()’ from ‘void’ to ‘bool’
       if(!device->stop())
                        ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:780:24: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:564:22: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::stop()’ from ‘void’ to ‘bool’
     if(!device->stop())
                      ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:564:22: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘void Kinect2Bridge::receiveColor()’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:1035:19: error: ‘class libfreenect2::Frame’ has no member named ‘format’
       this->color.format = colorFrame->format;
                   ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:1035:40: error: ‘class libfreenect2::Frame’ has no member named ‘format’
       this->color.format = colorFrame->format;
                                        ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘void Kinect2Bridge::processIrDepth(const cv::Mat&, std::vector<cv::Mat>&, const std::vector<Kinect2Bridge::Status>&)’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:1120:80: error: no matching function for call to ‘libfreenect2::Frame::Frame(int&, int&, int, uchar* const&)’
       libfreenect2::Frame depthFrame(sizeIr.width, sizeIr.height, 4, depth.data);
                                                                                ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:1120:80: note: candidates are:
In file included from /usr/local/include/libfreenect2/libfreenect2.hpp:33:0,
                 from /home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:50:
/usr/local/include/libfreenect2/frame_listener.hpp:58:3: note: libfreenect2::Frame::Frame(size_t, size_t, size_t)
   Frame(size_t width, size_t height, size_t bytes_per_pixel) :
   ^
/usr/local/include/libfreenect2/frame_listener.hpp:58:3: note:   candidate expects 3 arguments, 4 provided
/usr/local/include/libfreenect2/frame_listener.hpp:40:24: note: constexpr libfreenect2::Frame::Frame(const libfreenect2::Frame&)
 class LIBFREENECT2_API Frame
                        ^
/usr/local/include/libfreenect2/frame_listener.hpp:40:24: note:   candidate expects 1 argument, 4 provided
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘void Kinect2Bridge::callbackStatus()’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:767:25: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::start()’ from ‘void’ to ‘bool’
       if(!device->start())
                         ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:767:25: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:780:24: error: could not convert ‘((Kinect2Bridge*)this)->Kinect2Bridge::device->libfreenect2::Freenect2Device::stop()’ from ‘void’ to ‘bool’
       if(!device->stop())
                        ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:780:24: error: in argument to unary !
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘void Kinect2Bridge::receiveColor()’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:1035:19: error: ‘class libfreenect2::Frame’ has no member named ‘format’
       this->color.format = colorFrame->format;
                   ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:1035:40: error: ‘class libfreenect2::Frame’ has no member named ‘format’
       this->color.format = colorFrame->format;
                                        ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp: In member function ‘void Kinect2Bridge::processIrDepth(const cv::Mat&, std::vector<cv::Mat>&, const std::vector<Kinect2Bridge::Status>&)’:
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:1120:80: error: no matching function for call to ‘libfreenect2::Frame::Frame(int&, int&, int, uchar* const&)’
       libfreenect2::Frame depthFrame(sizeIr.width, sizeIr.height, 4, depth.data);
                                                                                ^
/home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:1120:80: note: candidates are:
In file included from /usr/local/include/libfreenect2/libfreenect2.hpp:33:0,
                 from /home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_bridge/src/kinect2_bridge.cpp:50:
/usr/local/include/libfreenect2/frame_listener.hpp:58:3: note: libfreenect2::Frame::Frame(size_t, size_t, size_t)
   Frame(size_t width, size_t height, size_t bytes_per_pixel) :
   ^
/usr/local/include/libfreenect2/frame_listener.hpp:58:3: note:   candidate expects 3 arguments, 4 provided
/usr/local/include/libfreenect2/frame_listener.hpp:40:24: note: constexpr libfreenect2::Frame::Frame(const libfreenect2::Frame&)
 class LIBFREENECT2_API Frame
                        ^
/usr/local/include/libfreenect2/frame_listener.hpp:40:24: note:   candidate expects 1 argument, 4 provided
make[2]: *** [iai_kinect2/kinect2_bridge/CMakeFiles/kinect2_bridge_nodelet.dir/src/kinect2_bridge.cpp.o] Error 1
make[1]: *** [iai_kinect2/kinect2_bridge/CMakeFiles/kinect2_bridge_nodelet.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
make[2]: *** [iai_kinect2/kinect2_bridge/CMakeFiles/kinect2_bridge.dir/src/kinect2_bridge.cpp.o] Error 1
make[1]: *** [iai_kinect2/kinect2_bridge/CMakeFiles/kinect2_bridge.dir/all] Error 2
In file included from /usr/include/pcl-1.7/pcl/visualization/pcl_visualizer.h:52:0,
                 from /usr/include/pcl-1.7/pcl/visualization/cloud_viewer.h:39,
                 from /home/banerjs/Code/iai_ws/src/iai_kinect2/kinect2_viewer/src/viewer.cpp:32:
/usr/include/pcl-1.7/pcl/visualization/area_picking_event.h: In member function ‘bool pcl::visualization::AreaPickingEvent::getPointsIndices(std::vector<int>&) const’:
/usr/include/pcl-1.7/pcl/visualization/area_picking_event.h:64:30: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
           if (nb_points_ == -1)
                              ^
Linking CXX executable /home/banerjs/Code/iai_ws/devel/lib/kinect2_calibration/kinect2_calibration
[100%] Built target kinect2_calibration
Linking CXX executable /home/banerjs/Code/iai_ws/devel/lib/kinect2_viewer/kinect2_viewer
[100%] Built target kinect2_viewer
make: *** [all] Error 2
Invoking "make -j8 -l8" failed

Any idea what's going on? I checked the installation of libfreenect2, it seems alright (atleast bin/Protonect returns data from the Kinect). I also installed PCL from the Linux Install URL and I don't think that's faulty.
At first I thought this was the same issue as #240, but my error is at different places than on that bug.

Hmm... It seems that the problem was caused by me installing libfreenect2 in /usr/local/freenect2 instead of in $HOME/freenect2. In the former case the function definition of Freenect2Device::stop() was void whereas in the latter case it was bool.
I don't know why that was the case, but perhaps it might be something that someone else might be able to check? For now, I'm closing this issue


fix(RpcChannel): allow client to provide specVersion

Motivation and Resolution
This saves an extra call on RPC for optionally-known information (like the chainId case).
also fixed speck -> spec typo
Checklist:

[v ] Performed a self-review of the code
[v ] Rebased to the last commit of the target branch (or merged it into my branch)
[ ] Linked the issues which this PR resolves
[ ] Documented the changes in code (API docs will be generated automatically)
[ ] Updated the tests
[ ] All tests are passing

This is ok, this change was introduced into https://github.com/starknet-io/starknet.js/pull/908 and needed to be merged.


NWIS Client: Possible case sensitivity issue

It appears the NWIS IV service is case-insensitive. For example, note the use of startDT and startdt in the two calls below:
https://waterservices.usgs.gov/nwis/iv/?format=json&sites=01646500&startDT=2021-11-01&endDT=2021-11-02&parameterCd=00060&siteStatus=all
https://waterservices.usgs.gov/nwis/iv/?format=json&sites=01646500&startdt=2021-11-01&enddt=2021-11-02&parameterCd=00060&siteStatus=all
This may be an issue for us because I think the current nwis_client validation is case-sensitive. This has caused some issues when using non-string datetime parameters. For example:
client = IVDataService()

# Ex1. This succeeds
obs = client.get(sites="01013500", startdt="2021-09-01T00:00Z")

# Ex2. This fails
obs = client.get(sites="01013500", startdt=pd.Timestamp("2021-09-01T00:00Z"))

# Ex.3 This suceeds
obs = client.get(sites="01013500", startDT=pd.Timestamp("2021-09-01T00:00Z"))

I'm hesitant to have the validator transform any arguments unexpectedly. However, it might be a good idea for the date converter/validator to be case-insensitive. So, if a user enters StArTdT as a parameter the key is passed onto NWIS un-altered, but the value is still treated the same way we currently treat startDT.
ping @aaraney

Hmm, so what it sounds like you are suggesting that we should normalize keyword argument keys? I am fine with dropping the case for keyword arguments. This seems like a feature that we would not advertise in effort to implicitly standardize user usage of the software, but I am all for supporting this. Something like this following should work:
#!/usr/bin/env python3

# typing imports
from typing import Callable, TypeVar

T = TypeVar("T")

def standardize_kwargs(fn: Callable[..., T]) -> Callable[..., T]:
    def wrapper(*args, **kwargs):
        normalized_kwargs = {k.lower(): v for k, v in kwargs.items()}

        return fn(*args, **normalized_kwargs)
    return wrapper

@standardize_kwargs
def foo(*, a: str):
    return a

def it_works():
    print(foo(A=2))

if __name__ == "__main__":
    it_works()

Actually I'm more partial to standardization. In which case, instead of
dropping case-sensitivity, we could raise a helpful error message.
On Tue, Nov 16, 2021, 09:55 Austin Raney @.***> wrote:

Hmm, so what it sounds like you are suggesting that we should normalize
keyword argument keys? I am fine with dropping the case for keyword
arguments. This seems like a feature that we would not advertise in effort
to implicitly standardize user usage of the software, but I am all for
supporting this. Something like this following should work:
#!/usr/bin/env python3
typing importsfrom typing import Callable, TypeVar
T = TypeVar("T")
def standardize_kwargs(fn: Callable[..., T]) -> Callable[..., T]:
def wrapper(*args, **kwargs):
normalized_kwargs = {k.lower(): v for k, v in kwargs.items()}
    return fn(*args, **normalized_kwargs)
return wrapper

@standardize_kwargsdef foo(*, a: str):
return a
def it_works():
print(foo(A=2))
if name == "main":
it_works()
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NOAA-OWP/hydrotools/issues/160#issuecomment-970411686,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADBZNXDPRBVWII3SZ6FCZF3UMJ5GXANCNFSM5IER3SNQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Actually I'm more partial to standardization. In which case, instead of dropping case-sensitivity, we could raise a helpful error message.

So, maybe something like this?
#!/usr/bin/env python3

# typing imports
from typing import Callable, TypeVar

T = TypeVar("T")

def improved_kwargs_error(fn: Callable[..., T]) -> Callable[..., T]:
    def wrapper(*args, **kwargs):
        fn_kwargs = set(fn.__annotations__)

        for k in kwargs.keys():
            if k not in fn_kwargs:
                error_message = (
                                              f"key word argument, {k}, not in accepted arguments.\n"
                                              f"accepted arguments are: {fn.__annotations__}"
                                             )

        return fn(*args, **kwargs)
    return wrapper

I like it. What do you think?

I like it. What do you think?

I also like it. I think it is more clear how to resolve an issue which is desirable given the intended audience.

A third option that we did not mention is removing **params from the method call and let python handle arguments that are incorrectly named. We could "forward" any applicable keyword arguments from get_raw to get's method signature. For example, the max_sites_per_request from get_raw could be added to get's method signature.


feat: create new dropdown upgrades

Draft because I'm waiting on a component in #2829, as well as wanting to gather some ideas / opinions on the overlayDropdown changes.
Related to #2830

I've rebased this to #2829 for now so that I can use the 'new' component

Followed up in https://github.com/camunda/camunda-modeler/pull/2843 to avoid git headaches.


Clean setup

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I think we don't want this as the first item you see when you look at the repsoitory. I'd add that as an option or additional entry in the dependency installation that you can use the virtual environment...

@gidden, how does this interact with your tackling a better install procedure?

The instructions here still provide source builds. We will probably want to revamp these instructions in general with the updated conda installs.

@zikolach do you want to add the .dylibs you need to this PR as well?

@gidden yes, sorry I thought I pushed it already. Now it is in PR

Hey @zikolach, we've had a lot of thrashing on the master branch recently before the YSSPers get here. Would you mind opening a new PR with changes you still think as relevant?


Update vae.py

Correct argument data interpretation in train_step. This is a tuple of form (tf.tensor, ) so needs to be unpacked before being passed to the loss function. The argument reconstruction is a tf.tensor already.

Thanks for the PR -- data in this case is a tensor representing one batch of data, not a tuple. You can try running the code on Colab, for instance.


[New Category] Html Crypto MIners

Аntі-WеbМіnеr Ѕtаrt 1.0 43011
<IP_ADDRESS> аzvјudwr.іnfо
<IP_ADDRESS> сnhv.со
<IP_ADDRESS> соіn-hіvе.соm
<IP_ADDRESS> guѕ.hоѕt
<IP_ADDRESS> јrоqvbvw.іnfо
<IP_ADDRESS> јѕесоіn.соm
<IP_ADDRESS> јуhfuqоh.іnfо
<IP_ADDRESS> kdоwqlрt.іnfо
<IP_ADDRESS> lіѕtаt.bіz
<IP_ADDRESS> lmоdr.bіz
<IP_ADDRESS> mаtаhаrіrаmа.хуz
<IP_ADDRESS> mіnесrunсh.со
<IP_ADDRESS> mіnеmуtrаffіс.соm
<IP_ADDRESS> mіnеr.рr0grаmm.соm
<IP_ADDRESS> rеаѕеdореr.рw
<IP_ADDRESS> хbаѕfbnо.іnfо
Аntі-WеbМіnеr Еnd

@AngelTs Do you have any interest in turning this into a proper list for this tool? Not for any negative reason, but because if you don't, I can toss it together and submit a pull request so it can be pulled upstream.

Found another list of coin miners here.

Having an up to date list of crypto minters is of course well and good and definitely falls within the mission statement of this project. However, I must insist if the list is going to continue to grow at this rate, can someone offer a curation solution for these, such as a scripted check to verify crypto mining is still actively being conducted by each of the listed entries? I know we are all doing our best to pull together and help out, but I just don't want to end up having something unmanageable and unchecked in the end.

Аntі-WеbМіnеr Ѕtаrt NoCoin v1
<IP_ADDRESS> coinhive.com
<IP_ADDRESS> coin-hive.com
<IP_ADDRESS> jsecoin.com
<IP_ADDRESS> server.jsecoin.com
<IP_ADDRESS> load.jsecoin.com
<IP_ADDRESS> static.reasedoper.pw
<IP_ADDRESS> mataharirama.xyz
<IP_ADDRESS> listat.biz
<IP_ADDRESS> lmodr.biz
<IP_ADDRESS> minecrunch.co
<IP_ADDRESS> minemytraffic.com
<IP_ADDRESS> crypto-loot.com
<IP_ADDRESS> 2giga.link
<IP_ADDRESS> 2giga.link
<IP_ADDRESS> ppoi.org
<IP_ADDRESS> coinerra.com
<IP_ADDRESS> coin-have.com
<IP_ADDRESS> kisshentai.net
<IP_ADDRESS> miner.pr0gramm.com
<IP_ADDRESS> kiwifarms.net
<IP_ADDRESS> anime.reactor.cc
<IP_ADDRESS> joyreactor.cc
<IP_ADDRESS> kissdoujin.com
<IP_ADDRESS> ppoi.org
<IP_ADDRESS> minero.pw
<IP_ADDRESS> coinnebula.com
<IP_ADDRESS> coinnebula.com
<IP_ADDRESS> afminer.com
<IP_ADDRESS> coinblind.com
<IP_ADDRESS> webmine.cz
<IP_ADDRESS> monerominer.rocks
<IP_ADDRESS> monerominer.rocks
<IP_ADDRESS> cdn.cloudcoins.co
<IP_ADDRESS> coinlab.biz
<IP_ADDRESS> papoto.com
<IP_ADDRESS> cookiescript.info
<IP_ADDRESS> cookiescriptcdn.pro
<IP_ADDRESS> rocks.io
<IP_ADDRESS> ad-miner.com
<IP_ADDRESS> party-nngvitbizn.now.sh
<IP_ADDRESS> cryptoloot.pro
<IP_ADDRESS> host.d-ns.ga
<IP_ADDRESS> baiduccdn1.com
<IP_ADDRESS> jsccnn.com
<IP_ADDRESS> jscdndel.com
<IP_ADDRESS> mine.nahnoji.cz
<IP_ADDRESS> goredirect.party
<IP_ADDRESS> miner.pr0gramm.com
<IP_ADDRESS> miner.cryptobara.com
<IP_ADDRESS> digger.cryptobara.com
<IP_ADDRESS> kickass.cd
<IP_ADDRESS> morningdigit.com
<IP_ADDRESS> webminepool.com
<IP_ADDRESS> webminepool.tk
<IP_ADDRESS> stackpathdns.com
<IP_ADDRESS> cpu2cash.link
<IP_ADDRESS> googleanalytcs.com
<IP_ADDRESS> coinpirate.cf
<IP_ADDRESS> webmine.pro
<IP_ADDRESS> a-o.ninja
<IP_ADDRESS> gasolina.ml
<IP_ADDRESS> gasolina.ml
<IP_ADDRESS> you.tubetitties.com
<IP_ADDRESS> hive.tubetitties.com
<IP_ADDRESS> playerassets.info
<IP_ADDRESS> tokyodrift.ga
<IP_ADDRESS> bewhoyouare.gq
<IP_ADDRESS> sen-to-zdrowie.ml
<IP_ADDRESS> freecontent.racing
<IP_ADDRESS> freecontent.loan
<IP_ADDRESS> chainblock.science
<IP_ADDRESS> hodling.faith
<IP_ADDRESS> freecontent.bid
<IP_ADDRESS> 1q2w3.fun
<IP_ADDRESS> candid.zone
<IP_ADDRESS> lewd.ninja
Аntі-WеbМіnеr Еnd

These are certainly helpful suggestions for those creating custom lists locally, however for this repo I think we are close to a solution in https://github.com/StevenBlack/hosts/issues/462 and we should focus our efforts on a curated list. And I'm not going to check every single one of the domains listed above, but I can say I personally recognize a good number as already being in our stevenblack/hosts. One we get a dedicated curator for these established, then all further suggestions can be forwarded there so these can be better managed with the focus and attention due them.

@AngelTs
a-o.ninja has not miner for more than half a year.
Please delete the domain a-o.ninja from the list.
Thank you

This is an old issue. We source our crypto miners from @ZeroDot1's CoinBlockerlists now, https://gitlab.com/ZeroDot1/CoinBlockerLists. Issues with crypto miners should be made over there, https://gitlab.com/ZeroDot1/CoinBlockerLists/issues.
@StevenBlack, I think this can be safely closed.

I agree @ScriptTiger.
Closing.


0.3.22 breaks functions downstream in GNU Octave

OpenBLAS 0.3.21 works perfectly with GNU Octave, but 0.3.22 causes several test failures in matrix inverses, eigenvector calculations, and sparse matrix operations. These include returning Inf instead of NaN and giving an answer that is wrong by several orders of magnitude. Affected Octave functions include inv(), eigs(), residue(), and det(), and any user code using those functions.
Thread: https://octave.discourse.group/t/make-check-fails-with-openblas-0-3-22/4289
How to reproduce:

Download & build a current version of GNU Octave from source: https://octave.org/download.
Run make check.
Observation: OpenBLAS 0.3.22 will give 10 test failures in the above functions. 0.3.21 will give zero test failures.

Workaround: I have reverted my system OpenBLAS to 0.3.21 for now, and Octave works properly again.
EDIT: Build settings for OpenBLAS, both versions:
FC=gfortran USE_OPENMP=1 USE_THREAD=1 USE_TLS=0 NO_STATIC=1 CPP_THREAD_SAFETY_TEST=1

If it helps: https://octave.discourse.group/t/make-check-fails-with-openblas-0-3-22/4289/4
The problem might be in  dpotrf or dpstrf due to the specific mix of Octave test failures.

Looks like i broke a division-by-zero test in zgetf2 with a quick fix lately (#3941 ,managed to change OR into AND and nobody/no test noticed)

That's probably off-topic to this issue. But @MehdiChinoune is probably referring to the following symbols that are no longer being exported (or exported with a different signature) by the OpenBLAS library compared to version 0.3.21:
  {b'libopenblas.dll': {b'dzamin_',
                        b'dzasum_',
                        b'dznrm2_',
                        b'dzsum1_',
                        b'dzsum_',
                        b'exec_blas',
                        b'gemm_thread_m',
                        b'gemm_thread_mn',
                        b'gemm_thread_n',
                        b'gemm_thread_variable',
                        b'get_num_procs',
                        b'goto_get_num_procs',
                        b'goto_set_num_threads',
                        b'gotoblas',
                        b'gotoblas_ATHLON',
                        b'gotoblas_BANIAS',
                        b'gotoblas_BARCELONA',
                        b'gotoblas_BULLDOZER',
                        b'gotoblas_COOPERLAKE',
                        b'gotoblas_COPPERMINE',
                        b'gotoblas_CORE2',
                        b'gotoblas_EXCAVATOR',
                        b'gotoblas_HASWELL',
                        b'gotoblas_KATMAI',
                        b'gotoblas_NEHALEM',
                        b'gotoblas_NORTHWOOD',
                        b'gotoblas_PILEDRIVER',
                        b'gotoblas_PRESCOTT',
                        b'gotoblas_SANDYBRIDGE',
                        b'gotoblas_SKYLAKEX'},

Is that intentional?

Looks like i broke a division-by-zero test in zgetf2 with a quick fix lately (#3941 ,managed to change OR into AND and nobody/no test noticed)

Would you like a test in Fortran which would have captured that? As scipy downstream points out, for them the regression comes from https://github.com/scipy/scipy/blob/main/scipy/linalg/src/det.f#L94
(cf. https://github.com/scipy/scipy/issues/18208#issuecomment-1487907458)

Using bug-63384.f  (see https://github.com/Reference-LAPACK/lapack/issues/763):
$ gfortran bug-63384.f -llapack -lblas
$ ./a.out
 x =
                       NaN                       NaN
                       NaN                       NaN
 anorm =                        NaN
 dgetrf info:            0
 ipiv =
           1           2
 dgecon info:            0
 rcond =    0.0000000000000000
 dgetri info:            0
 xinv =
                       NaN                       NaN
                       NaN                       NaN

That looks good. Now:
$ LD_PRELOAD=./libopenblas_zenp-r0.3.22.so ./a.out
 x =
                       NaN                       NaN
                       NaN                       NaN
 anorm =                        NaN
 dgetrf info:            1
 ipiv =
           1           2
 ** On entry to DGECON parameter number  5 had an illegal value
 dgecon info:           -5
 rcond =   -1.2345670461654663
 dgetri info:            0
 xinv =
                       NaN                       NaN
                       NaN                       NaN

Hmm. I wonder if the GETF2 fixes are trading a NumPy issue ( https://github.com/numpy/numpy/issues/22025 ) for an octave one (the DGECON fix for Reference-LAPACK issue 763 is carried in OpenBLAS 0.3.22)

Why does dgetrf return info=1 (and not 0)?

Good question. Seems temp was below DBL_MIN so singularity gets reported (original code before the fix used "is unequal ZERO" there, as does the original dgetf2.f of the reference LAPACK. A trap ?

This is directly in octave.
With 0.3.21:
octave:1> C = [   1.0000 +      0i        0 +      0i   1.0000 +      0i        0 +      0i
        0 + 1.0000i   0.3333 +      0i        0 - 1.0000i   0.3333 +      0i
   1.0000 +      0i        0 + 0.6667i   1.0000 +      0i        0 - 0.6667i
        0 + 1.0000i  -0.3333 +      0i        0 - 1.0000i  -0.3333 +      0i

];
octave:2> rcond(C)
ans = 0.1250

With 0.3.22:
octave:1> C = [  1.0000 +      0i        0 +      0i   1.0000 +      0i        0 +      0i
        0 + 1.0000i   0.3333 +      0i        0 - 1.0000i   0.3333 +      0i
   1.0000 +      0i        0 + 0.6667i   1.0000 +      0i        0 - 0.6667i
        0 + 1.0000i  -0.3333 +      0i        0 - 1.0000i  -0.3333 +      0i

];
octave:2> rcond(C)
ans = 0

I attached t_rcond.m with those two lines (to avoid cut-n-paste issues).
t_rcond.m.gz

@angsch what if I stack the old and new conditionals - if temp1 not exactly zero then check if large enough to do the division, swap&scale as needed, else set info ? I know it`s an engineering solution to a mathematical problem, but...

@angsch what if I stack the old and new conditionals - if temp1 not exactly zero then check if large enough to do the division, swap&scale as needed, else set info ? I know it`s an engineering solution to a mathematical problem, but...

My understanding of DLAMCH('S') alias DBL_MIN is that this is the smallest normal number. Of course, there are subnormals, but for the LU factorization there is no point in continuing here. So the check against DBL_MIN is in my opinion correct. Every subnormal entry (fabs(a) < DBL_MIN) should be treated as zero, ie set info. I do not know how exactly the criterion for complex numbers should look.
Do we know if the issue with rcond is related to the LU factorization or to some other problem?

Ok, thanks, I'm following through with that already - will also check if this takes care of the Octave issues once I've got it built

You do not need to build a new octave, you can use the binary provided by your distro and use LD_PRELOAD to
override blas libs.
LU also appears broken in octave (though the tests did not notice that):
octave:33> C
C =

   1.0000 +      0i        0 +      0i   1.0000 +      0i        0 +      0i
        0 + 1.0000i   0.3333 +      0i        0 - 1.0000i   0.3333 +      0i
   1.0000 +      0i        0 + 0.6667i   1.0000 +      0i        0 - 0.6667i
        0 + 1.0000i  -0.3333 +      0i        0 - 1.0000i  -0.3333 +      0i

octave:34> [L, U] = lu(C)
L =

   1.0000 +      0i        0 +      0i        0 +      0i        0 +      0i
   1.0000 +      0i        0 + 0.6667i   1.0000 +      0i        0 +      0i
        0 + 1.0000i   1.0000 +      0i        0 +      0i        0 +      0i
        0 + 1.0000i  -0.3333 +      0i   0.6000 + 0.8000i   1.0000 +      0i

U =

   1.0000 +      0i        0 +      0i   1.0000 +      0i        0 +      0i
        0 +      0i   0.3333 +      0i   1.0000 - 1.0000i        0 - 0.6667i
        0 +      0i        0 +      0i  -1.6667 - 1.6667i  -0.1112 +      0i
        0 +      0i        0 +      0i        0 +      0i  -0.2666 - 0.1333i

octave:35> L*U - C
ans =

        0 +      0i        0 +      0i        0 +      0i        0 +      0i
   1.0000 - 1.0000i  -0.3333 + 0.2222i  -0.0000 +      0i        0 +      0i
  -1.0000 + 1.0000i   0.3333 - 0.6667i        0 +      0i        0 +      0i
        0 +      0i   0.2222 +      0i        0 - 0.0000i        0 +      0i

with 0.3.21 (works as expected):
octave:26> C
C =

   1.0000 +      0i        0 +      0i   1.0000 +      0i        0 +      0i
        0 + 1.0000i   0.3333 +      0i        0 - 1.0000i   0.3333 +      0i
   1.0000 +      0i        0 + 0.6667i   1.0000 +      0i        0 - 0.6667i
        0 + 1.0000i  -0.3333 +      0i        0 - 1.0000i  -0.3333 +      0i

octave:27> [L, U] = lu(C)
L =

   1.0000 +      0i        0 +      0i        0 +      0i        0 +      0i
        0 + 1.0000i        0 - 0.4999i   1.0000 +      0i        0 +      0i
   1.0000 +      0i   1.0000 +      0i        0 +      0i        0 +      0i
        0 + 1.0000i        0 + 0.4999i   1.0000 +      0i   1.0000 +      0i

U =

   1.0000 +      0i        0 +      0i   1.0000 +      0i        0 +      0i
        0 +      0i        0 + 0.6667i        0 +      0i        0 - 0.6667i
        0 +      0i        0 +      0i        0 - 2.0000i   0.6666 +      0i
        0 +      0i        0 +      0i        0 +      0i  -1.3332 +      0i

octave:28> L*U - C
ans =

   0   0   0   0
   0   0   0   0
   0   0   0   0
   0   0   0   0

@dasergatskov Looks like something is wrong with the tiebreaker when pivoting is applied... Can you print P as well?

I assume P as permutation matrix. it is the same is both cases, but lu is still broken.
0.3.22:
octave:36> [L, U, P] = lu(C)
L =

   1.0000 +      0i        0 +      0i        0 +      0i        0 +      0i
        0 + 1.0000i   1.0000 +      0i        0 +      0i        0 +      0i
   1.0000 +      0i        0 + 0.6667i   1.0000 +      0i        0 +      0i
        0 + 1.0000i  -0.3333 +      0i   0.6000 + 0.8000i   1.0000 +      0i

U =

   1.0000 +      0i        0 +      0i   1.0000 +      0i        0 +      0i
        0 +      0i   0.3333 +      0i   1.0000 - 1.0000i        0 - 0.6667i
        0 +      0i        0 +      0i  -1.6667 - 1.6667i  -0.1112 +      0i
        0 +      0i        0 +      0i        0 +      0i  -0.2666 - 0.1333i

P =

Permutation Matrix

   1   0   0   0
   0   0   1   0
   0   1   0   0
   0   0   0   1

octave:37> P'*L*U - C
ans =

        0 +      0i        0 +      0i        0 +      0i        0 +      0i
   1.0000 - 1.0000i  -0.3333 + 0.2222i  -0.0000 +      0i        0 +      0i
  -1.0000 + 1.0000i   0.3333 - 0.6667i        0 +      0i        0 +      0i
        0 +      0i   0.2222 +      0i        0 - 0.0000i        0 +      0i

0.3.21:
octave:33> [L, U, P] = lu(C)
L =

   1.0000 +      0i        0 +      0i        0 +      0i        0 +      0i
   1.0000 +      0i   1.0000 +      0i        0 +      0i        0 +      0i
        0 + 1.0000i        0 - 0.4999i   1.0000 +      0i        0 +      0i
        0 + 1.0000i        0 + 0.4999i   1.0000 +      0i   1.0000 +      0i

U =

   1.0000 +      0i        0 +      0i   1.0000 +      0i        0 +      0i
        0 +      0i        0 + 0.6667i        0 +      0i        0 - 0.6667i
        0 +      0i        0 +      0i        0 - 2.0000i   0.6666 +      0i
        0 +      0i        0 +      0i        0 +      0i  -1.3332 +      0i

P =

Permutation Matrix

   1   0   0   0
   0   0   1   0
   0   1   0   0
   0   0   0   1

octave:34> P'*L*U - C
ans =

   0   0   0   0
   0   0   0   0
   0   0   0   0
   0   0   0   0

This is also a  repost from octave https://octave.discourse.group/t/make-check-fails-with-openblas-0-3-22/4289/7
I noticed that few test failures in octave are due to what appears to be a small (~eps) mutual RE <-->Im spillage.
E.g. with residue():
octave:1> b = [1, 0, 1];
octave:2> a = [1, 0, 18, 0, 81];
octave:3> [r, p, k, e] = residue (b, a)
warning: matrix singular to machine precision
warning: called from
    residue at line 257 column 5

r =

  -5.6656e-18 - 9.2593e-02i
   2.2222e-01 + 5.9111e-17i
  -6.6672e-17 + 9.2593e-02i
   2.2222e-01 + 1.3878e-16i
# w/ openblas 9.3.21
# r =
#
#        0 - 0.0926i
#   0.2222 +      0i
#        0 + 0.0926i
#  0.2222 +      0i
#

p =

   0 + 3i
   0 + 3i
   0 - 3i
   0 - 3i

k = [](0x0)
e =

   1
   2
   1
   2
octave:4> [br, ar] = residue (r, p, k)
br =

 Columns 1 and 2:

  -7.2338e-17 + 1.1102e-16i   1.0000e+00 + 3.8091e-16i

 Columns 3 and 4:

  -1.7304e-16 + 6.6613e-16i   1.0000e+00 - 1.3382e-16i

#
# br =
#
#   1   0   1

And test fail because br now 1x4 instead of 1x3

@martin-frbg
https://github.com/xianyi/OpenBLAS/blob/23f2c4ca5b0ee7f9a5bd1ec25206f4db19453c36/lapack/getf2/getf2_k.c#L104
My understanding of the build system is that the code gets instantiated for float and double. Does the constant change, too? DBL_MIN has its FLT_MIN counterpart. The compiler may promote the data type from float to double.

#3980 fixes the scipy error and also the rcond , lu und spurious residue errors from octave - I still see one scipy testsuite error:
TestFBLAS2Simple::test_spr_hpr AssertionError: Arrays are not almost equal to 6 decimals. Mismatched elements 9/9
but have not checked if it is/was already present with 0.3.21

With the latest patch, using d70895137589b61b6d892a74b3c793735bf58b31, it works again in Octave:
$ LD_PRELOAD=./libopenblas_zenp-r0.3.22.dev.so octave -q

octave:1> version -blas
ans = OpenBLAS (config: OpenBLAS 0.3.22.dev NO_AFFINITY USE_OPENMP ZEN MAX_THREADS=32)

octave:2> C = [1, 0, 1, 0;  i, 1/3, -i, 1/3; 1, 2*i/3, 1, -2*i/3; i, -1/3, -i, -1/3]
C =
   1.0000 +      0i        0 +      0i   1.0000 +      0i        0 +      0i
        0 + 1.0000i   0.3333 +      0i        0 - 1.0000i   0.3333 +      0i
   1.0000 +      0i        0 + 0.6667i   1.0000 +      0i        0 - 0.6667i
        0 + 1.0000i  -0.3333 +      0i        0 - 1.0000i  -0.3333 +      0i

octave:3> [L, U, P] = lu(C), residual = P'*L*U - C
L =
   1.0000 +      0i        0 +      0i        0 +      0i        0 +      0i
   1.0000 +      0i   1.0000 +      0i        0 +      0i        0 +      0i
        0 + 1.0000i        0 - 0.5000i   1.0000 +      0i        0 +      0i
        0 + 1.0000i        0 + 0.5000i   1.0000 -      0i   1.0000 +      0i

U =
   1.0000 +      0i        0 +      0i   1.0000 +      0i        0 +      0i
        0 +      0i        0 + 0.6667i        0 +      0i        0 - 0.6667i
        0 +      0i        0 +      0i        0 - 2.0000i   0.6667 +      0i
        0 +      0i        0 +      0i        0 +      0i  -1.3333 +      0i

P =
Permutation Matrix
   1   0   0   0
   0   0   1   0
   0   1   0   0
   0   0   0   1

residual =
   0   0   0   0
   0   0   0   0
   0   0   0   0
   0   0   0   0

Thanks for confirming. I am currently bisecting to find the cause of the lone remaining scipy test error

0.3.23 released now, Windows binaries may have to wait until Monday noon (CET) unless I convince myself that the MXE (mingw) dlltool is fully equivalent to a native lib.exe (for creating the export file and import library from the generated .def)

I guess I am late to the party, but I am still getting higher errors in complex chol (in octave)
I compiled 0.3.21 and 0.3.23 with the same compiler and with default params (Just typed make):
$ LD_PRELOAD=~/src/OpenBLAS-0.3.21/libopenblas_zenp-r0.3.21.so NUM_THREADS=16 octave -qf
octave:1> c1
ans = 4.8179e-16
octave:2> version -blas
ans = FlexiBLAS Version 3.3.0
OpenBLAS (config: OpenBLAS 0.3.21 NO_AFFINITY ZEN MAX_THREADS=32)
octave:3>

$ LD_PRELOAD=~/src/OpenBLAS-0.3.23/libopenblas_zenp-r0.3.23.so NUM_THREADS=16 octave -qf
octave:1> c1
ans = 2.2369e-15
error: assert (norm (R1 - R, Inf) < 1e1 * eps) failed
error: called from
    assert at line 107 column 11
    c1 at line 19 column 1
octave:2> version -blas
ans = FlexiBLAS Version 3.3.0
OpenBLAS (config: OpenBLAS 0.3.23 NO_AFFINITY ZEN MAX_THREADS=32)
octave:3>

The original test was for 10eps now it passes at 11eps, but the error increase by factor ~5.
Test script is attached.
c1.m.gz

@dasergatskov Mine is a little bit better, doesn't fail the assertion:
0.3.21:
$ octave -q
octave:1> version -blas
ans = OpenBLAS (config: OpenBLAS 0.3.21 NO_AFFINITY USE_OPENMP ZEN MAX_THREADS=32)
octave:2> c1
ans = 4.1489e-16
octave:3> exit

0.3.23:
$ LD_PRELOAD="./libopenblas_zenp-r0.3.23.so" octave -q
octave:1> version -blas
ans = OpenBLAS (config: OpenBLAS 0.3.23 NO_AFFINITY USE_OPENMP ZEN MAX_THREADS=32)
octave:2> c1
ans = 7.6690e-16
octave:3> exit

The options passed to building 0.3.23 didn't make a difference for me; I get the same results with these two sets of parameters. This:
CFLAGS="-pipe"  CXXFLAGS="-pipe"  FFLAGS="-pipe"  make -j all FC=gfortran USE_OPENMP=1 USE_THREAD=1 USE_TLS=0 NO_LAPACK=0 BUILD_LAPACK_DEPRECATED=1 NO_STATIC=1 BUILD_RELAPACK=1 CPP_THREAD_SAFETY_TEST=1 CONSISTENT_FPCSR=1

as well as this:
CFLAGS="-pipe"  CXXFLAGS="-pipe"  FFLAGS="-pipe"  make -j all

Compiler is gcc 12.2.1.

Getting ans = 5.4624e-16 from Octave 8.1.0 for a ZEN binary built with gcc 10.2.1 (the default in Debian 11), same as with 0.3.21, on a six-core Ryzen5-4600H running Linux - cpu model (Zen3 vs Zen4), compiler version and operating system will probably play a role.


Is it possible to drop dependency on logger version 2.066?

The logger package version 2.066 instead of std/experimental/logger has std/historical/logger file structure now and it prevents from successful compiling.

Yes it was renamed on purpose to avoid an unsolvable problem otherwise: https://github.com/burner/logger/issues/22
GFM is compatible up to DMD frontend 2.066, std.experimental.logger only exist in 2.067.
To be compatible with all frontends, include it like that:
static if( __VERSION__ >= 2067 )
    import std.experimental.logger;
else
    import std.historical.logger;

else to be only compatible with DMD 2.067+, include it normally:
import std.experimental.logger;

Once GFM drops compatibility with 2.066 frontend, the logger dependency won't be needed anymore.

Thanks!


processes in setup block are not guaranteed to execute before processes in when block

The nf-test  test attempts to run a subworkflow with a setup block that deletes s3 buckets using process DELETE_S3.
nextflow_workflow {

    name "Test Workflow CHECK_MD5_DOWNLOAD_UNICHEM_WORKFLOW"
    script "subworkflows/download_unichem_workflow.nf"
    workflow "CHECK_MD5_DOWNLOAD_UNICHEM_WORKFLOW"

    test("Should run without failures and should download all files") {

        setup {
            // Delete the test files in the s3 input files dir
            // so that we can test the download and upload
            run("DELETE_S3") {
                script "../../modules/util.nf"
                process {
                    """
                    input[0] =  Channel.value('s3://mybucket')
                    """
                }
            }

        }

        when {
            params {
                unichem {
                    s3_input_files_dir = 's3://mybucket'
                }
            }

            workflow {

                """
                input[0] = Channel.value(params.unichem.s3_input_files_dir)
                ....
                """
            }
...
        }
...

We see from the logs that the setup process DELETE_S3 is executed after other processes in the workflow block.
> Nextflow 24.10.0 is available - Please consider updating your version to it
   > N E X T F L O W  ~  version 24.04.4
   > Launching `[...]/.nf-test-3fa44d841839dc0c22a4f838a5652433.nf` [focused_mercator] DSL2 - revision: dc6044fdc0
   > WARN: Task runtime metrics are not reported when using macOS without a container engine
   > [a0/d1aefb] Submitted process > DOWNLOAD_UNICHEM_WORKFLOW:DOWNLOAD_UNICHEM
   > [fe/a5a33a] Submitted process > DELETE_S3
   > [86/8e38fa] Submitted process > DOWNLOAD_UNICHEM_WORKFLOW:WRITE_UNICHEM_ONTOLOGY_READER_CONFIG
   > [fc/122115] Submitted process > DOWNLOAD_UNICHEM_WORKFLOW:WRITE_MD5_FILE
   > Waiting for file transfers to complete (1 files)
...

Thanks, you are right. Since the test process is not using the output of the setup process, there is no dependency between them, and Nextflow could execute the setup process afterward. I will think about finding a way to fix this and guarantee the correct order 👍


Feature request: Less clutter in derivation trees.

I wanted to use your derivation trees as an educational tool here https://stackoverflow.com/a/53229580/502187. I have some wishes to reduce clutter, based on experience with clutter less Byrd box debugger:

I don't understand the ('=')/2 goals that are shown in the derivation. The source code does not contain these ('=')/2. I guess they are "synthetic". Can they be supressed? Are they result of (is)/2?

I don't understand why conjunctions are shown. Isn't the tree an AND/OR tree? Which would only have primitive goals as nodes? As the example shows, conjunctions tails are replicated down the tree, and don't give a lot of informations.

To make the issue self contained, here is the program I used:
length([], 0).
length([_|L], M) :- length(L, N), M is N+1.
And this is the tree I get:

I cannot really report how a solution would look like for a derivation tree. But for Byrd box, I have solved the problems 1) and 2) as follows:

Solution for (=)/2 problem:
I have a source level flag called sys_notrace. If this flag is set for a source, what happens inside this source is not tarced by the debugger.

Solution for (,)/2 problem:
I have a predicate level flag called sys_notrace (same name but other object). If this flag is set for a predicate, the predicate is not traced by the debugger.

Could be extended to parallel execution, found this nice paper:
Some Paradigms for Visualizing Parallel - Execution of Logic Programs
https://cliplab.org/papers/visandor-iclp93.pdf
But don't know what primitives Tau Prolog provides for parallel execution...

I found some trees in the wild, used by LPN:
http://www.learnprolognow.org/lpnpage.php?pagetype=html&pageid=lpn-htmlse43
The also use conjunctions, that redundantly are repeated. What I also noticed
is that they show partial substitutions, from unification along the arcs of the tree.
In your case you have after arc (16), (17), (18), (19), repeatedly showing the
same substitution. But the LPN example only shows substitution, when something new happens.


explore possibility of using python argparse to pass the parameters for kubectl

Current custom parser in completer.py is not really decent piece of code. Explore if it can offloaded to Python's argparse.

Ran into a bunch of troubles doing this. My initial idea was to read the command tree into a nested set of parsers created using argparse. However, argparse is not tolerant if parts of the command line don't adhere to the command tree it holds. It does a hard os.exit() which is problematic.
Going to switch back to a simple object tree representation. We will walk the tree and identify the extent of the traversal until tokens on command line are identified. The children of the node we reach point to the list of possible suggestions.

// CommandTree represents the Node (command) and its children (subcommands/options)
type CommandTree struct {
    Node string,
    Children []CommandTree
}

func Walk(t *CommandTree, parsed []string, unparsed []string) (suggestions []string) {
    token = unparsed.Pop()   // get first token
    if t.Node == token {
       parsed = append(parsed, token)
       for _, child := range(t.Children) {
            // walk all children with rest of the tokens
            Walk(child, parsed, unparsed)
       }
    } else {
        if len(unparsed) > 0 {
            suggestions := make([]string)
            for _, suggestion := range(t.Children) {
                   suggestions = append(suggestions, suggestion)
            }
            return  // suggestions come from children
        }
    }
}

I've taken the above mentioned approach and reduced the clutter in the completer.py module. There is now a parser.py (complementing a lexer.py) that reads the entire command set into a tree rooted at kubectl.  Given a command line, we walk the nodes of this tree as far as we can and return the children of the last node in the path as suggestions to fuzzyfinder.
TODO:

finish the global options parsing at any level of the subcommand
complete the get_resource(..) calls given a localFlag is used.

In case any one wants try out please check this brach


[HUDI-5250] use the estimate record size when estimation threshold is l…

Using the default value of estimate record size at the averageBytesPerRecord() when estimation threshold is less than 0
Change Logs
Currently, hudi obtains the average record size based on records written during previous commits. Used for estimating how many records pack into one file, and the code is about UpsertPartitioner.averageBytesPerRecord().
But we found that the single data file could become 600~700M and most other files are less than 200M.

Reason

the result of totalBytesWritten/totalRecordsWritten is very small when the last commit, but the next commit record is very large, then the data files will become very large.

Solve plan

Plan1: calculate avgSize of the past several commit not just only one, but the getCommitMetadata costs a lot of time, then this function might be slow, so we did not choose this.

Plan2: Use the estimated record size considering our data size is fixed in some sense, more the hudi community did not encourage adding a more boolean variable to control whether to use the last commit avgSize, then we use the estimation threshold, when it is less than 0, we use the default estimate record size.

Impact
UpsertPartitioner.averageBytesPerRecord(), small
Risk level (write none, low medium or high below)
low, this feature works only when estimation threshold is less than 0
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated

We use the previous commits' metadata to calculate the estimated record size and use it "
+ " to bin pack records into partitions.  Will use hoodie.copyonwrite.record.size.estimate value when this value is less than 0.");

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Will this change be applied to other clients?

@honeyaya : Changes look good. Can you add a test-case where getRecordSizeEstimationThreshold < 0

@honeyaya : Interested in taking this PR to completion ?


fix/pcd verifier interface

Motivation
PCD verifier use 'hard-code' URL, We should push it to parameters.
Change Summary
Change verifier interface and update test
Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review

[ ] PR title adheres to the conventional commits standard
[ ] PR has a change set
[ ] PR has been tagged with a change label(s) (i.e. documentation, feature, bug fix, or chore)
[ ] PR includes documentation if necessary.

Additional Context
If this is a relatively large or complex change, provide more details here that will help reviewers

I just wanted to have more insights on why this change, have you spotted some potential bugs?

It not really is a bug I think, I suggest change the verifier interface just for create more flexible for our SDK. When we hardcored a verify key as constant this make developers hard to make a env for testing, debug and extend.
More ever,  If we support more country identity this key will change for each country, I think push it to parameters help developers easy to control logic in source code.


rtx -ls hangs forever

Describe the bug
When I add azure-functions-core-tools = "4" to .rtx.toml, I can no longer successfully run rtx ls -current.
It always hangs indefinitely (Update: Actually, waited now longer and it just got killed after 8 minutes with exit code 137).
Neither the tool nor the plugin for azure-functions-core-tools needs to be installed. Also tried with the plugin and tool installed, but this didn't change anything.
I really have no idea why and if this is now related to my local environment, as I tried various things to understand this and at some point in time using my own fork of the azure-functions-core-tools asdf plugin.
I don't have this error, when just i.e. shfmt = "3" is in .rtx.toml, although I don't have the plugin and tool installed as well.
I only saw it with azure-functions-core-tools = "4" yet.
To Reproduce
.rtx.toml
[tools]
azure-functions-core-tools = "4"
shfmt = "3"

❯ cd ~/dev/tmp/rtx-azure-functions/not-working
[WARN] Tools not installed: azure-functions-core-tools@4, shfmt@3
❯ RTX_TRACE=1 rtx ls --current
[TRACE] (1) rtx::config::config_file::rtx_toml: [src/config/config_file/rtx_toml.rs:59] parsing: /Users/user/.config/rtx/config.toml
[TRACE] (1) rtx::plugins::rtx_plugin_toml: [src/plugins/rtx_plugin_toml.rs:29] parsing: /Users/user/.local/share/rtx/plugins/java/rtx.plugin.toml
[TRACE] (1) rtx::plugins::rtx_plugin_toml: [src/plugins/rtx_plugin_toml.rs:29] parsing: /Users/user/.local/share/rtx/plugins/ruby/rtx.plugin.toml
[TRACE] (3) rtx::config::config_file::rtx_toml: [src/config/config_file/rtx_toml.rs:59] parsing: /Users/user/.config/rtx/config.toml
[TRACE] (11) rtx::config::config_file::rtx_toml: [src/config/config_file/rtx_toml.rs:59] parsing: /Users/user/Projects/tmp/rtx-azure-functions/not-working/.rtx.toml
[TRACE] (1) rtx::config: [src/config/mod.rs:69] Settings: Settings {
    experimental: true,
    missing_runtime_behavior: Prompt,
    always_keep_download: false,
    always_keep_install: false,
    legacy_version_file: true,
    plugin_autoupdate_last_check_duration: 604800s,
    verbose: false,
    asdf_compat: false,
    jobs: 4,
    shorthands_file: None,
    disable_default_shorthands: false,
    log_level: Trace,
    raw: false,
}
[TRACE] (11) rtx::cache: [src/cache.rs:78] reading ~/Library/Caches/rtx/golang/legacy_filenames.msgpack.z
[DEBUG] Files: ~/Projects/tmp/rtx-azure-functions/not-working/.rtx.toml, ~/.config/rtx/config.toml
Installed Plugins: golang, java, ruby
[DEBUG] rtx ls --current
[TRACE] (11) rtx::cache: [src/cache.rs:78] reading ~/Library/Caches/rtx/java/aliases.msgpack.z
[TRACE] (5) rtx::cache: [src/cache.rs:78] reading ~/Library/Caches/rtx/golang/remote_versions.msgpack.z
[TRACE] (12) rtx::cache: [src/cache.rs:78] reading ~/Library/Caches/rtx/ruby/remote_versions.msgpack.z
[TRACE] (8) rtx::cache: [src/cache.rs:78] reading ~/Library/Caches/rtx/node/remote_versions.msgpack.z
[TRACE] (4) rtx::cache: [src/cache.rs:78] reading ~/Library/Caches/rtx/python/remote_versions.msgpack.z
[TRACE] (11) rtx::cache: [src/cache.rs:78] reading ~/Library/Caches/rtx/java/remote_versions.msgpack.z
[DEBUG] azure-functions-core-tools@4, shfmt@3, node@lts node@latest, python@latest!-0.1 python@latest, java@temurin-17, golang@prefix:1.20<EMAIL_ADDRESS>
HANGS here 8 minutes

❯ echo $?
137

Expected behavior
rtx ls or rtx ls --current working as expected or returning with a error message. Ideally, the error message would allow a developer to fix the plugin and to get rtx ls to work again.
rtx doctor output
❯ rtx doctor
rtx version:
  1.29.7 macos-arm64 (a540ac6 2023-05-10)

build:
  Target: aarch64-apple-darwin
  Features: BREW
  Built: Wed, 10 May 2023 16:25:07 +0000
  Rust Version: rustc 1.69.0 (84c898d65 2023-04-16)
  Profile: release

shell:
  /bin/zsh
  zsh 5.9 (x86_64-apple-darwin22.0)

rtx environment variables:
  RTX_USE_TOML=1
  RTX_SHELL=zsh

settings:
  {"experimental": "true", "missing_runtime_behavior": "prompt", "always_keep_download": "false", "always_keep_install": "false", "legacy_version_file": "true", "plugin_autoupdate_last_check_duration": "10080", "trusted_config_paths": "[\"~/sth\"]", "verbose": "false", "asdf_compat": "false", "jobs": "4", "disable_default_shorthands": "false", "log_level": "INFO", "raw": "false"}

config files:
  /Users/user/.config/rtx/config.toml
  /Users/user/dev/tmp/rtx-azure-functions/not-working/.rtx.toml

plugins:
  golang   https://github.com/kennyp/asdf-golang.git#f006a12
  java     https://github.com/rtx-plugins/rtx-java.git#24706b7
  node     (core)
  python   (core)
  ruby     https://github.com/rtx-plugins/rtx-ruby.git#e0728ce

toolset:
  azure-functions-core-tools@4, shfmt@3, node@lts node@latest, python@latest!-0.1 python@latest, java@temurin-17, golang@prefix:1.20<EMAIL_ADDRESS>
2 problems found:
plugin azure-functions-core-tools is not installed

plugin shfmt is not installed

I actually got a panic when I used this plugin, does rtx ls --json work?

this may be related to #539, I suspect github may have been timing out since it's been having a lot of issues lately

I actually got a panic when I used this plugin, does rtx ls --json work?

Yes, rtx ls --json is working:
❯ rtx ls --json
{
  "azure-functions-core-tools": [
    {
      "version": "4",
      "requested_version": "4",
      "install_path": "/Users/user/.local/share/rtx/installs/azure-functions-core-tools/4",
      "source": {
        "type": ".rtx.toml",
        "path": "/Users/user/dev/tmp/rtx-azure-functions/not-working/.rtx.toml"
      }
    }
  ],
  ...
}

this may be related to #539, I suspect github may have been timing out since it's been having a lot of issues lately

Of course, I don't know exactly, what rtx is doing for an rtx ls, but I thought it is not related to network issues.
Especially, as I don't have the azure-functions-core-tools plugin installed at all.
I thought it would just list a missing tool version. Similar to shfmt in this example. For shfmt, I also have no plugin installed.
❯ rtx ls --current
shfmt  3 (missing)     ~/dev/tmp/rtx-azure-functions/working/.rtx.toml 3

In addition, when I install the plugin, run list-all of the plugin it is quite fast. But a second later rtx ls --current hangs again. But perhaps rtx is calling sth else in addition to list-all. This is something, I wasn't able to figure out yet.
❯ cd ../not-working
[WARN] Tools not installed: azure-functions-core-tools@4, shfmt@3
❯ rtx plugins install azure-functions-core-tools
rtx azure-functions-core-tools https://github.com/daveneeley/asdf-azure-functions-core-tools.git#✓ 1s %
❯ time ~/.local/share/rtx/plugins/azure-functions-core-tools/bin/list-all
0.10 0.91 0.92 0.93 0.94 0.95 0.96 0.97 0.98 0.99 0.100 1.0 1.0.1 1.0.3 1.0.4 1.0.6 1.0.7 1.0.8 1.0.9 1.0.10 1.0.11 1.0.12 1.0.14 1.0.15 1.0.17 1.0.18 1.0.19 1.0.20 1.0.25 1.0.27 1.0.28 2.0.1-beta.8 2.0.1-beta.22 2.0.1-beta.23 2.0.1-beta.24 2.0.1-beta.25 2.0.1-beta.26 2.0.1-beta.27 2.0.1-beta.28 2.0.1-beta.29 2.0.1-beta.30 2.0.1-beta.31 2.0.1-beta.32 2.0.1-beta.33 2.0.1-beta.34 2.0.1-beta.35 2.0.1-beta.36 2.0.1-beta.37 2.0.1-beta.38 2.0.1-beta.39 2.0.2 2.0.3 2.0.1948 2.0.2359 2.1.725 2.1.746 2.1.748 2.2.32 2.2.70 2.3.148 2.3.199 2.4.249 2.4.290 2.4.299 2.4.317 2.4.401 2.4.419 2.4.498 2.4.499 2.5.553 2.6.666 2.6.1048 2.6.1064 2.6.1071 2.6.1080 2.6.1106 2.7.1149 2.7.1158 2.7.1373 2.7.1415 2.7.1468 2.7.1471 2.7.1480 2.7.1505 2.7.1524 2.7.1558 2.7.1575 2.7.1585 2.7.1633 2.7.1704 2.7.1724 2.7.1812 2.7.1846 2.7.1948 2.7.2045 2.7.2100 2.7.2184 2.7.2254 2.7.2359 2.7.2508 2.7.2628 2.7.2748 2.7.2796 2.7.2855 2.7.2883 2.7.2913 2.7.2936 2.7.3023 2.7.3188 3.0.1740 3.0.1975 3.0.2009 3.0.2106 3.0.2245 3.0.2358 3.0.2534 3.0.2630 3.0.2750 3.0.2798 3.0.2852 3.0.2881 3.0.2912 3.0.2931 3.0.2996 3.0.3160 3.0.3216 3.0.3233 3.0.3245 3.0.3284 3.0.3331 3.0.3354 3.0.3388 3.0.3442 3.0.3477 3.0.3528 3.0.3568 3.0.3734 3.0.3785 3.0.3873 3.0.3904 3.0.4378 3.0.4425 3.0.4484 3.0.4502 3.0.4585 3.0.4626 3.0.4669 3.0.4727 3.0.4753 3.0.4806 3.0.4837 3.0.4868 3.0.4899 4.0.3780 4.0.3893 4.0.3928 4.0.3971 4.0.4385 4.0.4426 4.0.4483 4.0.4544 4.0.4590 4.0.4629 4.0.4653 4.0.4670 4.0.4704 4.0.4736 4.0.4785 4.0.4829 4.0.4865 4.0.4895 4.0.4915 4.0.5030 4.0.5085 4.0.5095 4.0.5110 4.0.5148
~/.local/share/rtx/plugins/azure-functions-core-tools/bin/list-all  0.02s user 0.03s system 11% cpu 0.471 total
❯ rtx ls --current

# hangs here

Not sure, if it is relevant, but calling rtx current is working as expected:
❯ rtx current
[WARN] azure-functions-core-tools@4 is specified in ~/Projects/tmp/rtx-azure-functions/not-working/.rtx.toml, but not installed
azure-functions-core-tools 4
[WARN] shfmt@3 is specified in ~/Projects/tmp/rtx-azure-functions/not-working/.rtx.toml, but not installed
shfmt 3
node 18.16.0 20.1.0

OK, a workaround for this issue is to rename the plugin in the .rtx.toml file.
Using the following config, rtx ls and rtx ls --current immediately start working again (linking to the same azure-functions repository the azure-functions-core-tools should be using.
So I think the issue is not really plugin code itself, but perhaps related to the shorthand handling or sth like this in rtx?
Not sure.
Non-blocking .rtx.toml config:
[tools]
azure-functions = "4"
shfmt = "3"

[plugins]
azure-functions = 'https://github.com/daveneeley/asdf-azure-functions-core-tools.git'

This issue resolved itself with latest 1.30.x release. I expect #555 resolved this. Thanks.

I kind of doubt #555 had an impact, though I also don't understand why --current would have an affect. This is all it does and that bit of code should not be capable of causing a hang: https://github.com/jdxcode/rtx/blob/c6680500e0e410e577510d375c98ab80ce521a03/src/cli/ls.rs#L58
I suppose we can just wait and see if it crops up again.


Keys are not working as expected

many keys are not working properly on the editor.
Like copy paste, enter . etc are not working on an already created script

/needsMoreInfo
Please provide more details about the problem and you can follow troubleshooting steps from https://github.com/microsoft/vscode/wiki/Keybinding-Issues to gather more info or resolve problem.
Also try with all extensions disabled
You can try this with F1 and >Developer: Reload Window With Extensions Disabled

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

Hi,
I had install vim editor extension due to which is was going into insert,
normal and visual modes by some keys
I have removed that extension and that have resolved the issue.
Thanks
Regards,
Vandana
On Mon, Jan 1, 2024, 10:42 AM VSCode Triage Bot @.***>
wrote:

Thanks for creating this issue! We figured it's missing some basic
information or in some other way doesn't follow our issue reporting
guidelines https://aka.ms/vscodeissuereporting. Please take the time to
review these and update the issue.
Happy Coding!
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/201642#issuecomment-1873159466,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2GZCBLCFQU2W2YQZHN4LRTYMJATLAVCNFSM6AAAAABBIFZ5NCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZTGE2TSNBWGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

/causedByExtension
Yep, that's common with Vim extension.
Just out of curiosity - why did you install Vim extension? It is some tutorial, extension pack, or something else?

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


added GetHeapStatistics - monitor isolate heap statistics

added GetHeapStatistics method to monitor isolate heap statistics

Thanks for the contribution @mehrdadrad
Are you able to add a test for this method?

Thanks for the contribution @mehrdadrad
Are you able to add a test for this method?

@rogchap I just pushed a test pls check.


Can’t scroll tarot album left/right (JavaScript)

The JavaScript carousel album slider (for the tarot_key template) is ‘jammed’. The expected behavior is to scroll all the way back and forth from beginning to end. However right now it only budges a little bit.
The issue is better demonstrated at narrower resolutions (especially mobile)

Resolved in PR #100


BUG-103531 - Zeppelin is not respecting the absolute hdfs path for no…

What changes were proposed in this pull request?
Zeppelin was appending /user/<zeppelin_user> to user provided notebook directory even when user provides the absolute hdfs path
The issue is happening since check for 'hdfs' path is missing for zeppelin.notebook.dir.
So added the similar check which exists for zeppelin.config.dir https://github.com/apache/ambari/blob/trunk/ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py#L349
How was this patch tested?
Manually tested the fix. Manually made the changes to master.py and tested below scenarios :

HDFS path is respected when an absolute path is  provided
Path is unaltered if it starts with '/' ie when a complete path is provided
User provided path is appended with '/user/<zeppelin_user>' when the user provides a relative path.

Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2421/
Test PASSed.

Closing PR as it is over 6 months old.


fix fee_history crash

Codecov Report

Merging #192 (230a15a) into master (d7323c2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #192   +/-   ##
=======================================
  Coverage   78.85%   78.85%
=======================================
  Files          60       60
  Lines       17151    17151
=======================================
  Hits        13524    13524
  Misses       3627     3627

Impacted Files
Coverage Δ

runtime/src/lib.rs
21.99% <ø> (ø)

Continue to review full report at Codecov.

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


feat(universe.ts): add fetch release history function

:tada: This PR is included in version 4.226.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: usage of node12 which is deprecated

fixes #334.

Please undo the unrelated formatting changes.

Okay, they're undone.


Add containerd and CRI-O to glossary

2 new glossary entries:

containerd
CRI-O

Relevant to #5993. Could help with a suggestion I made for PR #14255.

NB containerd sorts after the initial-caps entries due to #7491

/lgtm

/lgtm
/approve


🤖 GitHub Bot: Rich markup error and git diff issues

Current Behavior
Currently experiencing two main errors when running the GitHub Bot:

Rich markup error:

ERROR:gptme.message:Error printing message
Traceback (most recent call last):
  File "/workspace/gptme/message.py", line 313, in print_msg
    console.print(s)
  File "/usr/local/lib/python3.10/site-packages/rich/console.py", line 1679, in print
    renderables = self._collect_renderables(
  File "/usr/local/lib/python3.10/site-packages/rich/console.py", line 1539, in _collect_renderables
    self.render_str(
  File "/usr/local/lib/python3.10/site-packages/rich/console.py", line 1429, in render_str
    rich_text = render_markup(
  File "/usr/local/lib/python3.10/site-packages/rich/markup.py", line 167, in render
    raise MarkupError(
rich.errors.MarkupError: closing tag '[/<m>]' at position 3246 doesn't match any open tag

Git diff command error:

error: unknown option `cached'
usage: git diff --no-index [<options>] <path> <path>

@ErikBjare Thank you for checking.
I am currently using the following GitHub Actions configurations:

.github/actions/bot/action.yml: https://github.com/Sunwood-ai-labs/gptme-github-bot-demo1/blob/main/.github/actions/bot/action.yml
.github/workflows/bot.yml: https://github.com/Sunwood-ai-labs/gptme-github-bot-demo1/blob/main/.github/workflows/bot.yml

Note that I have modified some parts of action.yml due to permission-related errors.
This is the setup where the git diff error you mentioned is occurring.


Fix a bug that occurred if a dbserver was unluckily.

If the dbserver was supposed to resign, the restart was unlucky.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/9382/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/9399/ new run because jenkins was failing.


[flutter_plugin_tools] Migrate firebase-test-lab to new base command

Migrates firebase-test-lab to use the new package-looping base command.
Other changes:

Extracts several helpers to make the main flow easier to follow
Removes support for finding and running *_e2e.dart files, since we no longer use that file structure for integration tests.

Part of https://github.com/flutter/flutter/issues/83413
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

Landing on "red" since submit-queue isn't updating correctly.


Enhancement request: searchable comments field

I think it would be good to have an application where users could enter text comments, and those comments could be searched. We might consider doing this as an enrichment of the data from the db, rather than by modifying the db itself, thus further demonstrating how Redis can be wrapped around a legacy system - this would also allow us to talk about durability!

Yes extending the model should be done, and comments is what I used in other demo. But did not have the time to do it yet

@TobyHFerguson


Handing unauthorized on runMethod

Fix for one of the #1889 issues:

Error handling inside rest.runMethod's method API call

Closing because it has been committed another pr: #1897


Swap out the directions in the export statements;

Description
Let's give this a shot. I've been stumped by what the issue is
considering my manual testing is working just fine. I'm thinking that
these export statements are backwards considering how they're being used
by the step named deploy_app_client_tls_steps which reads the TLS_*
variables and the not the named variables being saved in the CircleCI
environment variables.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against acd04725951b1613fd64faa29da3f449aba20235

Closing this cause we committed the changes at the same time! lol
https://github.com/transcom/mymove/commit/0922a833462e8be28210f66c9a0177257723195c


30 Nov 2021 - Financial Objects SIG Meeting Minutes

Date
Tuesday, Nov 30 2021 - 10am ET / 3pm GMT
Untracked attendees

Fullname, Affiliation, GitHub username

Meeting notices

FINOS Project leads are responsible for observing the FINOS guidelines for running project meetings. Project maintainers can find additional resources in the FINOS Maintainers Cheatsheet.

All participants in FINOS project meetings are subject to the LF Antitrust Policy, the FINOS Community Code of Conduct and all other FINOS policies.

FINOS meetings involve participation by industry competitors, and it is the intention of FINOS and the Linux Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws. Please contact<EMAIL_ADDRESS>with any questions.

FINOS project meetings may be recorded for use solely by the FINOS team for administration purposes. In very limited instances, and with explicit approval, recordings may be made more widely available.

Agenda

[ ] Convene & roll call (5mins)
[ ] Display FINOS Antitrust Policy summary slide
[ ] Review Meeting Notices (see above)
[ ] Approve past meeting minutes
[ ] Structured Products Working Group update
[ ] FINOS OSSF Feedback
[ ] FIMA - ESG Focus
[ ] ISDA Post Trade Panel - 7th Dec
[ ] AOB, Q&A & Adjourn (5mins)

Decisions Made

[ ] Decision 1
[ ] Decision 2
[ ] ...

Action Items

[ ] Action 1
[ ] Action 2
[ ] ...

WebEx info
Meeting Details
•	Meeting link: https://finos.webex.com/finos/j.php?MTID=mc56f76ae5a93c44709379b3077207f2c
•	Meeting number:<PHONE_NUMBER>
•	Password: xXGEQgJH562
•	Join by video system: Dial<EMAIL_ADDRESS>You can also dial <IP_ADDRESS> and enter your meeting number.
•	Join by phone: +1-415-655-0003 US Toll | +44-20319-88141 UK Toll | Access code:<PHONE_NUMBER>

Julia / FINOS

David Jones / ipushpull

Jon Nance / Axoni


SGDK 1.2 changes needed to build successfully

I ported my project to the new SGDK version and had to make a couple changes to gendev:

Update sega.s in the skeleton (project) to the new one
Copy sgdk-read-only/bin/lz2w.jar to $(GENDEV)/bin (BEFORE compiling libmd, it's a dependency)

There's 2 less mysteries for you to solve hopefully.

Thanks for the tips, this should all be incorporated (actually a bit ago but I've been lazy with issues)


$animator and ngShow: initial state

I am using ngShow directive to show some elements on the page, that are initially hidden. As 1.1.4 added animations, I've decided to try to add them to page and used example from documentations.
But as result when initial value is 'false' and document loaded, I see that elements, that should be hidden by ngShow are visible and classes "..-hide-setup", "...-hide-start" are been applied.
This brings to undesired behavior: hidden elements start hiding themselves instead of been hidden initially.
Here is Plunker that illustrate that situation:  http://plnkr.co/edit/67Q96KmExV78Kd84tq4g
Is this 'by design' or should be fixed? Now I am checking ngShowDirective and $animator sources to find a solution how to inform $animator that it should skip animation when it is initial value...

Thank you @robwalch !


Contact us form

Description
Contact us form should not be on every page!
It should be only on the home page and if required the contact us page!

Agreed. :+1: We can make this a separate issue


Handling run entity label for 4D sequences

Currently, ezBIDS applies a run entity to sequences in a scan session that contain the same series group ID, meaning they have the same values for the SeriesDescription, ImageType, EchoTime, and RepetitionTime metadata fields. When users reach the Dataset Review page, ezBIDS performs a volume threshold check on all 4D sequences (most commonly, func/bold). This is for instances where a func/bold sequence may not have run to completion, either due to a stimulus presentation software issue, participant asked to stop early, MRI technician needed to restart the sequence, etc. As such, the researcher is unlikely to use this data because it is too truncated be of value and likely re-runs the sequence. ezBIDS uses a simple heuristic for volume threshold (60/TR), the expected number of volumes collected in a 60-sec duration given the TR.
Here's how the run entity labels are generated, given an instance where a func/bold sequence doesn't reach the threshold:
Case 1
func/bold (task-bart_run-1)
func/bold (task-bart_run-2) [excluded, only 10 volumes]
func/bold (task-bart_run-3)
func/bold (task-bart_run-4)
func/bold (task-bart_run-5)
As we can see, run 2 was restarted, yet ezBIDS increases the run entity value. I feel the expected behavior should be:
Case 2
func/bold (task-bart_run-1)
func/bold (task-bart_run-2) [excluded, only 10 volumes]
func/bold (task-bart_run-2)
func/bold (task-bart_run-3)
func/bold (task-bart_run-4)
This way, the final run numbers for the func/bold sequences in this scan are [1,2,3,4] rather an [1,3,4,5]. The latter appears odd from a data sharing perspective. Furthermore, if users upload timing files to be converted to events.tsv, proper mapping necessitates correct matching of the run entity label.
For what it's worth, the bids-validator doesn't complain if the run labels are disjointed in (e.g, run-1, run-3), so this is mainly an issue of data organization.
@anibalsolon and @bhatiadheeraj, do you have any thoughts/comments on this?

Alternatively, users may prefer Case 1 in situations where they explicitly don't want a specific run and want to denote that (e.g., we anticipated running 5 func/bold sequences but the 2nd run had poor motion but we still conducted runs 3,4,5).

I've pushed a commit (88d25d2 that will have ezBIDS adhere to Case 2, as in my experience this is the most common way files would be ordered. This thus follows the 80/20 rule.


Add warning about C-space on Mac

Modern OS X seems to reserve C-space for toggling keyboard encodings. The VS Code plugin suggests C-c C-r as usually a reasonable replacement for C-c C-space, so I've suggested adding the same tip. I don't know enough about C-c C-r to know whether this is dangerous advice or needs more expansion, or whether it's worth trying to understand how to override the system's control of C-space entirely.

Yes, that seems fine. I’m confused about who exactly this affects as one reading of this post is that it ought to affect everyone. Possibly that was briefly true and now it only affects people with multiple keyboards enabled. In fact given that it doesn’t affect all Macs maybe this is fine to just let people Google. https://www.reddit.com/r/MacOSBeta/s/PhcqVbBc2f


Custom Widget Example

Can you please add an example to show how to create our own custom widget using ttauri?

It is a good idea.
But a lot of documentation needs to be done for this.
Although i stabilised how widgets should be used, the internals of widgets are still a bit in flux. Specifically the drawing context is still a bit inconsistent.

Another update; working on the draw_context and making all draw function consistent with each other.
I want all function to accept a quad, so transforms need not be passed. Already got that working for box and sdf  shaders,but not yet the image shader.

Another update; reworked draw_context::draw_image(). Now all draw commands except draw_text will take a quad as argument to draw inside the quad whatever its orientation or location. draw_text still accepts a transform, because it draws multiple quads inside it (one for each glyph), I believe it should remain how it works now.
The icon_widget is now a bit simpler due to it not needing knowledge of vulkan anymore.

update: looks most of the programming work is done.
Next will be to make custom widget examples and documentation.

Update: first two (non interactive) example widgets


Readme looks a bit wonky on mobile

The readme looks great on desktop screen sizes, but this is what I saw when opening the repository from my phone:

Opening this issue with the hope that there's a solution which allows us to keep the nice-looking desktop readme layout.

Addressing this in https://github.com/dfinity/portal/pull/2633


delete token

Signed-off-by: Ulrica Zhang<EMAIL_ADDRESS>
What this PR does / why we need it: delete token
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:
Release note:

ACK and review in progress.

ACK and review in progress.

@Ulricacacaca
Please pay attention to the status of this PR.
The code involved in this PR currently conflicts with the main branch.
Please resolve these conflicts.

Hi @Ulricacacaca,
I'am going to approve these changes and merge this pull request to our master branch, but here are some conflicts need to be resolved.


Add supported DLMS/COSEM devices

Devices that can work in combination with the Open Smart Grid Platform added.

 You should sign our Contributor License Agreement in order to get your pull request merged.

All is well.


Github actions

If I understand correctly the global logic (I only infer my understanding from the syntax of the workflows... As you know I haven't read the github's documentation yet), this only runs the tests on pull-requests.
I'd prefer to have tests running on everything, way before pushing anything.
Just in case: the logic we'd like is that for any workflow, running from any branch, tag, release or anything: if tests don't pass, we don't push npm packages, binaries, nor docker images nor anything.

Yes agree with that.


Bootstrap path config

This adds a bootstrap_path Behat config, in case the bootstrap/app.php file is in custom location.

+1 :+1:

Can this happen?

It would be good if this could come true :)

I no longer use or maintain this package. Would one of you like to take over management?

I will have to pull this down meege it and review since.
Let me get to that later this week
Basically I will try and get travis in place as well for a faster PR workflow

This would be really useful to have! It seems like there's an option for it here, but I can't trace it well enough and it doesn't seem to work for me.


get all feeds

how I can get the expleore feed?
Example:
https://pasteboard.co/IVe6N05.jpg

Use ig.feed.discover().

Use ig.feed.discover().
thnks but is not  get feeds
please check this
https://pasteboard.co/IVe6N05.jpg

The explore feed isn't implemented yte. You can open a PR and add it.

@Nerixyz The explore feed is actually implemented, but maybe not fully. If you call ig.discover.topicalExplore() that returns the explore feed, but the issue is that it always returns the same results. Any idea why that's the case?

is actually implemented,

I know, but it's only for prefetching and not as a feed.

@Nerixyz Ohh I see. Thanks for the quick response.


HHH-17388 - Add comment @Where annotation only works in persist

https://hibernate.atlassian.net/browse/HHH-17388

This proposed documentation change is not correct.

@gavinking
I discovered while debugging between versions, the problem seems to have been solved with Hibernate 6.0's support for SQM in HQL and the evolution of AST!
I think this change will have a side effect on projects in pre-6.0 environments that were using @Where annotation.
Please note. Thanks 👍


Better error on input type missmatch (fix #459)

As instructed in #459 i made changes. I don't know if tests are in correct place - if need to be differ please tell me.

Closing in favour of a1dc96f.


Fixed grammar error

Changed line 49 from "that: groups.etcd|length is not divisibleby 2" to "that: groups.etcd|length is not divisible by 2".

CLA signed.

Think I misinterpreted the syntax being used in the Ansible play.