Skip to content

add chewtree tool: phylogenetic tree from chewBBACA allele profiles - #1973

Open
bgruening wants to merge 5 commits into
masterfrom
chewtree
Open

add chewtree tool: phylogenetic tree from chewBBACA allele profiles#1973
bgruening wants to merge 5 commits into
masterfrom
chewtree

Conversation

@bgruening

@bgruening bgruening commented Aug 30, 2026

Copy link
Copy Markdown
Owner

closes #1937

Adds the chewtree tool (split out from the vienna_rna update commit) for building phylogenetic trees from chewBBACA allele profiles.

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)

There are two labels that allow to ignore specific (false positive) tool linter errors:

  • skip-version-check: Use it if only a subset of the tools has been updated in a suite.
  • skip-url-check: Use it if github CI sees 403 errors, but the URLs work.

bgruening and others added 4 commits August 30, 2026 13:01
@bgruening

Copy link
Copy Markdown
Owner Author

Should we combine the two Python scripts into one?

python '$__tool_directory__/scripts/mentalist_tree.py' '$phantcec_dm' > '$phantcec_tree'
]]></command>
<inputs>
<param name="input" format="tabular" type="data" label="chewBBACA allele profile TSV file" help="Tab-separated file with allele calls; first column sample names, subsequent columns loci, last two columns (CC, clonal_complex) excluded"/>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ChewBBACA allelcall output is a collection. Probably it is better if this tool also accepted input as a collection

Comment on lines +25 to +28
<param name="input_type" type="select" label="Input matrix type">
<option value="allele" selected="true">Allele profile - calculate Hamming distance matrix</option>
<option value="distance">Distance matrix - use pre-computed distance matrix directly</option>
</param>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can the dm be calculated from other tools?
If it is only calculated here, then the input should be only the allele profile.

Comment thread tools/chewtree/macros.xml Outdated
Comment thread tools/chewtree/.shed.yml
description: Builds phylogenetic trees from allelic profiles using ChewBBACA
homepage_url: https://github.com/B-UMMI/ChewBBACA
long_description: |
ChewTree is part of the ChewBBACA suite. It builds phylogenetic trees from allelic profiles computed by ChewBBACA's AlleleCall algorithm.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
ChewTree is part of the ChewBBACA suite. It builds phylogenetic trees from allelic profiles computed by ChewBBACA's AlleleCall algorithm.
ChewTree is part of the Galaxy ChewBBACA tool suite. It builds phylogenetic trees from allelic profiles computed by ChewBBACA's AlleleCall algorithm.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

For me it sounded like it is part of the tool upstream.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

mh, I can add this ... but this raises a new question. Should we move this tool into the chewbbaca folder under IUC?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

I don't even know where those scripts are coming from. @paulzierep ?

Do you, @nilchia, know if those are part of chewbacca upstream somewhere?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I also don't know where they are from.
But I know they are not part of ChewBBACA upstream code

Co-authored-by: Amirhossein Nilchi <66441226+nilchia@users.noreply.github.com>
@nilchia

nilchia commented Aug 30, 2026

Copy link
Copy Markdown
Collaborator

https://github.com/bgruening/cgmlst_filtering_and_clustering/blob/main/README.md
We already have this tool, though! I think they overlap.

Actually this is the one you shared with me before.

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