CliMA / CliMA/ClimaAnalysis.jl
Add Base.merge and Base.merge! for `NCCatalog`
Open
good first issue
- Dominant language
- Julia
- Stars
- 12
- Forks
- 5
- Avg merge
- 7d 2h
- Merged PRs (30d)
- 2
Description
While looking at the `CompositeDataLoader` in ClimaCoupler (at https://github.com/CliMA/ClimaCoupler.jl/blob/02a374599bd828383a58dd8bf61c8ccb3f04891d/src/CalibrationTools.jl#L211), I realized the code can be simplified if a `merge` function is available.
This function would just merge two `NCCatalog`s together. If there are duplicate names, then we can follow the same behavior for merging dictionaries and the rightmost argument that owns the key gets its value in the dictionary.
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.