search

autorope / donkeycar

Open source hardware and software platform to build a small scale self driving car.
http://www.donkeycar.com
MIT License
3.16k stars 1.3k forks source link

Allow multiple vehicle profiles via multiple .ini files #73

Closed yconst closed 7 years ago

yconst commented 7 years ago

The vehicle could be passed as an optional parameter when running the script files, otherwise the default (vehicle.ini) would be used.

wroscoe commented 7 years ago

When would you need multiple profiles on a single vehicle? Also, you should join our slack channel to discuss and share your builds. Enter your email here and I'll add you: http://donkeycar.weebly.com/

yconst commented 7 years ago

Thanks, I've sent a request.

The case I was facing was that of having two vehicles and changing the SD card between them. One had different servo/motor settings so I believe multiple configs can be helpful in such cases.

But I imagine this could also go another way: Offering different stock configuration files for different vehicle models (e.g. different rc car models) and types (diff steering, normal steering), can open Donkey to a wider audience (e.g. people who already have different models of rc cars/vehicles).

wroscoe commented 7 years ago

This should be done with an optional argument "--config" that can specify a different config file. The diff drive does not yet use config files.

yconst commented 7 years ago

I've added --config parsing and a couple of vehicle configs based on the google doc manual in the following commit: https://github.com/yconst/donkey/commit/5fb192c443eb0b43ea6d37134cb0a800bc263ff8

I did not create a pull request because this commit also includes the changes to start-server.sh, part of another mr. Once that mr is resolved, I'm going to add one for this too.

yconst commented 7 years ago

This has been merged to dev. Closing