Skip to content

docs: rendering the heatmap properly in ehrapy longitudinal notebook - #1094

Merged
eroell merged 7 commits into
mainfrom
docs/heatmap-rendering
Jul 15, 2026
Merged

docs: rendering the heatmap properly in ehrapy longitudinal notebook#1094
eroell merged 7 commits into
mainfrom
docs/heatmap-rendering

Conversation

@eroell

@eroell eroell commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Reran the notebook. Closes #1090

@github-actions github-actions Bot added the chore label Jul 14, 2026
@eroell
eroell marked this pull request as draft July 14, 2026 16:17
@eroell
eroell marked this pull request as ready for review July 15, 2026 11:31
@eroell
eroell requested a review from Zethson July 15, 2026 11:31
@Zethson

Zethson commented Jul 15, 2026

Copy link
Copy Markdown
Member
image

I thought that's fixed now in EHRData. Have you rerun this with the latest EHRData version? If not, maybe we can rerun it with the EHRData version on main so that the notebook is clean? I guess you'll make another release super soon anyways.

@Zethson

Zethson commented Jul 15, 2026

Copy link
Copy Markdown
Member

I don't think there's anything we can do here but it's super hard to find the options on this plot to e.g. save or reset it:

image

The 3 dots are almost invisible

@Zethson Zethson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much! I actually used the preview URL yesterday so your work was not in vain and the timing was right.

@eroell

eroell commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator Author

I thought that's fixed now in EHRData

It is, just mixed up a checked out branch. Looks pretty now

The 3 dots are almost invisible

aye, first impression is that this is hard to modify

@eroell
eroell merged commit 5c69f54 into main Jul 15, 2026
20 checks passed
@eroell
eroell deleted the docs/heatmap-rendering branch July 15, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Heatmap doesn't render

2 participants