[python] Strip needless wrapper-class docstrings #3234
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue and/or context: As tracked on issue #2407 / [sc-51048].
Note that the intended Python and R API changes are all agreed on and finalized as described in #2407.
Changes:
At present we have four layers:
DataFrame
_tdb_handles.py
pybind11
as inapis/python/soma_dataframe.cc
libtiledbsoma
as inlibtiledbsoma/src/soma/soma_array.h
Throughout the new-shape project (#2407) I've been putting docstrings in the
_tdb_handles.py
layer which is silly because:Notes for Reviewer:
This PR is a work in progress. It is not ready for review.