JuliaDynamics/DrWatson.jl

Savename expand for nested structs is not working

Open

#215 aperta il 19 nov 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Julia (94 fork)batch import
buggood first issuesavename

Metriche repository

Star
 (915 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

As savename works for structs, I think it should also be possible to expand them. This doesn't work at the moment because of https://github.com/JuliaDynamics/DrWatson.jl/blob/1eece239953ab2a5aa6de1c9cb67236df0665264/src/naming.jl#L119 which, unless defined for the struct, throws an error for non-iterable types.

Guida contributor

Direzione di ricerca
Il problema indica la riga 119 in src/naming.jl, dove la funzione savename presuppone che le struct siano iterabili. Le struct annidate potrebbero non essere iterabili, causando errori. Indaga l'implementazione attuale e considera l'aggiunta di un fallback utilizzando Base.fieldnames o simile per espandere le struct non iterabili. Verifica se esiste già una gestione per altri tipi e assicura la compatibilità con le versioni precedenti.
Tech stack
Nessuno
Dominio
databackend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Julia basicsFamiliarity with DrWatson.jl
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60