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