smith-chem-wisc / MetaMorpheus

Proteomics search software with integrated calibration, PTM discovery, bottom-up, top-down and LFQ capabilities
MIT License
90 stars 45 forks source link

Eliminate PsmCount from PEP calculation #2347

Closed trishorts closed 3 months ago

trishorts commented 4 months ago

cuz some smart people think that introduces bias.

https://pubmed.ncbi.nlm.nih.gov/21391616/

image

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 92.84065% with 62 lines in your changes are missing coverage. Please review.

Project coverage is 92.74%. Comparing base (8ab02bc) to head (d3e9bc2). Report is 1 commits behind head on master.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347/graphs/tree.svg?width=650&height=150&src=pr&token=m0HXyK8tFy&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc)](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc) ```diff @@ Coverage Diff @@ ## master #2347 +/- ## ========================================== + Coverage 92.65% 92.74% +0.08% ========================================== Files 136 137 +1 Lines 21243 21199 -44 Branches 2930 2932 +2 ========================================== - Hits 19682 19660 -22 + Misses 1083 1056 -27 - Partials 478 483 +5 ``` | [Files](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc) | Coverage Δ | | |---|---|---| | [...ineLayer/Calibration/DataPointAquisitionResults.cs](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc#diff-TWV0YU1vcnBoZXVzL0VuZ2luZUxheWVyL0NhbGlicmF0aW9uL0RhdGFQb2ludEFxdWlzaXRpb25SZXN1bHRzLmNz) | `80.43% <100.00%> (ø)` | | | [...rpheus/EngineLayer/Calibration/LabeledDataPoint.cs](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc#diff-TWV0YU1vcnBoZXVzL0VuZ2luZUxheWVyL0NhbGlicmF0aW9uL0xhYmVsZWREYXRhUG9pbnQuY3M=) | `100.00% <100.00%> (ø)` | | | [...s/EngineLayer/ClassicSearch/ClassicSearchEngine.cs](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc#diff-TWV0YU1vcnBoZXVzL0VuZ2luZUxheWVyL0NsYXNzaWNTZWFyY2gvQ2xhc3NpY1NlYXJjaEVuZ2luZS5jcw==) | `97.84% <100.00%> (ø)` | | | [...gineLayer/CrosslinkSearch/CrosslinkSearchEngine.cs](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc#diff-TWV0YU1vcnBoZXVzL0VuZ2luZUxheWVyL0Nyb3NzbGlua1NlYXJjaC9Dcm9zc2xpbmtTZWFyY2hFbmdpbmUuY3M=) | `91.08% <100.00%> (ø)` | | | [...EngineLayer/CrosslinkSearch/CrosslinkedPeptides.cs](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc#diff-TWV0YU1vcnBoZXVzL0VuZ2luZUxheWVyL0Nyb3NzbGlua1NlYXJjaC9Dcm9zc2xpbmtlZFBlcHRpZGVzLmNz) | `100.00% <100.00%> (ø)` | | | [MetaMorpheus/EngineLayer/FdrAnalysis/PsmData.cs](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc#diff-TWV0YU1vcnBoZXVzL0VuZ2luZUxheWVyL0ZkckFuYWx5c2lzL1BzbURhdGEuY3M=) | `100.00% <100.00%> (+7.69%)` | :arrow_up: | | [...aMorpheus/EngineLayer/GlycoSearch/GlycoPeptides.cs](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc#diff-TWV0YU1vcnBoZXVzL0VuZ2luZUxheWVyL0dseWNvU2VhcmNoL0dseWNvUGVwdGlkZXMuY3M=) | `88.03% <100.00%> (ø)` | | | [...pheus/EngineLayer/GlycoSearch/GlycoSearchEngine.cs](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc#diff-TWV0YU1vcnBoZXVzL0VuZ2luZUxheWVyL0dseWNvU2VhcmNoL0dseWNvU2VhcmNoRW5naW5lLmNz) | `92.67% <100.00%> (ø)` | | | [MetaMorpheus/EngineLayer/Gptmd/GptmdEngine.cs](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc#diff-TWV0YU1vcnBoZXVzL0VuZ2luZUxheWVyL0dwdG1kL0dwdG1kRW5naW5lLmNz) | `89.34% <100.00%> (ø)` | | | [...eus/EngineLayer/Localization/LocalizationEngine.cs](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc#diff-TWV0YU1vcnBoZXVzL0VuZ2luZUxheWVyL0xvY2FsaXphdGlvbi9Mb2NhbGl6YXRpb25FbmdpbmUuY3M=) | `98.36% <100.00%> (ø)` | | | ... and [37 more](https://app.codecov.io/gh/smith-chem-wisc/MetaMorpheus/pull/2347?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=smith-chem-wisc) | |