scalanlp/breeze

SparseArray/SparseVector.compact should be split into 2-3 functions

Open

#422 aperta il 23 lug 2015

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Scala (690 fork)batch import
good first issuehelp wanted

Metriche repository

Star
 (3453 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Currently, SparseArray/SparseVector.compact does two separate things which are usually not intended to be done together. It should be split into three functions:

1: Add a def filter(predicate: T => Boolean): Unit function that allows specifying a predicate which is then used to strip certain values from the array/vector without reallocation.

2: Add a function called def tailor(): Unit that cuts down the memory usage to whatever is required to hold the values.

3: For backwards compatibility, replace the function compact with a function that calls filter(x => x == zero.zero) and then tailor().

Guida contributor

Direzione di ricerca
Implementa le tre funzioni: filter, tailor e aggiorna compact per chiamarle entrambe. Assicura la compatibilità all'indietro.
Tech stack
Nessuno
Dominio
backenddata
Tipo issue
Funzionalità
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Scala
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70