apache / apache/datasketches-java
Compress TDigest can generate centroid with NaN mean
- 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