Skip to content
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

Numpy 2.0 allowed in pyproject.toml #8306

Merged
merged 4 commits into from
Dec 9, 2024
Merged

Numpy 2.0 allowed in pyproject.toml #8306

merged 4 commits into from
Dec 9, 2024

Conversation

Ipuch
Copy link
Contributor

@Ipuch Ipuch commented Dec 3, 2024

Related

What

A very small commit to see if CI still pass with Numpy 2.0 and over allowed, as python projects are moving quickly to it.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hi! Thanks for opening this pull request.

Because this is your first time contributing to this repository, make sure you've read our Contributor Guide and Code of Conduct.

@abey79 abey79 added dependencies concerning crates, pip packages etc include in changelog labels Dec 4, 2024
@@ -14,7 +14,7 @@ classifiers = [
dependencies = [
# Must match list in `.github/workflows/reusable_test_wheels.yml`
"attrs>=23.1.0",
"numpy>=1.23,<2",
"numpy>=1.23",
Copy link
Member

Choose a reason for hiding this comment

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

This invalidates the pixi.lock file, which explains the CI failure. Could you run any pixi command (e.g. pixi run rerun) to update the lock file and push it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@teh-cmc
Copy link
Member

teh-cmc commented Dec 5, 2024

@rerun-bot approve

@abey79
Copy link
Member

abey79 commented Dec 5, 2024

It looks like you are using a pixi version that is newer than the one we currently have in CI. I'll check if we can migrate to the new version to make this PR mergeable.

@abey79
Copy link
Member

abey79 commented Dec 5, 2024

@Ipuch
Copy link
Contributor Author

Ipuch commented Dec 5, 2024

Can I update my branch?

@abey79
Copy link
Member

abey79 commented Dec 5, 2024

Can I update my branch?

Yes, should be good now.

@Ipuch
Copy link
Contributor Author

Ipuch commented Dec 6, 2024

C++ CI failed : The error in the logs is: final link failed: No space left on device.

@abey79 abey79 merged commit b3e35a3 into rerun-io:main Dec 9, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies concerning crates, pip packages etc include in changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants