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

Bug: Cutadapt 3.5+galaxy1 parameter --strip-suffix #481

Closed
jennaj opened this issue Mar 10, 2022 · 2 comments
Closed

Bug: Cutadapt 3.5+galaxy1 parameter --strip-suffix #481

jennaj opened this issue Mar 10, 2022 · 2 comments
Labels
bug Something isn't working

Comments

@jennaj
Copy link
Member

jennaj commented Mar 10, 2022

Message for end-users:

  1. How to notice if you hit the bug?
  • The "submit" button for CutAdapt redirects to the top of the tool form. Does not actually submit.
  • No confirmation messages in the center panel, no new dataset(s) in the history.
  1. How to workaround the bug?
  • Click on "Read Modification Options" to expand the section.
  • Scroll down to the "Strip suffix" section (will be highlighted by a light blue box).
  • Type one or more characters (any!) into that parameter, then remove those characters until empty again.
  • Scroll down and click on the "submit" button again.
  • The tool will now execute!

Developers:

The option --strip-suffix is for some reason flagged as required upon job submission (blocks the submission, and shouldn't). But if the user "touches" that parameter's free text input (adds then removes any content, even a single character), the requirement is satisfied, and the job submits.

  1. That shouldn't be needed
  2. It isn't obvious what is going wrong because that section is collapsed at the default and the rerun tool form view (I had help from a person that submitted a bug report who noticed/described the exact problem!)
  3. Only happens at usegalaxy.org -- could this be related to 22.01??
  4. Another odd workaround -- load up the tool form from a prior version of this tool using "rerun", switch to the 3.5+galaxy1 version, adjust the inputs/settings (doesn't need to include this param) -- and submit works.
  5. This tool is included in tutorials.
  6. Introduced in this version of the tool added to org in Dec 2021 Update cutadapt on usegalaxy.org #447

Test histories:

Tests (those failures are due to a metadata issue, but the tests that never ran might be due to this)

ping @dannon @davebx @mvdbeek @natefoo


Screenshots

Before submit

Screen Shot 2022-03-10 at 10 33 16 AM

After submit AND expanding that section

Screen Shot 2022-03-10 at 10 33 51 AM

Fiddle with the parameter, and the job submits

Screen Shot 2022-03-10 at 10 37 17 AM

@jennaj jennaj added the bug Something isn't working label Mar 10, 2022
@mvdbeek
Copy link
Member

mvdbeek commented Mar 11, 2022

This isn't needed anymore, the fix is deployed on usegalaxy.org

@mvdbeek mvdbeek closed this as completed Mar 11, 2022
@jennaj
Copy link
Member Author

jennaj commented Mar 11, 2022

Fixed in galaxyproject/galaxy#13523

Will be live at usegalaxy.org before Monday, March 24, 2022

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants