You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Store PeptideIndexer settings as metavalues in SearchParameters (OpenMS#8489)
* Initial plan
* Add PeptideIndexer settings as metavalues in ProteinIdentification
Co-authored-by: jpfeuffer <8102638+jpfeuffer@users.noreply.github.com>
* Extract metavalue keys as constants to improve maintainability
Co-authored-by: jpfeuffer <8102638+jpfeuffer@users.noreply.github.com>
* Remove trailing whitespace from PeptideIndexing.h
Co-authored-by: jpfeuffer <8102638+jpfeuffer@users.noreply.github.com>
* Use !empty() instead of size() > 0 for better readability
Co-authored-by: jpfeuffer <8102638+jpfeuffer@users.noreply.github.com>
* Store PeptideIndexer settings in SearchParameters instead of ProteinIdentification
Co-authored-by: jpfeuffer <8102638+jpfeuffer@users.noreply.github.com>
* Remove unnecessary static constants for metavalue keys
Co-authored-by: jpfeuffer <8102638+jpfeuffer@users.noreply.github.com>
* Fix PeptideIndexing test and update test output files
- Fix unit test to provide ProteinIdentification before running
(the test was creating an empty prot_ids vector but expecting
metavalues to be set - the code only updates existing
ProteinIdentifications)
- Update all PeptideIndexer TOPP test expected output files to
include the new metavalues in SearchParameters
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
* Add PeptideIndexer settings to TOPP test expected output files (OpenMS#8509)
Update expected output files to include PeptideIndexer settings that are
now written to SearchParameters. This includes decoy_string, enzyme,
enzyme_specificity, and related parameters.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jpfeuffer <8102638+jpfeuffer@users.noreply.github.com>
Co-authored-by: Timo Sachsenberg <timo.sachsenberg@uni-tuebingen.de>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
0 commit comments