-
Notifications
You must be signed in to change notification settings - Fork 448
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
VCFfilter (Galaxy Version 1.0.0_rc1+galaxy1) -- tool help is a mismatch versus the updated form query structure, impacts Coursera tutorials #1910
Comments
Related, maybe should be considered together: #1917 |
Above changes are for next rev. PR for this versions updates: #1791 |
The following PR may fix this issue: #2209 |
Final testing once the PR is done, tool in MTS and updated on main will happen in this ticket: galaxyproject/usegalaxy-playbook#182 IUC can close this one out once dev steps are done. Can reopen if later tests on Main fail that are tool related (not server related). Works in a local. |
1.0.0_rc1+galaxy2: https://toolshed.g2.bx.psu.edu/view/devteam/vcffilter/de885f4afb7f |
Example of current form usage:
Associated tool form help:
Users are confused with whether to quote the filter value or not (they shouldn't or the tool errors). An additional complication is that this tool is included in the Corsica classes and use the prior tool version with a different form (and those tools were completely replaced at usegalaxy.org due to bugs). Things change, but we can help in small ways like this.
Another option is to have the tool be smart enough to ignore quotes and/or clearly report what is wrong (the filter term) ... would address problems users had before and now when a filter was constructed incorrectly. Right now the tool doesn't state what went wrong (bug preview or in the full bug report or in job info stderr/stdout). Something like "Filter term is malformed, see the tool help for examples" would be useful.
This is what currently is reported, for any unexpected user entered "Specify filtering value" term. We get bug reports about this frequently.
The text was updated successfully, but these errors were encountered: