-
Notifications
You must be signed in to change notification settings - Fork 4
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
Remove CentOS 7 and pin NumPy version #70
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JCGoran : Is CentOS 7 causing trouble in CI or it's still working OK?
The reason we have kept this is that we have our own users on CentOS/RH7. BB5 is still RH7! :)
@pramodk it's causing problems in the build CI: https://github.com/neuronsimulator/nrn-build-ci/actions/runs/10055747236/job/27793876304 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Readme should be changed, too.
CentOS 7 reached EOL on June 30th, 2024. Also pin NumPy because NMODL tries to upgrade it.
EDIT: pinning makes no difference for 8.2.5 since it stubbornly tries to install NumPy 2 later, which is why we need to ship 8.2.6.