search

yt-project / yt

Main yt repository
http://yt-project.org
Other
454 stars 272 forks source link

Do Not Merge: Checking enzo answer fail with np >= 2 #4927

Closed chrishavlin closed 3 weeks ago

chrishavlin commented 3 weeks ago

This PR is to double check my comment here https://github.com/yt-project/yt/pull/4874#issuecomment-2168530832 for why a couple of enzo answer tests were failing with the latest numpy. This PR should not be merged -- IMO it is more appropriate to bump the answer tests, but this PR will confirm that the change in type promotion with np 2.0 is indeed the cause for the failures.

This PR branch is off of the branch for https://github.com/yt-project/yt/pull/4874

chrishavlin commented 3 weeks ago

passed! the one failure in the py3.12 windows test is unrelated.