About Handling Error in cellsToMultiPolygon

Abierto
#158 2 comentarios 1 reacción 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
48/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
c, java
Área
api, backend

Línea de trabajo

Comienza en src/main/java/com/uber/h3core/H3Core.java alrededor de la línea 610 y compara la implementación nativa de cellsToLinkedMultiPolygon en src/h3lib/lib/algos.c alrededor de la línea 1168, junto con el PR de corrección de errores #129. Confirma cómo los índices duplicados producen SIGSEGV y si el resultado previsto es la validación de la entrada, la documentación o un código de error de H3Exception; se considera terminado cuando el comportamiento elegido está cubierto y el fallo deja de ser opaco.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

I discovered that when using the cellsToMultiPolygon method, a SIGSEGV error can occur if there are duplicate indices in the input array. And this issue was addressed in a recent bug fix.

However, since the cellsToLinkedMultiPolygon method is designed to take a set as input, i think it would be beneficial to add a pre-processing step to ensure the input array has unique elements before calling h3Api.cellsToLinkedMultiPolygon.

Alternatively, there should be at least a comment in the code indicating that the input array must contain unique elements. Additionally, it would be helpful to specify an error code for the H3Exception, as it currently returns only 1.

It was really hard for me to find out the reason for the error 😭

Lenguaje dominante
Java
Estrellas
333
Forks
68
Sin PR fusionados en 30 d

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de uber/h3-java

Todos los issues de uber/h3-java

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.