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

feat(python): add missing width property to LazyFrame #5431

Merged
merged 2 commits into from
Nov 5, 2022
Merged

feat(python): add missing width property to LazyFrame #5431

merged 2 commits into from
Nov 5, 2022

Conversation

alexander-beedie
Copy link
Collaborator

@alexander-beedie alexander-beedie commented Nov 5, 2022

Brings in-line with DataFrame (spotted while starting another pass through the docs).

import polars as pl
df = pl.DataFrame({"foo": [1, 2, 3], "bar": [4, 5, 6]})

df.width
2

df.lazy().width
2

@github-actions github-actions bot added enhancement New feature or an improvement of an existing feature python Related to Python Polars rust Related to Rust Polars labels Nov 5, 2022
@alexander-beedie alexander-beedie changed the title feat(python, rust): add missing width property to LazyFrame feat(python): add missing width property to LazyFrame Nov 5, 2022
@ritchie46
Copy link
Member

Thanks! Can you add a docs entry? ;)

@alexander-beedie
Copy link
Collaborator Author

alexander-beedie commented Nov 5, 2022

Thanks! Can you add a docs entry? ;)

Lol... already had in the larger PR - incoming! (It's the follow-up to the previous one containing Series/Expr structural/nav improvements (#5376), this time doing the same for DataFrame and LazyFrame. That should be ready to roll in another 15-30 mins or so).

@alexander-beedie
Copy link
Collaborator Author

@ritchie46: width added to the docs here now - going to try and do some more with the Sphinx update, so that'll come tomorrow now - never enough hours in the day... :)

@ritchie46
Copy link
Member

@ritchie46: width added to the docs here now - going to try and do some more with the Sphinx update, so that'll come tomorrow now - never enough hours in the day... :)

I know! :)

@ritchie46 ritchie46 merged commit 6e33883 into pola-rs:master Nov 5, 2022
@alexander-beedie alexander-beedie deleted the lazyframe-width branch November 6, 2022 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or an improvement of an existing feature python Related to Python Polars rust Related to Rust Polars
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants