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

Add Fracture Functionality #26

Open
inkenbrandt opened this issue Jul 9, 2019 · 3 comments
Open

Add Fracture Functionality #26

inkenbrandt opened this issue Jul 9, 2019 · 3 comments

Comments

@inkenbrandt
Copy link

inkenbrandt commented Jul 9, 2019

This is more of a suggested improvement rather than an issue. In their 2012 paper (https://pdfs.semanticscholar.org/5a91/b7e3dc884928d1923bf6be28228bee42ae44.pdf) Marin, Wendland, and Strack show how fractures can be added to Analytic Element Models. If one of the elements, like inhomogeneity1d, already satisfies this request, please disregard this issue.

@mbakker7
Copy link
Owner

mbakker7 commented Jul 9, 2019

Thanks for the suggestion. Do I understand it correctly that the fractures are modeled with a constant width and uniform hydraulic conductivity? What would you suggest for vertical exchange between layers? Currently, the LineSinkDitch and LineSinkDitchString can do highly conductive fractures where the head is uniform in the fracture.

@inkenbrandt
Copy link
Author

inkenbrandt commented Jul 9, 2019

I believe your assumptions are correct in terms of width and conductivity. Unfortunately, the vertical exchange is not considered in the paper! Maybe @ispmarin can help?

This looks like the repo to start with: https://github.com/ispmarin/academic/tree/master/phd/pyaem

@ispmarin
Copy link

Hello @inkenbrandt, it's been a while. The paper does not address the vertical exchange, and as far as I know this fracture model is not capable of simulating it in its current form. Maybe you can find a way to integrate the vertical discharge on Strack's book (https://www.amazon.com/Analytical-Groundwater-Mechanics-Otto-Strack/dp/1107148839).

It would be great if you you could integrate this solution on the PyAEM code that Bakker provided above!

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

No branches or pull requests

3 participants