apache / apache/datasketches-java

Compress TDigest can generate centroid with NaN mean

Fechada
#702 4 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
Java
Estrelas
958
Forks
226
Merge médio
3d 40min
PRs com merge (30d)
13

Descrição

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?

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Direção de pesquisa

Comece pela discussão da pull request vinculada e rastreie os caminhos de compactação e desserialização do TDigest mencionados na issue. Verifique como as médias e os pesos dos centroides são validados e confirme que os centroides armazenados não podem conter médias NaN nem pesos não positivos, enquanto o comportamento de rank/quantile continua sendo tratado separadamente.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
java
Domínio
data
Tipo de issue
Bug
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Ativa
Clareza
Razoavelmente clara
Facilidade para iniciantes
52/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.