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

test(prt): omit name field (object dtype) from snapshots #1887

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

wpbonelli
Copy link
Contributor

@wpbonelli wpbonelli commented Jun 19, 2024

NumPy 2.0.0 changes the way arrays with object dtype are pickled: https://numpy.org/devdocs/release/2.0.0-notes.html#highlights. Workaround is to keep snapshots numeric and check string/object columns explicitly when needed: MODFLOW-USGS/modflow-devtools@1883647.


Checklist of items for pull request

  • Added new test or modified an existing test
  • Ran black on new and modified autotests
  • Removed checklist items not relevant to this pull request

@wpbonelli wpbonelli marked this pull request as ready for review June 19, 2024 03:53
@wpbonelli wpbonelli merged commit 22c0fd1 into MODFLOW-USGS:develop Jun 19, 2024
18 checks passed
@wpbonelli wpbonelli deleted the numpy2-fixes branch June 19, 2024 04:03
wpbonelli added a commit to modflowpy/flopy that referenced this pull request Jun 19, 2024
Remove NumPy upper bound to close #2153. Also fix `@requires_pkg` usage in `test_export.py`, replace deprecated `itemset` in `mfdatastorage.py`, and bump some CI jobs to Python 3.9.

This PR required updates on the MF6 side:

* test(test_gwf_maw04.py): switch deprecated np.unicode_ -> np.str_ MODFLOW-USGS/modflow6#1886
* test(prt): omit name field (object dtype) from snapshots MODFLOW-USGS/modflow6#1887
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.

None yet

1 participant