-
Notifications
You must be signed in to change notification settings - Fork 72
Add ML predictor #1303
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Add ML predictor #1303
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fix IndexError: string index out of range in reference proteome similarity step
Improve handling of inframe indels in repetitive regions
…ng to a shared variable
… unnecessary usage
Ensure that prediction_algorithms list gets copied so it's not pointing to a shared block variable
Update --top-score-metric2 parameter help text for clarity and remove unnecessary usage (hotfix base)
Fix per-allele counts in the aggregated report
…topes/filtered file
…_epitopres/filtered file as an input_tsv
Fix a few bugs where standalone commands don't work with all_epitopes/filtered TSVs
Update example and demo data for 5.5.4 release
- Updated _resolve_artifact_paths() to use numpy126 versions of model artifacts - Added numpy126 model artifacts: rf_downsample_model_numpy126.pkl, trained_imputer_numpy126.joblib, label_encoders_numpy126.pkl - Ensures compatibility with numpy 1.26 and pvactools_ml_integration environment
- Resolved conflicts by accepting master's version for test data files - Preserved ML predictor numpy126 artifacts and updated ml_predictor.py - Successfully integrated latest pVACtools updates with ML model compatibility
Folowing changes was just added to make the test script for ml_predictor to work - Add is_preferred_transcript() function - Add transcript_prioritization_strategy() function - Add pandas import - Resolves import errors in ML predictor tests
- Remove label_encoders.pkl (old version) - Remove rf_downsample_model.pkl (old version) - Remove trained_imputer.joblib (old version) - Keep only numpy126-compatible artifacts for ML predictor integration
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This project aims to create a ML model for facilitating the ITB decisions for peptide vaccines. The model takes the following as input and predicts decisions in the "Evaluation" column in the pvacview file: