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

ElmerGrid, add input checks when reading gmsh msh files. #611

Merged
merged 2 commits into from
Dec 2, 2024

Conversation

richb2k
Copy link
Contributor

@richb2k richb2k commented Nov 30, 2024

Detect currently valid gmsh formats and if ElmerGrid doesn't support the format, print an informative message, including how to fix it, and call fatal error. This change enables reading of gmsh format 1 msh files, which wasn't working before. Gmsh type 3 msh files are detected and reported as unsupported. As before, binary files are detected. This PR is an extension related to Issue #603 and PR #604, where detection and reporting of binary files was added. This completes upgrading reading of current gmsh file formats, so the user will know positively if a format is or is not supported. A small archive with eight test cases is included.
gmsh-all-formats-examples.zip

…ntly

valid gmsh formats and if ElmerGrid doesn't support the format, print an
informative message including how to fix it, and call fatal error.  This
change enables reading of gmsh format 1 msh files, which wasn't working.
Gmsh type 3 msh files are detected and reported as unsupported.
@richb2k
Copy link
Contributor Author

richb2k commented Nov 30, 2024

Synchronized ElmerGui by copying in the three source files from ElmerGrid. This will update ElmerGui with the fix for parentheses, in addition to the update to gmsh input file checking.

@raback
Copy link
Contributor

raback commented Dec 2, 2024

Thanx Rich! Looks good.

@raback raback merged commit bc22551 into ElmerCSC:devel Dec 2, 2024
3 of 10 checks passed
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