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

Spherical refinement_callback is experiencing some issues #189

Open
gthyagi opened this issue Apr 23, 2024 · 1 comment
Open

Spherical refinement_callback is experiencing some issues #189

gthyagi opened this issue Apr 23, 2024 · 1 comment

Comments

@gthyagi
Copy link
Contributor

gthyagi commented Apr 23, 2024

It appears that the refinement_callback is the culprit when it comes to issues in spherical models, especially when both refinement and refinement_callback are enabled. However, when I ran the model with only the refinement option, it worked, and the Stokes iterations converged.

First call to refinement_callback happens here:

if callable(refinement_callback):

Here is the script
https://github.com/gthyagi/UW3_Annulus_Spherical_Benchmarks/blob/f1f1f1cf7ee71e737f66232d3a81ad70d7278c94/Spherical_Benchmarks/Ex_Stokes_Spherical_Benchmark_Kramer_ref_1_ref_cb_0.py

@gthyagi gthyagi changed the title The spherical refinement_callback is experiencing some issues, but the refinement option alone is working perfectly Spherical refinement_callback is experiencing some issues Apr 23, 2024
@lmoresi
Copy link
Member

lmoresi commented Apr 26, 2024

The implementation is naive and fragile - it assumes that it is fine just to move any labeled nodes (post refinement) so that they are at the required radius. Could be all sorts of things going wrong, including something in how we catch labels on nodes (it's the edges that are labelled for refinement) or it could be that we don't do something important before we rebuild the mesh discretisation.

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

2 participants