JuliaDynamics/DrWatson.jl

Savename expand for nested structs is not working

Open

#215 aberto em 19 de nov. de 2020

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)Julia (94 forks)batch import
buggood first issuesavename

Métricas do repositório

Stars
 (915 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
O problema aponta para a linha 119 em src/naming.jl, onde a função savename assume que structs são iteráveis. Structs aninhadas podem não ser iteráveis, causando erros. Investigue a implementação atual e considere adicionar um fallback usando Base.fieldnames ou similar para expandir structs não iteráveis. Verifique se já existe tratamento para outros tipos e garanta a compatibilidade com versões anteriores.
Pilha de tecnologia
None
Domain
databackend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Julia basicsFamiliarity with DrWatson.jl
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60