cvra / robot-software

CVRA monorepo - All software running on our bots lives here
MIT License
44 stars 21 forks source link

Robot specific parameters storage and loading #23

Closed SyrianSpock closed 7 years ago

SyrianSpock commented 7 years ago

We need to choose how to store and load the configuration of different robots. Do we use config.yaml for everything and remove all the defines. There are common parameters and ones that are specific to each robot. How do we separate those ?

antoinealb commented 7 years ago

Do you have an example for common parameters ?

I would put everything in config.yaml but keep default values define'd. I think we could separate in 3 files:

We will always load parameters from the common file. Then we will load the robot specific one.

SyrianSpock commented 7 years ago

Do you have an example for common parameters ?

All calibrated parameters (wheelbase, encoder pulses per mm, etc)

antoinealb commented 7 years ago

Aren't those ones the perfect examples of per-robot parameters ?

SyrianSpock commented 7 years ago

Completely misread the question :neutral_face: Common parameters are all non-calibrated parameters (motor wheel radius, motor track length), map / strategy specific stuff (opponent obstacle size)

antoinealb commented 7 years ago

ok makes more sense !

SyrianSpock commented 7 years ago

For now, we'll keep one config file for each robot.