Several improvements to OpenSCENARIO support

[fabianoboril]

• Added controller using CARLA's autopilot (in replacement for ActivateControllerAction)
• Added support for storyboards with multiple stories
• Extended SimpleVehicleController (OSC) to handle traffic lights, turn on brake lights and use reasonable acceleration/deceleration
• Improved reloading of OpenDRIVE maps

---

This change is 

[fpasch]

Reviewed 6 of 6 files at r1.
Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @fabianoboril)

---

Docs/CHANGELOG.md, line 24 at r1 (raw file):

* Extended SimpleVehicleController (OSC) to handle traffic lights
* Generalized visualizer attached to OSC controllers

newlines?

---

srunner/scenariomanager/carla_data_provider.py, line 596 at r1 (raw file):

                _spawn_point.location.x = transform.location.x
                _spawn_point.location.y = transform.location.y
                if blueprint.has_tag('walker'):

always valid? Maybe at least have some description

---

srunner/scenariomanager/actorcontrols/carla_autopilot.py, line 38 at r1 (raw file):

        super(CarlaAutoPilotControl, self).__init__(actor)
        self._actor.set_autopilot(enabled=True)

route + target_speed cannot be set, right? Maybe it would help if you override those BasicControl methods and print out that it's not supported.

---

srunner/scenariomanager/actorcontrols/simple_vehicle_control.py, line 56 at r1 (raw file):

                                        (consider_trafficlights, true/false) - Enable consideration of traffic lights
                                        (max_deceleration, float)            - Use a reasonable deceleration value for
                                                                               this vehicle

also add max_accel

[fabianoboril]

Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @fpasch)

---

Docs/CHANGELOG.md, line 24 at r1 (raw file):

Previously, fpasch (fpasch) wrote…

newlines?

done

---

srunner/scenariomanager/carla_data_provider.py, line 596 at r1 (raw file):

Previously, fpasch (fpasch) wrote…

always valid? Maybe at least have some description

Done.

---

srunner/scenariomanager/actorcontrols/carla_autopilot.py, line 38 at r1 (raw file):

Previously, fpasch (fpasch) wrote…

route + target_speed cannot be set, right? Maybe it would help if you override those BasicControl methods and print out that it's not supported.

The base class provides these messages already.

---

srunner/scenariomanager/actorcontrols/simple_vehicle_control.py, line 56 at r1 (raw file):

Previously, fpasch (fpasch) wrote…

also add max_accel

Done.

[fpasch]

Reviewed 3 of 3 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @fabianoboril)