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
Feature/51 publish heartbeat #55
Conversation
Introduced in 0c41ad75a169864cf24eb522c08136be99122019 of https://github.com/tue-robotics/diagnostics
1611c81
to
b801b02
Compare
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.
Please call the force update methods if you are timing it yourself
updater = diagnostic_updater.Updater() | ||
updater.setHardwareID("none") | ||
updater.add(diagnostic_updater.Heartbeat()) | ||
rospy.Timer(rospy.Duration(1), lambda event: updater.update()) |
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.
Please use the force_update method, the update method is already throttled on 1hz
|
||
ros::Timer t = nh.createTimer(ros::Duration(1.0), | ||
(const ros::TimerCallback &) [](const ros::TimerEvent& te)->void { | ||
g_diagnostic_updater->update(); |
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.
please call forece update
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.
We should add diagnostics to all running nodes
Also the Jenkins build is using indigo on trusty, I think this causes problems with the C++11 syntax |
Yes
…Sent from my iPhone
On 7 Jul 2018, at 10:42, Loy ***@***.***> wrote:
@LoyVanBeek commented on this pull request.
In openpose_ros/src/openpose_ros_node.cpp:
> @@ -162,6 +166,17 @@ int main(int argc, char** argv)
g_pub = nh.advertise<sensor_msgs::Image>("result_image", 1);
}
+ g_diagnostic_updater = std::shared_ptr<diagnostic_updater::Updater>(new diagnostic_updater::Updater());
+ g_diagnostic_updater->setHardwareID("none");
+
+ diagnostic_updater::Heartbeat heartbeat = diagnostic_updater::Heartbeat();
+ g_diagnostic_updater->add(heartbeat);
+
+ ros::Timer t = nh.createTimer(ros::Duration(1.0),
+ (const ros::TimerCallback &) [](const ros::TimerEvent& te)->void {
@reinzor this lambda is the only C++11 thing here, right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Nice work, I think we should address the other nodes as well in this PR. Then we can merge next Tuesday :) |
@LoyVanBeek, are you planning to add these in the near future? |
Would be nice, but my time the coming weeks is severely limited and then I'm off on vacation for a couple of weeks, so fist time I can get around to it will be end of October. |
Tested |
This depends on using our fork of ROS diagnostics
Fixes #51