Skip to content

Fix bug reclvl file parser to account for ragged columns#302

Merged
wtbarnes merged 1 commit intomainfrom
reclvl-parser-fix
Sep 10, 2024
Merged

Fix bug reclvl file parser to account for ragged columns#302
wtbarnes merged 1 commit intomainfrom
reclvl-parser-fix

Conversation

@wtbarnes
Copy link
Owner

This is a bugfix for the reclvl parser to account for possible ragged columns. In some versions of the database, a given file may have rate and temperature arrays of unequal length. This deals with that case.

@wtbarnes wtbarnes added I/O bugfix Fixes a bug. PRs with this label may be backported. Run v7 tests Run the test suite using the CHIANTI database version 7 backport 0.3 labels Sep 10, 2024
@codecov
Copy link

codecov bot commented Sep 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.17%. Comparing base (5e730e5) to head (694e127).
Report is 24 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #302      +/-   ##
==========================================
+ Coverage   92.70%   93.17%   +0.46%     
==========================================
  Files          37       37              
  Lines        3111     3119       +8     
==========================================
+ Hits         2884     2906      +22     
+ Misses        227      213      -14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@wtbarnes wtbarnes merged commit bc5b05f into main Sep 10, 2024
@wtbarnes wtbarnes deleted the reclvl-parser-fix branch September 10, 2024 20:04
meeseeksmachine pushed a commit to meeseeksmachine/fiasco that referenced this pull request Sep 10, 2024
wtbarnes added a commit that referenced this pull request Sep 10, 2024
…lumns (#303)

Co-authored-by: Will Barnes <will.t.barnes@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes a bug. PRs with this label may be backported. I/O Run v7 tests Run the test suite using the CHIANTI database version 7

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant