Closed wfondrie closed 3 years ago
Merging #9 (481a737) into master (d188408) will increase coverage by
5.83%
. The diff coverage is50.00%
.
@@ Coverage Diff @@
## master #9 +/- ##
==========================================
+ Coverage 70.44% 76.28% +5.83%
==========================================
Files 15 15
Lines 1357 1362 +5
==========================================
+ Hits 956 1039 +83
+ Misses 401 323 -78
Impacted Files | Coverage Δ | |
---|---|---|
mokapot/parsers/pepxml.py | 82.20% <50.00%> (+69.81%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update d188408...481a737. Read the comment docs.
The PepXML parser lacked the option to bin modification masses in the case of an open modification search. This is necessary when grouping peptides for peptide-level confidence estimation, so that peptides with the same modification, but with slight mass deviations, are considered together.
This PR adds the
open_modification_bin_size
parameter to theread_pepxml()
function.