Skip to content

feat: log Heatmap sample filtering - #101

Merged
phoman14 merged 2 commits into
DEVfrom
DEV-heatmap-sample-filter-messaging
Oct 13, 2026
Merged

phoman14 merged 2 commits into
DEVfrom
DEV-heatmap-sample-filter-messaging

Conversation

@phoman14

@phoman14 phoman14 commented Oct 13, 2026 •

Copy link
Copy Markdown
Member

Changes

  • Log the exact metadata column and requested sample names before Heatmap filtering.
  • Log displayed filtered/total cell counts after a successful filter.
  • Add focused Heatmap coverage for the custom-column filter log.

Issues

None.

PR Checklist

  • This comment contains a description of changes with justifications, with any relevant issues linked.
  • Write unit tests for any new features, bug fixes, or other code changes.
  • Update the docs if there are any API changes (roxygen2 comments, vignettes, readme, etc.). No API changes.
  • Update NEWS.md with a short description of any user-facing changes and reference the PR number.
  • Run devtools::check() locally and fix all notes, warnings, and errors. Deferred: only modified-function tests were run.

@phoman14
phoman14 merged commit 88f669d into DEV Oct 13, 2026
7 of 12 checks passed
@phoman14
phoman14 deleted the DEV-heatmap-sample-filter-messaging branch October 13, 2026 22: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