Skip to content

Conversation

NicolasHug
Copy link
Member

@NicolasHug NicolasHug commented May 11, 2022

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.

Looks great to me. I've proposed some "better" dummy text so that we can merge the PR instead of leaving it as a POC. Can come back later and put more meaningful text.

@datumbox
Copy link
Contributor

The schema test is failing and it's related. We just need to add _docs on the permitted fields.

Co-authored-by: Vasilis Vryniotis <datumbox@users.noreply.github.com>
@NicolasHug NicolasHug merged commit e6edcef into pytorch:main May 11, 2022
facebook-github-bot pushed a commit that referenced this pull request May 20, 2022
Summary:
* POC

* Update torchvision/models/resnet.py

* Apply suggestions from code review

* Fix tests

* ufmt

* Remove useless docstring

Reviewed By: datumbox

Differential Revision: D36413350

fbshipit-source-id: c7f648a9877195b7fd580ec8a25b281fb8eea9fa

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