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

Improve TraCI shutdown. #2140

Merged
merged 18 commits into from
Feb 5, 2024
Merged

Improve TraCI shutdown. #2140

merged 18 commits into from
Feb 5, 2024

Conversation

Gamenot
Copy link
Collaborator

@Gamenot Gamenot commented Jan 20, 2024

This should be a general improvement which will help prevent sumo|sumo-gui processes from leaking or becoming zombie processes.

Fixes #2139

@Gamenot Gamenot mentioned this pull request Jan 20, 2024
@Gamenot
Copy link
Collaborator Author

Gamenot commented Jan 25, 2024

The server now manages to get through a large scale stress test (60k instances, 32 parallel) without without errors on the client or server side.

Previously there would be errors in roughly 3/1000 instances.

@Gamenot Gamenot force-pushed the tucker/improve_traci_shutdown branch from ed047c7 to 636a06f Compare January 26, 2024 16:23
@Gamenot Gamenot merged commit 5d9328d into master Feb 5, 2024
26 checks passed
@Gamenot Gamenot deleted the tucker/improve_traci_shutdown branch February 5, 2024 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Help Request] traci
1 participant