-
Notifications
You must be signed in to change notification settings - Fork 58
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
calibration #44
calibration #44
Conversation
… added service to request calibration data
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thankyou for creating your first PR! Contributions like yours is what it is all about. Keep it up!
@@ -40,6 +40,7 @@ | |||
from rclpy.qos import QoSProfile | |||
from sensor_msgs.msg import Imu, MagneticField, Temperature | |||
from std_msgs.msg import String | |||
from example_interfaces.srv import Trigger |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Towflos we should probably add example_interfaces
as a dependency to the package.xml
then, right?
@Towflos wow this is a great feature! adding a service to calibrate the IMU. Do you think you could add a short section to the |
@flynneva the service only reads the calibration values and returns them in a message. I added the command to call the service to the |
@Towflos great stuff again, well done 🍻 |
#22 added mag_radius and acc_radius to parameters, added Trigger service to request calibration data