Infinite loop in coverer builder
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
Comienza en S2RegionCoverer.builder y sigue cómo minLevel y maxLevel se almacenan y se utilizan en getCovering. Reproduce la configuración proporcionada e inspecciona las pruebas existentes de validación del builder o de covering. La tarea estará terminada cuando se impida que la combinación de niveles no válida entre en el bucle infinito de refinamiento y la regresión quede cubierta por una prueba.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
S2RegionCoverer coverer = S2RegionCoverer.builder()
.setMinLevel(20)
.setMaxLevel(10)
.build();
S2Cap cap = S2Cap.fromAxisAngle(new S2Point(1, 0, 0), S1Angle.degrees(5));
S2CellUnion covering = coverer.getCovering(cap); // OutOfMemoryError
The coverer keeps trying to refine cells to reach minLevel(20), but maxLevel(10) prevents going past level 10, so it never satisfies the min level constraint and spirals until the JVM runs out of heap. Could be prevented by checking that min < max
- 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
-
Possible bug in compareEdgesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
google/s2-geometry-library-java#56 · 1 comentario · 1 reacción ·
-
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 183 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
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
waiting-for-triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
spring-cloud/spring-cloud-openfeign#1443 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 84/100
ADORSYS-GIS/keycloak-oid4vp-plugin#221 ·
Los mantenedores suelen responder en 2 días
-
Upgrade to Spring Pulsar 2.0.8Abiertostatus: team-only type: dependency-upgrade
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
spring-projects/spring-boot#52099 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
tchiotludo/akhq#3307 · 1 reacción ·
Los mantenedores suelen responder en 1 día