Skip to content

Commit 40726b8

Browse files
rlyclaude
andauthored
Fix load_namespace when default_type_map is None (#95)
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent c75cd0e commit 40726b8

2 files changed

Lines changed: 15 additions & 4 deletions

File tree

hdmf_docutils/doctools/renderrst.py

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
from collections import OrderedDict
88
from .rst import RSTDocument, RSTTable, RSTSectionLabelHelper
99
import os
10+
import warnings
1011

1112

1213
class DataTypeSection(dict):
@@ -328,7 +329,17 @@ def render_namespace(namespace_catalog,
328329
ns_src_label = "hdmf-type-namespace-src"
329330
# Create the target doc
330331
if namespace_name is None:
331-
namespace_name = namespace_catalog.default_namespace
332+
core = namespace_catalog.core_namespaces
333+
if not core:
334+
raise ValueError(
335+
"namespace_name not provided and namespace_catalog has no core namespaces"
336+
)
337+
if len(core) > 1:
338+
warnings.warn(
339+
"namespace_name not provided; namespace_catalog has multiple core "
340+
"namespaces %r. Using the first (%r)." % (core, core[0])
341+
)
342+
namespace_name = core[0]
332343
curr_namespace = namespace_catalog.get_namespace(namespace_name)
333344
# Section heading
334345

hdmf_docutils/generate_format_docs.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -118,10 +118,10 @@ def load_namespace(namespace_file,
118118
pass
119119
# Load the namespace separately if it already exists or we don't have a default type map specified
120120
if namespace_catalog is None:
121-
namespace_catalog = NamespaceCatalog(default_namespace,
122-
group_spec_cls=group_spec_cls,
121+
namespace_catalog = NamespaceCatalog(group_spec_cls=group_spec_cls,
123122
dataset_spec_cls=dataset_spec_cls,
124-
spec_namespace_cls=spec_namespace_cls)
123+
spec_namespace_cls=spec_namespace_cls,
124+
core_namespaces=[default_namespace])
125125
namespace_catalog.load_namespaces(namespace_file, resolve=resolve)
126126

127127
return namespace_catalog

0 commit comments

Comments
 (0)