Skip to content

Commit

Permalink
Correct TODO text on test not running on EventsExecutor yet
Browse files Browse the repository at this point in the history
I think I missed this one was trying to do async callbacks.

Signed-off-by: Brad Martin <[email protected]>
  • Loading branch information
Brad Martin committed Jan 20, 2025
1 parent b1b3c62 commit 29c5109
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion rclpy/test/test_executor.py
Original file line number Diff line number Diff line change
Expand Up @@ -649,7 +649,7 @@ def test_multi_threaded_spin_once_until_future(self) -> None:

def test_not_lose_callback(self) -> None:
self.assertIsNotNone(self.node.handle)
# TODO(bmartin427) EventsExecutor has some kind of explosion here I haven't figured out yet
# TODO(bmartin427) EventsExecutor doesn't yet properly handle async callbacks
executor = SingleThreadedExecutor(context=self.context)

callback_group = ReentrantCallbackGroup()
Expand Down

0 comments on commit 29c5109

Please sign in to comment.