apache / apache/datasketches-java
Compress TDigest can generate centroid with NaN mean
- Langage dominant
- Java
- Étoiles
- 958
- Forks
- 226
- Merge moyen
- 3 j 9 h
- PR mergées (30 j)
- 8
Description
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?
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Piste de recherche
Commencez par la discussion de la pull request liée et suivez les chemins de compression et de désérialisation de TDigest mentionnés dans l’issue. Vérifiez comment les moyennes et les poids des centroïdes sont validés, et confirmez que les centroïdes stockés ne peuvent pas contenir de moyennes NaN ni de poids non positifs, tandis que le comportement de rank/quantile reste traité séparément.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java
- Domaine
- data
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- Active
- Clarté
- Plutôt claire
- Accessibilité débutants
- 52/100