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

Double-boundary points warning #166

Open
mmichelis opened this issue Feb 7, 2024 · 2 comments
Open

Double-boundary points warning #166

mmichelis opened this issue Feb 7, 2024 · 2 comments
Milestone

Comments

@mmichelis
Copy link

I've been running a few FSI examples with OpenFOAM and FEniCS, and have been working into the FEniCS documentation, and like the simplicity of the framework so far. With some of the simulations, mostly soft bodies with neohookean material models, right after the precice coupling is done, some warnings are printed for "Found a double-boundary point at (x,y,z)".

I was wondering what causes this warning, and how I can look more into where this occurs. Even with a very simple BoxMesh, something is causing this warning. I will look further into the adapter and my code to understand what could potentially cause this warning, but perhaps others have seen this warning before. Thanks a lot!

@BenjaminRodenberg
Copy link
Member

Hi @mmichelis!

The warning is raised at this place:

logger.warning("Found a double-boundary point at {location}.".format(location=point))

Some history about the warning can be found here: #90

I think we should definitely improve this warning, because at the moment it is obviously confusing and not clear when this warning is something to worry about and when it can be safely ignored.

If you want to dig deeper into this issue: Any PR improving on this warning or (simple) test cases where this warning actually leads to a problematic situation would be highly appreciated. I unfortunately only vaguely remember the context and cannot spend a lot of time on it right now.

@mmichelis
Copy link
Author

Hey Benjamin,

Thanks a lot for this, that will help me look into it a bit more! I will dig a bit as well, the simulation coupling is currently still exploding after a few iterations, so I'll dig a bit to see what is causing this (perhaps it's my fluid simulation). If it is because of this warning, I'll try to follow-up on this!

Cheers,
Mike

@BenjaminRodenberg BenjaminRodenberg modified the milestones: v2.x.x, v2.1.0, v2.2.0 Mar 12, 2024
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

2 participants