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

ENH: Transition to Python 3.10 #59

Merged
merged 1 commit into from
Jul 1, 2023

Conversation

jhlegarreta
Copy link
Contributor

@jhlegarreta jhlegarreta commented May 6, 2023

Transition to Python 3.10.

Bump scilpy to eb5d3fb and upgrade dependencies accordingly.

Remove the dependencies force-upgrade in the package testing workflow,
as the versions required by scilpy are now modern enough to avoid the
primary issues they intended to fix.

Specifically, fix the BLAS option warning in numpy triggered by
python-spams when using an earlier version of numpy. Fixes:

AttributeError: module 'numpy.__config__' has no attribute 'blas_opt_info'

ERROR: Failed building wheel for python-spams

reported in:
https://github.com/scil-vital/tractolearn/actions/runs/4901866945/jobs/8753373506#step:5:945

Also, avoid using the scilpy-patched dipy version installation
issues reporting Cython is not being found by installing a more recent
version of scilpy using the commit hash. Fixes

RuntimeError: Need cython>=0.29.24 to build extensions but cannot import "Cython"

repored in
https://github.com/scil-vital/tractolearn/actions/runs/5384681010/jobs/9772886628#step:5:996

@github-actions github-actions bot added type:infrastructure Infrastructure/ecosystem-related changes type:enhancement New feature or request labels May 6, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels May 6, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels May 6, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels May 6, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels May 6, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels May 6, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels May 6, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels May 6, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels May 6, 2023
@jhlegarreta
Copy link
Contributor Author

OK, all failed attempts maybe due to scilpy having transitioned to Python 3.10. So we either try to fix the python-spams-related issue without upgrading or we upgrade to Python 3.10.

@github-actions github-actions bot added the type:documentation Improvements or additions to documentation label May 20, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels May 21, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels May 21, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels Jun 27, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels Jun 27, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels Jun 27, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels Jun 27, 2023
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels Jul 1, 2023
Transition to Python 3.10.

Bump `scilpy` to eb5d3fb and upgrade dependencies accordingly.

Remove the dependencies force-upgrade in the package testing workflow,
as the versions required by `scilpy` are now modern enough to avoid the
primary issues they intended to fix.

Specifically, fix the BLAS option warning in `numpy` triggered by
`python-spams` when using an earlier version of `numpy`. Fixes:
```
AttributeError: module 'numpy.__config__' has no attribute 'blas_opt_info'

ERROR: Failed building wheel for python-spams
```

reported in:
https://github.com/scil-vital/tractolearn/actions/runs/4901866945/jobs/8753373506#step:5:945

Also, avoid using the `scilpy`-patched `dipy` version installation
issues reporting Cython is not being found by installing a more recent
version of `scilpy` using the commit hash. Fixes

```
RuntimeError: Need cython>=0.29.24 to build extensions but cannot import "Cython"
```

repored in
https://github.com/scil-vital/tractolearn/actions/runs/5384681010/jobs/9772886628#step:5:996
@github-actions github-actions bot added type:enhancement New feature or request and removed type:enhancement New feature or request labels Jul 1, 2023
@jhlegarreta jhlegarreta changed the title ENH: Bump scilpy to version 1.5.0 and upgrade dependencies ENH: Transition to Python 3.10 Jul 1, 2023
@jhlegarreta
Copy link
Contributor Author

Going ahead and merging @felixdumais.

@jhlegarreta jhlegarreta merged commit 2f94df0 into scil-vital:main Jul 1, 2023
@jhlegarreta jhlegarreta deleted the UpgradeScilpy branch July 1, 2023 18:35
@jhlegarreta
Copy link
Contributor Author

Using a recent commit hash for SCILPY is also motivated by the fact that new versions are released regularly, and given that the DIPY hot fix is no longer necessary for the chosen recent commit hash, it is not worthwhile spending more time on it, and choosing a commit close to a soon-to-be-released newer SCILPY version is a reasonable choice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:documentation Improvements or additions to documentation type:enhancement New feature or request type:infrastructure Infrastructure/ecosystem-related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant