JuliaDynamics/DrWatson.jl

Savename expand for nested structs is not working

Open

#215 ouverte le 19 nov. 2020

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Julia (94 forks)batch import
buggood first issuesavename

Métriques du dépôt

Stars
 (915 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Le problème pointe vers la ligne 119 dans src/naming.jl, où la fonction savename suppose que les structures sont itérables. Les structures imbriquées peuvent ne pas être itérables, ce qui provoque des erreurs. Étudiez l'implémentation actuelle et envisagez d'ajouter une solution de repli utilisant Base.fieldnames ou similaire pour développer les structures non itérables. Vérifiez s'il existe déjà une gestion pour d'autres types et assurez la compatibilité ascendante.
Stack technique
Aucun
Domaine
databackend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Julia basicsFamiliarity with DrWatson.jl
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60