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

PRT with quadtree and voronoi grids #1434

Closed
rhugman opened this issue Nov 9, 2023 · 4 comments
Closed

PRT with quadtree and voronoi grids #1434

rhugman opened this issue Nov 9, 2023 · 4 comments
Labels

Comments

@rhugman
Copy link

rhugman commented Nov 9, 2023

In regard to #1389. I've been experimenting with PRT in @aprovost-usgs 's fork and hit (what I think are) a few bugs for DISV grids.

This is for a case with high conductivity in a few cells, lots of layers and pass-through cells in some of them (idomain==-1). I have not been able to generate a synthetic case that replicates the issue. Can provide versions of the model files if it helps.

Describe the bug

With a GRIDGEN generated quadtree grid, some particle locations (x,y) are recorded outside the model domain for some tstps, but then "bounce" back. The simulation completes successfully:
image

Same model, but using a Voronoi grid, MF6 crashes with the following error message:

image

If using ATS, this message is printed to the command line, then eventually crashes with the same issue above:

image

Environment

  • Windows11
  • VERSION 6.5.0.dev0+prt (preliminary) 10/07/2023
  • MODFLOW 6 compiled Oct 07 2023 02:19:02 with Intel(R) Fortran Intel(R) 64
    Compiler Classic for applications running on Intel(R) 64, Version 2021.7.0
    Build 20220726_000000
@rhugman rhugman added the bug label Nov 9, 2023
@aprovost-usgs
Copy link
Contributor

@rhugman Thank you for your interest in PRT and for reporting the issue. We believe the problems stem from the fact that the PRT tracking algorithm is not currently formulated to handle adjacent cell faces that meet at a 180-degree angle, and this is exactly the situation that arises from quadtree refinement, as well as (somewhat surprisingly) in Voronoi grids generated by flopy. (We'll update the description in #1389 to be more explicit about that.) We believe we have a solution to that limitation of PRT and are planning to update the algorithm prior to release, but we're in the early stages of that effort. Thanks for offering to provide your model files - they might be helpful for testing once we get the algorithm updated. Please feel free to email me directly at [email protected], and we can arrange for an efficient transfer of the files.

@rhugman
Copy link
Author

rhugman commented Nov 9, 2023

thanks @aprovost-usgs; I'll be in touch via email.

@aprovost-usgs
Copy link
Contributor

@rhugman Thank you

@langevin-usgs
Copy link
Contributor

Am closing this as it is being handled separately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants