Skip to content

Commit

Permalink
Fix case where SumoTrafficSimulation could get locked up in simulatio…
Browse files Browse the repository at this point in the history
…n step. (#2088)

* Fix case where simulationStep could become infinitely large.

* Update changelog.
  • Loading branch information
Gamenot authored Sep 28, 2023
1 parent d09f0fe commit 55075d0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ Copy and pasting the git commit messages is __NOT__ enough.
### Changed
### Deprecated
### Fixed
- Fixed issue where `SumoTrafficSimulation` could get locked up on reset if a scenario had only 1 map but multiple scenario variations.
### Removed
### Security

Expand Down
3 changes: 1 addition & 2 deletions smarts/core/sumo_traffic_simulation.py
Original file line number Diff line number Diff line change
Expand Up @@ -430,8 +430,7 @@ def teardown(self):
except traci.exceptions.FatalTraCIError:
pass

if self._allow_reload:
self._cumulative_sim_seconds = 0
self._cumulative_sim_seconds = 0
self._non_sumo_vehicle_ids = set()
self._sumo_vehicle_ids = set()
self._hijacked = set()
Expand Down

0 comments on commit 55075d0

Please sign in to comment.