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

Large app.ron causes significant slowdown during shutdown #3156

Closed
jleibs opened this issue Aug 30, 2023 · 5 comments
Closed

Large app.ron causes significant slowdown during shutdown #3156

jleibs opened this issue Aug 30, 2023 · 5 comments
Labels
🪳 bug Something isn't working 📺 re_viewer affects re_viewer itself

Comments

@jleibs
Copy link
Member

jleibs commented Aug 30, 2023

Describe the bug
Rerun takes 2-3 seconds to shutdown at quit.

To Reproduce
Not clear how I got into this state, but the attached app.ron definitely creates the issue for me. This app.ron is currently about 7.2M
app.ron.zip

Update: you can reproduce the behavior by opening any view and then repeatedly clicking the "repopulate viewport" button. Seems proportional to the number of space-views created.

  • For structure_from_motion/main.py (3 space views) the file only grows at 500 bytes / repopulate.
  • For open_photogrammetry_format/main.py --no-frames (118 space views) the file grows at 10k bytes / repopulate.

Expected behavior
app.ron should stay a reasonable size and rerun should shutdown quickly.

Screenshots
image

Backtrace

Desktop (please complete the following information):

  • OS: Ubuntu 22.10

Rerun version
main

@jleibs jleibs added 🪳 bug Something isn't working 📺 re_viewer affects re_viewer itself labels Aug 30, 2023
@emilk
Copy link
Member

emilk commented Aug 30, 2023

I'll add some GC:ing to the next egui release

@teh-cmc
Copy link
Member

teh-cmc commented Aug 30, 2023

Tangent question: where are these 3.8s going? It shouldn't take anywhere near that to write 7M of data to page cache? Is it the RON serializer?

@emilk
Copy link
Member

emilk commented Aug 30, 2023

Yeah, I think so. I should switch to bincode or msgpack.

@emilk
Copy link
Member

emilk commented Sep 4, 2023

@emilk
Copy link
Member

emilk commented Sep 18, 2023

Here is another fix:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪳 bug Something isn't working 📺 re_viewer affects re_viewer itself
Projects
None yet
Development

No branches or pull requests

3 participants