Have extract_array() accept non-integer dim() outputs
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 35/100
Rechercherichtung
Beginne damit, extract_array() und die Prüfung zu finden, die Ergebnisse von dim() ablehnt, die keine Ganzzahlen sind. Sieh dir an, wie big.matrix und andere Matrixpakete für auf dem Datenträger gespeicherte Matrizen Dimensionen bereitstellen, und untersuche anschließend die bestehenden Tests rund um extract_array(). Als erledigt gilt die Aufgabe, wenn numerische Dimensionsausgaben akzeptiert werden, ohne den aktuellen Ganzzahlfall zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- R
- Sterne
- 29
- Forks
- 12
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Bioconductor/DelayedArray
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 38/100
Bioconductor/DelayedArray#129 · 11 Kommentare ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
Bioconductor/DelayedArray#127 · 1 Kommentar ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
Bioconductor/DelayedArray#125 · 1 Kommentar ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
Bioconductor/DelayedArray#123 ·
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 20/100
Bioconductor/DelayedArray#122 ·
Alle Issues in Bioconductor/DelayedArray
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
robjhyndman/forecast#1220 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
JamesHWade/deputy#192 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
bug triage_needed
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
pharmaverse/rtables#1123 · 1 Kommentar · 1 Reaktion ·