google / google/s2-geometry-library-java
Possible bug in compareEdges
- Lenguaje dominante
- Java
- Estrellas
- 587
- Forks
- 231
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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
https://github.com/google/s2-geometry-library-java/blob/57151fc0f7fe543b75e9f743bf897e81928d5c3f/library/src/com/google/common/geometry/S2EdgeUtil.java#L3101
Guía de contribución
Línea de trabajo
Start with compareEdges in library/src/com/google/common/geometry/S2EdgeUtil.java around line 3101, then compare it with src/s2/s2edge_crossings.h around line 63 in the referenced C++ implementation. Trace the surrounding arguments and existing tests to determine which comparison is intended; done means the Java behavior matches the verified implementation and tests pass.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp, java
- Área
- computer-graphics
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 62/100