Skip to content

Conversation

emorway-usgs
Copy link
Contributor

MT3DMS and MT3D-USGS support NSS = -1, flopy should as well

addresses #743

…for multiple lakes. This should account for that condition
…versions of MT3D-USGS, but didn't make the final cut and was never removed from flopy until now. Fixes and closes issue #423
# Conflicts:
#	examples/Notebooks/flopy3_mt3d-usgs_example_with_sft_lkt_uzt.ipynb
Uzt instantiation code was initially copied from SFT, and the two deleted lines are for SFT, not UZT
Instantiation of UZT class was generating IUZBND as a 3D array, should be 2D
When a single value and a user comment appears on uzgag line, wrong len() calculated
MT3D-USGS and MT3DMS source codes allow for -1, flopy should too

(#743)
@coveralls
Copy link

coveralls commented Dec 5, 2019

Coverage Status

Coverage decreased (-41.8%) to 15.25% when pulling eb5c11d on emorway-usgs:new_develop into aa4b667 on modflowpy:develop.

@christianlangevin
Copy link

@emorway-usgs, do you have a simple test you can add so we see how this works?

@christianlangevin
Copy link

This PR is no longer needed. The fix for this issue was implemented in 51d8064

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.

3 participants