Skip to content

docs: add Claude Code context files and DRAGEN pipeline reference - #39

Open
joshuamschmidt wants to merge 3 commits into
mainfrom
feat/claude
Open

docs: add Claude Code context files and DRAGEN pipeline reference#39
joshuamschmidt wants to merge 3 commits into
mainfrom
feat/claude

Conversation

@joshuamschmidt

Copy link
Copy Markdown

Summary

  • Adds CLAUDE.md — Claude Code project instructions giving the AI assistant full context on architecture, the stage DAG, key modules, configuration, and how to navigate the DRAGEN 3.7 guide. Includes a section map of guide line numbers for topics relevant to this pipeline (QC outputs, caller options, input/reference, full option reference).
  • Adds docs/dragen_guide.md — the full DRAGEN 3.7 user guide (21 103 lines), checked in so Claude Code can Read specific sections by offset without hitting the filesystem outside the repo.
  • Adds docs/dragen_pipeline_reference.md — a concise, repo-maintained reference for our specific pipeline configuration: enabled modules, mode-specific inputs, expected output files (QC CSVs, variant/call files, what's excluded from bulk download), ICA parameter conventions, and config key reference.
  • docs/dragen_pipeline_reference.md also lists ICA help doc URLs in .md format that Claude Code can fetch directly via WebFetch for live ICA platform documentation (API reference, Nextflow pipeline details, CLI data transfer, etc.) without needing a checked-in copy.

Why

This helps those who use our DRAGEN outputs and potential contributors (and Claude) understand what DRAGEN outputs to expect, what gets downloaded vs skipped, and how ICA parameters map to DRAGEN CLI flags.
Previously this lived only in the 21k-line DRAGEN guide and in scattered code comments. This surfaces the relevant subset in a single place and keeps it versioned with the code.

joshuamschmidt and others added 2 commits April 28, 2026 12:35
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

@michael-harper michael-harper left a comment

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.

This looks good and will definitely save tokens. I'm wondering if docs/dragen_guide.md is actually allowed to be distributed like this, is it proprietary?

--qc-coverage-count-soft-clipped-bases true
--qc-coverage-reports-1 cov_report,cov_report
--qc-coverage-filters-1 'mapq<1,bq<0,mapq<1,bq<0'
--vc-gvcf-gq-bands 13 20 30 40

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.

These have just been changed to 10 20 30 40

```
--qc-coverage-reports-1 cov_report,cov_report
--qc-coverage-filters-1 'mapq<1,bq<0,mapq<1,bq<0'
--vc-gvcf-gq-bands 13 20 30 40

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.

These have just been changed to 10 20 30 40

### QC metric CSVs — expected

MultiQC collects these via `rglob('*.csv')` — there is no validation step; missing files are silently
absent from the report rather than causing an error.

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.

I believe MultiQC is going to be removed from this pipeline as it lives in the single_sample_qc_popgen pipeline

| gVCF mode | `vc_emit_ref_confidence` | `GVCF` | gVCF only; excluded from bulk download |
| VCF output | `vc_enable_vcf_output` | `false` | No VCF, only gVCF |
| CNV calling | `enable_cnv` | `true` | |
| CNV segmentation | `cnv_segmentation_mode` | `SLM` | Shifting Level Models |

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.

Just a note, in WGS this is SLM, in WES it is recommended to set to HSLM (Heterogeneous SLM). How would we like to capture these kinds of differences?

Comment thread CLAUDE.md
pytest test

# Bump version (updates Dockerfile, config TOML, README, creates git tag)
bump2version <major|minor|patch>

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.

This might differ among systems. I use bump-myversionbecause it looked likebumpversionandbump2versnio` were both deprecated when I set it up originally.

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.

3 participants