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

Feat predefined stress periods #73

Merged
merged 8 commits into from
Oct 17, 2023

Conversation

aleaf
Copy link
Collaborator

@aleaf aleaf commented Oct 14, 2023

This PR adds the ability to pre-define stress periods in a CSV table. Pages describing time and space discretization input and interpolation methods were also added to the docs.

 * Add automatic writing of lake polygon and lake cell connection shapefiles
 * Add name_column arg to shapefile input (which adds names to lake auxiliary tables)
 * Allow PRISM input to be specified for all lakes (via single filename instead of dictionary)
 * Allow specification of lakes_shapefile: without include_ids: item
 * Move output tables to tables/ folder
… be specified for evaluating np.any along the 0th axis; the returned scalar 'True' was resulting in no head observation input getting written
…s can be set to fail if invalid arguments are passed, with the valid args printed
…me and end_datetime cols to datetime64[ns] so that pd.concat doesn't fail with heterogenous dtypes
…weird apparent (and hopefully transient) shapely/GEOS bug with error message about MultiPolygon linear rings not being closed
…defined in a CSV file, which allows for more complicated or irregular stress period configurations that would otherwise require many group blocks; for example 7-day periods that always start on the same day of the year, which results in an extra period of 1 or 2 days at the end of each year.
@codecov-commenter
Copy link

Codecov Report

Merging #73 (1d6aca1) into develop (3a6ec42) will decrease coverage by 0.18%.
Report is 4 commits behind head on develop.
The diff coverage is 87.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@             Coverage Diff             @@
##           develop      #73      +/-   ##
===========================================
- Coverage    89.14%   88.96%   -0.18%     
===========================================
  Files           64       64              
  Lines        12305    12378      +73     
===========================================
+ Hits         10969    11012      +43     
- Misses        1336     1366      +30     
Files Coverage Δ
mfsetup/__init__.py 100.00% <100.00%> (ø)
mfsetup/mf6model.py 97.72% <100.00%> (ø)
mfsetup/tdis.py 87.46% <100.00%> (+0.95%) ⬆️
mfsetup/tests/test_mf6_tmr_shellmound.py 63.22% <100.00%> (+0.77%) ⬆️
mfsetup/tests/test_pfl_mfnwt_inset.py 97.19% <100.00%> (+<0.01%) ⬆️
mfsetup/utils.py 67.70% <66.66%> (-0.38%) ⬇️
mfsetup/grid.py 86.56% <50.00%> (-0.30%) ⬇️
mfsetup/mfmodel.py 89.75% <72.72%> (-0.44%) ⬇️
mfsetup/lakes.py 93.72% <82.50%> (-1.30%) ⬇️

... and 4 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@aleaf aleaf merged commit c278ba2 into DOI-USGS:develop Oct 17, 2023
1 of 5 checks passed
@aleaf aleaf deleted the feat_predefined_stress_periods branch October 17, 2023 19:50
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.

2 participants