Skip to content

feat(dis_properties): add grid method to get discretization properties - #2795

Open
jlarsen-usgs wants to merge 5 commits into
modflowpy:developfrom
jlarsen-usgs:develop
Open

feat(dis_properties): add grid method to get discretization properties#2795
jlarsen-usgs wants to merge 5 commits into
modflowpy:developfrom
jlarsen-usgs:develop

Conversation

@jlarsen-usgs

Copy link
Copy Markdown
Contributor
  • returns dictionary of keyword arguments to build DIS, DISV, and DISU depending on grid type
  • added cell .area calculation via shoelace algorithm to Grid
  • remove deprecated flopy.mf6.utils/reference.py which housed "pre-modelgrid" spatial reference support for MF6 models

Mirrors support outlined in removed PR #2700

* replace numpy .shape = with .reshape() in Raster
* replace .utcfromtimestamp with fromtimestamp(timestamp, timezone.utc) in ModelTime
* numpy will be deprecating `.shape =` reshaping, migrating to reshape convention to prevent DeprecationWarnings
* returns dictionary of keyword arguments to build DIS, DISV, and DISU depending on grid type
* added cell `.area` calculation via shoelace algorithm to `Grid`
* remove deprecated flopy.mf6.utils/reference.py which housed "pre-modelgrid" spatial reference support for MF6 models
@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 55.10204% with 22 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.2%. Comparing base (556c088) to head (6375f7e).
⚠️ Report is 190 commits behind head on develop.

Files with missing lines Patch % Lines
flopy/discretization/unstructuredgrid.py 57.1% 9 Missing ⚠️
flopy/discretization/grid.py 20.0% 8 Missing ⚠️
flopy/discretization/structuredgrid.py 66.6% 4 Missing ⚠️
flopy/discretization/vertexgrid.py 83.3% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2795      +/-   ##
===========================================
+ Coverage     55.5%    73.2%   +17.7%     
===========================================
  Files          644      660      +16     
  Lines       124135   132182    +8047     
===========================================
+ Hits         68947    96886   +27939     
+ Misses       55188    35296   -19892     
Files with missing lines Coverage Δ
flopy/discretization/vertexgrid.py 80.1% <83.3%> (-3.6%) ⬇️
flopy/discretization/structuredgrid.py 54.1% <66.6%> (+6.7%) ⬆️
flopy/discretization/grid.py 74.5% <20.0%> (-1.4%) ⬇️
flopy/discretization/unstructuredgrid.py 80.5% <57.1%> (-1.0%) ⬇️

... and 581 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jlarsen-usgs

Copy link
Copy Markdown
Contributor Author

PR will close
#2388
#2661

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