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

Applications that exit too quickly fail to flush their data #2124

Closed
jleibs opened this issue May 15, 2023 · 1 comment · Fixed by #2125
Closed

Applications that exit too quickly fail to flush their data #2124

jleibs opened this issue May 15, 2023 · 1 comment · Fixed by #2125
Assignees
Labels
🦟 regression A thing that used to work in an earlier release

Comments

@jleibs
Copy link
Member

jleibs commented May 15, 2023

Looks like a fairly standard shutdown race condition:

fast.py

import rerun as rr

rr.init("fast", spawn=True)
rr.log_rect("rect", [16, 16, 64, 64], label="Rect1", color=(255, 0, 0))

No data shows up.

slow.py:

import rerun as rr

rr.init("fast", spawn=True)
rr.log_rect("rect", [16, 16, 64, 64], label="Rect1", color=(255, 0, 0))
import time
time.sleep(0.1)

Data shows up.

@jleibs jleibs added the 🦟 regression A thing that used to work in an earlier release label May 15, 2023
@teh-cmc teh-cmc self-assigned this May 15, 2023
@abey79
Copy link
Member

abey79 commented Jun 5, 2023

I'm getting what appears to be sporadic instances of this behaviour, i.e. script returns quickly with exit code 0 and viewer is stuck on the "Ready - listening to port xxx". Got that at least with blueprint and car. Funnily, adding time.sleep(1) to car didn't seem to fix the issue.

Edit: it's probably something else as it occurs only when running from PyCharm.

emilk pushed a commit that referenced this issue Jun 6, 2023
### What

Removed unnecessary workaround for #2124 in blueprint python example

### Checklist
* [x] I have read and agree to [Contributor
Guide](https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md) and
the [Code of
Conduct](https://github.com/rerun-io/rerun/blob/main/CODE_OF_CONDUCT.md)
* [ ] ~~I've included a screenshot or gif (if applicable)~~

<!-- This line will get updated when the PR build summary job finishes.
-->
PR Build Summary: https://build.rerun.io/pr/2316

<!-- pr-link-docs:start -->
Docs preview: https://rerun.io/preview/e74edcd/docs
<!-- pr-link-docs:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🦟 regression A thing that used to work in an earlier release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants