Skip to content

[Docs] Fix outdated source install and API reference docs#19409

Merged
mshr-h merged 3 commits intoapache:mainfrom
tlopex:doc333
Apr 16, 2026
Merged

[Docs] Fix outdated source install and API reference docs#19409
mshr-h merged 3 commits intoapache:mainfrom
tlopex:doc333

Conversation

@tlopex
Copy link
Copy Markdown
Member

@tlopex tlopex commented Apr 16, 2026

as per title

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the TVM documentation, providing clearer instructions for making TVM importable, detailing CI documentation scripts, and expanding the API reference for tvm.contrib and tvm.support. It also updates the TFLite frontend environment requirements and modifies how CUDA-enabled builds are presented in tutorials. Feedback includes correcting a typo in a build command and removing an invalid Sphinx directive in the new API documentation.

Comment thread docs/README.md
You can run the following script to reproduce the CI sphinx pre-check stage.
This script skips the tutorial executions and is useful to quickly check the content.
The following script mirrors the CI docs pipeline: it runs a sphinx pre-check
(when not running locally) and then performs a full `make htmldepoly` build,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

There is a typo in the command name: htmldepoly should be htmldeploy.

Comment on lines +21 to +23
:members:
:imported-members:
:autosummary:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The :autosummary: option is not a valid option for the automodule directive in Sphinx. This will likely trigger a warning during the documentation build and the option will be ignored. If you intended to show a summary table, you should use the .. autosummary:: directive separately.

Suggested change
:members:
:imported-members:
:autosummary:
:members:
:imported-members:

@mshr-h mshr-h merged commit e0e9315 into apache:main Apr 16, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants