Skip to content

Add Polar Charts and full axis support #113

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

Merged
merged 19 commits into from
Aug 13, 2021
Merged

Add Polar Charts and full axis support #113

merged 19 commits into from
Aug 13, 2021

Conversation

kMutagene
Copy link
Collaborator

@kMutagene kMutagene commented Aug 8, 2021

This PR adds many things for #42 and #44 :

  • Full support for the polar object on the chart layout
  • Full Radial Axis coverage
  • Full Angular Axis coverage
  • Full Linear Axis coverage
  • Unified Axis parameter casing
  • All polar charts based on the plotly.js 2.0 spec (current ones are deprecated):
    • Logic to render scatterpolar derived traces via WegGL
    • Chart.ScatterPolar
    • Chart.PointPolar
    • Chart.LinePolar
    • Chart.SplinePolar
    • Chart.BubblePolar
    • Chart.BarPolar
  • A fix on the title object. it now uses the correct, non-deprecated way to set any kind of title (a title object instead of just setting a string)
  • Docs
  • Tests

This PR
closes #115
closes #116
closes #117
closes #118
closes #119

This is breaking AF btw.

@kMutagene
Copy link
Collaborator Author

ScatterPolar derived traces with a few options set on the polar object:

image

[
    Chart.PointPolar(r,t,Name="PointPolar")
    Chart.LinePolar(r2,t,Name="LinePolar", ShowMarkers = true)
    Chart.SplinePolar(r3,t,Name="SplinePolar", ShowMarkers = true)
]
|> Chart.Combine
|> Chart.withPolar(
    Polar.init(
        Sector= (0., 270.),
        Hole=0.1
    )
)
|> Chart.withAngularAxis(
    Axis.AngularAxis.init(
        Color="darkblue"
    )
)
|> Chart.withRadialAxis(
    Axis.RadialAxis.init(
        Title = Title.init("Hi, i am the radial axis"),
        Color="darkblue",
        SeparateThousands = true
    )
)
|> Chart.Show

@kMutagene kMutagene linked an issue Aug 9, 2021 that may be closed by this pull request
@kMutagene kMutagene marked this pull request as ready for review August 13, 2021 12:25
@kMutagene kMutagene merged commit 2ef7e17 into dev Aug 13, 2021
@kMutagene kMutagene deleted the polar branch October 21, 2021 06:36
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.

Title Polar Bar Polar (reference) Scatter Polar GL (reference) Scatter Polar (reference)
1 participant