-
Notifications
You must be signed in to change notification settings - Fork 990
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 support for time synchronisation with FCU #159
Conversation
public: | ||
SystemTimePlugin(): | ||
uas(nullptr) | ||
{ |
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.
Don't put init code to constructor, must be in initialize(). How it might build?..
You forget add it to mavros/CMakeLists.txt (look list of plugin sources) and to |
Add: #include <sensor_msgs/TimeReference.h> |
@vooon Cannot fix last problem. Great if you could look into it. Leave me a comment, I'll follow up tomorrow. |
At line 132: you trying to divide message smart pointer (that don't implement |
Also i think that ros::Duration is preffered type for offset value (and for message, because time_offset can not be used as TimeReference). |
All done and implemented |
Add support for time synchronisation with FCU.
Looks good. Merged. |
@vooon Thanks. APM compatibility should be OK, but please check if required. |
This PR adds timesync support. I've finished FCU implementation, but needs testing and hotfix if needed.
@vooon Please go through this, and let me know.
To use the offset anywhere else, simply subscribe to the mavros/time_offset topic and add this offset to time_boot_ms of incoming mavlink messages.
Kabir