feat: add DataFrame.pipe() method#421
Conversation
| that function as a keyword argument whose key is the value of the second | ||
| element of the tuple. | ||
|
|
||
| Parameters |
There was a problem hiding this comment.
google style docstring please.
Also could you please screenshots of the API doc rendering?
There was a problem hiding this comment.
Fixed to follow google style docstrings.
There was a problem hiding this comment.
Examples are not rendering (they go before Args/Returns). also the args and kwargs appear funky https://screenshot.googleplex.com/97cktrgpWUyMzxL
There was a problem hiding this comment.
fixed docstring. haven't got the rendering process figure out yet though
There was a problem hiding this comment.
Thanks! [nit] The style in this file (common.py) itself is not google style, but it isn't leading to the website as the one in generic.py is taking effect, so we can remove it from here, or update to google style for consistency.
There was a problem hiding this comment.
sorry, forgot that I had two docstrings to fix. changed this one to google style too
…type (#442) * feat: support datetime related casting in (Series|DataFrame|Index).astype * chore: add deferred exec code samples (#439) * chore: add deferred exec code samples * fix tests * fix tests * feat: add DataFrame.pipe() method (#421) * addressed comments --------- Co-authored-by: Garrett Wu <6505921+GarrettWu@users.noreply.github.com> Co-authored-by: TrevorBergeron <tbergeron@google.com>
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
Fixes #<issue_number_goes_here> 🦕