Skip to content

TST: Do not test for signalling NaNs not raising warnings#19477

Closed
seberg wants to merge 1 commit intonumpy:mainfrom
seberg:adjust-signalling-nan-tests
Closed

TST: Do not test for signalling NaNs not raising warnings#19477
seberg wants to merge 1 commit intonumpy:mainfrom
seberg:adjust-signalling-nan-tests

Conversation

@seberg
Copy link
Copy Markdown
Member

@seberg seberg commented Jul 14, 2021

This is technically even incorrect in IEEE, signalling NaNs should
pretty much always give a warning. But overall, it seems unlikely
NumPy should bother about it (we never create them after all).

This is split off from gh-19476. Mainly, to check whether s390x is
broken...

This is technically even incorrect in IEEE, signalling NaNs should
pretty much always give a warning.  But overall, it seems unlikely
NumPy should bother about it (we never create them after all).

This is split off from numpygh-19476.  Mainly, to check whether s390x is
broken...
@seberg
Copy link
Copy Markdown
Member Author

seberg commented Jul 14, 2021

Nvm, I missed that the test was written in a way so that it would only be valid on little-endian machines.

@seberg seberg closed this Jul 14, 2021
@seberg seberg deleted the adjust-signalling-nan-tests branch July 14, 2021 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant