google / google/s2-geometry-library-java

Possible bug in compareEdges

Aperta
#56 1 commento 1 reazione 0 assegnatari Vedi su GitHub
Lingua principale
Java
Stelle
587
Fork
231
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

The C++ and Java versions of `compareEdges` don't match as the C++ compare `a1` and `b1`, while the Java compare `b0` and `b1`.
I guess the C++ version is the correct one?

https://github.com/google/s2geometry/blob/fde443b85c018d2991b7459630932add72ef6f76/src/s2/s2edge_crossings.h#L63

Image

https://github.com/google/s2-geometry-library-java/blob/57151fc0f7fe543b75e9f743bf897e81928d5c3f/library/src/com/google/common/geometry/S2EdgeUtil.java#L3101

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.