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

openPMD: Simplified Match #50

Merged
merged 1 commit into from
Apr 20, 2023

Conversation

ax3l
Copy link
Collaborator

@ax3l ax3l commented Apr 20, 2023

Although we do advertise recommended namings in the ED-PIC extension, in practice slight variations are common, e.g., lowercase j for current densities.

Adds a case-insensitive match.

Although we do advertise recommended namings in the ED-PIC extension,
in practice slight variations are common, e.g., lowercase `j` for
current densities.

Adds a case-insensitive match.
@ax3l ax3l requested a review from AngelFP April 20, 2023 18:13
@AngelFP
Copy link
Owner

AngelFP commented Apr 20, 2023

More PRs pls :D

@AngelFP AngelFP merged commit 288682f into AngelFP:dev Apr 20, 2023
@ax3l ax3l deleted the openPMD-fields-simplified-match branch April 20, 2023 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants