Repository navigation
Add sativaepang test fixtures (tiny + GTDB small tier) - #2266
Merged
Merged
Conversation
Two tiers for sativaepang/{reference,lootasks,looplace,looscore} module
tests: the tool's own bundled 38-sequence example (fast smoke test), and
the existing curated GTDB archaeal 16S dataset from the sativa branch
(121 sequences, with a deliberate injected mislabel pair the full
leave-one-out chain correctly flags at Phylum level).
Also includes pre-built RAxML-NG tree+model and sativa-epang refjson+
model fixtures for both tiers, since each downstream module's test
takes the prior stage's real output as a static input rather than
chaining live processes.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014eSqkuYZXfNiWy87y3srup
danilodileo
approved these changes
Sep 9, 2026
danilodileo
left a comment
danilodileo
left a comment
There was a problem hiding this comment.
LGTM, is it right to merge to nf-core/modules repo?
Member
Author
Yes, this is what we're supposed to do for modules. I'm adding a set of modules in a minute that will use this data. |
13 of 14 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR checklist
README.mdREADME.mdis still up-to-dateDescription
Test fixtures for four new modules being proposed to nf-core/modules
(
sativaepang/reference,sativaepang/lootasks,sativaepang/looplace,sativaepang/looscore, wrapping Aaramis/sativa-epang)and the
fasta_tax_detectmislabels_sativaepangsubworkflow that chains them withraxmlng/search.Added under
data/delete_me/sativaepang/(956K total), two tiers:test), plus a RAxML-NG tree/model built from it, for fast smoke testing.
injected mislabel pair (swapped taxonomy on two maximally-divergent sequences), used
for a real mislabel-detection assertion rather than just "the process completed".
Full provenance/generation details for every file are in
data/delete_me/sativaepang/README.md.🤖 Generated with Claude Code
https://claude.ai/code/session_014eSqkuYZXfNiWy87y3srup