Merge pull request #10 from JuliaDataCubes/zarrattrs

Allow looser dict types when creating Zarr variables

Author: meggart

src/datasets/zarr.jl

@@ -20,8 +20,8 @@ end
 
 function add_var(p::ZarrDataset, T::Type, varname, s, dimnames, attr;
   chunksize=s, kwargs...)
-  attr["_ARRAY_DIMENSIONS"]=reverse(collect(dimnames))
-  za = zcreate(T, p.g, varname, s...;attrs=attr,chunks=chunksize,kwargs...)
+  attr2 = merge(attr,Dict("_ARRAY_DIMENSIONS"=>reverse(collect(dimnames))))
+  za = zcreate(T, p.g, varname, s...;attrs=attr2,chunks=chunksize,kwargs...)
   za
 end
 

test/datasets.jl

@@ -41,8 +41,8 @@ end
 function test_write(T)
     p = tempname()
     ds = create_empty(T, p)
-add_var(ds, 0.5:1:9.5, "lon", ("lon",), Dict{String,Any}("units"=>"degrees_east"))
-add_var(ds, 20:-1.0:1, "lat", ("lat",), Dict{String,Any}("units"=>"degrees_north"))
+add_var(ds, 0.5:1:9.5, "lon", ("lon",), Dict("units"=>"degrees_east"))
+add_var(ds, 20:-1.0:1, "lat", ("lat",), Dict("units"=>"degrees_north"))
 v = add_var(ds, Float32, "tas", (10,20), ("lon", "lat"), Dict{String,Any}("units"=>"Celsius"))
 
 v[:,:] = collect(reshape(1:200, 10, 20))