Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UMI-tools: support extract command #2296

Merged
merged 16 commits into from Feb 19, 2024
Merged

UMI-tools: support extract command #2296

merged 16 commits into from Feb 19, 2024

Conversation

vladsavelyev
Copy link
Member

@vladsavelyev vladsavelyev commented Feb 8, 2024

Adds a new subcommand extract, so now both dedup and extract are supported. Adds a general stats column fro the number of extracted reads, as well as a bar plot showing the split between regex matches.

Fix #2284

@vladsavelyev vladsavelyev added awaits-review Awaiting final review and merge. module: enhancement labels Feb 8, 2024
@vladsavelyev vladsavelyev added this to the MultiQC v1.20 milestone Feb 8, 2024
Copy link
Contributor

github-actions bot commented Feb 8, 2024

🚀 Deployed on https://mqc-pr-2296--multiqc.netlify.app

@vladsavelyev vladsavelyev merged commit c5ef275 into main Feb 19, 2024
7 checks passed
@vladsavelyev vladsavelyev deleted the umi-tools-extract branch February 19, 2024 22:50
vladsavelyev added a commit that referenced this pull request Feb 23, 2024
* UMI-tools: support extract command

* [automated] Update CHANGELOG.md

* Add barplot

* Barplot with extract stats

* Doc

* Changelog

* Lint

* Fix duplicated IDs in linting

* Merge conflict

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>
Co-authored-by: Phil Ewels <phil.ewels@seqera.io>
ewels added a commit that referenced this pull request Feb 24, 2024
* Rewrite version checker to use new API endpoint

* Move version_check_url into the config, for local testing

* Version check URL

* Clean up

* Typo

* Parse CI env var

* Fix

* Catch timeout error

* UMI-tools: support `extract` command (#2296)

* UMI-tools: support extract command

* [automated] Update CHANGELOG.md

* Add barplot

* Barplot with extract stats

* Doc

* Changelog

* Lint

* Fix duplicated IDs in linting

* Merge conflict

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>
Co-authored-by: Phil Ewels <phil.ewels@seqera.io>

* Infinite dmax or dmin fail JSON dump load in JavaScript (#2354)

* Infinite dmax or dmin fail JSON dump load in JavaScript

* [automated] Update CHANGELOG.md

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>

* Fix changelog.py for module updates

* Whatshap: robust to stdout appended to TSV (#2361)

* Drop `full_figure_for_development` the call to avoid Kaleido issues (#2359)

* Wrap `full_figure_for_development` in try-except to handle Kaleido errors

* [automated] Update CHANGELOG.md

* Get around read-only /tmp

* Remove the full_figure_for_development call for good, just re-calcualte range from data

* Changelog

* Comments

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>

* PURPLE: support v4.0.1 output without `version` column (#2366)

* Purple: support v4.0.1 output without version column

* [automated] Update CHANGELOG.md

* [automated] Update CHANGELOG.md

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>

* Ignore `.parquet` files. More robust `htseq` file reading loop (#2364)

* Ignore .parquet files. Handle htseq unicode errors

* [automated] Update CHANGELOG.md

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>

* Add box plot (#2358)

* Add box plot

* [automated] Update CHANGELOG.md

* Fix linting

* CSP

* Fix sample order

* Allow overriding title for a dataset tab

* Violin: color dots blue in monochrome violins to indiciate interactivity

* Mono color for boxes, support highliting

* Rename beeswarm to violin

* Barplot: stacking: normal should mean stacked, not grouped

* Update plotting documentation

* Docs

* Redirect beeswarm

* CSP

* Update the flat plot section and remove highcharts mention

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>

* Replace `setup.py` with `pyproject.toml` (#2353)

* Replace setup.py with pyproject.toml

* [automated] Update CHANGELOG.md

* Add myself as author

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>

* Put matplotlib back into main deps (#2370)

* Put matplotlib back into main deps

* [automated] Update CHANGELOG.md

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>

* Generic font family for Plotly (#2368)

* Generic font family for Plotly

* [automated] Update CHANGELOG.md

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>
Co-authored-by: Phil Ewels <phil.ewels@seqera.io>

* Samtools `coverage` submodule (#2356)

* Allow override title per-dataset

* Add samtools coverage submodule

* [automated] Update CHANGELOG.md

* Fix changelog

* Update multiqc/modules/samtools/rmdup.py

Co-authored-by: Phil Ewels <phil.ewels@seqera.io>

* Fix rmdup regex

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>
Co-authored-by: Phil Ewels <phil.ewels@seqera.io>

* Update `modules.md`: `pyproject.toml`, `"shared_key": "read_count"`, ruffify (#2371)

* Update modules.md: `pyproject.toml`, `"shared_key": "read_count"`, ruffify

* [automated] Update CHANGELOG.md

* Apply suggestions from code review

---------

Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>
Co-authored-by: Phil Ewels <phil.ewels@seqera.io>

---------

Co-authored-by: Vlad Savelyev <vladislav.sav@gmail.com>
Co-authored-by: MultiQC Bot <multiqc-bot@seqera.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaits-review Awaiting final review and merge. module: enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UMI-Tools: support extract command
3 participants