Automotive demo car inclusion refact
@agalbachicar is already working on this.
Since Jun 4, 2019.
- Dominant language
- C++
- Stars
- 16
- Forks
- 5
- PR merge metrics
- No merged PRs in 30d
Description
## Description
In PR #9173, per reviewer and author agreement, it was pushed a refactor on how to improve the way cars were added to the simulation. Right now, `multilane_onram_merge` only allows to add `MaliputRailCars` (see #9193). After that issue is solved, it will be good to have IDM cars into it as well as trying other car controllers. For that, some things might me reviewed:
- Scenario selection.
- Flags for the cars (speed, number, type, naming, etc.)
- Refactor on how load cars to the environments.
I have a WIP branch where I sketched a solution [here](https://github.com/ekumenlabs/drake/commit/b2dd8307ddfa3f5276493cce687a200e348ad5a2) which can be used to kick off the discussion (note it was branched while I was working on #9173 and it is not be updated).
CC @liangfok @jadecastro
This ticket is a copy of: https://github.com/RobotLocomotion/drake/issues/9226
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.