-
Notifications
You must be signed in to change notification settings - Fork 13
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
add wrapper for training Spec2Vec in Galaxy #314
Conversation
@maximskorik can you please explain the role of docker image? Is it just to get the newest version of the tool available in |
Exactly. We recently added functionality for Spec2Vec to export a model to |
The CI fails because the linter is concerned about suffixes in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks pretty good! Great job @maximskorik
Can we maybe change this to be compliant with the CI so that it passes? I know it is not a fault on our side, but still as soon as the CI fails and you merge stuff, it becomes harder to see when it actually starts failing for a different reason. |
Updates
spec2vec_training
tool to GalaxyDescription
This tool can train a Spec2Vec model from
msp
ormgf
spectra files. The outputs arejson
model-metadata andnpy
peak-embeddings, which can be read byspec2vec.serialization
module (in the future downstream step) to compute Spec2Vec similarity scores. Optional outputs are Pythonpickle
file & checkpoints of a model at user-defined iterations.Closes #315