Skip to content

Do not initially emplace any value for the cached Iteration index#1873

Merged
franzpoeschel merged 1 commit intoopenPMD:devfrom
franzpoeschel:fix-initial-cached-iteration-index
Apr 20, 2026
Merged

Do not initially emplace any value for the cached Iteration index#1873
franzpoeschel merged 1 commit intoopenPMD:devfrom
franzpoeschel:fix-initial-cached-iteration-index

Conversation

@franzpoeschel
Copy link
Copy Markdown
Contributor

this avoids subtle bugs
followup to #1860

Copy link
Copy Markdown
Contributor

@guj guj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice catch!

@franzpoeschel franzpoeschel enabled auto-merge (squash) April 20, 2026 17:20
@franzpoeschel franzpoeschel merged commit 2540d26 into openPMD:dev Apr 20, 2026
31 checks passed
franzpoeschel added a commit to franzpoeschel/openPMD-api that referenced this pull request Apr 21, 2026
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