Skip to content

Conversation

@svekars
Copy link
Contributor

@svekars svekars commented May 4, 2023

Fix #178
Test plan:

Screenshot 2023-05-03 at 4 03 34 PM

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 a lot for fixing this @svekars .

I have checked out this PR locally and built the torchvision docs with it. I can confirm the docs look much better and I'm not observing any regression on the rest of the docs.

FWIW, here's a before and after renderring of the torchvision docs. They look much better with this PR.

Currently on main:
image

With this PR:
image

@svekars
Copy link
Contributor Author

svekars commented May 9, 2023

Thanks so much for verifying @NicolasHug !

@svekars svekars merged commit a1dbd5c into master May 9, 2023
@svekars svekars deleted the fix-178 branch May 9, 2023 21:36
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.

API docs have formatting issues

4 participants