apache / apache/datasketches-java

Compress TDigest can generate centroid with NaN mean

Fermée
#702 4 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.