Skip to content

[C++ API][Docs] Improve docs for torch::nn::Module#11115

Closed
goldsborough wants to merge 1 commit intopytorch:masterfrom
goldsborough:api-docs
Closed

[C++ API][Docs] Improve docs for torch::nn::Module#11115
goldsborough wants to merge 1 commit intopytorch:masterfrom
goldsborough:api-docs

Conversation

@goldsborough
Copy link
Contributor

Added some documentation. Will rebuild docs to make sure it looks good. Can already accept approvals.

@ebetica @apaszke @ezyang

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

goldsborough has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.


/// Provides a means to traverse the `Module` tree.
///
/// See the documentation for `CursorBase` for information on how to operate

This comment was marked as off-topic.

This comment was marked as off-topic.

/// Attempts to cast this `Module` to the given `ModuleType`.
///
/// This method is useful when calling `apply()` on a `ModuleCursor`.
/// \rst

This comment was marked as off-topic.

This comment was marked as off-topic.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

goldsborough is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

PenghuiCheng pushed a commit to PenghuiCheng/pytorch that referenced this pull request Sep 11, 2018
Summary:
Added some documentation. Will rebuild docs to make sure it looks good. Can already accept approvals.

ebetica apaszke ezyang
Pull Request resolved: pytorch#11115

Differential Revision: D9597880

Pulled By: goldsborough

fbshipit-source-id: 56b701da631702ba56e281a0de0f7ebe490f5c5a
@ezyang ezyang added the merged label Jun 26, 2019
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.

4 participants