Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove C models #214

Open
wants to merge 6 commits into
base: releases
Choose a base branch
from
Open

Remove C models #214

wants to merge 6 commits into from

Conversation

RalfG
Copy link
Member

@RalfG RalfG commented Feb 9, 2024

  • Replace remaining C model files with .xgboost models
  • Use more accurate mass values
  • Fix various C compile-time warnings
  • Format C and Cython code
  • Fix calculating correlations if some peptides could not be processed (e.g., invalid amino acids). Now returns nan for those spectrum correlations
  • Clean up setup.py

Base automatically changed from v4.0.0 to releases July 17, 2024 15:27
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant