Order of decomposes coordinates is not counter/clockwise

[yakir12]

The Point2fs that decompose(Point2f, Rect(0, 0, 1, 1)) returns are not sorted in a clockwise order (nor counter clockwise). This is not ideal because plotting a decomposed rectangle doesn't work:

poly(decompose(Point2f, Rect(0, 0, 1, 1)))

nor does plotting polygons with a hole:

using GeometryBasics
p = Polygon(decompose(Point2f, Rect(0, 0, 1, 1)), [decompose(Point2f, Circle(Point2f(0.5), 0.25))])
poly(p)

Only specifying the coordinates manually:

p = Polygon(Point2f[(0,0), (1,0), (1,1), (0,1)], [decompose(Point2f, Circle(Point2f(0.5), 0.25))])

worked:

[ffreyer]

Should probably be considered in JuliaGeometry/GeometryBasics.jl#173

[asinghvi17]

Yeah, I've run into this issue as well....

[ffreyer]

Right, this is probably going to be a part of that pr now. Question is which way? Does Cairo have a preference?

[ffreyer]

I guess with reverse() existing it doesn't matter that much as long as it is consistent?

[asinghvi17]

If you're talking about geospatial, I can find you a standard which fits any winding order you might want to do :D but for now, we could go with the standard counterclockwise order? Cairo is fine with either CW or CCW.

Yeah consistency is definitely the key here!