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

refactor(netcdf): updates focused on flopy parity #2163

Draft
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

mjreno
Copy link
Contributor

@mjreno mjreno commented Jan 24, 2025

Adjustment to model NetCDF input file creation that align with FloPy #2417:

  • Fix DIS mesh_face_x array, which was off by an index
  • Rotate mesh x and y coordinates when angle of rotation has been specified
  • change global attributes "modflow6_grid" and "modflow6_model" to "modflow_grid" and "modflow_model", as these identifiers might be more generally applied
  • After FloPy PR is merged, a follow up PR will update autotests to use new functionality

Checklist of items for pull request

  • Replaced section above with description of pull request
  • Added new test or modified an existing test
  • Ran ruff on new and modified python scripts in .doc, autotests, doc, distribution, pymake, and utils subdirectories.
  • Formatted new and modified Fortran source files with fprettify
  • Updated develop.tex with a plain-language description of the bug fix, change, feature; required for changes that may affect users
  • Removed checklist items not relevant to this pull request

For additional information see instructions for contributing and instructions for developing.

@mjreno mjreno marked this pull request as draft January 24, 2025 19:00
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.

1 participant