JuliaDataCubes / YAXArrays.jl

Yet Another XArray-like Julia package
https://juliadatacubes.github.io/YAXArrays.jl/
Other
89 stars 14 forks source link

Inconsistency in syntax between `savecube` and `savedataset` #345

Open Qfl3x opened 9 months ago

Qfl3x commented 9 months ago

Why is the path savecube a "normal" argument, while path in savedataset a kwarg?

lazarusA commented 9 months ago

Indeed. Things should be unified. There is an open issue about it as well: https://github.com/JuliaDataCubes/YAXArrays.jl/issues/309. Some work needs to be done also here.

Qfl3x commented 9 months ago

Another thing is that path isn't a mandatory kwarg, and it has a default value of "", wouldn't that create unexpected behavior?