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
One might expect that we would want graph dict to be Dict{Int, Vector{Int})
or Dict{String, Vector{String}
but JSON will go to Dict(String, Vector{Any}) by default, and so user will have to convert this manually after for some consistency.
Possible solution
If we wanted integers we could add the line
graph_dict =Dict(eachrow([[parse(Int,x) for x inkeys(graph_dict)] collect(values(graph_dict))]))
likewise for strings, and adjust any conflicts in the get_upstream_basins function
The text was updated successfully, but these errors were encountered:
Issue
ClimaRivers.jl/src/Environments.jl
Line 54 in 52f95ed
One might expect that we would want graph dict to be
Dict{Int, Vector{Int})
or
Dict{String, Vector{String}
but JSON will go to
Dict(String, Vector{Any})
by default, and so user will have to convert this manually after for some consistency.Possible solution
If we wanted integers we could add the line
likewise for strings, and adjust any conflicts in the
get_upstream_basins
functionThe text was updated successfully, but these errors were encountered: