Update doc to reflect current behavior of pack_padded_sequences #46937
Conversation
albanD
force-pushed
the
pack_padded_seq_doc
branch
from
564f233 to
3407d55
Compare
torch/nn/utils/rnn.py
Outdated
There was a problem hiding this comment.
"sequences" -> "sequence"
"if provided as a Tensor" -> "if provided as a tensor" (tensor is lowercase when used in descriptions in the docs)
Codecov Report
@@ Coverage Diff @@
## master #46937 +/- ##
===========================================
- Coverage 60.81% 48.43% -12.39%
===========================================
Files 2748 2735 -13
Lines 254027 252151 -1876
===========================================
- Hits 154489 122125 -32364
- Misses 99538 130026 +30488 |
facebook-github-bot
left a comment
facebook-github-bot
left a comment
There was a problem hiding this comment.
@albanD has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
💊 CI failures summary and remediationsAs of commit d02adda (more details on the Dr. CI page):
codecov.io: 1 failed
This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group. This comment has been revised 4 times. |
albanD
force-pushed
the
pack_padded_seq_doc
branch
from
950fedb to
d02adda
Compare
facebook-github-bot
left a comment
facebook-github-bot
left a comment
There was a problem hiding this comment.
@albanD has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
albanD
changed the title
3 participants
This behavior was changed by side effect by #41984
Update the doc to reflect the actual behavior of the function.