-
Notifications
You must be signed in to change notification settings - Fork 122
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
RFC, chore: shrink series
module docstrings
#1976
Conversation
series
module docstrings
>>> nw.from_native(s_native, series_only=True)[ | ||
... :2 | ||
... ].to_native() # doctest:+ELLIPSIS |
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.
>>> nw.from_native(s_native, series_only=True)[ | |
... :2 | |
... ].to_native() # doctest:+ELLIPSIS | |
>>> s = nw.from_native(s_native, series_only=True) | |
>>> s[:2].to_native() # doctest:+ELLIPSIS |
narwhals/series.py
Outdated
>>> nw.from_native( | ||
... s_native, series_only=True | ||
... ).to_native() # doctest: +NORMALIZE_WHITESPACE |
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.
>>> nw.from_native( | |
... s_native, series_only=True | |
... ).to_native() # doctest: +NORMALIZE_WHITESPACE | |
>>> s = nw.from_native(s_native, series_only=True) | |
>>> s.to_native() # doctest: +NORMALIZE_WHITESPACE |
narwhals/series.py
Outdated
>>> nw.from_native(s_native, series_only=True).pipe( | ||
... lambda x: x + 2 | ||
... ).to_native() # doctest: +NORMALIZE_WHITESPACE |
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.
>>> nw.from_native(s_native, series_only=True).pipe( | |
... lambda x: x + 2 | |
... ).to_native() # doctest: +NORMALIZE_WHITESPACE | |
>>> s = nw.from_native(s_native, series_only=True) | |
>>> s.pipe(lambda x: x + 2).to_native() # doctest: +NORMALIZE_WHITESPACE |
narwhals/series.py
Outdated
>>> nw.from_native(s_native, series_only=True).is_in( | ||
... [3, 2, 8] | ||
... ).to_native() # doctest: +ELLIPSIS |
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.
>>> nw.from_native(s_native, series_only=True).is_in( | |
... [3, 2, 8] | |
... ).to_native() # doctest: +ELLIPSIS | |
>>> s = nw.from_native(s_native, series_only=True) | |
>>> s.is_in([3, 2, 8]).to_native() # doctest: +ELLIPSIS |
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.
thanks a tonne for doing this @FBruzzesi !
Thanks @MarcoGorelli ! I hope this gives us some buffer for new features now π |
π definitely |
What type of PR is this? (check all applicable)
Related issues
If you have comments or can explain your changes, please do so below
to_native()
is mainly ArrowSeries repr not being great when ELLIPSIS are required to pass the doctest