Skip to content

Reduce StopWatch_TEST.py sleep times (backport #800) - #805

Merged
scpeters merged 1 commit into
gz-math9from
mergify/bp/gz-math9/pr-800
Aug 20, 2026
Merged

Reduce StopWatch_TEST.py sleep times (backport #800)#805
scpeters merged 1 commit into
gz-math9from
mergify/bp/gz-math9/pr-800

Conversation

@mergify

@mergify mergify Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

🦟 Bug fix

Ports #326 to python

Summary

The sleep durations in Stopwatch_TEST.cc were reduced in #326, but the python test still takes nearly 20 seconds. This updates the python test to match the sleep durations used in c++, reducing the duration to 1 second:

Backport Policy

  • This is safe to backport to the following versions:
    • Jetty
    • Ionic
    • Harmonic
    • Fortress
  • This should not be backported
  • I am not sure
  • Other (fill in yourself)

Checklist

  • Signed all commits for DCO
  • Added a screen capture or video to the PR description that demonstrates the fix (as needed)
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • Updated Bazel files (if adding new files). Created an issue otherwise.
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Assisted-by" to your commits. (See this policy for more info.)

Assisted-by: Gemini Next

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Backports: If this is a backport, please use Rebase and Merge instead.


This is an automatic backport of pull request #800 done by [Mergify](https://mergify.com).

Match test timing used in Stopwatch_TEST.cc

Assisted-by: Gemini Next

Signed-off-by: Steve Peters <scpeters@intrinsic.ai>
(cherry picked from commit 8292148)
@github-project-automation github-project-automation Bot moved this from Inbox to In review in Core development Aug 20, 2026
@scpeters
scpeters merged commit 647c632 into gz-math9 Aug 20, 2026
16 checks passed
@scpeters
scpeters deleted the mergify/bp/gz-math9/pr-800 branch August 20, 2026 23:50
@github-project-automation github-project-automation Bot moved this from In review to Done in Core development Aug 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🪵 jetty Gazebo Jetty

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants