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

Hotfix #5

Closed
wants to merge 3 commits into from
Closed

Hotfix #5

wants to merge 3 commits into from

Conversation

KumoLiu
Copy link
Owner

@KumoLiu KumoLiu commented Jun 25, 2024

Fixes # .

Description

A few sentences describing the changes proposed in this pull request.

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • New tests added to cover the changes.
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.
  • In-line docstrings updated.
  • Documentation updated, tested make html command in the docs/ folder.

ericspod and others added 3 commits June 24, 2024 11:48
### Description

This fixes the version of Numpy to be less than 2.0 so that we can get a
working setup guaranteed. Numpy 2.0 isn't supported yet by MONAI and
will need a number of changes to be compatible.

This is only sometimes a problem because a number of dependencies we
have require Numpy<2.0. When these are absent however, such as with
minimal installs, Numpy 2.0 can be installed and causes problems. This
is a temporary fix until we have worked out an update that's compatible
with versions before and after 2.0.

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [ ] Breaking change (fix or new feature that would cause existing
functionality to change).
- [ ] New tests added to cover the changes.
- [ ] Integration tests passed locally by running `./runtests.sh -f -u
--net --coverage`.
- [ ] Quick tests passed locally by running `./runtests.sh --quick
--unittests --disttests`.
- [ ] In-line docstrings updated.
- [ ] Documentation updated, tested `make html` command in the `docs/`
folder.

Signed-off-by: Eric Kerfoot <[email protected]>
…ply.github.com>

I, Eric Kerfoot <[email protected]>, hereby add my Signed-off-by to this commit: 6a64249

Signed-off-by: Eric Kerfoot <[email protected]>
I, Eric Kerfoot <[email protected]>, hereby add my Signed-off-by to this commit: 589f1b4

Signed-off-by: Eric Kerfoot <[email protected]>
@KumoLiu KumoLiu deleted the branch releasing/1.3.2 June 25, 2024 14:30
@KumoLiu KumoLiu closed this Jun 25, 2024
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.

2 participants