apache / apache/datasketches-java

Compress TDigest can generate centroid with NaN mean

Cerrado
#702 4 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Java
Estrellas
958
Forks
226
Merge medio
3 d 15 h
PR fusionados (30 d)
10

Descripción

See this thread https://github.com/apache/datasketches-rust/pull/23#discussion_r2622754228

Also we don't verify that the centroid mean is not NaN and the weight is not zero (should be positive even?) on deserialization. (Generally we can't assume the data to be deserialized is safe.)

I think the rank/quantile method can return NaN if the calculation result in that. But the stored centroid should not be NaN or else stable sort is unsound?

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Comienza con la discusión del pull request enlazada y sigue las rutas de compresión y deserialización de TDigest mencionadas en el issue. Verifica cómo se validan las medias y los pesos de los centroides, y confirma que los centroides almacenados no pueden contener medias NaN ni pesos no positivos, mientras que el comportamiento de rank/quantile sigue tratándose por separado.

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

Evaluación

Stack tecnológico
java
Área
data
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Activo
Claridad
Bastante claro
Aptitud para principiantes
52/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.