-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Docs link python classes to docs #2633 #2739
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
Docs link python classes to docs #2633 #2739
Conversation
@Bibo-Joshi Is there anything else i should add to this pr? |
Hi. As mentioned in #2738 and offline, it would be good if you build the docs and checked for places where linking to the python/APS docs is not yet used or used wrongly. Ofc I don't expect you to check every docstring manually, that's just not feasible. However, |
|
done |
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.
pretty much how I would do it. If there are missing/wrong references in other parts of code, we can just fix it later
Co-authored-by: Harshil <37377066+harshil21@users.noreply.github.com>
Co-authored-by: Harshil <37377066+harshil21@users.noreply.github.com>
Thanks for the contribution :) |
Resolves item number 2 on the original message in #2633 - link to python docs.
Added intersphinx to conf.py, also minor docs mending.