Skip to content

Update top/traj metadata to append rather than replace logfile extrac… - #51

Merged
jkalayan merged 2 commits into
mainfrom
50-bug-populated-metadata-overwritten
Sep 25, 2026
Merged

jkalayan merged 2 commits into
mainfrom
50-bug-populated-metadata-overwritten

Conversation

@jkalayan

Copy link
Copy Markdown
Collaborator

…ted metadata

Summary

Ensure extracted metadata from top/traj files is appended to already populated metadata, rather than replacing it.

@jkalayan jkalayan linked an issue Sep 25, 2026 that may be closed by this pull request
@jkalayan
jkalayan merged commit 35846a0 into main Sep 25, 2026
12 checks passed
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 36154954674

Coverage decreased (-0.6%) to 90.829%

Details

  • Coverage decreased (-0.6%) from the base build.
  • Patch coverage: 7 uncovered changes across 1 file (7 of 14 lines covered, 50.0%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
biosim_extractor/helpers/metadata_utils.py 10 3 30.0%
Total (2 files) 14 7 50.0%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1025
Covered Lines: 931
Line Coverage: 90.83%
Coverage Strength: 8.17 hits per line

💛 - Coveralls

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.

[Bug]: Populated metadata overwritten

2 participants