Skip to content

Comments

fix histogram in polaraxes in CairoMakie#5511

Draft
bjarthur wants to merge 1 commit intoMakieOrg:masterfrom
bjarthur:bja/cairopolarhist
Draft

fix histogram in polaraxes in CairoMakie#5511
bjarthur wants to merge 1 commit intoMakieOrg:masterfrom
bjarthur:bja/cairopolarhist

Conversation

@bjarthur
Copy link
Contributor

fixes #3175

tests pass locally.

gemini wrote this. seems reasonable but i'm not that familiar with this part of Makie so please review thoroughly.

@github-project-automation github-project-automation bot moved this to Work in progress in PR review Jan 28, 2026
@ffreyer
Copy link
Collaborator

ffreyer commented Jan 29, 2026

#5412 also fixes this iirc

@bjarthur
Copy link
Contributor Author

#5412 is close! the r axis though doesn't start at 0. is that maybe why it isn't merged yet? is there a workaround using maybe rlimits or rticks?

Screenshot 2026-01-29 at 8 08 48 AM

with this code from #3175

f = Figure()
ax = PolarAxis(f[1, 1])
p = hist!(ax, 2pi .* rand(300), strokewidth = 1, bins = range(0, 2pi, length=37)) # direction = :x for ax.theta_as_x = false
f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Work in progress

Development

Successfully merging this pull request may close these issues.

PolarAxis histogram fails in CairoMakie

2 participants