Ben Guild

I’ve been playing around with S2 and wanted to suggest updating the documentation (perhaps via a pull request) to highlight that `S2Polygon` is considered obsolete on the page that describes...

Took a shot at this, but I'm not sure about the polygon building. Needs code review and thoughts. Noting the absence of `S2Builder` from C++ so it's not a clear/direct...

"CentroidCellID()" method for `CellUnion`

1

I find this useful occasionally, but open to thoughts including about whether it belongs in the library.

Removing "earth" overlap from S2 tests

1

Noting the "circular" implementation and import conflict in https://github.com/golang/geo/pull/239 … removing the "earth" subject matter (and their own tests like "TestKmToAngle") from the S2 tests to instead be about general...

I find this useful occasionally, but open to thoughts including about whether it belongs in the library.

Two neighbor cells give error with "cellsToLinkedMultiPolygon"

6

Hi, I was playing around with this, and noticed that there seems to be a bug with two neighboring cells in the ocean? ### Minimal C Example ```c #include #include...

"polygonToCellsExperimental" can give incorrect results near antimeridian

1

Related to #1046, I was playing around with this library and realized that there presently seem to be some subtleties in how -180º longitude is handled compared to +180º …because...

"cellsToMultiPolygon" occasionally produces degenerate polygons with only 1 vertex

4

For example, when calling with a specific set of 164 valid resolution-2 cells, the function returns 14 polygons where: - **12 polygons are valid** (≥ 3 vertices) - **2 polygons...

Possibly related to #1029, #1046, and #1048… both calls below return`E_FAILED` codes. ### Demo ```c #include #include #include int main() { LatLng verts[] = { {.lat = degsToRads(-89.99993255000588), .lng =...