Skip to content

Bump uv to make CI run with the new metatensor releases#1146

Merged
pfebrer merged 5 commits into
mainfrom
pfebrer-patch-2
May 18, 2026
Merged

Bump uv to make CI run with the new metatensor releases#1146
pfebrer merged 5 commits into
mainfrom
pfebrer-patch-2

Conversation

@pfebrer
Copy link
Copy Markdown
Contributor

@pfebrer pfebrer commented May 18, 2026

Trying to fix recent CI failures, which are apparently related to pytorch/pytorch#165448

We had uv pinned, which installed python 3.13.8 when the python version was "3.13". This python version has problems with torch >= 2.11. For some reason the CI in main is using torch==2.10 at most and this is why we were not seeing any problem. However, the dependency bump in #1144 makes CI use torch >= 2.11 and therefore the CI was failing.

This PR bumps both the setup-uv action and uv itself, which we leave unpinned (latest version is used) so that we don't get problems from old uvs in the future.


📚 Documentation preview 📚: https://metatrain--1146.org.readthedocs.build/en/1146/

@pfebrer
Copy link
Copy Markdown
Contributor Author

pfebrer commented May 18, 2026

@HaoZeke do you know why you left the step of setting up python in tests.yaml when you added uv?

@sofiia-chorna
Copy link
Copy Markdown
Collaborator

thanks a lot for fixing! i was afraid if it is something with release 😁

@pfebrer
Copy link
Copy Markdown
Contributor Author

pfebrer commented May 18, 2026

It's still not fixed, it doesn't fail in main because it installs torch==2.10, but CI still uses python 3.13.8 which will make your PR fail 😅

@pfebrer
Copy link
Copy Markdown
Contributor Author

pfebrer commented May 18, 2026

Now it is using python 3.13.13, this will solve the failure in your PR in principle 👍

@pfebrer pfebrer changed the title Remove setup python from CI Bump uv to make CI run with the new metatensor releases May 18, 2026
@pfebrer pfebrer requested a review from Luthaf May 18, 2026 10:40
@pfebrer
Copy link
Copy Markdown
Contributor Author

pfebrer commented May 18, 2026

cscs-ci run

Copy link
Copy Markdown
Member

@Luthaf Luthaf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look good to me

@pfebrer pfebrer enabled auto-merge (squash) May 18, 2026 11:33
pfebrer added 2 commits May 18, 2026 14:47
Updated the tests workflow to install additional dependencies.
@pfebrer
Copy link
Copy Markdown
Contributor Author

pfebrer commented May 18, 2026

cscs-ci run

@pfebrer pfebrer merged commit 687aeb8 into main May 18, 2026
21 checks passed
@pfebrer pfebrer deleted the pfebrer-patch-2 branch May 18, 2026 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants