daphne-project/daphne

`CSRMatrix::append()` on empty matrix

Aberta

#727 aberto em 16 de mai. de 2024

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)C++ (84 forks)auto 404
buggood first issue

Métricas do repositório

Stars
 (80 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

The current implementation of the append mechanism of CSRMatrix (prepareAppend(), append(), finishAppend()) seems to be incorrect when the number of rows and/or the number of columns of the matrix is zero.

A similar problem existed for DenseMatrix and was fixed in 2b4f086ce45df10419c00d05e56433c30999a91a.

Thanks to @AlexRTer for pointing out this problem in #722.

Guia do colaborador

Direção de pesquisa
Examine a correção para DenseMatrix::append() no commit 2b4f086 e aplique alterações semelhantes a CSRMatrix::append() para lidar com zero linhas ou colunas.
Pilha de tecnologia
cpp
Domain
backenddata
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60