JuliaGPU/CUDA.jl

Should compact `show` mark `CuArray`s?

Open

#1190 aperta il 6 ott 2021

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Julia (274 fork)batch import
enhancementgood first issue

Metriche repository

Star
 (1408 star)
Metriche merge PR
 (Merge medio 5g 5h) (16 PR mergiate in 30 g)

Descrizione

The compact printing used within a Tuple etc. doesn't at present distinguish a CuArray from an ordinary Array:

julia> (; x = cu([1f0]))  # CUDA v3.4.2
(x = Float32[1.0],)

julia> (; x = [1f0])
(x = Float32[1.0],)

Maybe it should? This came up in https://github.com/FluxML/Zygote.jl/issues/1095 , but more generally Zygote likes to return tuples of arrays, and having a visible distinction might be helpful.

This could just be cu(Float32[1.0]) but perhaps that's not right for all eltypes.

Guida contributor

Direzione di ricerca
Indaga sul metodo show per CuArray nel repository CUDA.jl. Cerca il comportamento di stampa compatta, probabilmente in base/show.jl. Il problema suggerisce di modificare come CuArray viene visualizzato all'interno delle tuple, forse aggiungendo un prefisso 'cu'. Controlla i metodi show esistenti per gli array GPU e considera una soluzione che indichi che l'array si trova sulla GPU. Discuti con i maintainer il formato desiderato.
Tech stack
Nessuno
Dominio
developer experience
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.
Abbastanza chiara
Prerequisiti
JuliaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60