-
Notifications
You must be signed in to change notification settings - Fork 834
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
Subscribing & Publishing from ROS nodes does not work #1391
Comments
This report is a bit too vague: the error message you include is ROS-specific, so isn't going to be much help to WSL devs. I would include at least some
The |
here are the straces of roscore and rostopic. unfortunately i am not experienced in interpreting them. i will get hold of logs where everything works tomorrow |
Here are two similar logs from a working Ubuntu 16.04 LTS + ROS Kinetic installation When comparing them, the HTTP response received immediately before the error message is emitted does have a nonzero body. In fact the body length is identical on WSL and Ubuntu 16.04... |
The issue persists in 14986. |
The issue persists in 15002. Starting a roscore works with the output
roswtf yields
Again, roswtf complains about the unconnected subscription of /rosout. Finally, rostopic echo /rosout yields
|
Same behavior on 15014.
|
While testing with roscpp and rospy talker_listener tutorial examples at https://github.com/ros/ros_tutorials on build 15025 it seems like rospy nodes works, but only sending works for roscpp (talker). I can run roscpp_tutorial talker together with rospy_tutorial listener. I still get I ran strace on roscpp listener on wsl and my working ubuntu 16.04. I would happy to provide any additional information need. |
The issue persists in 15048
|
Issue still exists in Version 1703 (OS Build 15063.138)
|
Issue still exists in Version 1703 (OS Build 16188.1000)
Part of
It looks like that the TCP connection from
Server python code which raise exception(
and
I'm new to ROS. I have not found the code of |
I have determined the problem. The problem lies in the ROS.
It assumes that the |
Interesting. Have you tested this already by patching |
@gavanderhoorn I'm trying to do this. |
Easiest is probably to use an overlay workspace with just |
Replace
in
TOPIC of ROS works. Test success in ROS/Tutorials/WritingPublisherSubscriber(python), ROS/Tutorials/ExaminingPublisherSubscriber and turtlesim |
Debugged by @bxwllzz. Ref: microsoft/WSL#1391 (comment).
Just tested this and it seems to work. I'm not entirely sure about commenting that See gavanderhoorn/ros_comm@c83508d for the diff. Nice work @bxwllzz 👍. Would be nice if you could submit a PR against |
I have submit a PR to |
Thanks. I've commented there as well. |
I wrote down a step by step guide to get ros running on wsl here. In my view, the issue can be closed as fixed in the Creators Update. |
Linking to #1450 with my input there. |
Just an update that |
FYI : As of today, ROS kinetic (ros_comm 1.12.7) work out of box on WSL 16299.19, without changing source tree. https://answers.ros.org/question/238646/installing-ros-on-ubuntu-bash-in-windows-10/ |
@k-okada - Thanks for the validation. |
Just making sure: @k-okada: did you test this with ROS pubs/subs running the a single machine only (ie: the WSL machine), or were there more involved? I ask because a critical fix for Kinetic hasn't been backported from Lunar yet (and another is being reviewed for Lunar) which -- in the past at least -- was absolutely required to get things working. |
@gavanderhoorn What's the critical fix you mentioned? I tested ROS Kinetic about a week ago. It works in single machine but didn't work in multi machine bacause of the reason I mentioned in ros/ros_comm#1202. I have open a new PR (ros/ros_comm#1202) to ros_comm (lunar-devel) several days ago to fix a bug about TransportTCP. I'm sure my PR completely solve this problem for both Lunar and Kinetic. I hava tested it in both single machine and multi machines. |
The critical fix is your previous PR (ros/ros_comm#1050), which won't be available on Kinetic until ros/ros_comm#1205 is merged. |
@gavanderhoorn I upgraded ROS about a week ago. That version of ros_comm works in single machine. But It didn't work in multi machines ( I tested this about a week ago). I will test the latest version of ros_comm to check if it work in multi machines or not tomorrow. Update 2017/10/30: |
yes, that's correct, and I haven't tested with multi machine environment. |
ROS Lunar doesn't work for me. See this issue: #2560 |
@rowanG077 - Point noted. Let's continue the discussion on #2560. |
Hey everyone. The issue still sometimes appears when using the ROSBridge TCP Server. That happens with ROS Kinetic and Melodic.
I cannot exactly understand what the solution was to the initial problem, but it looks like it still exists to some degree in the rosbridge TCP server. Any ideas where to look into to find the problem? |
Can you share a little more detail on your environment? Windows version? Version of Ubuntu? Version of ros? Version of ros_comm? |
Thanks for your fast reaction. Windows: 1803 When I run natively in ubuntu 16 or 18 this never happened. |
Any Ideas where I should look to find the culprit? |
@Hemofektik You can run |
Thanks for your input. I will give it a try. |
Hi there! I have just apt-installed ros kinetic on WSL Ubuntu 16.04. The python talker/listener example only works as long as I start both from the same bash. Is that still the issue here? Wasn't it fixed in October 2017? A bit more on the symptoms:
or
and everything works as expected. When I run Any help appreciated! |
The current state of WSL makes working with ROS nodes on WSL impossible.
What works
roscore
starts without displaying errors (since build 14965)What doesn't work
rostopic echo /rosout
fails with the message[WARN] [1479567965.827686]: Inbound TCP/IP connection failed: connection from sender terminated before handshake header received. 0 bytes were received. Please check sender for additional details.
It seems that the implementation of network sockets on WSL is still incomplete.
I tested this on builds 14965 and 14971.
More information
To help prioritize this issue, I created a post on User Voice
The text was updated successfully, but these errors were encountered: