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

Revert #1154 (Fix NumPy DocTests) #1180

Conversation

pseudo-rnd-thoughts
Copy link
Member

Description

Reverts #1153 which had reverted #1144, why the NumPy string formats are not working is strange and unexplained as I don't believe it is directly related to NumPy releases

@Kallinteris-Andreas
Copy link
Collaborator

Hello Darkness, my old friend

@Kallinteris-Andreas
Copy link
Collaborator

Kallinteris-Andreas commented Sep 29, 2024

Ok, so it appears to pass CI for numpy<2 but fail for numpy>=2 (from what I can tell)

image

since we will be dropping py3.8 soon, we can also drop numpy<2

@pseudo-rnd-thoughts
Copy link
Member Author

Ohh, doctest is disabled for 3.8 which is why it passed.

The reason I closed the PR is that reverted the changes, the PR failed on all the changes.
So I reran the main CI that had just failed on doctest and it passed just two days without any changes.

This really is a mystery to understand what is causing the problem.

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