tutorials icon indicating copy to clipboard operation
tutorials copied to clipboard

[BUG] [Documentation][Typo] : https://pytorch.org/tutorials/intermediate/dist_tuto.html

Open ankushjqc opened this issue 1 year ago • 1 comments

Add Link

https://pytorch.org/tutorials/intermediate/dist_tuto.html

Describe the bug

Documentation Bug. It seems like there is a typo in following statement. https://github.com/pytorch/tutorials/blob/f1380d39af94c52e291c4eebf20d9ce2b137f3e9/intermediate_source/dist_tuto.rst?plain=1#L159

  • reading from tensor after dist.irecv() will result in undefined behaviour. It shall be
  • reading from tensor before dist.irecv() will result in undefined behaviour.

Describe your environment

Not applicable.

cc @wconstab @osalpekar @H-Huang @kwen2501

ankushjqc avatar Sep 11 '24 16:09 ankushjqc

I feel it should rather be like this https://github.com/pytorch/tutorials/pull/3022. This seems more accurate as we need to wait until req.wait() has been executed

venkatram-dev avatar Sep 11 '24 19:09 venkatram-dev