JuliaDynamics/DrWatson.jl

parse_savename MethodError

Open

#361 opened on Sep 29, 2022

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Julia (94 forks)batch import
bughelp wantedsavename

Repository metrics

Stars
 (915 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

I try encoding symbols with DrWatson.jl savename. The encoding works, but I have troubles getting the NamedTuple back from the encoded string, but only get MethodError on tryparse.

using DrWatson
slc = (site = :LUE, targetlim = :lim_P, scenario = (:prior_sep,))

tmp1 = DrWatson.savename(slc; allowedtypes=(Symbol, Tuple))
slc1 = DrWatson.parse_savename(tmp1, parsetypes = (Symbol, Tuple))

tmp2 = DrWatson.savename(slc; allowedtypes=(Symbol, Tuple), expand=["scenario"])
slc2 = DrWatson.parse_savename(tmp2, parsetypes = (Symbol, Tuple))

Since, I this is my first use of DrWatson, I think its just that I did not fully get the API yet, rather than a bug and posted this first at discourse.

How do I correctly specify the arguments to savename and parse_savename to make the example work for the example slc NamedTuple? I am using Julia 1.8.1 and DrWatson v2.9.1.

Contributor guide

Research direction
Inspect the parse savename function implementation in DrWatson.jl, focusing on how it handles Symbol and Tuple types. Check for existing tests for parsing with parsetypes argument. Consider adding a test case that reproduces the error.
Tech stack
None
Domain
data
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Julia
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70