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

polars collect breakes further operations #12828

Open
maxim-uvarov opened this issue May 10, 2024 · 1 comment
Open

polars collect breakes further operations #12828

maxim-uvarov opened this issue May 10, 2024 · 1 comment
Labels
🐛 bug Something isn't working dataframe issues related to the dataframe implementation

Comments

@maxim-uvarov
Copy link
Contributor

maxim-uvarov commented May 10, 2024

Describe the bug

@ayax79 I found another bug? (Execute in the order as given below.) Without $a | polars collect, the last command works as expected.

Though, I don't fully understand if polars collect is needed now. But anyway, it would be better if you personally took a look into it; maybe it is important.

How to reproduce

> let a = [[a b]; [1 a] [2 b] [3 c] [4 d]] | polars into-df

> $a | polars collect
╭─#─┬─a─┬─b─╮
│ 0 │ 1 │ a │
│ 1 │ 2 │ b │
│ 2 │ 3 │ c │
│ 3 │ 4 │ d │
╰─#─┴─a─┴─b─╯

> $a
 | polars reverse
 | polars with-column [
      ((polars col a) * 2 | polars as double_a)
      ((polars col a) / 2 | polars as half_a)
 ]
 | polars collect
Error:   × Dataframe 13a4ee8a-5bea-41f2-9e1c-67dda5c789b9 not found in cache

Expected behavior

polars to work

Screenshots

No response

Configuration

key value
version 0.93.1
major 0
minor 93
patch 1
branch main
commit_hash c54d223
build_os macos-aarch64
build_target aarch64-apple-darwin
rust_version rustc 1.77.2 (25ef9e3d8 2024-04-09)
rust_channel 1.77.2-aarch64-apple-darwin
cargo_version cargo 1.77.2 (e52e36006 2024-03-26)
build_time 2024-05-07 05:50:32 +00:00
build_rust_channel release
allocator mimalloc
features dataframe, default, sqlite, system-clipboard, trash, which
installed_plugins explore, image, inc, polars

Additional context

No response

@maxim-uvarov maxim-uvarov added the needs-triage An issue that hasn't had any proper look label May 10, 2024
@fdncred fdncred added the dataframe issues related to the dataframe implementation label May 10, 2024
@sholderbach sholderbach added 🐛 bug Something isn't working and removed needs-triage An issue that hasn't had any proper look labels May 11, 2024
fdncred pushed a commit that referenced this issue May 24, 2024
This reverts commit 68adc46.

# Description

Reverts the lazyframe refactor (#12669) for the next release, since
there are still a few lingering issues. This temporarily solves #12863
and #12828. After the release, the lazyframes can be added back and
cleaned up.
@maxim-uvarov
Copy link
Contributor Author

maxim-uvarov commented May 25, 2024

here was a wrong idea that I deleted later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working dataframe issues related to the dataframe implementation
Projects
None yet
Development

No branches or pull requests

3 participants