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

meshgen optionality #266

Merged
merged 4 commits into from
Jun 22, 2022
Merged

meshgen optionality #266

merged 4 commits into from
Jun 22, 2022

Conversation

WPringle
Copy link
Collaborator

@WPringle WPringle commented May 16, 2022

  1. adding option improve_with_reduced_quality to meshgen for allowing for mesh improvements even when quality is decreasing or large number of nodes eliminated, which sometimes is necessary to force the advancement in mesh quality.
  2. adding option delaunay_elim_on_exit to meshgen to potentially skip the last call to delaunay_elim to potentially avoid deleting boundary elements.
  3. fixing recursive cleaning issues.

WPringle added 3 commits May 16, 2022 17:08
…h improvements even when quality is decreasing or large number of nodes eliminated, which sometimes is necessary to force the advancement in mesh quality. 2) adding option delaunay_elim_on_exit to to potentially skip the last call to delaunay_elim to potentiall avoid deleting boundary elements
- make sure projected fixed points are passed to preserve fixed points in the smoothing operation
- avoid repeating cleaning in an infinite loop by not calling when the triangulation is unchanged
@krober10nd
Copy link
Collaborator

I think this looks good. Based on the contents of this change, I assume you've also seen the final call to delaunay_elmin delete boundary elements? Did you come to any conclusion why that may be happening?

@WPringle
Copy link
Collaborator Author

WPringle commented Jun 22, 2022

I tried with and without the final call to delaunay_elim on what I was doing and didn't find any real difference, but I think the final call should still generally happen otherwise you might get triangles outside of meshing area. May be some case as in what you saw it helps to turn off, so that's why made it into an option.

@krober10nd
Copy link
Collaborator

Okay please update the changelog.

@krober10nd krober10nd self-requested a review June 22, 2022 15:53
@WPringle WPringle requested a review from krober10nd June 22, 2022 15:53
@WPringle WPringle merged commit 903948d into Projection Jun 22, 2022
@WPringle WPringle deleted the meshgen_patch branch June 22, 2022 15:55
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 this pull request may close these issues.

2 participants