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.

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

geoPath.curve ? #199

Open
Fil opened this issue May 21, 2020 · 2 comments
Open

geoPath.curve ? #199

Fil opened this issue May 21, 2020 · 2 comments
Assignees
Labels

Comments

@Fil
Copy link
Member

Fil commented May 21, 2020

It would be great to be able to specify geoPath.curve(d3.curveBasis) to make sketchy maps https://observablehq.com/@d3/context-to-curve

A difficulty might be to find how to automatically switch to curveBasisClosed for rings, so maybe the signature would have to be geoPath.curve(d3.curveBasis, d3.curveBasisClosed); it's a bit ugly.

@Fil Fil self-assigned this May 21, 2020
@Fil Fil added the idea label Jul 10, 2020
@Fil
Copy link
Member Author

Fil commented Jun 6, 2021

It would also make the chart at https://observablehq.com/@d3/contourdensity-linear-binning much nicer

linear curve

image

curveBasisClosed

image

@Fil
Copy link
Member Author

Fil commented Dec 17, 2022

For reference, this is addressed in https://observablehq.com/@d3/context-to-curve
You still need to adjust the curve to the type of shape (closed or not) that you are drawing.

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

No branches or pull requests

1 participant