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

Minimap2 enhancements #3061

Merged
merged 7 commits into from Jun 5, 2020
Merged

Conversation

wm75
Copy link
Contributor

@wm75 wm75 commented Jun 4, 2020

FOR CONTRIBUTOR:

  • - I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • - License permits unrestricted use (educational + commercial)
  • - This PR adds a new tool or tool collection
  • - This PR updates an existing tool or tool collection
  • - This PR does something else (explain below)

wm75 added 7 commits June 4, 2020 13:07
Catches minimap's [ERROR] messages, which indicate failure of the
tool in the pipe. Parameter defaults can now also be overwritten
with zero values, which were previously ignored. The minimum value
got increased to 1 for the -O and -E options, for which zero is
not allowed.
Enforces sr presets for PE reads to ensure encoding of mate info in the output
FLAG field. Makes profile selection optional for single-end reads and adds
minimap's splice:hq profile as an option.
Exposes more minimap2 settings related to spliced alignments, bundles
all of them inone place, and makes spliced alignment mode mutually
exclusive with --frag mode.
Exposes the following additional minimap2 parameters:
--max-chain-skips
--max-chain-iter
--no-end-flt

Cleanups and fixes:
- moves -G option into spliced alignment section and fixes parameter help
- reallows zeros for the second values of -O and -E
- introduces a second value for -z
@wm75
Copy link
Contributor Author

wm75 commented Jun 4, 2020

Fixes various odds and ends in the minimap2 wrapper.

@bgruening bgruening merged commit ffcf398 into galaxyproject:master Jun 5, 2020
@bgruening
Copy link
Member

Thanks @wm75!

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.

None yet

2 participants