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

Bug in AdTpfaFlux #1104

Closed
keileg opened this issue Jan 22, 2024 · 3 comments · Fixed by #1111
Closed

Bug in AdTpfaFlux #1104

keileg opened this issue Jan 22, 2024 · 3 comments · Fixed by #1111
Assignees

Comments

@keileg
Copy link
Contributor

keileg commented Jan 22, 2024

There seems to be a bug in the AdTpfaFlux mixin class related to Neumann conditions on internal domains.

A likely partial cause is that, in AdTpfaFlux.diffusive_flux, filtering on Neumann conditions is only done on external boundaries (contrast this with AdTpfaFlux.pressure_trace, where internal boundaries is also filtered). It could also be that both methods need filtering on fracture tips, it seems likely this is currently not done.

To get this right, and properly tested, we need to solve a flow problem with both intersections and fracture tips.

@IvarStefansson: Feel free to add to the problem description.

@keileg keileg self-assigned this Jan 22, 2024
@IvarStefansson
Copy link
Contributor

Agree with the description. This setup might be relevant:

class ThermoporomechanicsWell(

If you use it, could you be bothered to fix #1108?

@IvarStefansson
Copy link
Contributor

Or this one, depending on whether you want to include Fourier or not:

class PoromechanicsWell(

@keileg
Copy link
Contributor Author

keileg commented Jan 23, 2024

Or this one, depending on whether you want to include Fourier or not:

class PoromechanicsWell(

Will do.

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 a pull request may close this issue.

2 participants