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

Broke the methods apart to make it cleaner if the user wants more meshing control #6

Merged
merged 5 commits into from
Jan 29, 2025

Conversation

jmwright
Copy link
Member

@shimwell Please review this as an alternative to #5 and let me know what you think

@jmwright jmwright requested a review from shimwell January 29, 2025 11:53
@shimwell
Copy link
Collaborator

This looks like a good approach to me. I can close #5

@jmwright
Copy link
Member Author

@shimwell Did you man to close this PR too?

@jmwright jmwright reopened this Jan 29, 2025
jmwright and others added 4 commits January 29, 2025 07:41
Fix gmsh name

Co-authored-by: Jonathan Shimwell <[email protected]>
Co-authored-by: Jonathan Shimwell <[email protected]>
Co-authored-by: Jonathan Shimwell <[email protected]>
@jmwright
Copy link
Member Author

@shimwell I have added an example of how to use getTaggedGmsh in the readme.

@shimwell
Copy link
Collaborator

@shimwell Did you man to close this PR too?

sorry that was my mistake

@jmwright jmwright merged commit a47ce50 into main Jan 29, 2025
6 checks passed
@jmwright jmwright deleted the composition branch January 29, 2025 13:40
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