Skip to content

feat: add create_gnomad_sites_vcf tool#6

Open
ameynert wants to merge 15 commits intomainfrom
am_05_create_gnomad_sites_vcf
Open

feat: add create_gnomad_sites_vcf tool#6
ameynert wants to merge 15 commits intomainfrom
am_05_create_gnomad_sites_vcf

Conversation

@ameynert
Copy link
Copy Markdown
Collaborator

Summary

  • Ports create_gnomad_sites_vcf.py from human-diversity-reference/scripts as a defopt-compatible toolkit tool
  • Exports gnomAD variant sites above a given population frequency threshold as a VCF file, with per-population frequency fields written to the INFO column
  • Renames positional arguments path/vcf_path to sites_table_path/output_vcf_path for clarity

Test plan

  • uv run --directory divref poe check-all passes

🤖 Generated with Claude Code

ameynert and others added 10 commits April 10, 2026 09:59
Add hail, polars, duckdb, pandas, pydantic, and tqdm as Python
dependencies in divref/pyproject.toml. Add mypy ignore_missing_imports
overrides for hail and tqdm (no type stubs). Add openjdk>=11 as a
conda dependency in a new pixi 'hail' feature and include it in the
default environment.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ameynert ameynert had a problem deploying to github-actions-snakemake-linting April 10, 2026 22:58 — with GitHub Actions Failure
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 10, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d9468685-5d6f-426b-b963-bb2f69fa3a1c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch am_05_create_gnomad_sites_vcf

Comment @coderabbitai help to get the list of available commands and usage tips.

ameynert and others added 4 commits April 10, 2026 16:14
Replaces the template setup placeholder with actual project guidance:
environment setup (uv + pixi), common commands, architecture overview,
CLI registration pattern, and tool pipeline execution order.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add HailPath type alias and shared helper functions used across
multiple Hail-based pipeline tools: to_hashable_items, get_haplo_sequence,
variant_distance, and split_haplotypes.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Port rewrite_fasta.py from human-diversity-reference/scripts as a
defopt-compatible toolkit tool. Filters a FASTA file to keep only
canonical chromosomes (chr1-22, X, Y, MT). Fixes a potential
UnboundLocalError by initialising keep=False before the loop and
removes the unused header_line variable from the original.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ameynert ameynert force-pushed the am_04_rewrite_fasta branch from 3c51270 to 88ee1de Compare April 10, 2026 23:17
Port create_gnomad_sites_vcf.py from human-diversity-reference/scripts as a
defopt-compatible toolkit tool. Exports gnomAD variant sites above a given
population frequency threshold as a VCF file with per-population frequency
fields in the INFO column. Renames positional args path/vcf_path to
sites_table_path/output_vcf_path for clarity.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ameynert ameynert force-pushed the am_05_create_gnomad_sites_vcf branch from 7c7eaed to af1052d Compare April 10, 2026 23:17
@ameynert ameynert temporarily deployed to github-actions-snakemake-linting April 10, 2026 23:17 — with GitHub Actions Inactive
@ameynert ameynert force-pushed the am_04_rewrite_fasta branch 3 times, most recently from a2f8fa4 to d07c2c6 Compare April 14, 2026 20:47
Base automatically changed from am_04_rewrite_fasta to main April 14, 2026 20:52
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.

1 participant