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

Add bwa-mem2-idx #6558

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

duartetorreserick
Copy link

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)

Often we use bwa in workflows for multiple samples, where the same index is created repeatedly—-hundreds or even thousands of times. This tool will allow the creation of an index before running bwa-mem2, so multiple bwa-mem2 instances can use the same index without creating it over and over again. This will require a modification to bwa-mem2 as well to be able to take advantage of the index.

</inputs>

<outputs>
<data name="output" format="text" label="Test Testov"/>
Copy link
Member

@mvdbeek mvdbeek Nov 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you create a subclass of the directory datatype for this in https://github.com/galaxyproject/galaxy/blob/dev/lib/galaxy/config/sample/datatypes_conf.xml.sample?

Something like

<datatype extension="bwa_mem_2_index" type="galaxy.datatypes.data:Directory" display_in_upload="true" subclass="true"/>

Then you can move the index directory to $output.files_path, and in the same way the bwa_mem2 tool can consume the index directory from ``$output.extra_files_path`

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