Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add received message age metric to topic statistics #1080

Merged
merged 15 commits into from
Apr 24, 2020
Merged

Add received message age metric to topic statistics #1080

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
c25a916
Add received message age metric to topic statistics
Apr 21, 2020
fbee7b0
Add unit tests
Apr 22, 2020
c930563
Add IMU messages in unit test
Apr 23, 2020
1cba4b5
Use system time instead of steady time
dabonnie Apr 23, 2020
faff047
Fix test warnings
dabonnie Apr 23, 2020
2b90d72
Replace IMU messages with new dummy messages
Apr 23, 2020
d9186c1
Remove outdated TODO and unused test variables
Apr 23, 2020
2e3fa8a
Address review comments
dabonnie Apr 23, 2020
0d9ecf4
Address review comments
Apr 23, 2020
393a0d3
Re-add message with header for unit testing
Apr 23, 2020
8cd1361
Remove extra newline
Apr 23, 2020
920c23a
Address message review feedback
dabonnie Apr 23, 2020
f43f91e
Address more review feedback
dabonnie Apr 23, 2020
92810d0
Fix Windows failure
dabonnie Apr 24, 2020
b7e674b
Only set append_library_dirs once
dabonnie Apr 24, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion rclcpp/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -619,8 +619,12 @@ if(BUILD_TESTING)
target_link_libraries(test_wait_set ${PROJECT_NAME})
endif()

set(append_library_dirs "${CMAKE_CURRENT_BINARY_DIR}")
if(WIN32)
set(append_library_dirs "${append_library_dirs}/$<CONFIG>")
endif()
jacobperron marked this conversation as resolved.
Show resolved Hide resolved
ament_add_gtest(test_subscription_topic_statistics test/topic_statistics/test_subscription_topic_statistics.cpp
APPEND_LIBRARY_DIRS "${CMAKE_CURRENT_BINARY_DIR}"
APPEND_LIBRARY_DIRS "${append_library_dirs}"
)
if(TARGET test_subscription_topic_statistics)
ament_target_dependencies(test_subscription_topic_statistics
Expand Down