You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
EDIT: My mistake, this issue isn't directly to do with the clockwise confusion, just examining it in a testbed.
I'm not absolutely sure I understand what you are describing:
In both cases, it seems to be correctly merging two polygons to form one, there are no holes, so the polygon is returned in the same orientation in each case (counterclockwise it seems).
The doc states:
The operation may result in an outer polygon (boundary) and multiple inner polygons (holes) produced which could be distinguished by calling is_polygon_clockwise().
It doesn't actually specify which is which (if I am reading it correctly), so maybe a counter clockwise result is expected.
The doc could arguably be clarified here though to say e.g. counterclockwise for the boundary, and clockwise for the hole. Although this could be confusing due to #92154, so it is understandable why this might not have been mentioned.
In both cases, it seems to be correctly merging two polygons to form one, there are no holes, so the polygon is returned in the same orientation in each case (counterclockwise it seems).
Yes this is the spirit.
In my case, I input some cw polygons and expected a cw one. And I don't know why the union of two Clock-Wise polygons will produce a counter-clock-wise polygon. The document just ignore this point, so my solution is to reserve the output array.
Tested versions
Godot v4.3.dev6.mono
System information
Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3050 Laptop GPU (NVIDIA; 31.0.15.2698) - AMD Ryzen 7 5800U with Radeon Graphics (16 Threads)
Issue description
This code merges two unit (clock-wise / counter-clock-wise) sqaure polygons, but returns the same result.
The result polygons are counter-clock-wise.
According to the document, it should returns a cw polygon if it is a boundary and a ccw one if a hole.
logs:
Steps to reproduce
Create a node, copy and apply the script, and run.
Minimal reproduction project (MRP)
N/A
The text was updated successfully, but these errors were encountered: