-
-
Notifications
You must be signed in to change notification settings - Fork 313
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allow segment-specific radius
and inner_radius
for pie
#4028
Conversation
Yes! I have transfered the fork into my own account, and see the checkbox now. Hope it works now! Thank you! |
pie!(ax, 20, 0, vs; radius=rs_inner, inner_radius=0, kw..., color=lp) | ||
pie!(ax, 20, 0, vs; radius=rs, inner_radius=rs_inner, kw..., color=Makie.wong_colors(0.8)[eachindex(vs)]) | ||
|
||
display(fig) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is causing the test failures?
display(fig) | |
fig |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! fixed.
Something is wrong with docs here. 360m runtime... |
seems that similar issues appear in docs/src/reference/plots/pie.md :( |
The test has passed. Thanks! |
This PR is based on #4027, and thus please don't merge it before #4027 is finished. [Update: Done]
I extend the
radius
andinner_radius
ofpie
so that we now can easily plot the following figure: