Skip to content

io.pepxml: Fix inferring score when first spectrum_query item does not have search_hit items.#143

Merged
RalfG merged 2 commits intomainfrom
fix/pepxml-searchhit-error
Feb 12, 2026
Merged

io.pepxml: Fix inferring score when first spectrum_query item does not have search_hit items.#143
RalfG merged 2 commits intomainfrom
fix/pepxml-searchhit-error

Conversation

@RalfG
Copy link
Member

@RalfG RalfG commented Feb 12, 2026

Fixed

@RalfG RalfG added this to the v1.5.2 milestone Feb 12, 2026
@RalfG RalfG self-assigned this Feb 12, 2026
@RalfG RalfG added the bug Something isn't working label Feb 12, 2026
@RalfG RalfG merged commit faeadc5 into main Feb 12, 2026
5 checks passed
@RalfG RalfG deleted the fix/pepxml-searchhit-error branch February 12, 2026 12:45
@codecov
Copy link

codecov bot commented Feb 12, 2026

Codecov Report

❌ Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.97%. Comparing base (c5960ee) to head (c51c946).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
psm_utils/io/pepxml.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #143      +/-   ##
==========================================
- Coverage   72.01%   71.97%   -0.05%     
==========================================
  Files          32       32              
  Lines        3245     3247       +2     
==========================================
  Hits         2337     2337              
- Misses        908      910       +2     
Flag Coverage Δ
unittests 71.97% <33.33%> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Development

Successfully merging this pull request may close these issues.

io.pepxml: PSM score cannot be inferred if first spectrum_query items is empty

1 participant