CommonDatamodel.parentdataset for root group for a multi-file should …

…be nothing
JuliaGeo · Feb 16, 2024 · e95dc06 · e95dc06
1 parent 16d30cc
commit e95dc06
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/dataset.jl b/src/dataset.jl
@@ -17,7 +17,14 @@ only a single group, this will be always the root group `"/"`.
 """
 name(ds::AbstractDataset) = "/"
 
+"""
+    pds = CommonDatamodel.parentdataset(ds::AbstractDataset)
+
+The data set `pds` containing `ds` as a sub-group.
+`pds` is nothing for the root group.
+"""
 parentdataset(ds::AbstractDataset) = nothing
+
 sync(ds::AbstractDataset) = nothing
 
 """

diff --git a/src/multifile.jl b/src/multifile.jl
@@ -62,7 +62,12 @@ end
 
 function parentdataset(mfds::MFDataset)
     ds = parentdataset.(mfds.ds)
-    return MFDataset(ds,mfds.aggdim,mfds.isnewdim,mfds.constvars)
+
+    if isnothing(ds[1])
+        return nothing
+    else
+        return MFDataset(ds,mfds.aggdim,mfds.isnewdim,mfds.constvars)
+    end
 end
 
 Base.Array(v::MFVariable) = Array(v.var)