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

Modify failed pjlib test to exclusive #4258

Merged
merged 2 commits into from
Jan 22, 2025
Merged

Modify failed pjlib test to exclusive #4258

merged 2 commits into from
Jan 22, 2025

Conversation

sauwming
Copy link
Member

@sauwming sauwming commented Jan 21, 2025

Re #4246 (CI: change sleep_test() and timestamp_test() to exclusive on Windows), the failure also happens on other platforms such as Mac, not just on Windows.

Example log from CI Mac test:

06:09:03.889 ------------ Logs for sleep_test [rc:-30]: ------------
05:57:37.097 ...2025-00-21 05:57:37.097
05:57:38.139 ...2025-00-21 05:57:38.139
05:57:39.229 ...2025-00-21 05:57:39.229
05:57:39.229 ..running sleep duration test
05:57:41.379 ...error: slept for 2150 ms instead of 2000 ms (outside 20 msec tolerance)
05:57:42.403 ...error: slept for 1024 ms instead of 1000 ms (outside 20 msec tolerance)
05:57:42.977 ...error: slept for 574 ms instead of 500 ms (outside 20 msec tolerance)
05:57:43.305 ...error: slept for 121 ms instead of 100 ms (outside 20 msec tolerance)
05:57:43.305 ...avg/max slippage: 54/150 ms

@sauwming sauwming self-assigned this Jan 21, 2025
@sauwming sauwming merged commit 52915ed into master Jan 22, 2025
39 of 41 checks passed
@sauwming sauwming deleted the pj-test-fail branch January 22, 2025 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants