Skip to content

Including subgroup size#20

Merged
leanayu merged 6 commits into
mainfrom
ly/17-include-n
Jun 18, 2026
Merged

Including subgroup size#20
leanayu merged 6 commits into
mainfrom
ly/17-include-n

Conversation

@leanayu

@leanayu leanayu commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Description

  • Addresses issue Include "n=" #17
  • Added the subgroup size under each normative plot title: the sample size is dynamically computed based on the current UI filters (sex, age range, and vertebral level).

Changes
Added a helper function in callbacks.py to compute the subgroup size from the raw data.

image

@valosekj

Copy link
Copy Markdown
Member

Thanks! Looks good!

Just an idea: how about showing "n" also for the "Demographics" figure?

Screenshot 2026-06-15 at 2 59 30 PM

@valosekj

valosekj commented Jun 15, 2026

Copy link
Copy Markdown
Member

it might be nice to link the raincloudplot sex selection with the table below

image

UPDATE: low priority, skip for now

@leanayu

leanayu commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator Author

Now, N appears when hovering the plots.
For example, in the heatmap:
image

Or, in the raincloud plot:
image

@valosekj

Copy link
Copy Markdown
Member

Nice! Looks good!

@leanayu leanayu merged commit 497c380 into main Jun 18, 2026
@leanayu leanayu deleted the ly/17-include-n branch June 18, 2026 15:25
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