Skip to content

Remove C models#214

Closed
RalfG wants to merge 6 commits intoreleasesfrom
remove-c-models
Closed

Remove C models#214
RalfG wants to merge 6 commits intoreleasesfrom
remove-c-models

Conversation

@RalfG
Copy link
Copy Markdown
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
@RalfG
Copy link
Copy Markdown
Member Author

RalfG commented Apr 15, 2026

Superseded by #260.

@RalfG RalfG closed this Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant