Skip to content

Standardize filter output figures across MOSuite function calls#3

Open
phoman14 wants to merge 10 commits into
mainfrom
FigOutSync
Open

Standardize filter output figures across MOSuite function calls#3
phoman14 wants to merge 10 commits into
mainfrom
FigOutSync

Conversation

@phoman14

@phoman14 phoman14 commented Jul 2, 2026

Copy link
Copy Markdown
Member

Summary

This PR aligns the filter-counts capsule with the package-level figure standardization work. The intent is for figures produced from this capsule to match figures produced by the corresponding MOSuite function calls, rather than carrying capsule-specific color or legend defaults.

Changes

  • Use the same default plot color palette as the MOSuite package for filter-counts outputs.
  • Leave the histogram legend font size blank by default so filter_counts() can use the package's automatic legend text scaling for many or long labels.
  • Parse an explicit histogram legend font size only when the user provides one.
  • Keep the capsule parameter descriptions focused on the user-facing defaults.
  • Update CHANGELOG.md for the user-facing default changes.

Validation: local capsule tests passed (PASS 7).

Issues

No linked issue.

PR Checklist

(Strikethrough any points that are not applicable.)

  • This comment contains a description of changes with justifications, with any relevant issues linked.
  • Run the capsule in Code Ocean and ensure it completes successfully with no errors. Local capsule tests passed.
  • Update CHANGELOG.md with a short description of any user-facing changes and reference the PR number. Follow the style described in https://style.tidyverse.org/news.html
  • The tests workflow succeeds on the latest commit.

@phoman14 phoman14 changed the title Align filter plot defaults with MOSuite package Standardize filter output figures across MOSuite function calls Jul 2, 2026
@phoman14 phoman14 requested a review from kelly-sovacool July 2, 2026 20:00
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