Skip to content

More docs for Conv1d Conv2d#3870

Merged
ezyang merged 2 commits intopytorch:masterfrom
iaroslav-ai:patch-1
Nov 27, 2017
Merged

More docs for Conv1d Conv2d#3870
ezyang merged 2 commits intopytorch:masterfrom
iaroslav-ai:patch-1

Conversation

@iaroslav-ai
Copy link
Copy Markdown
Contributor

For a first time user of Conv1d, it is not clear from documentation what N, C and L exactly mean. This should clarify this. Same for Conv2d.

For a first time user of Conv1d, it is not clear from documentation what N, C and L exactly mean. This should clarify this. Same for Conv2d.
Copy link
Copy Markdown
Contributor

@apaszke apaszke left a comment

Choose a reason for hiding this comment

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

Looks good, but please fix the linter errors (you have trailing whitespace).

@iaroslav-ai
Copy link
Copy Markdown
Contributor Author

Should be fixed now

@ezyang ezyang merged commit 558516f into pytorch:master Nov 27, 2017
@soumith soumith added the 0.3.1 label Feb 4, 2018
soumith pushed a commit that referenced this pull request Feb 7, 2018
* Add a bit of notation explanation

For a first time user of Conv1d, it is not clear from documentation what N, C and L exactly mean. This should clarify this. Same for Conv2d.
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
* Add a bit of notation explanation

For a first time user of Conv1d, it is not clear from documentation what N, C and L exactly mean. This should clarify this. Same for Conv2d.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants