Closed avishniakov closed 3 weeks ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
Package | New capabilities | Transitives | Size | Publisher |
---|---|---|---|---|
pypi/numpy@2.0.0 | environment, eval, filesystem, shell, unsafe | 0 |
102 MB | charlesr.harris, matthew.brett, mattip, ...2 more |
Also fixing tenacity
update error (as discussed in https://github.com/jd/tenacity/issues/471).
I'd recommend enabling the ruff
rule NPY201 ahead of attempting to upgrade to numpy
2.0, it will catch some code pieces that will break upon upgrading (and more often than not can be fixed in a numpy
1.x compatible way)
LGTM, but do we want to definitively delete the lines from the
integration-test-slow.yml
instead of just commenting out?
Yeah, will do so, once CI is green
Describe changes
Temporarily pin numpy to versions below 2. Support for numpy 2.0 will be done in the course of other PRs.
Pre-requisites
Please ensure you have done the following:
develop
and the open PR is targetingdevelop
. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.Types of changes