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

Fragger Result reading #770

Merged
merged 8 commits into from
Apr 23, 2024
Merged

Conversation

nbollis
Copy link
Contributor

@nbollis nbollis commented Mar 21, 2024

Added classes to readers for MsFragger Psm, Protein, and Peptide reading.

The protein and peptide files have two possible formats. One for the individual file result and one for the combined results. The individual file format is more verbose and has additional fields. the PeptideFile and ProteinFile can be used for either result.

Copy link

codecov bot commented Mar 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.14%. Comparing base (cf854ac) to head (da79acf).
Report is 1 commits behind head on master.

❗ Current head da79acf differs from pull request most recent head b2e83b6. Consider uploading reports for the commit b2e83b6 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #770      +/-   ##
==========================================
+ Coverage   75.06%   75.14%   +0.07%     
==========================================
  Files         189      191       +2     
  Lines       30235    30307      +72     
  Branches     3035     3042       +7     
==========================================
+ Hits        22697    22773      +76     
+ Misses       7024     7022       -2     
+ Partials      514      512       -2     
Files Coverage Δ
...nalResults/IndividualResultRecords/MsFraggerPsm.cs 100.00% <100.00%> (ø)
...rs/ExternalResults/ResultFiles/MsFraggerPsmFile.cs 100.00% <100.00%> (ø)
mzLib/Readers/Util/SupportedFileTypes.cs 100.00% <100.00%> (+7.69%) ⬆️

Alexander-Sol
Alexander-Sol previously approved these changes Apr 3, 2024
elaboy
elaboy previously approved these changes Apr 7, 2024
@nbollis nbollis dismissed stale reviews from elaboy and Alexander-Sol via 8bf8099 April 12, 2024 21:20
@nbollis nbollis changed the title Fragger psm reading Fragger Result reading Apr 12, 2024
@trishorts trishorts merged commit 9d2cd28 into smith-chem-wisc:master Apr 23, 2024
1 check 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