-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
Add Rover Simulation and sim config #6615
Conversation
Sexy rover model there ;) |
Likely not helpful here, but FYI there'a an old abandoned PR for adding ground firmware here - #6050 |
I'm trying to test this and having issues. I cloned the pr-rover branches of both Firmware and sitl_gazebo and I get the same errors as above here: This can be easily fixed by changing line 637 of src/systemcmds/pwm/pwm.c storing the return in a variable (I used ret as done below). After that the simulation compiles and runs but I encur in errors saying: [Err] [gazebo_mavlink_interface.cpp:933] joint_control_type[velocity_kinematic] undefined. These disappear if the joint type in rover.sdf are changed to velocity instead of velocity_kinematic. In this case, the simulation is not working, I'm still working on a fix. |
13d962b
to
7e26655
Compare
@marcozorzi Worth re-trying. I removed the special joint type and added a velocity PID controller (which is also closer to what is around in HW). |
@LorenzMeier Yes now it's working perfectly |
Interesting pull request here ! It really is beginning to look like ROS/Gazebo simulations :) |
Thanks! |
@Kiwa21 The rover parts right now are mostly for driving the vehicle. High-level navigation should be done in a higher-level software component. |
@coveralls took a while 😄. |
@sarahToonsi please don't necro-bump. Open a new issue an properly describe your issue, thank you. |
Sorry :) |
This adds a full rover simulation for SITL and Gazebo. The physical config will follow this week for a Traxxas Stampede. This is to enable student projects at ETH.
Running it:
make posix gazebo_rover
It kind of works already (tested manual and missions), needs further fine-tuning here and there. But I'm confident we can drive this outdoors already.