Skip to content

Commit 4546bb8

Browse files
Merge pull request #172 from amd/alex_doc_forzenset
Documentation update
2 parents 681d493 + 3270389 commit 4546bb8

2 files changed

Lines changed: 17 additions & 2 deletions

File tree

docs/PLUGIN_DOC.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -448,7 +448,7 @@ Collect raw output from niccli (Broadcom) and nicctl (Pensando) commands.
448448
- **CMD_NICCLI_DISCOVERY_LEGACY**: `['niccli --listdev', 'niccli --list']`
449449
- **CMD_NICCLI_DISCOVERY_NEW**: `['niccli --list_devices', 'niccli --list']`
450450
- **CMD_NICCLI_DISCOVERY**: `['niccli --listdev', 'niccli --list']`
451-
- **CMD_NICCLI_DISCOVERY_ALL**: `frozenset({'niccli --listdev', 'niccli --list', 'niccli --list_devices'})`
451+
- **CMD_NICCLI_DISCOVERY_ALL**: `frozenset({'niccli --list', 'niccli --list_devices', 'niccli --listdev'})`
452452
- **CMD_NICCLI_SUPPORT_RDMA_TEMPLATE_LEGACY**: `niccli -dev {device_num} nvm -getoption support_rdma -scope 0`
453453
- **CMD_NICCLI_PERFORMANCE_PROFILE_TEMPLATE_LEGACY**: `niccli -dev {device_num} nvm -getoption performance_profile`
454454
- **CMD_NICCLI_PCIE_RELAXED_ORDERING_TEMPLATE_LEGACY**: `niccli -dev {device_num} nvm -getoption pcie_relaxed_ordering`

docs/generate_plugin_doc_bundle.py

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -405,6 +405,21 @@ def annotations_for_model(model_cls: type) -> List[str]:
405405
return [f"**{k}**: `{format_type_annotation(v)}`" for k, v in anns.items()]
406406

407407

408+
def format_class_var_value(val: Any) -> str:
409+
"""Stable string for docs. set/frozenset repr order depends on PYTHONHASHSEED."""
410+
if isinstance(val, frozenset):
411+
if not val:
412+
return "frozenset()"
413+
items = sorted(val, key=lambda x: (type(x).__name__, repr(x)))
414+
return "frozenset({" + ", ".join(repr(x) for x in items) + "})"
415+
if isinstance(val, set):
416+
if not val:
417+
return "set()"
418+
items = sorted(val, key=lambda x: (type(x).__name__, repr(x)))
419+
return "{" + ", ".join(repr(x) for x in items) + "}"
420+
return str(val)
421+
422+
408423
def class_vars_dump(cls: type, exclude: set) -> List[str]:
409424
ignore = {"abc_impl", "_abc_impl", "__abstractmethods__"}
410425
exclude = set(exclude) | ignore
@@ -427,7 +442,7 @@ def class_vars_dump(cls: type, exclude: set) -> List[str]:
427442
else:
428443
out.append(f"**{name}**: `{val}`")
429444
else:
430-
out.append(f"**{name}**: `{val}`")
445+
out.append(f"**{name}**: `{format_class_var_value(val)}`")
431446
return out
432447

433448

0 commit comments

Comments
 (0)