Skip to content

fix: default ONT model to QSHMM-ONT-HQ (high-quality)#95

Merged
berntpopp merged 1 commit into
mainfrom
fix/ont-default-hq-model
Apr 7, 2026
Merged

fix: default ONT model to QSHMM-ONT-HQ (high-quality)#95
berntpopp merged 1 commit into
mainfrom
fix/ont-default-hq-model

Conversation

@berntpopp
Copy link
Copy Markdown
Owner

Summary

Changes the default ONT amplicon model from ERRHMM-ONT.model to QSHMM-ONT-HQ.model (quality-score model for modern ONT chemistry like Kit14/Dorado). Updates model_type default from errhmm to qshmm accordingly.

Changes

  • config.json: ont_amplicon_params.model_fileQSHMM-ONT-HQ.model, model_typeqshmm
  • Pipeline, CLI, docs, tests: all defaults updated consistently
  • Added config_experiment.json to .gitignore

Test plan

  • 1,431 tests pass (88.78% coverage)
  • ruff, mypy, format clean
  • CI passes

🤖 Generated with Claude Code

Changed default from ERRHMM-ONT.model to QSHMM-ONT-HQ.model
(quality-score model for modern ONT chemistry like Kit14/Dorado).
Updated model_type default from errhmm to qshmm accordingly.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@berntpopp berntpopp merged commit f32dabb into main Apr 7, 2026
6 checks passed
@berntpopp berntpopp deleted the fix/ont-default-hq-model branch April 7, 2026 12:55
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