Skip to content
This repository has been archived by the owner on Nov 22, 2023. It is now read-only.

Polygon type? #124

Open
goretkin opened this issue Nov 30, 2017 · 1 comment
Open

Polygon type? #124

goretkin opened this issue Nov 30, 2017 · 1 comment

Comments

@goretkin
Copy link
Contributor

Should there be an explicit Polygon type? Right now the code uses AbstractArray{Point} and Vector{Point} (in only one place each) to describe a Polygon.

@SimonDanisch
Copy link
Member

Yes, that would be appreciated! We had some discussions about this, but never came to a conclusion, since people seem to have very different needs for polygons. I was thinking to make Vector{Vector{Point}} our first polygon type for now, since that's what e.g. https://github.com/JuliaGeometry/EarCut.jl expects right now, and I wish to integrate that better into the geometry infrastructure! But feel free to go ahead with any polygon type you want! We are pretty undecided on the exact definition, so it's a bit first come, first serve kind of situation ;)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants