Noble-Lab / casanovo

De Novo Mass Spectrometry Peptide Sequencing with a Transformer Model
https://casanovo.readthedocs.io
Apache License 2.0
116 stars 40 forks source link

Correctly report AA precision and recall during validation #253

Closed bittremieux closed 1 year ago

bittremieux commented 1 year ago

Fixes #252.

AA precision and recall were swapped due to swapped input of true and predicted peptides to the evaluation.

codecov[bot] commented 1 year ago

Codecov Report

Merging #253 (1d12bc9) into dev (9a44630) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #253   +/-   ##
=======================================
  Coverage   89.24%   89.24%           
=======================================
  Files          12       12           
  Lines         902      902           
=======================================
  Hits          805      805           
  Misses         97       97           
Files Coverage Δ
casanovo/denovo/model.py 97.45% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more