Skip to content
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

Picard SamToFastq tool fails, even prior test re-runs, at both EU and ORG #2747

Closed
jennaj opened this issue Dec 11, 2019 · 3 comments
Closed

Picard SamToFastq tool fails, even prior test re-runs, at both EU and ORG #2747

jennaj opened this issue Dec 11, 2019 · 3 comments

Comments

@jennaj
Copy link
Member

@jennaj jennaj commented Dec 11, 2019


Workaround for End-users


Issue details

The tool creates two or more outputs:

  1. Active datasets with datatype fastqsanger. How many depends on params.
  2. A hidden dataset with either BAM or SAM in a dataset with datatype txt

Problems:

  • Rerunning prior test fails now (at ORG, direct rerun or in a workflow)
  • Running that same prior test at EU also fails
  • Running with the updated tool version or a different BAM also fails
  • All failures are a single dataset, no matter what the params are, instead of active + hidden datasets
  • Fails whether run in workflow or directly from history
  • Tool was migrated from tools-devteam to tools-iuc sometime in the last year or so
  • MTS: https://toolshed.g2.bx.psu.edu/view/devteam/picard/7d34178f2812

Test histories:
Expand the annotation to find share links to workflows used for some of the tests. Everything is tagged/labeled with the input, params, tool versions.

@martenson was able to reproduce
@jmchilton confirmed this tool is a bit special, uses "dataset discovery"
@nsoranzo @mvdbeek commented on Ghelp post that it looks like bug and gave user a workaround (post job action in workflow). That is fine but is from using some much older version of the tool.
@jennaj tested it in 7/2018 when the output datatype was updated. Test passed back then: galaxyproject/tools-devteam#414

A user reported that they were getting txt instead of fastqsanger outputs, but that must be from some much older version of the tool (we fixed that). They haven't confirmed which version yet or where they are working (probably ORG), but those don't matter. That was fixed and the current versions are failing, including a version that did work before (the "corrected datatype" fix). Post: https://help.galaxyproject.org/t/workflow-canvas-why-is-the-output-for-samtofastq-txt-instead-of-fastq/2630

@jennaj
Copy link
Member Author

@jennaj jennaj commented Jan 10, 2020

Any updates? Thx!

Loading

@bernt-matthias
Copy link
Contributor

@bernt-matthias bernt-matthias commented May 16, 2021

I guess this is solved in #2855.. Can you retry with the most recent version and reopen if needed?

Loading

@jennaj
Copy link
Member Author

@jennaj jennaj commented Aug 24, 2021

Tested version 2.18.2.2 -- works great now both eu and org, sorry for delay

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants