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.

Sign up for GitHub

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

Jump to bottom

Saving through the file menu is broken on 0.5.0 #2005

Closed
jleibs opened this issue Apr 29, 2023 · 1 comment
Closed

Saving through the file menu is broken on 0.5.0 #2005

jleibs opened this issue Apr 29, 2023 · 1 comment
Labels
🪳 bug Something isn't working ⛃ re_datastore affects the datastore itself 🦟 regression A thing that used to work in an earlier release

Comments

@jleibs
Copy link
Member

jleibs commented Apr 29, 2023 •
edited
Loading

Run api_demo main and then try to save via the menu.

The terminal prints out an error along the liens of:

[2023-04-29T18:56:11Z WARN  re_log_types::data_table] failed to concatenate cells for column rerun.radius
[2023-04-29T18:56:11Z ERROR re_viewer::app] File saving failed: Failed to export to data tables

and no file is produced.

Definitely reproduces on Linux; other OS's to be confirmed.

Not clear exactly what conditions cause it to repro. The simple demos in api-demo work, but longer demos such as transforms_3d fail.
@jleibs jleibs added 🪳 bug Something isn't working ⛃ re_datastore affects the datastore itself 🦟 regression A thing that used to work in an earlier release labels Apr 29, 2023
This was referenced Apr 29, 2023
Closed
Merged
@teh-cmc
Copy link
Member

teh-cmc commented May 9, 2023

Fixed in #2007

@teh-cmc teh-cmc closed this as completed May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🪳 bug Something isn't working ⛃ re_datastore affects the datastore itself 🦟 regression A thing that used to work in an earlier release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@teh-cmc @jleibs