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

Fix double mesh.read() call in parsing procedures #1535

Merged
merged 1 commit into from
Oct 6, 2023

Conversation

franzpoeschel
Copy link
Contributor

First call was outside try{}, so errors would not be caught

First call was outside try{}, so errors would not be caught
@ax3l
Copy link
Member

ax3l commented Oct 6, 2023

@franzpoeschel thank you!

Just for labeling: is this a bug in an existing release as well or purely on dev?

@ax3l ax3l merged commit 2e0b016 into openPMD:dev Oct 6, 2023
@ax3l ax3l self-assigned this Oct 6, 2023
@franzpoeschel
Copy link
Contributor Author

@franzpoeschel thank you!

Just for labeling: is this a bug in an existing release as well or purely on dev?

Yes, it is also in the latest release

eschnett added a commit to eschnett/openPMD-api that referenced this pull request Oct 13, 2023
* dev:
  [pre-commit.ci] pre-commit autoupdate (openPMD#1536)
  Fix double mesh.read() call (openPMD#1535)
  Python 3.12: Remove Distutils (openPMD#1508)
  openpmd-pipe: fix handling of constant components (openPMD#1530)
  Bug fix: Linear read mode cannot directly access specific file of file-based Series (openPMD#1533)
  Unknown openPMD version in data: Add upgrade hint (openPMD#1528)
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