Have extract_array() accept non-integer dim() outputs
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia individuando extract_array() e il controllo che rifiuta i risultati di dim() non interi. Esamina come big.matrix e gli altri pacchetti per matrici su disco espongono le dimensioni, quindi controlla i test esistenti relativi a extract_array(). Il lavoro è completo quando gli output delle dimensioni numeriche vengono accettati senza compromettere l'attuale caso degli interi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Upon a little experimentation, it seems the only thing preventing a big.matrix from the bigmemory package to be used as a Seed inside a DelayedArray is that dim() on a big.matrix returns a numeric and not an integer. This is caught by the check in extract_array() raising an error.
Would it be possible to relax this check to accept numeric as well as integer? Many on-disk matrix packages return numeric via dim() (bigmemory, filematrix, feather, fst, etc). It seems inconvenient to have to define a new Seed class for these packages just to reimplement the simple dim() function.
- Lingua principale
- R
- Stelle
- 29
- Fork
- 12
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Bioconductor/DelayedArray
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
Bioconductor/DelayedArray#129 · 11 commenti ·
-
Custom delayed operations Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
Bioconductor/DelayedArray#127 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
Bioconductor/DelayedArray#125 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
Bioconductor/DelayedArray#123 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
Bioconductor/DelayedArray#122 ·
Tutte le issue di Bioconductor/DelayedArray
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
robjhyndman/forecast#1220 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
JamesHWade/deputy#192 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
bug triage_needed
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
pharmaverse/rtables#1123 · 1 commento · 1 reazione ·