JuliaLang/julia

re-evaluate/remove `Random.UnsafeView`

Open

#36.718 aperta il 17 lug 2020

Vedi su GitHub
 (7 commenti) (3 reazioni) (0 assegnatari)Julia (5773 fork)batch import
good first issuehelp wantedrandomnessstdlib

Metriche repository

Star
 (48.709 star)
Metriche merge PR
 (Merge medio 20g 6h) (157 PR mergiate in 30 g)

Descrizione

I just noticed the Random stdlib has an UnsafeView type for filling arrays. With the new layout optimizations, this should no longer be necessary. We should try using the normal SubArray type for this. If it's still too slow/complex, then at least the type can be made safe by holding the underlying array directly instead of a Ptr.

Guida contributor

Direzione di ricerca
Esamina l'implementazione di Random.UnsafeView e il suo utilizzo nella libreria standard. Prova a sostituirlo con SubArray e testa le prestazioni. Se non funziona, modifica UnsafeView per mantenere direttamente l'array sottostante invece di un Ptr.
Tech stack
Nessuno
Dominio
backend
Tipo issue
Refactoring
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
JuliaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35