-
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
[WIP] Add retip tool suit wrapper(s) #3
Conversation
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 very good @smartx-usman, thanks!
One thing I'd like to ask is to design a Galaxy tool test for each tool. It does not need to go in this repository yet, since the container is still private, but we will need them soon.
There are already some example data listed here: https://gitlab.ics.muni.cz/recetox/mass-spectrometry/retip#testing-data
add missing requirement drop extra whitespace
…. Modify datatype to use more generic type rather than custom defined type. Use docker image from DockerHub.
update biotransformer versioning to iuc standard
[WIP] add xmsannotator
[WIP] add apLCMS
Error in stderr is actually correct behavior.
Adapt apLCMS to HDF5 files
Change xmsannotator error detection policy
Cleaned xml wrapper and added test.
@smartx-usman you pulled a conflicting commit into this branch, this needs a merge/rebase with/on |
…. Modify datatype to use more generic type rather than custom defined type. Use docker image from DockerHub.
# Conflicts: # tools/biotransformer/biotransformer.xml # tools/biotransformer/wrapper_biotransformer.py # tools/retip/macros.xml # tools/retip/retip_apply.xml # tools/retip/retip_descriptors.xml
@martenson conflicting commit should be removed now. |
At a glance it seems correct to me, but github has trouble displaying it. I'll try closing this PR and reopening. |
Alright, the new PR displays fine in GH. Link above ^ |
No description provided.