Skip to content

Taxonomic Update for obitools4 #7921

Open
kpayet26 wants to merge 27 commits into
galaxyproject:mainfrom
kpayet26:obitools4_taxonomic_update
Open

Taxonomic Update for obitools4 #7921
kpayet26 wants to merge 27 commits into
galaxyproject:mainfrom
kpayet26:obitools4_taxonomic_update

Conversation

@kpayet26
Copy link
Copy Markdown
Contributor

Hello,
This PR add a few more tools for taxonomic attribution, manage taxonomic database and more.
Tools added : obitaxonomy, obipcr, obitag, obirefidx and obicomplement.
Additional options for the handling of taxonomic databases will be integrated into all prior tool versions.

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.

@kpayet26
Copy link
Copy Markdown
Contributor Author

I think I need that skip-version-check-label, and I'm not able to add it myself. Can someone help me with it?

@bgruening @bernt-matthias

change the test in obirefidx and obitag
@bernt-matthias bernt-matthias added the skip-version-check Allow IUC members to skip the version linter in PR reviews (use only for partially updated suites). label Apr 24, 2026
Comment thread tools/obitools/obicomplement.xml Outdated
Comment thread tools/obitools/obipcr.xml Outdated
Comment thread tools/obitools/obipcr.xml Outdated
Comment thread tools/obitools/obirefidx.xml Outdated
Comment thread tools/obitools/obirefidx.xml Outdated
Comment thread tools/obitools/obitaxonomy.xml Outdated
Comment thread tools/obitools/obitaxonomy.xml
@kpayet26 kpayet26 marked this pull request as ready for review May 4, 2026 12:18
@kpayet26 kpayet26 requested a review from bernt-matthias May 7, 2026 12:29
Comment thread tools/obitools/macros.xml Outdated
Comment thread tools/obitools/macros.xml Outdated
Comment thread tools/obitools/macros.xml Outdated
Comment thread tools/obitools/macros.xml Outdated
</param>
<when value="no_taxo"></when>
<when value="use_taxo">
<param name="rank" type="select" label="Restrict to the given taxonomic rank" multiple="true" optional="true">
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.

You can also reuse the taxo_macro here. You could add this additional parameter here by using <yield/> (maybe named): see https://planemo.readthedocs.io/en/latest/writing_advanced.html#parameterizing-xml-macros-with-yield

Copy link
Copy Markdown
Contributor Author

@kpayet26 kpayet26 May 13, 2026

Choose a reason for hiding this comment

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

Tried something using <yield/>. Now there is only one single taxo_macro. Please let me know if it's okay.

Comment thread tools/obitools/obicomplement.xml Outdated
Comment thread tools/obitools/obipcr.xml Outdated
Comment thread tools/obitools/obirefidx.xml Outdated
Comment thread tools/obitools/obitag.xml Outdated
Comment thread tools/obitools/obitag.xml Outdated
Comment thread tools/obitools/obitag.xml Outdated
@kpayet26 kpayet26 requested a review from bernt-matthias May 18, 2026 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-version-check Allow IUC members to skip the version linter in PR reviews (use only for partially updated suites).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants