Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/Update driving controller for Finals #158

Closed
Carter90 opened this issue Feb 12, 2021 · 3 comments
Closed

Fix/Update driving controller for Finals #158

Carter90 opened this issue Feb 12, 2021 · 3 comments
Assignees
Labels
invalid This doesn't seem right

Comments

@Carter90
Copy link
Contributor

Carter90 commented Feb 12, 2021

So the ability to drive the rover was completely broken they even removed the controller, luckily there are many off the shelf that should do a better job currently take your pick https://github.com/ros-controls/ros_controllers/

useful files to know where they live

./SC2/srcp2-competitors/ros_workspace/install/share/srcp2_models_final/urdf/rovers/small_scout/small_scout.xacro
./SC2/srcp2-competitors/ros_workspace/install/share/srcp2_models_final/urdf/rovers/common/universal_wheel.xacro

install me

sudo apt install ros-noetic-four-wheel-steering-controller
sudo apt install ros-noetic-teleop-twist-keyboard

files to start with

wget https://raw.githubusercontent.com/ros-controls/ros_controllers/noetic-devel/four_wheel_steering_controller/test/config/four_wheel_steering_controller_twist_cmd.yaml
wget https://raw.githubusercontent.com/ros-controls/ros_controllers/noetic-devel/four_wheel_steering_controller/test/launch/four_wheel_steering_common.launch

NOTE

I was unsuccessful restoring the old controller to a working state I got it to run but was unable to connect to it with teleop after lots of digging probably for the best
roslaunch simple_robot_controller.launch robot_name:=small_scout_1 many other files and launches modified

@Carter90 Carter90 added the invalid This doesn't seem right label Feb 12, 2021
@Carter90 Carter90 self-assigned this Feb 12, 2021
@Carter90 Carter90 changed the title Update driving conroller for Finals Fix/Update driving conroller for Finals Feb 12, 2021
@Carter90 Carter90 assigned gmfricke and abbypribis and unassigned Carter90 Feb 15, 2021
@Carter90 Carter90 changed the title Fix/Update driving conroller for Finals Fix/Update driving controller for Finals Feb 22, 2021
@Carter90
Copy link
Contributor Author

So Runaround Robotics says there is a pull req https://gitlab.com/scheducation/srcp2-final-public/-/issues/62 I'm guessing its ros-controls/ros_controllers#441

@abbypribis
Copy link
Contributor

Check driver-controller branch for beginnings of controller
To test:

  1. cd SC2, catkin clean -y, source srcp2-final-public/ros_workspace/install/setup.bash, catkin build, source ./devel/setup.bash
  2. Start sim - srcp2-final-public/docker/scripts/run_srcp2_final
  3. Start drive controller - rosrun drive-controller drive-controller.py - should get message init_node
  4. Drive robot around - rosrun teleop_twist_keyboard teleop_twist_keyboard.py cmd_vel:=/small_scout_1/cmd_vel - hold down Shift for strafing
  5. Kill sim - docker kill $(docker ps -q)

@abbypribis
Copy link
Contributor

Robot does not go backwards correctly (wheels freak out) and overturns.

Implement the turning limits from the code we borrowed - see if that fixes it.

Also, gotta fix that semaphore/blocking so that drive messages aren't getting written over/interrupted

@Carter90 Carter90 closed this as completed Jul 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

3 participants