-
Notifications
You must be signed in to change notification settings - Fork 16
Add Sphinx gallery and fixes to the doc build #273
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
docs/source/conf.py
Outdated
"exclude-members": "__weakref__", | ||
} | ||
|
||
# Mock imports for dependencies that aren't available during doc building |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just for my understanding, once we have wheels that work in CI then we can go ahead and remove all of this right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it might be problematic for a local installation that doesn't have all these deps more than the CI since we are installing all this. Also, just for API docs these import might not be necessarily needed. Since I'm only adding doc build changes and sphinx gallery in this PR, I will delete this part for now.
This pull request updates documentation build system and structure for torchforge