Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Pie charts #1

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Pie charts #1

wants to merge 1 commit into from

Conversation

arcresu
Copy link
Contributor

@arcresu arcresu commented May 5, 2023

My initial attempt at pie charts needs some more polish before it gets added back in. Specifically

  1. It should be inset-aware and probably based on StatAutomap to position the charts.
  2. It should be able to draw the pies as properly circular. There are some interactions with the coordinate projection that aren't working but I couldn't find a combination of transformations that would keep the sectors in the right place and also avoid distortions. It would be nice to make it more of an annotation that doesn't need to be aware of the CRS at all.
  3. There should be a smarter default for the radius.

pies-1

@arcresu
Restore pie charts
56478a3 
This reverts commit 8715b45.
@arcresu arcresu added the enhancement New feature or request label May 5, 2023
@arcresu arcresu changed the title Restore pie charts Pie charts May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@arcresu