-
Notifications
You must be signed in to change notification settings - Fork 18
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
manta tool does not output any file by default #660
Comments
Yeah... You probably did not notice it @bgruening but I am systematically refreshing the tools. Last weeks I was on Lumpy-Smoove and the next one on the list is... Manta ! I fear the worst ! Shall I jump directly to |
@bgruening I have reviewed the manta wrapper. To me this looks like a monstrous wrapper of wrapper of wrapper. Beside the issue you mentioned last week, it's a miracle that 4/5 tests pass... We can clarify and fix, for sure.
Although I can admit that none of these points are really strong and the Manta wrapper has been installed 165 times (not by me !), I have mixed feelings. Do you think it's worth maintaining the Galaxy Manta wrapper ? Tell me what you think (other opinions are also welcome), I will definitely take it into account. |
I agree we should probably deprecate it. But I'm unsure we we should fix this last problem. It seems to be relatively easy to rearrange the outputs in the Galaxy wrapper, no? I will hide the tool on the EU server, do you agree? |
OK, I'll do it
No problem with that |
See PR #669, @bgruening |
The outputs are all booleans and it is possible to submit a job with NO output. Even worse its the default setting :)
The text was updated successfully, but these errors were encountered: