Skip to content

adds a new workflow for assessing coverage of genomic features#59

Merged
bwlang merged 9 commits into
masterfrom
feature_cov_v2
May 27, 2026
Merged

adds a new workflow for assessing coverage of genomic features#59
bwlang merged 9 commits into
masterfrom
feature_cov_v2

Conversation

@bwlang

@bwlang bwlang commented May 21, 2026

Copy link
Copy Markdown
Member

also includes a visualization of feature coverage

@bwlang bwlang requested a review from adrianreich May 21, 2026 20:00
Comment thread bin/feature_depth_bokeh.py Fixed

@adrianreich adrianreich 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.

Looks good. You may consider adding a screenshot of one of the visualizations in the readme, and perhaps adding a blurb about it there.

Comment thread feature_cov_meth.nf Outdated
Comment thread feature_cov_meth.nf Outdated
Comment thread feature_cov_meth.nf
Comment thread bin/feature_depth_bokeh.py Outdated
Comment thread bin/feature_depth_bokeh.py
Comment thread bin/feature_depth_bokeh.py Outdated
Comment thread feature_cov_meth.nf Outdated
Comment thread feature_cov_meth.nf Outdated
Comment thread feature_cov_meth.nf Outdated
Comment thread feature_cov_meth.nf Outdated
bwlang and others added 8 commits May 24, 2026 10:03
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
feature_depth_bokeh.py

Addresses PR comments:
- Add detailed usage and parameter documentation for feature_cov_meth.nf
  in README.md
- Refactor feature_depth_bokeh.py to generalize library/group parsing
  and color mapping
- Add --strip-suffix option to feature_depth_bokeh.py for flexible BAM
  label handling
- Make genome parameter handling in feature_cov_meth.nf more robust and
  customizable
- Validate required genome parameters and improve error messages for
  custom assemblies
@bwlang bwlang merged commit 47c9fa7 into master May 27, 2026
2 checks passed
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