Skip to content
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

Clean up autosummary usage #54457

Merged
merged 2 commits into from
Aug 18, 2023
Merged

Clean up autosummary usage #54457

merged 2 commits into from
Aug 18, 2023

Conversation

vyasr
Copy link
Contributor

@vyasr vyasr commented Aug 8, 2023

This PR stops numpydoc from producing the summary tables for a class's methods and attributes, preferring to instead rely on autosummary. This approach allows using a class.rst template that isn't relying on hacks to get autosummary to still generate stub pages. In a follow-up PR we can look into removing more of those hacks throughout the code base, this PR is just scoped to resolve the issues with the main class.rst template.

@vyasr
Copy link
Contributor Author

vyasr commented Aug 8, 2023

In my local builds this PR does not appear to increase the number of warnings at all, suggesting that the changes are safe.

@mroeschke mroeschke mentioned this pull request Aug 8, 2023
@vyasr vyasr force-pushed the feat/cleanup_docs branch from efefa47 to 19ec625 Compare August 8, 2023 20:00
@vyasr vyasr changed the base branch from main to 0.19.x August 15, 2023 19:41
@vyasr vyasr changed the base branch from 0.19.x to main August 15, 2023 19:41
@mroeschke
Copy link
Member

I pushed an update to add accessor attributes to the API reference

@vyasr vyasr force-pushed the feat/cleanup_docs branch from 2570167 to c4c2277 Compare August 17, 2023 18:09
@vyasr vyasr force-pushed the feat/cleanup_docs branch from c4c2277 to 33f78a0 Compare August 17, 2023 18:17
@vyasr vyasr marked this pull request as ready for review August 17, 2023 20:22
@mroeschke mroeschke added the Docs label Aug 17, 2023
@mroeschke mroeschke added this to the 2.2 milestone Aug 17, 2023
@mroeschke mroeschke merged commit eb98a47 into pandas-dev:main Aug 18, 2023
@mroeschke
Copy link
Member

Awesome, thanks @vyasr

mroeschke pushed a commit to mroeschke/pandas that referenced this pull request Aug 18, 2023
* Disable numpydoc and use autosummary instead

* Add toctree for accessors
@vyasr vyasr deleted the feat/cleanup_docs branch May 21, 2024 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DOC: autosummary usage to avoid warnings and generate extra pages seems incorrect
2 participants