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.

Sign up for GitHub

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

Jump to bottom

Fix backpressure deadlock #496

Merged
merged 4 commits into from
May 13, 2020
Merged

Fix backpressure deadlock #496

merged 4 commits into from
May 13, 2020

Conversation

mvollrath
Copy link
Contributor

See #425

@mvollrath mvollrath mentioned this pull request May 12, 2020
Closed
@mvollrath mvollrath changed the title Fix unsubscribe deadlock Fix backpressure deadlock May 12, 2020
@J-Rojas
Copy link

J-Rojas commented May 13, 2020

I've tested these changes and they have solved the deadlock.

@mvollrath mvollrath merged commit f45156e into RobotWebTools:develop May 13, 2020
@jihoonl
Copy link
Member

jihoonl commented May 14, 2020

Hey @J-Rojas , I see you frequently in PRs and comments in RWT. I invited in maintainers group. So, you can have more access to RWT repos. Thanks for the support.

@mvollrath mvollrath mentioned this pull request May 14, 2020
Closed
DomenicP added a commit to GeckoRobotics/rosbridge_suite that referenced this pull request Sep 30, 2021
@DomenicP
Port RobotWebTools#464, RobotWebTools#496, and RobotWebTools#502 from…
4fb65c8 
… ROS1 branch

RobotWebTools#464
RobotWebTools#496
RobotWebTools#502
DomenicP added a commit to GeckoRobotics/rosbridge_suite that referenced this pull request Sep 30, 2021
@DomenicP
Port RobotWebTools#464, RobotWebTools#478, RobotWebTools#496, and Rob…
8edf267 
…otWebTools#502 from ROS1 branch

RobotWebTools#464
RobotWebTools#496
RobotWebTools#502
@DomenicP DomenicP mentioned this pull request Sep 30, 2021
Merged
DomenicP added a commit to GeckoRobotics/rosbridge_suite that referenced this pull request Sep 30, 2021
@DomenicP
Port RobotWebTools#464, RobotWebTools#478, RobotWebTools#496, and Rob…
17a114b 
…otWebTools#502 from ROS1 branch

RobotWebTools#464
RobotWebTools#496
RobotWebTools#502
DomenicP added a commit to GeckoRobotics/rosbridge_suite that referenced this pull request Oct 1, 2021
@DomenicP
Port RobotWebTools#464, RobotWebTools#478, RobotWebTools#496, and Rob…
74450f6 
…otWebTools#502 from ROS1 branch

RobotWebTools#464
RobotWebTools#496
RobotWebTools#502
DomenicP added a commit to GeckoRobotics/rosbridge_suite that referenced this pull request Oct 1, 2021
@DomenicP
Port RobotWebTools#464, RobotWebTools#478, RobotWebTools#496, and Rob…
172c7fb 
…otWebTools#502 from ROS1 branch

RobotWebTools#464
RobotWebTools#496
RobotWebTools#502
amacneil pushed a commit that referenced this pull request Oct 7, 2021
@DomenicP
Port #464, #478, #496, and #502 from ROS1 branch (#663)
cc5209c 
#464
#496
#502
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@J-Rojas J-Rojas J-Rojas approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mvollrath @J-Rojas @jihoonl