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.

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 cache_benchmark #679

Merged
merged 3 commits into from
May 15, 2024
Merged

Conversation

EricCousineau-TRI
Copy link
Contributor

Towards #676

@clalancette Would you be OK reviewing this?

tf2/test/cache_benchmark.cpp Outdated Show resolved Hide resolved
tf2/test/cache_benchmark.cpp Show resolved Hide resolved
Copy link
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more minor change, then I'll run CI on it.

tf2/test/cache_benchmark.cpp Show resolved Hide resolved
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Eric Cousineau <eric.cousineau@tri.global>
@EricCousineau-TRI
Copy link
Contributor Author

done, thanks!

@EricCousineau-TRI
Copy link
Contributor Author

EricCousineau-TRI commented May 14, 2024

is it possible to run this through full platform testing? (Linux, Mac, Win) for it to then merge?

@clalancette
Copy link
Contributor

is it possible to run this through full platform testing? (Linux, Mac, Win) for it to then merge?

We don't run CI on macOS, but yes, here is the rest of the CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@ahcorde ahcorde requested a review from clalancette May 15, 2024 07:54
@clalancette clalancette merged commit 381d381 into ros2:rolling May 15, 2024
2 checks passed
@ahcorde
Copy link
Contributor

ahcorde commented May 28, 2024

https://github.com/Mergifyio backport jazzy

Copy link
Contributor

mergify bot commented May 28, 2024

backport jazzy

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request May 28, 2024
* Add cache_benchmark

Signed-off-by: Eric Cousineau <eric.cousineau@tri.global>
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit 381d381)
ahcorde pushed a commit that referenced this pull request May 28, 2024
* Add cache_benchmark

Signed-off-by: Eric Cousineau <eric.cousineau@tri.global>
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit 381d381)

Co-authored-by: Eric Cousineau <eric.cousineau@tri.global>
@ahcorde
Copy link
Contributor

ahcorde commented May 28, 2024

https://github.com/Mergifyio backport humble iron

Copy link
Contributor

mergify bot commented May 28, 2024

backport humble iron

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request May 28, 2024
* Add cache_benchmark

Signed-off-by: Eric Cousineau <eric.cousineau@tri.global>
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit 381d381)
mergify bot pushed a commit that referenced this pull request May 28, 2024
* Add cache_benchmark

Signed-off-by: Eric Cousineau <eric.cousineau@tri.global>
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit 381d381)
ahcorde added a commit that referenced this pull request May 28, 2024
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Co-authored-by: Eric Cousineau <eric.cousineau@tri.global>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
ahcorde added a commit that referenced this pull request May 28, 2024
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Co-authored-by: Eric Cousineau <eric.cousineau@tri.global>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants