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

Parse lower/mixed case input files #47

Merged
merged 1 commit into from
May 16, 2019

Conversation

benzwick
Copy link
Collaborator

Abaqus input files are case insensitive.

Previously, the element type (e.g. C3D4 == c3d4)
and the NSET and ELSET options were not recognized
if lower or mixed case.

Note: Boundary conditions and loads are read verbatim
(i.e. symbols :P or :p are created for pressure loads
depending on the case used in the input file)
which may cause confusion.

Abaqus input files are case insensitive.

Previously, the element type (e.g. C3D4 == c3d4)
and the NSET and ELSET options were not recognized
if lower or mixed case.

Note: Boundary conditions and loads are read verbatim
(i.e. symbols :P or :p are created for pressure loads
depending on the case used in the input file)
which may cause confusion.
@coveralls
Copy link

Coverage Status

Coverage decreased (-33.09%) to 65.472% when pulling 099547a on benzwick:lower-case-keywords into 49a46b1 on JuliaFEM:master.

@ahojukka5
Copy link
Member

LGTM, thanks a lot for this contribution also.

@ahojukka5 ahojukka5 merged commit d74535d into JuliaFEM:master May 16, 2019
@avigliotti
Copy link
Contributor

Hi,
thanks for the great work!

would it possible to add an option that prevents the verbose info outputs when reading a model or a mesh?

@benzwick
Copy link
Collaborator Author

Hi,
thanks for the great work!

would it possible to add an option that prevents the verbose info outputs when reading a model or a mesh?

See issue #45

@avigliotti
Copy link
Contributor

just saw it, thanks

@benzwick benzwick deleted the lower-case-keywords branch December 13, 2019 11:34
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.

4 participants