Have extract_array() accept non-integer dim() outputs
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza localizando extract_array() y la comprobación que rechaza los resultados de dim() que no sean enteros. Revisa cómo big.matrix y otros paquetes de matrices en disco exponen las dimensiones y, después, inspecciona las pruebas existentes relacionadas con extract_array(). Se considera terminado cuando se acepten salidas de dimensiones numéricas sin romper el caso actual de enteros.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- R
- Estrellas
- 29
- Forks
- 12
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Bioconductor/DelayedArray
-
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
Bioconductor/DelayedArray#129 · 4 comentarios ·
-
Custom delayed operations Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Bioconductor/DelayedArray#127 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
Bioconductor/DelayedArray#125 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
Bioconductor/DelayedArray#123 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
Bioconductor/DelayedArray#122 ·
Todos los issues de Bioconductor/DelayedArray
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
r-lib/pkgdepends#485 · 3 comentarios ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
-
beginners blocker
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
enviPathR AbiertoBuild Error Build OK Build Warning policies-accepted pre-review precheck-passed
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
Bioconductor/BiocContributions#207 · 6 comentarios ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
datacarpentry/semester-biology#1255 ·