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

CSV_OUTPUT for advanced stress packages #357

Closed
tandreasr opened this issue Mar 23, 2020 · 3 comments
Closed

CSV_OUTPUT for advanced stress packages #357

tandreasr opened this issue Mar 23, 2020 · 3 comments
Assignees

Comments

@tandreasr
Copy link

Hi Chris,
do you see any chance to provide an output of the convergence history for SFR, LAK & MAW packages similiar to the one provided by the CSV_OUTPUT option of the IMS package?

In my opinion this is quite important in order to obtain information, why a certain advanced package doesn't coverge.

Currently only the last value of head and residual change are written to list file and this only, if they exceed the limits, e.g. in routine sfr_cc().

Thanks and best regards
Andreas

@langevin-usgs
Copy link
Contributor

Hi @tandreasr, I can see why this might be beneficial. Most of the convergence problems that we are seeing are due to one of the advanced stress packages (MAW, LAK, SFR, UZF). As you know, the _cc() routines are called within the picard iteration loop of the IMS solver, so that would be one logical place to write convergence history if requested by the user.

Our problem at the moment is that we have a number of ongoing MODFLOW 6 efforts that are taking most of our time. Once we are further along with those, we can start to work on some of these feature requests. Alternatively, you are welcome to submit a pull request if you feel that this is something you can implement yourself. You can contact me directly if you'd like to discuss this further.

@tandreasr
Copy link
Author

Hi Chris,
thanks for your fast answer.
Unfortunately I haven't got the time to imerge myself deeper into the MODFLOW6 sources,
because such a change would require more than just a view lines of code
(flags, output options, proper documentation...).

So I'm just waiting until you've got time left for those requests :-)

Thanks
Andreas

@jdhughes-usgs jdhughes-usgs self-assigned this Apr 2, 2020
@jdhughes-usgs
Copy link
Contributor

closed by 19cfad2 and a3246bb

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

No branches or pull requests

3 participants