Skip to content

Conversation

prabhat00155
Copy link
Contributor

Discussion: #4344 (comment)

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

Thanks @prabhat00155 , LGTM modulo the clang issue

@datumbox datumbox merged commit 5772590 into pytorch:main Sep 1, 2021
@prabhat00155 prabhat00155 deleted the prabhat00155/fix_minor_comments branch September 1, 2021 14:13
facebook-github-bot pushed a commit that referenced this pull request Sep 9, 2021
Summary:
* Fixed a couple of typos and removed unnecessary bracket

* Fixed clang error

Reviewed By: fmassa

Differential Revision: D30793325

fbshipit-source-id: 6846ce295fcb144e4d936de759da8934d35d2059
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