Skip to content

Remove titles from the subplots#18

Merged
ypriverol merged 2 commits intomainfrom
dev
Mar 18, 2026
Merged

Remove titles from the subplots#18
ypriverol merged 2 commits intomainfrom
dev

Conversation

@ypriverol
Copy link
Contributor

@ypriverol ypriverol commented Mar 18, 2026

Summary by CodeRabbit

  • New Features
    • Added configurable option to remove plot titles from quality control report visualizations including PCA, t-SNE, volcano plots, and summary charts. The option is enabled by default and applies consistently across all report outputs for cleaner formatting while preserving layout properties.

@coderabbitai
Copy link

coderabbitai bot commented Mar 18, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This pull request adds a configurable strip_plot_title parameter (defaulting to True) to the QC report generation functions. When enabled, it removes Plotly figure titles across PCA, t-SNE, volcano, and other plots while preserving layout properties.

Changes

Cohort / File(s) Summary
Title-Stripping Enhancement
src/pyprideap/viz/qc/report.py
Introduced strip_plot_title boolean parameter to qc_report and qc_report_split functions. Implements title removal logic across PCA, t-SNE, volcano, and generic plot rendering paths by setting layout title to empty string. Updates method signatures and docstrings. Backward compatible via default True value.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 A Rabbit's Ode to Tidier Plots

Plot titles vanish with a gentle touch,
Clean visualizations mean so much!
Stripped and polished, lean and bright,
Our QC reports now look just right! 🎨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing plot titles from Plotly figures in the QC report module via a new strip_plot_title parameter.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@ypriverol ypriverol merged commit a8ae914 into main Mar 18, 2026
12 of 13 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Mar 19, 2026
Merged
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