daphne-project/daphne

`CSRMatrix::append()` on empty matrix

Aperta

#727 aperta il 16 mag 2024

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (84 fork)auto 404
buggood first issue

Metriche repository

Star
 (80 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina la correzione per DenseMatrix::append() nel commit 2b4f086 e applica modifiche simili a CSRMatrix::append() per gestire righe o colonne zero.
Tech stack
cpp
Dominio
backenddata
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60