Release 0.9.15 merge

Docs/CHANGELOG.md

@@ -1,5 +1,5 @@
 ## Table of Contents
-* [Latest Changes](#latest-changes)
+* [CARLA ScenarioRunner 0.9.15](#carla-scenariorunner-0915)
 * [CARLA ScenarioRunner 0.9.13](#carla-scenariorunner-0913)
 * [CARLA ScenarioRunner 0.9.12](#carla-scenariorunner-0912)
 * [CARLA ScenarioRunner 0.9.11](#carla-scenariorunner-0911)
@@ -12,18 +12,100 @@
 * [CARLA ScenarioRunner 0.9.5](#carla-scenariorunner-095)
 * [CARLA ScenarioRunner 0.9.2](#carla-scenariorunner-092)
 
-## Latest changes
+## CARLA ScenarioRunner 0.9.15
 ### :rocket: New Features
-* Minor improvements to some example scenarios. These include FollowLeadingVehicle, VehicleTurning, DynamicObjectCrossing and SignalizedJunctionRightTurn and RunningRedLight. Their behaviors are now more smooth, robust and some outdated mechanics have been removed
-* SignalizedJunctionLeftTurn has been remade. It now has an actor flow on which the ego has to merge into, instead of a single vehicle.
-* The BackgroundActivity has been readded to the routes, which the objective of creating the sensation of traffic around the ego
 * Add waypoint reached threshold so that the precision of the actor reaching to waypoints can be adjusted based on object types.
 * OpenSCENARIO support:
     - Added both init and story support for `EntityAction`
     - Added story support for `TrafficSignalControllerAction`
     - Added both init and story support for `LateralDistanceAction`
     - Added support for `TrafficSignalControllerCondition`
 * Supported OpenSCENARIO 2.0 standard.
+* New scenarios:
+    - InvadingTurn: vehicles at the opposite direction lane partially invade the ego's one, forcing it to leave space for them,moving slightly off-center.
+    - EnterActorFlow: the ego has to enter a highway lane filled with incoming traffic
+    - MergerIntoslowtraffic. variation of `EnterActorFlow` but with slow traffic.
+    - InterurbanActorFlow and InterurbanAdvancedActorFlow: actor flow scenarios for the new interurban intersections with dedicated lanes present at Town12 and Town13.
+    - Accident: the ego is met with an accident, forcing it to lane change to avoid it.
+    - AccidentTwoWays:  same as `Accident` but having to invade an opposite direction lane.
+    - ParkedObstacle: similar to `Accident` but with a parked obstacle instead.
+    - ParkedObstacleTwoWays:  same as `ParkedObstacle` but having to invade an opposite direction lane.
+    - HazardAtSideLane: similar to `Accident` but with a moving group of bicycles in the rightmost park of the lane
+    - HazardAtSideLaneTwoWays:  same as `HazardAtSideLane` but having to invade an opposite direction lane.
+    - ConstructionObstacleTwoWays: same as `ConstructionObstacle` but having to invade an opposite direction lane.
+    - VehicleOpensDoorTwoWays: similar to `Accident` but this time the blockage is cause by a vehicle opening its door.
+    - StaticCutIn: the ego is meant with an adversary that exits a stopped lane, cutting in front of the ego.
+    - ParkingCutIn: similar to `StaticCutIn` but the adversary starts at a parking lane.
+    - HighwayCutIn: the ego is met with a vehicle that tries to enter the highway by cutting in front of it.
+    - ParkingExit: Only usable at the beginning of the routes, makes the ego start at a parking lane.
+    - HardBreakRoute: uses the BackgroundActivity to make all vehicles in front of the ego hard break.
+    - YieldToEmergencyVehicle: the ego finds an emergency vehicle behind, having to lane chane to give way.
+    - VehicleTurningRoutePedestrian: variation of `VehicleTurningRoute` but with a pedestrian crossing instead of a bycicle.
+    - BlockedIntersection: with low visibility, the ego performs a turn only to find out that the end is blocked by another vehicle.
+    - CrossingBicycleFlow: the ego has to do a turn at an intersection but it has to cross a bycicle lane full of traffic.
+    - PedestrianCrossing: a group of pedestrians crossing a crosswalk. Easier version of `DynamicObjectCrossing` with no occluder.
+    - ParkingCrossingPedestrian: variation of `DynamicObjectCrossing`, but using a parked vehicle as the occluder.
+    - OppositeVehicleTakingPriority: variation of `OppositeVehicleRunningRedLight` but without traffic lights.
+    - NonSignalizedJunctionLeftTurn: variation of `SignalizedJunctionLeftTurn` but without traffic lights.
+    - NonSignalizedJunctionRightTurn: variation of `SignalizedJunctionRightTurn` but without traffic lights.
+    - PriorityAtJunction: utility scenario during routes to add a green traffic light at the next intersection.
+    - NoSignalJunctionCrossingRoute: Does nothing but wait for the ego to exit an intersection.
+* Improvements to old scenarios:
+    - ControlLoss: Added actual noise to the ego's control (currently only during routes).
+    - All VehicleTurning variations: more robustness and better synchronization.
+    - OppositeVehicleRunningRedLight: Improvement synchronization and the opposite vehicle's behavior.
+    - SignalizedJunctionLeftTurn: it is now an actor flow that ego has to cross.
+    - SignalizedJunctionRightTurn. it is now an actor flow that the ego has to merge into.
+    - Renamed `ConstructionSetupCrossing` to `ConstructionObstacle`, and prepared it for routes.
+* Improvements to the CarlaDataProvider:
+    - Added a lock when checking the dictionaries to avoid issues in multithreading
+    - Added the `transform` argument to all register function to avoid returning None during the first frame
+    - Added the `get_global_route_planner` and `get_all_actors` to avoid repeating these costly calls more than necessary
+    - Added `set_runtime_init_mode` and `is_runtime_init_mode`, used by the Leaderboard to initialize scenarios during the simulation
+    - At the `create_blueprint` function, replaced the `safe` argument with the `attribute_filter`, for a more generic parsing of any of the blueprint attributes.
+    - Removed the `CarlaDataProvider.get_ego_vehicle_route()` and `CarlaDataProvider.set_ego_vehicle_route()` functions as this is now information available to all scenarios.
+* Improvements to the routes:
+    - Scenarios are no longer position based, but instead part of a route's xml.
+    - Routes now also include the criteria of its scenarios.
+    - `waypoint` have been renamed to `position` and are part of the `waypoints` category.
+    - More than one `weather` are allowed, creating a dynamic one based on the ego vehicle's completed percentage of the route.
+    - Changed the timeout to also be dependent on the distance driven by the ego vehicle.
+    - Added the `RouteLightsBehavior` to control of all scene and vehicle lights during runtime
+    - Added a new criteria for routes, `CheckMinSpeed`, that checks the ego's speed and compares it with the rest of the traffic
+    - Separated the route argument into two, `route` for the file path, and `route-id`, for the name of route. the functionality remains unchanged.
+    - Simplified the overall parsing.
+* The BackgroundActivity part of the routes has been completely remade, with the objective of creating the sensation of traffic around the ego will increasing the performance
+* Added a Backgroundmanager to interact with the new BackgroundActivity, to allow it to adapt to incoming scenarios
+* Added new atomic behaviors:
+    - SyncArrivalWithAgent
+    - CutIn
+    - AddNoiseToRouteEgo
+    - ConstantVelocityAgentBehavior
+    - AdaptiveConstantVelocityAgentBehavior
+    - WaitForever
+    - BatchActorTransformSetter
+    - OppositeActorFlow
+    - InvadingActorFlow
+    - BicycleFlow
+    - OpenVehicleDoor
+    - SwitchWrongDirectionTest
+    - SwitchMinSpeedCriteria
+    - WalkerFlow
+    - AIWalkerBehavior
+    - ScenarioTimeout
+    - MovePedestrianWithEgo
+* Improved the Criterion class for a more comprehensive base criteria and easier use in the `results_writer` class.
+* Added new atomic criteria:
+    - MinimumSpeedRouteTest
+    - YieldToEmergencyVehicleTest
+    - ScenarioTimeoutTest
+* Added new atomic trigger conditions
+    - WaitUntilInFrontPosition
+* Merged the `Scenario` class into the `BasicScenario` one.
+* Scenarios can now have parameters as part of the their xml definition, which is saved as a dictionary at `config.other_parameters`
+* Simplified and improved how routes are parsed.
+* Added the `wait-for-repetitions` argument at the manual control for a smoother transition between scenarios / repetitions
+* Updated numpy's version to avoid issues with newer version of Python 3
 
 ### :bug: Bug Fixes
 * Fixed bug at OtherLeadingVehicle scenario causing the vehicles to move faster than intended
@@ -459,3 +541,4 @@
     - WaitForTrafficLightState: wait for the traffic light to have a given state
     - SyncArrival: sync the arrival of two vehicles to a given target
     - AddNoiseToVehicle: Add noise to steer as well as throttle of the vehicle
+    - CutInWithStaticVehicle:Based on the code of ParkingCutIn,realized the cutin function of a static vehicle on the highway

README.md

@@ -21,6 +21,7 @@ branch contains the latest fixes and features, and may be required to use the la
 
 It is important to also consider the release version that has to match the CARLA version.
 
+* [Version 0.9.15](https://github.com/carla-simulator/scenario_runner/releases/tag/v0.9.15) and the 0.9.15 Branch: Compatible with [CARLA 0.9.15](https://github.com/carla-simulator/carla/releases/tag/0.9.15)
 * [Version 0.9.13](https://github.com/carla-simulator/scenario_runner/releases/tag/v0.9.13) and the 0.9.13 Branch: Compatible with [CARLA 0.9.13](https://github.com/carla-simulator/carla/releases/tag/0.9.13)
 * [Version 0.9.12](https://github.com/carla-simulator/scenario_runner/releases/tag/v0.9.12) and the 0.9.12 Branch: Compatible with [CARLA 0.9.12](https://github.com/carla-simulator/carla/releases/tag/0.9.12)
 * [Version 0.9.11](https://github.com/carla-simulator/scenario_runner/releases/tag/v0.9.11) and the 0.9.11 Branch: Compatible with [CARLA 0.9.11](https://github.com/carla-simulator/carla/releases/tag/0.9.11)

manual_control.py

@@ -67,7 +67,6 @@
     from pygame.locals import K_F1
     from pygame.locals import KMOD_CTRL
     from pygame.locals import KMOD_SHIFT
-    from pygame.locals import K_BACKSPACE
     from pygame.locals import K_TAB
     from pygame.locals import K_SPACE
     from pygame.locals import K_UP
@@ -114,8 +113,6 @@ def get_actor_display_name(actor, truncate=250):
 
 class World(object):
 
-    restarted = False
-
     def __init__(self, carla_world, hud, args):
         self.world = carla_world
         try:
@@ -140,10 +137,6 @@ def __init__(self, carla_world, hud, args):
 
     def restart(self, args):
 
-        if self.restarted:
-            return
-        self.restarted = True
-
         self.player_max_speed = 1.589
         self.player_max_speed_fast = 3.713
 
@@ -161,7 +154,7 @@ def restart(self, args):
                     print("Ego vehicle found")
                     self.player = vehicle
                     break
-        
+
         self.player_name = self.player.type_id
 
         # Set up the sensors.
@@ -177,9 +170,14 @@ def restart(self, args):
 
         self.world.wait_for_tick()
 
-    def tick(self, clock):
+    def tick(self, clock, wait_for_repetitions):
         if len(self.world.get_actors().filter(self.player_name)) < 1:
-            return False
+            if not wait_for_repetitions:
+                return False
+            else:
+                self.player = None
+                self.destroy()
+                self.restart()
 
         self.hud.tick(self, clock)
         return True
@@ -232,13 +230,6 @@ def parse_events(self, client, world, clock):
             elif event.type == pygame.KEYUP:
                 if self._is_quit_shortcut(event.key):
                     return True
-                elif event.key == K_BACKSPACE:
-                    if self._autopilot_enabled:
-                        world.player.set_autopilot(False)
-                        world.restart()
-                        world.player.set_autopilot(True)
-                    else:
-                        world.restart()
                 elif event.key == K_F1:
                     world.hud.toggle_info()
                 elif event.key == K_TAB:
@@ -917,7 +908,7 @@ def game_loop(args):
             clock.tick_busy_loop(60)
             if controller.parse_events(client, world, clock):
                 return
-            if not world.tick(clock):
+            if not world.tick(clock, args.wait_for_repetitions):
                 return
             world.render(display)
             pygame.display.flip()
@@ -978,6 +969,10 @@ def main():
         '--keep_ego_vehicle',
         action='store_true',
         help='do not destroy ego vehicle on exit')
+    argparser.add_argument(
+        '--wait-for-repetitions',
+        action='store_true',
+        help='Avoids stopping the manual control when the scenario ends.')
     args = argparser.parse_args()
 
     args.width, args.height = [int(x) for x in args.res.split('x')]

requirements.txt

@@ -1,12 +1,12 @@
 py-trees==0.8.3
+numpy==1.18.4; python_version >= '3.0'
 networkx==2.2
 Shapely==1.7.1
 psutil
 xmlschema==1.0.18
 ephem
 tabulate
 opencv-python==4.2.0.32
-numpy
 matplotlib
 six
 simple-watchdog-timer

scenario_runner.py

@@ -93,8 +93,8 @@ def __init__(self, args):
         self.client = carla.Client(args.host, int(args.port))
         self.client.set_timeout(self.client_timeout)
         dist = pkg_resources.get_distribution("carla")
-        if LooseVersion(dist.version) < LooseVersion('0.9.12'):
-            raise ImportError("CARLA version 0.9.12 or newer required. CARLA version found: {}".format(dist))
+        if LooseVersion(dist.version) < LooseVersion('0.9.15'):
+            raise ImportError("CARLA version 0.9.15 or newer required. CARLA version found: {}".format(dist))
 
         # Load agent if requested via command line args
         # If something goes wrong an exception will be thrown by importlib (ok here)
@@ -236,7 +236,10 @@ def _prepare_ego_vehicles(self, ego_vehicles):
 
             for i, _ in enumerate(self.ego_vehicles):
                 self.ego_vehicles[i].set_transform(ego_vehicles[i].transform)
-                CarlaDataProvider.register_actor(self.ego_vehicles[i])
+                self.ego_vehicles[i].set_target_velocity(carla.Vector3D())
+                self.ego_vehicles[i].set_target_angular_velocity(carla.Vector3D())
+                self.ego_vehicles[i].apply_control(carla.VehicleControl())
+                CarlaDataProvider.register_actor(self.ego_vehicles[i], ego_vehicles[i].transform)
 
         # sync state
         if CarlaDataProvider.is_sync_mode():
@@ -401,11 +404,11 @@ def _load_and_run_scenario(self, config):
                                         timeout=100000)
             else:
                 scenario_class = self._get_scenario_class_or_fail(config.type)
-                scenario = scenario_class(self.world,
-                                          self.ego_vehicles,
-                                          config,
-                                          self._args.randomize,
-                                          self._args.debug)
+                scenario = scenario_class(world=self.world,
+                                          ego_vehicles=self.ego_vehicles,
+                                          config=config,
+                                          randomize=self._args.randomize,
+                                          debug_mode=self._args.debug)
         except Exception as exception:                  # pylint: disable=broad-except
             print("The scenario cannot be loaded")
             traceback.print_exc()
@@ -471,15 +474,8 @@ def _run_route(self):
         """
         result = False
 
-        if self._args.route:
-            routes = self._args.route[0]
-            scenario_file = self._args.route[1]
-            single_route = None
-            if len(self._args.route) > 2:
-                single_route = self._args.route[2]
-
         # retrieve routes
-        route_configurations = RouteParser.parse_routes_file(routes, scenario_file, single_route)
+        route_configurations = RouteParser.parse_routes_file(self._args.route, self._args.route_id)
 
         for config in route_configurations:
             for _ in range(self._args.repetitions):
@@ -576,11 +572,10 @@ def main():
     parser.add_argument('--openscenario', help='Provide an OpenSCENARIO definition')
     parser.add_argument('--openscenarioparams', help='Overwrited for OpenSCENARIO ParameterDeclaration')
     parser.add_argument('--openscenario2', help='Provide an openscenario2 definition')
+    parser.add_argument('--route', help='Run a route as a scenario', type=str)
+    parser.add_argument('--route-id', help='Run a specific route inside that \'route\' file', default='', type=str)
     parser.add_argument(
-        '--route', help='Run a route as a scenario (input: (route_file,scenario_file,[route id]))', nargs='+', type=str)
-
-    parser.add_argument(
-        '--agent', help="Agent used to execute the scenario. Currently only compatible with route-based scenarios.")
+        '--agent', help="Agent used to execute the route. Not compatible with non-route-based scenarios.")
     parser.add_argument('--agentConfig', type=str, help="Path to Agent's configuration file", default="")
 
     parser.add_argument('--output', action="store_true", help='Provide results on stdout')