You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The in memory version of run_omniscape has type Dict{String, String} for the cfg arg. Might it make sense to allow Bool values too, for e.g. calc_normalized_current?
The text was updated successfully, but these errors were encountered:
It would be easy to update the dict type requirement Dict{String, Any} or Dict{String, Union{String, Bool}, but we'd have to make sure we're parsing things correctly. My original thought was that it's a bit easier to just enforce String as the value type for all keys so we always know what type to expect when parsing things.
The in memory version of
run_omniscape
has typeDict{String, String}
for thecfg
arg. Might it make sense to allow Bool values too, for e.g.calc_normalized_current
?The text was updated successfully, but these errors were encountered: