Skip to content

Add autogenerated API reference content to the documentation#190

Merged
hemildesai merged 4 commits intoNVIDIA-NeMo:mainfrom
mckimn:autogenerate-api-doc
Apr 28, 2025
Merged

Add autogenerated API reference content to the documentation#190
hemildesai merged 4 commits intoNVIDIA-NeMo:mainfrom
mckimn:autogenerate-api-doc

Conversation

@mckimn
Copy link
Contributor

@mckimn mckimn commented Apr 3, 2025

This PR adds API reference content from project source files to the documentation build and cleans up some of the doc source files to remove build errors/warnings.

mckimn added 3 commits April 3, 2025 11:43
Signed-off-by: Nathan McKimpson <nmckimpson@nvidia.com>
Signed-off-by: Nathan McKimpson <nmckimpson@nvidia.com>
Signed-off-by: Nathan McKimpson <nmckimpson@nvidia.com>
# See the License for the specific language governing permissions and
# limitations under the License.

"""The NeMO-Run API."""
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this needed? If so can we rename to NeMo-Run?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This docstring is used as an introduction to the generated API reference. A docstring is needed, but it can use any wording you'd like.

Here is an example of how it's used in the output: http://sw-mobile-docs/Nathan/nemo-run/api/nemo_run/index.html

Copy link

Choose a reason for hiding this comment

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

Suggested doc string: This module serves as the main entrypoint for the NeMo-Run Python library,
providing programmatic access to its core functionalities for configuring,
packaging, and launching experiments across various execution environments.

Signed-off-by: Hemil Desai <hemild@nvidia.com>
@hemildesai hemildesai marked this pull request as ready for review April 28, 2025 22:08
@hemildesai hemildesai merged commit 7474649 into NVIDIA-NeMo:main Apr 28, 2025
18 of 20 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.

3 participants

Comments