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

Waitable should check callback_group if it can be executed. #1001

Conversation

fujitatomoya
Copy link
Collaborator

address #1000

Signed-off-by: Tomoya Fujita Tomoya.Fujita@sony.com

Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya
Copy link
Collaborator Author

@iuhilnehc-ynos can you help review?

Copy link
Contributor

@iuhilnehc-ynos iuhilnehc-ynos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fujitatomoya
Copy link
Collaborator Author

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@fujitatomoya
Copy link
Collaborator Author

a couple of unstable warnings are unrelated. i will go ahead to merge this.

@fujitatomoya fujitatomoya merged commit 7780fa1 into ros2:rolling Sep 29, 2022
@fujitatomoya
Copy link
Collaborator Author

@Mergifyio backport humble galactic foxy

mergify bot pushed a commit that referenced this pull request Sep 29, 2022
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 7780fa1)

# Conflicts:
#	rclpy/rclpy/executors.py
mergify bot pushed a commit that referenced this pull request Sep 29, 2022
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 7780fa1)

# Conflicts:
#	rclpy/rclpy/executors.py
mergify bot pushed a commit that referenced this pull request Sep 29, 2022
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 7780fa1)

# Conflicts:
#	rclpy/rclpy/executors.py
@mergify
Copy link
Contributor

mergify bot commented Sep 29, 2022

backport humble galactic foxy

✅ Backports have been created

fujitatomoya added a commit that referenced this pull request Sep 29, 2022
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 7780fa1)
fujitatomoya added a commit that referenced this pull request Sep 29, 2022
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 7780fa1)
fujitatomoya added a commit that referenced this pull request Sep 29, 2022
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 7780fa1)
fujitatomoya added a commit that referenced this pull request Sep 29, 2022
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 7780fa1)
fujitatomoya added a commit that referenced this pull request Nov 3, 2022
…1013)

Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 7780fa1)

Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
fujitatomoya added a commit that referenced this pull request Nov 3, 2022
…1014)

Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 7780fa1)

Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
fujitatomoya added a commit that referenced this pull request Nov 3, 2022
…1015)

Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 7780fa1)

Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
iuhilnehc-ynos pushed a commit to iuhilnehc-ynos/rclpy that referenced this pull request Apr 10, 2023
… (ros2#1013)

Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 7780fa1)

Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
Signed-off-by: Chen Lihui <lihui.chen@sony.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants