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

c++ replay: graceful shutdown #22280

Merged
merged 11 commits into from
Oct 4, 2021
Merged

c++ replay: graceful shutdown #22280

merged 11 commits into from
Oct 4, 2021

Conversation

deanlee
Copy link
Contributor

@deanlee deanlee commented Sep 19, 2021

No description provided.

@deanlee deanlee force-pushed the replay_shutdown branch 2 times, most recently from 1cb3f33 to fbf0da2 Compare September 21, 2021 16:04
@deanlee deanlee marked this pull request as ready for review September 21, 2021 19:50
@pd0wm pd0wm added the tools label Sep 21, 2021
@deanlee deanlee mentioned this pull request Sep 24, 2021
11 tasks
@deanlee deanlee force-pushed the replay_shutdown branch 2 times, most recently from 7004fe7 to b6eae58 Compare October 4, 2021 13:16
@deanlee deanlee marked this pull request as draft October 4, 2021 13:49
@deanlee deanlee marked this pull request as ready for review October 4, 2021 15:08
@pd0wm
Copy link
Contributor

pd0wm commented Oct 4, 2021

I'm still seeing some camerad server/VisionIPC shutdown messages after the replay destructor. Is that intended?

@deanlee
Copy link
Contributor Author

deanlee commented Oct 4, 2021

The camera is released after print "shutdown: done" caused output's order problem

@pd0wm pd0wm merged commit 2253cba into commaai:master Oct 4, 2021
@deanlee deanlee deleted the replay_shutdown branch October 4, 2021 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants