prantogg commented on issue #53: URL: https://github.com/apache/sedona-spatialbench/issues/53#issuecomment-3481704315
Thanks for surfacing this @timbow78. I ran some checks on the SF1000 buildings and zones: - All geometries are closed - ~0.4% of buildings (889 out of 219k) are invalid due to `ST_Area = 0` - likely caused by degenerate polygons from the random coordinate generator. I’ll patch this. - The rest pass Sedona's `ST_IsValid`, which under the hood uses JTS’s `IsValidOp` with standard OGC SFS rules — it checks topology (e.g. self-intersections, closure), but not ring orientation. - Buildings have mixed winding (mostly clockwise); zones are all CCW. I couldn’t find where OGC strictly enforces ring orientation - some engines like Oracle and PostGIS interpret it more strictly though, so I’ll update the generator to standardize on CCW for exterior rings. Appreciate you flagging this! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
