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

Unstable test, re_arrow_store::store_polars::to_dataframe fails #894

Closed
Wumpf opened this issue Jan 24, 2023 · 1 comment · Fixed by #1549 or #1801
Closed

Unstable test, re_arrow_store::store_polars::to_dataframe fails #894

Wumpf opened this issue Jan 24, 2023 · 1 comment · Fixed by #1549 or #1801
Assignees
Labels
🪳 bug Something isn't working 🧑‍💻 dev experience developer experience (excluding CI) ⛃ re_datastore affects the datastore itself

Comments

@Wumpf
Copy link
Member

Wumpf commented Jan 24, 2023

Describe the bug
Test instability on CI. One instance of it:
https://github.com/rerun-io/rerun/actions/runs/3995045925/jobs/6853468210

Backtrace

 thread 'gc_correct' panicked at 'called `Option::unwrap()` on a `None` value', /usr/local/cargo/registry/src/git.luolix.top-1ecc6299db9ec823/polars-core-0.26.1/src/utils/mod.rs:601:22
stack backtrace:
   0: rust_begin_unwind
             at /rustc/897e37553bba8b42751c67658967889d11ecd120/library/std/src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/897e37553bba8b42751c67658967889d11ecd120/library/core/src/panicking.rs:142:14
   2: core::panicking::panic
             at /rustc/897e37553bba8b42751c67658967889d11ecd120/library/core/src/panicking.rs:48:5
   3: core::option::Option<T>::unwrap
             at /rustc/897e37553bba8b42751c67658967889d11ecd120/library/core/src/option.rs:775:21
   4: polars_core::utils::concat_df
             at /usr/local/cargo/registry/src/git.luolix.top-1ecc6299db9ec823/polars-core-0.26.1/src/utils/mod.rs:601:22
   5: polars_core::functions::diag_concat_df
             at /usr/local/cargo/registry/src/git.luolix.top-1ecc6299db9ec823/polars-core-0.26.1/src/functions.rs:289:5
   6: re_arrow_store::store_polars::<impl re_arrow_store::store::DataStore>::to_dataframe
             at ./src/store_polars.rs:94:18
   7: correctness::gc_correct
             at ./tests/correctness.rs:377:9
   8: correctness::gc_correct::{{closure}}
             at ./tests/correctness.rs:345:1
   9: core::ops::function::FnOnce::call_once
             at /rustc/897e37553bba8b42751c67658967889d11ecd120/library/core/src/ops/function.rs:248:5
  10: core::ops::function::FnOnce::call_once
             at /rustc/897e37553bba8b42751c67658967889d11ecd120/library/core/src/ops/function.rs:248:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
@Wumpf Wumpf added 🪳 bug Something isn't working 🧑‍💻 dev experience developer experience (excluding CI) labels Jan 24, 2023
@teh-cmc teh-cmc self-assigned this Jan 26, 2023
@teh-cmc teh-cmc removed their assignment Feb 24, 2023
@teh-cmc teh-cmc self-assigned this Mar 9, 2023
@emilk
Copy link
Member

emilk commented Mar 20, 2023

I don't think this is fixed

@emilk emilk reopened this Mar 20, 2023
@emilk emilk added the ⛃ re_datastore affects the datastore itself label Mar 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪳 bug Something isn't working 🧑‍💻 dev experience developer experience (excluding CI) ⛃ re_datastore affects the datastore itself
Projects
None yet
3 participants