Tomoya Fujita

Results 1252 comments of Tomoya Fujita
trafficstars

> Sounds like self.executor within rclpy needs to be an internal self._executor to remove that kind of collision potential from user-space code. this sounds reasonable to me. but not sure,...

looking again at the current implementation weakref is intentionally used to avoid circular reference, probably example code is not good. any thoughts? > In rclcpp, each call of spin() instantiates...

+1 great effort 👍

> Current fixing code is very simple. Is it necessary to re-write new code for Humble and Galactic? i would say let's do that. i think it would be better...

CI: * Linux [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux&build=18102)](http://ci.ros2.org/job/ci_linux/18102/) * Linux-aarch64 [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux-aarch64&build=12634)](http://ci.ros2.org/job/ci_linux-aarch64/12634/) * Windows [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_windows&build=18746)](http://ci.ros2.org/job/ci_windows/18746/)

@adityapande-1995 could you do another review on this?

@iuhilnehc-ynos can you review this? i do need another review for this, btw, CI unstable failure is not related to this PR. CC: @clalancette @audrow @mjcarroll

@iuhilnehc-ynos @Barry-Xu-2018 please start CI once all comments are resolved.

@iuhilnehc-ynos i rebased this barnch to humble latest today.