Skip to content

Conversation

@yassineAlouini
Copy link
Contributor

In this PR, I add the updated quantized inception documentation.

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 for the PR @yassineAlouini !

Looks great, only a few comments, on top of those from @oke-aditya :)

@yassineAlouini
Copy link
Contributor Author

Thanks for the PR @yassineAlouini !

Looks great, only a few comments, on top of those from @oke-aditya :)

Thanks for catching these. Will fix very soon. 👌

yassineAlouini and others added 3 commits May 13, 2022 15:14
[ENH] Improve doc thanks to various code review comments.

Co-authored-by: Aditya Oke <[email protected]>
Co-authored-by: Nicolas Hug <[email protected]>
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.

Thank you @yassineAlouini !

@NicolasHug NicolasHug merged commit e7abd3b into pytorch:main May 13, 2022
@yassineAlouini yassineAlouini deleted the inception branch May 16, 2022 09:50
facebook-github-bot pushed a commit that referenced this pull request May 24, 2022
Summary:
* [DOC] Add quantized inception updated documentation.

* Add missing file.

* Apply suggestions from code review

[ENH] Improve doc thanks to various code review comments.

Reviewed By: datumbox

Differential Revision: D36413355

fbshipit-source-id: 2820f76f65d98570b5f432a3337d01a4a9fee0c3

Co-authored-by: Aditya Oke <[email protected]>
Co-authored-by: Nicolas Hug <[email protected]>
Co-authored-by: Aditya Oke <[email protected]>
Co-authored-by: Nicolas Hug <[email protected]>
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