Skip to content

[26.1] Add kraken2 datatype - #22455

Draft
pvanheus wants to merge 9 commits into
galaxyproject:release_26.1from
pvanheus:add_kraken2_datatype
Draft

[26.1] Add kraken2 datatype#22455
pvanheus wants to merge 9 commits into
galaxyproject:release_26.1from
pvanheus:add_kraken2_datatype

Conversation

@pvanheus

Copy link
Copy Markdown
Contributor

The update in galaxyproject/tools-iuc#7257 for the kraken2 tool allows it to use a custom database passed as a parameter. Currently this uses the Directory datatype. This datatype allows a more specific input type to be specified and also allows a compressed archive to be uploaded and unpacked to the kraken2 database datatype.

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

Comment thread lib/galaxy/datatypes/binary.py Outdated
@mvdbeek
mvdbeek marked this pull request as draft April 15, 2026 07:56
@mvdbeek mvdbeek moved this from Needs Review to In Progress in Galaxy Dev - weeklies Apr 15, 2026
@pvanheus pvanheus changed the title Add kraken2 datatype [DRAFT] Add kraken2 datatype Apr 23, 2026
@mvdbeek mvdbeek modified the milestones: 26.1, 26.2 May 19, 2026
@pvanheus

Copy link
Copy Markdown
Contributor Author

Update from GCC 2026 - this now uses only subclasses, so only changes the datatypes_conf.xml.sample.

Wins and losses:

  1. This involves no code changes to the Galaxy code
  2. This loses the ability to "sniff" the archive or the directory to check that it has the files expected from a Kraken2 database

@pvanheus
pvanheus changed the base branch from dev to release_26.1 June 26, 2026 12:14
@github-actions github-actions Bot changed the title Add kraken2 datatype [26.1] Add kraken2 datatype Jun 26, 2026
@@ -360,6 +359,12 @@
<datatype extension="tar" auto_compressed_types="gz,bz2" type="galaxy.datatypes.binary:CompressedArchive" subclass="true" display_in_upload="true">
<converter file="archive_to_directory.xml" target_datatype="directory"/>
</datatype>
<datatype extension="kraken2db.tar" auto_compressed_types="gz,bz2" type="galaxy.datatypes.binary:Binary" subclass="true" display_in_upload="true" description="Kraken2 database archive">
<converter file="archive_to_directory.xml" target_datatype="kraken2db"/>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does this really work? That is does it produce a kraken2db or a directory.

<converter file="archive_to_directory.xml" target_datatype="kraken2db"/>
</datatype>
<datatype extension="kraken2db" type="galaxy.datatypes.data:Directory" subclass="true" description="Kraken2 database directory">
<converter file="directory_to_archive.xml" target_datatype="kraken2db.tar.gz"/>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Would this convert any directory?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

3 participants