bulebule icon indicating copy to clipboard operation
bulebule copied to clipboard

API-related changes

Open cua-cua opened this issue 7 years ago • 0 comments

module_name style

  • modify the public function names so they include the name of their module that it is also the name of the file where are defined

move.c / speed.c functions:

  • calibration.c: change cases with move_side to move
  • "We should probably move turn_back to speed module. Turn back should probably be a speed_turn() with 180-degree turn. If there is a front-wall adjustment it should be called move_back(). We have a function named like that to get out of a dead-end, so if we want to have a static turn back with front-wall adjustment we should probably call it turn_back_static() or something like that."
  • Implement speed parametrization on front movements
  • Rename or redesign the method 'parametric_move_front(float distance, float end_linear_speed)' to differentiate side_sensors_control behaviour between diagonals and straight cases.

control.c:

  • Should we create a new abstraction layer to "float get_measured_angular_speed" type of functions?

cua-cua avatar Nov 25 '18 17:11 cua-cua