-
Notifications
You must be signed in to change notification settings - Fork 717
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
{data}[gfbf/2023a] polars v0.19.19 #19770
{data}[gfbf/2023a] polars v0.19.19 #19770
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Test report by @lexming |
Test report by @lexming |
…asyconfigs into 20240201120931_new_pr_polars01919
@boegelbot: please test @ generoso |
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1936697091 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@lexming failing checksum for |
@boegel Indeed, |
Superseeded by #20275 |
@akesandgren I would still like to get this merged, we are using this version in our clusters |
@boegelbot: please test @ generoso |
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2038453176 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
the bot cannot use the |
@boegelbot please test @ jsc-zen3 |
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2057490882 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
I cannot reproduce the failures with the checksum of I'll see if can inspect the tarball in the bot systems. |
@boegelbot please test @ generoso |
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2077547550 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegel found the issue with all the failed tests for this PR, fix in easybuilders/easybuild-framework#4517 |
c78afea
to
3c579a9
Compare
@boegelbot please test @ generoso |
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2079788168 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
The update: actually
update 2: support for branches seems to be better on jsc-zen3, trying that one... |
@boegelbot please test @ jsc-zen3 |
@lexming: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2079826826 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @lexming |
@boegelbot please test @ jsc-zen3 |
@lexming: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2084534051 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @lexming |
@boegel all tests are good with most recent |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
(created using
eb --new-pr
)This PR is for v0.19.19 instead of the more recent v0.20.x because the latter need nightly features in Rust that are not yet available in Rust v1.75.
Depends on:
tar
command used inget_source_tarball_from_git
to get reproducible tarballs easybuild-framework#4248Cargo
easyblock to extract crates into a vendor subdir and overwrite git repo URLs with local paths easybuild-easyblocks#3118