Skip to content

v0.10.3: per-chemistry flexiplex edit distances and config file reorg… - #51

Merged
ch99l merged 1 commit into
develfrom
bug_fix_flexiplex
Oct 5, 2026
Merged

ch99l merged 1 commit into
develfrom
bug_fix_flexiplex

Conversation

@ch99l

@ch99l ch99l commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator

…anisation

@ch99l
ch99l requested a balanced review from Copilot October 5, 2026 02:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The configuration reorganization is consistently applied across workflow consumers, with no unresolved correctness issues found.

Review effort: Balanced
Findings: None

What changed in this PR

Moves chemistry configuration files into assets/ and makes Flexiplex edit distances chemistry-configurable.

Changes:

  • Added per-chemistry Flexiplex flank and barcode edit distances.
  • Renamed and reorganized preprocessing configuration files.
  • Removed superseded global Flexiplex parameters.
File Description
modules/​preprocess_fastq.nf Loads chemistry-specific preprocessing settings.
main.nf Uses reorganized asset paths.
conf/​dev.config Removes obsolete Flexiplex parameters.
CHANGELOG.md Documents configuration changes.
assets/​flexiplex_config.csv Adds per-chemistry edit distances.
assets/​cutadapt_config.csv Provides renamed adapter configuration.
assets/​barcode_coordinate_config.csv Relocates barcode and spatial paths.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ch99l
ch99l merged commit f026c26 into devel Oct 5, 2026
23 checks passed
@ch99l
ch99l deleted the bug_fix_flexiplex branch October 5, 2026 02:57
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