Configurable setup of arguments to pass to SUMO executable

[roireshef]

Feature Description

Please provide:

flow.core.kernel.simulation.traci.TraCISimulation has start_simulation method that spawns a SUMO instance. The configuration of arguments passed to the sumo instance is not exposed to the developer, and we need to go and change library code, which is not a good practice IMHO. Just like:

            # command used to start sumo
            sumo_call = [
                sumo_binary, "-c", scenario.cfg,
                "--remote-port", str(sim_params.port),
                "--num-clients", str(sim_params.num_clients),
                "--step-length", str(sim_params.sim_step)
            ]

I suggest movinga any and all SUMO executable parameters to some default configuration class (SimParams would be a good candidate) and expose it to users.

Thanks!

[eugenevinitsky]

Oh yeah, this is a good idea!

[roireshef]

Anyone working on that? Is there any expected timeframe?