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

First micro-ROS app on Linux reviewed. #183

Draft
wants to merge 4 commits into
base: master
from
Draft

Conversation

@FranFin
Copy link
Contributor

@FranFin FranFin commented Jul 16, 2020

No description provided.

@pablogs9
Copy link
Member

@pablogs9 pablogs9 commented Jul 16, 2020

LGTM

Copy link
Contributor

@ralph-lange ralph-lange left a comment

Thank you for the update!

First of all, let's take a Ubuntu 18.04 LTS computer and install **ROS 2 Dashing Diademata**:
First of all, install **ROS 2 Dashing Diademata** on your Ubuntu 18.04 LTS computer.
To do so from binaries, via Debian packages, follow the instructions detailed
[here](https://index.ros.org/doc/ros2/Installation/Dashing/Linux-Install-Debians/) instead.

This comment has been minimized.

@ralph-lange

ralph-lange Jul 16, 2020
Contributor

Suggested change
[here](https://index.ros.org/doc/ros2/Installation/Dashing/Linux-Install-Debians/) instead.
from the [ROS Index](https://index.ros.org/doc/ros2/Installation/Dashing/Linux-Install-Debians/) website.
@@ -7,29 +7,22 @@ This tutorial teaches you how to create a first micro-ROS application on Linux f

## Installing ROS 2 and the micro-ROS build system

First of all, let's take a Ubuntu 18.04 LTS computer and install **ROS 2 Dashing Diademata**:
First of all, install **ROS 2 Dashing Diademata** on your Ubuntu 18.04 LTS computer.
To do so from binaries, via Debian packages, follow the instructions detailed

This comment has been minimized.

@ralph-lange

ralph-lange Jul 16, 2020
Contributor

Suggested change
To do so from binaries, via Debian packages, follow the instructions detailed
To do so from binaries, via Debian packages, follow the instructions
We can then proceed to build the firmware and source the local installation:
```bash
# Build step

This comment has been minimized.

@ralph-lange

ralph-lange Jul 16, 2020
Contributor

Suggested change
# Build step

I propose to remove this comment given the explanation in previous paragraph.

First of all, create a micro-ROS agent:
To do that, let's first of all create a micro-ROS agent:
```bash
# Download micro-ROS-Agent packages

This comment has been minimized.

@ralph-lange

ralph-lange Jul 16, 2020
Contributor

Suggested change
# Download micro-ROS-Agent packages

Comment is explained in the previous paragraph already.

Now, we want to check that everything is working.
Open another command line. We are going to listen to the `ping` topic
with ROS 2 to check whether the micro-ROS Ping Pong node is correctly publishing the expected pings:
```bash
source /opt/ros/$ROS_DISTRO/setup.bash

This comment has been minimized.

@ralph-lange

ralph-lange Jul 16, 2020
Contributor

Suggested change
source /opt/ros/$ROS_DISTRO/setup.bash
source /opt/ros/dashing/setup.bash

... to be consistent with similar changes above.

@@ -187,7 +189,7 @@ source /opt/ros/$ROS_DISTRO/setup.bash
ros2 topic echo /microROS/pong
```
At this point, we know that our app is publishing pings. Let's check if it also answers to someone else pings in a new command line:
And now, let's publish a `fake_ping` with ROS 2 from yet another command line:
```bash
source /opt/ros/$ROS_DISTRO/setup.bash

This comment has been minimized.

@ralph-lange

ralph-lange Jul 16, 2020
Contributor

Suggested change
source /opt/ros/$ROS_DISTRO/setup.bash
source /opt/ros/dashing/setup.bash
One of the advantages of having an Linux micro-ROS app is that you don't need to buy a bunch of hardware in order to
test some multi-node micro-ROS apps.
So, with the same micro-ROS agent of the last section, let's open four different command lines and run the following on
each:
```bash
cd microros_ws
source /opt/ros/$ROS_DISTRO/setup.bash

This comment has been minimized.

@ralph-lange

ralph-lange Jul 16, 2020
Contributor

Suggested change
source /opt/ros/$ROS_DISTRO/setup.bash
source /opt/ros/dashing/setup.bash
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.