Skip to content

Conversation

oke-aditya
Copy link
Contributor

@oke-aditya oke-aditya commented May 16, 2022

Well let's categorize the information. It will make it easier to understand.
Open to ideas 😃

cc @NicolasHug

Copy link
Contributor Author

@oke-aditya oke-aditya left a comment

Choose a reason for hiding this comment

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

Thoughts?

Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

Thanks @oke-aditya looking good already. Let me know your thoughts on the comments below.

@oke-aditya
Copy link
Contributor Author

I have tried adding one line descriptions. Please have a look :)

@oke-aditya oke-aditya requested a review from datumbox May 17, 2022 18:46
@oke-aditya
Copy link
Contributor Author

Co-authored-by: Vasilis Vryniotis <datumbox@users.noreply.github.com>
@oke-aditya oke-aditya requested a review from datumbox May 18, 2022 08:21
Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@datumbox datumbox merged commit 49ee65f into pytorch:main May 18, 2022
@oke-aditya oke-aditya deleted the ops_docs branch May 18, 2022 08:24
facebook-github-bot pushed a commit that referenced this pull request Jun 1, 2022
Summary:
* try re arranging docs

* rewrite docs a bit

* Apply suggestions from code review

Reviewed By: NicolasHug

Differential Revision: D36760947

fbshipit-source-id: 1be9d48137af1bfdf03c35436aace23f4abf58e1

Co-authored-by: Vasilis Vryniotis <datumbox@users.noreply.github.com>
Co-authored-by: Vasilis Vryniotis <datumbox@users.noreply.github.com>
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.

3 participants