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

Sniffles: Update VCF info field table in help and add --detect-large-ins flag option #6380

Merged
merged 5 commits into from
Sep 29, 2024

Conversation

fubar2
Copy link
Collaborator

@fubar2 fubar2 commented Sep 28, 2024

FOR CONTRIBUTOR:

  • This PR does something else (explain below)
  • Add an option to set --detect-large-ins flag false.
  • enforce str() conversion for select values
  • Replace the old help VCF information table - now mostly out of date.
    • Fastidiously formatted the info fields from an output VCF (!) - such fragility, both human and restructured text...

@fubar2 fubar2 changed the title Sniffles Sniffles: Update VCF info field table in help and add --detect-large-ins flag option Sep 29, 2024
@bgruening
Copy link
Member

This might help next time: https://rsted.info.ucl.ac.be/

Can you please bump the version?

@fubar2
Copy link
Collaborator Author

fubar2 commented Sep 29, 2024

This might help next time: https://rsted.info.ucl.ac.be/

Nice - thanks!
Useful but doesn't make a table which is what this needed. They are painful in html too but not as bad as restructured text. Patience and precision that can probably be made easier - and it only works when it is right.

Everything else in Galaxy is slick these days. Tool help seems an ugly wart by comparison and although the trend seems to be to link to tutorials, some people prefer at least a few useful hints on the actual tool form.
No idea what the optimal solution is but pretty sure it is not restructured text.

@bgruening bgruening merged commit 03fbb13 into galaxyproject:main Sep 29, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants