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

Fix another Plexon overflow + Limit Doc build to NumPy < 2.0 while we finish updating individual IOs #1612

Merged
merged 6 commits into from
Dec 13, 2024

Conversation

zm711
Copy link
Contributor

@zm711 zm711 commented Dec 13, 2024

@alejoe91 I found another plexon overflow. could you merge this one. This is the problem with the documentation.

cc @h-mayorquin (this will fix the PR. I found another overflow).

@zm711
Copy link
Contributor Author

zm711 commented Dec 13, 2024

Okay so the problem was casting behavior with numpy 2.0. So for docs I limited to < 2.0 because we depend on IOs and we have not updated the IOS yet. This is ready to go when tests pass @alejoe91. Heberto and I did this together to figure out it was a dependency issue. And this reminds me that each individual IO will need additional confirmation to be Numpy 2.0 ready.

@zm711 zm711 changed the title Fix another Plexon overflow Fix another Plexon overflow + Limit Doc build to < 2.0 while we finish updating individual IOs Dec 13, 2024
@zm711 zm711 changed the title Fix another Plexon overflow + Limit Doc build to < 2.0 while we finish updating individual IOs Fix another Plexon overflow + Limit Doc build to NumPy < 2.0 while we finish updating individual IOs Dec 13, 2024
@alejoe91 alejoe91 merged commit 9fd9ef1 into NeuralEnsemble:master Dec 13, 2024
72 checks passed
@zm711 zm711 deleted the plexon-overflow-2-fix branch December 13, 2024 22:39
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