-
Notifications
You must be signed in to change notification settings - Fork 16
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
Dev/latency tracking #452
base: master
Are you sure you want to change the base?
Dev/latency tracking #452
Conversation
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.
Make sure to run black .
in the root of the angel workspace so the style-check CI passes:
$ ./angel-workspace-shell.sh
...
$ black .
ros/angel_system_nodes/angel_system_nodes/activity_classification/activity_classifier_tcn.py
Show resolved
Hide resolved
ros/angel_system_nodes/angel_system_nodes/activity_classification/activity_classifier_tcn.py
Show resolved
Hide resolved
ros/angel_system_nodes/angel_system_nodes/latency_tracker/latency_tracker.py
Outdated
Show resolved
Hide resolved
ros/angel_system_nodes/angel_system_nodes/latency_tracker/latency_tracker.py
Outdated
Show resolved
Hide resolved
ros/angel_system_nodes/angel_system_nodes/pose_estimation/pose_estimator.py
Show resolved
Hide resolved
all_poses_msg.header.stamp = self.get_clock().now().to_msg() | ||
self.patient_pose_publisher.publish(all_poses_msg) |
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.
Wait, this is inside the for-loop. This and the following line (297) should be left indented a level to outside the for keypoints_ in keypoints:
loop...
@cameron-a-johnson Wasn't this something that we looked at earlier in the summer? I thought we fixed this earlier. Was this publish inside the "maybe iterates once" for-loop intentional to only publish a message if there were keypoints? As this currently stands, messages are not being published for images that were processed, but no poses were estimated. It would not be "wrong" to publish a pose message for an image explicitly indicating there were no poses.
This is an MVP of a latency tracking node that can be run to review the latency of the pose estimation, object detection, and activity classification nodes.