Skip to content

Fix docs build errors - #81

Merged
maanug-nv merged 6 commits into
mainfrom
maanug/fix-docs-errors
Jun 13, 2025
Merged

maanug-nv merged 6 commits into
mainfrom
maanug/fix-docs-errors

Conversation

@maanug-nv

Copy link
Copy Markdown
Contributor

Fixed several errors:

WARNING: Duplicate item hub.models.gpt.GPTConfig.tp_comm_overlap_cfg (attribute) [autodoc2.dup_item] [autodoc2.dup_item]

Removed duplicate tp_comm_overlap_cfg attribute in `GPTConfig.

/builds/nemo-framework-documentation/megatron-hub-build/docs/apidocs/hub/hub.tokenizers.tokenizer.md:933: ERROR: Unknown target name: "\\r\\n\\p{L}\\p{N}". [docutils]

Seems this was caused by PATTERN_TIKTOKEN and PATTERN_TIKTOKEN_V2, no need for these to be global vars.

/builds/nemo-framework-documentation/megatron-hub-build/src/megatron/hub/converters/state_transform.py.rst:12: WARNING: Document headings start at H2, not H1 [myst.header]

Removed megatron.hub.converters from docs, since it will be replaced in the future.

/builds/nemo-framework-documentation/megatron-hub-build/src/megatron/hub/utils/import_utils.py.rst:9: WARNING: Document headings start at H2, not H1 [myst.header]

Fixed docstrings in megatron.hub.import_utils to follow Google style.

Signed-off-by: Maanu Grover <maanug@nvidia.com>
Signed-off-by: Maanu Grover <maanug@nvidia.com>
Signed-off-by: Maanu Grover <maanug@nvidia.com>
Signed-off-by: Maanu Grover <maanug@nvidia.com>
Signed-off-by: Maanu Grover <maanug@nvidia.com>
Signed-off-by: Maanu Grover <maanug@nvidia.com>
@maanug-nv
maanug-nv merged commit 95b367c into main Jun 13, 2025
@maanug-nv
maanug-nv deleted the maanug/fix-docs-errors branch June 13, 2025 23:24
yaoyu-33 pushed a commit that referenced this pull request Jul 10, 2025
Signed-off-by: Hemil Desai <hemild@nvidia.com>
@yaoyu-33 yaoyu-33 added docs Documentation-only updates or documentation debt and removed documentation labels Mar 10, 2026

This branch was previously deployed

2 inactive deployments
nemo-ci — f0c04fec Deployed Jun 13, 2025 by copy-pr-bot[bot]
test — f0c04fec Deployed Jun 13, 2025 by copy-pr-bot[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation-only updates or documentation debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants