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

cut not clip #4

Closed
Fil opened this issue Feb 22, 2018 · 1 comment · Fixed by #67
Closed

cut not clip #4

Fil opened this issue Feb 22, 2018 · 1 comment · Fixed by #67
Labels
idea

Comments

@Fil
Copy link
Member

Fil commented Feb 22, 2018 •
edited
Loading

Can we use this to cut (instead of clipping) to the polygon.

Would be useful to avoid disappearing points that are just on the edge of the polygon in polyhedral projections.
See d3/d3-geo-projection#132 (comment)
@Fil Fil mentioned this issue Feb 22, 2018
Closed
@Fil
Copy link
Member Author

Fil commented Feb 23, 2018 •
edited
Loading

Proposed API:

  • clip.clipPoint(false) would not clip the points only lines and polygons.

or:

  • clip.pointTolerance(epsilon) keep points close to the edge (default: epsilon)

🤔

@Fil Fil added the idea label Jul 10, 2020
Fil added a commit that referenced this issue Jun 16, 2024
@Fil
clipPoint
137d048 
closes #4
@Fil Fil mentioned this issue Jun 16, 2024
Merged
Fil added a commit that referenced this issue Jun 16, 2024
@Fil
clipPoint
b4f3646 
clipPoint = false is applied to all polyhedrals when the clip polygon covers (almost) the whole sphere

closes #4
Fil added a commit that referenced this issue Jun 23, 2024
@Fil
clipPoint
ab8b33d 
clipPoint: false is applied to all polyhedrals when the clip polygon covers (almost) the whole sphere

closes #4
@Fil Fil closed this as completed in #67 Jun 23, 2024
@Fil Fil closed this as completed in 69963b3 Jun 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
idea
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

clipPoint d3/d3-geo-polygon
1 participant
@Fil