-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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: Document DataFrame.__getitem__
and Series.__getitem__
#18309
Conversation
@@ -8,6 +8,7 @@ Export DataFrame data to other formats: | |||
.. autosummary:: | |||
:toctree: api/ | |||
|
|||
DataFrame.__array__ |
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.
drive-by
- `memory_resource`: Set an RMM memory resource for | ||
device-side allocations. | ||
device-side allocations. |
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.
same
DataFrame.__getitem__
and Series.__getitem__
DataFrame.__getitem__
and Series.__getitem__
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #18309 +/- ##
==========================================
+ Coverage 80.23% 80.35% +0.11%
==========================================
Files 1501 1501
Lines 199180 199214 +34
Branches 2837 2837
==========================================
+ Hits 159822 160076 +254
+ Misses 38830 38610 -220
Partials 528 528 ☔ View full report in Codecov by Sentry. |
Good idea; there's quite a bit going on in |
cool, thanks for your review! 🙏 |
I was adding docs for these in Narwhals and noticed they don't appear in the Polars docs. Let's add them!