Skip to content
This repository has been archived by the owner on Feb 4, 2021. It is now read-only.

Windows repeated test_timer_number_callbacks100hertz mostly fails #139

Closed
clalancette opened this issue Jul 5, 2018 · 2 comments
Closed

Comments

@clalancette
Copy link

For the past week, the Windows repeated test of test_timer_number_callbacks100hertz has been failing in CI:

Error Message

AssertionError: should have received 4 callbacks, received [0, 1, 2]

Stacktrace

multiprocessing.pool.RemoteTraceback: 
"""
Traceback (most recent call last):
  File "C:\Python36\lib\multiprocessing\pool.py", line 119, in worker
    result = (True, func(*args, **kwds))
  File "C:\J\workspace\nightly_win_rep\ws\src\ros2\rclpy\rclpy\test\test_timer.py", line 29, in run_catch_report_raise
    return func(*args, **kwargs)
  File "C:\J\workspace\nightly_win_rep\ws\src\ros2\rclpy\rclpy\test\test_timer.py", line 79, in func_number_callbacks
    assert len(callbacks) == 4, 'should have received 4 callbacks, received %s' % str(callbacks)
AssertionError: should have received 4 callbacks, received [0, 1, 2]
"""

The above exception was the direct cause of the following exception:

    def test_timer_number_callbacks100hertz():
        func_launch(
>           func_number_callbacks, ['0.01'], "didn't receive the expected number of callbacks")

test\test_timer.py:167: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
test\test_timer.py:136: in func_launch
    args=(function, args)
C:\Python36\lib\multiprocessing\pool.py:259: in apply
    return self.apply_async(func, args, kwds).get()
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

self = <multiprocessing.pool.ApplyResult object at 0x000002C7E7CA0D68>
timeout = None

    def get(self, timeout=None):
        self.wait(timeout)
        if not self.ready():
            raise TimeoutError
        if self._success:
            return self._value
        else:
>           raise self._value
E           AssertionError: should have received 4 callbacks, received [0, 1, 2]

C:\Python36\lib\multiprocessing\pool.py:644: AssertionError
@dhood
Copy link
Member

dhood commented Aug 17, 2018

1khz tests disabled in ros2/rclpy#223. from discussion in ros2/rclcpp#457 sounds like 100hz can be disabled too if it's still causing issues since the test isn't supposed to rely on hardware being fast enough

@clalancette
Copy link
Author

As far as I can tell, this hasn't happened in at least the last week. Thus, I'm going to close this out for now and we can reopen if we see the problem again.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants