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

Allowed reading of scan descriptions from Raw Files #736

Merged
merged 2 commits into from
Nov 27, 2023

Conversation

nbollis
Copy link
Member

@nbollis nbollis commented Oct 25, 2023

The scan description was a field that was not read into the MsDataScan object, it now is read in when present in raw files

@codecov
Copy link

codecov bot commented Oct 25, 2023

Codecov Report

Merging #736 (03c27a6) into master (d147f54) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #736   +/-   ##
=======================================
  Coverage   74.40%   74.40%           
=======================================
  Files         171      171           
  Lines       28862    28870    +8     
  Branches     2862     2863    +1     
=======================================
+ Hits        21474    21482    +8     
  Misses       6902     6902           
  Partials      486      486           
Files Coverage Δ
mzLib/MassSpectrometry/MsDataScan.cs 83.95% <100.00%> (+0.20%) ⬆️
mzLib/Readers/Thermo/ThermoRawFileReader.cs 81.32% <100.00%> (+0.32%) ⬆️

@nbollis nbollis merged commit 6c77ca0 into smith-chem-wisc:master Nov 27, 2023
3 checks passed
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.

4 participants