Possible bug in compareEdges
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 62/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Área
- computer-graphics
Línea de trabajo
Comienza con compareEdges en library/src/com/google/common/geometry/S2EdgeUtil.java alrededor de la línea 3101 y luego compáralo con src/s2/s2edge_crossings.h alrededor de la línea 63 en la implementación de C++ referenciada. Sigue los argumentos circundantes y las pruebas existentes para determinar qué comparación se pretende; la tarea está terminada cuando el comportamiento de Java coincide con la implementación verificada y las pruebas pasan.
Escrito por el modelo de indexación a partir del texto del issue.
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?
- Lenguaje dominante
- Java
- Estrellas
- 587
- Forks
- 232
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de google/s2-geometry-library-java
-
Infinite loop in coverer builderAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Build failure when mvn compileAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
google/s2-geometry-library-java#47 · 2 comentarios ·
-
Added new ant target docs: "Generate documentation"Posiblemente ocupada @ThomasThelen la tomó hace 180 días. Abiertoauto-migrated Type-Patch
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
google/s2-geometry-library-java#2 · 1 comentario ·
-
FR: Add bazel supportAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
-
S1ChordAngleTest FailureAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
google/s2-geometry-library-java#44 · 9 comentarios ·
Todos los issues de google/s2-geometry-library-java
Issues similares
-
bug status: needs triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
LawnchairLauncher/lawnchair#7360 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Temporal transforms put pre-epoch timestamps at `.999999` into the previous unitPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
apache/iceberg#18371 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
objectionary/hone-maven-plugin#1297 ·
Los mantenedores suelen responder en 1 día
-
bug documentation iam
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
objectionary/lints#1541 ·
Los mantenedores suelen responder en 1 día