-
Notifications
You must be signed in to change notification settings - Fork 458
AIRO-1617 Private ros params #348
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
Merged
Merged
Changes from all commits
Commits
Show all changes
2 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -337,16 +337,19 @@ void Start() | |
| ## Add niryo_moveit Package | ||
| The Niryo One ROS stack is already installed on the robot and only the `niryo_moveit` package will need to be added. | ||
|
|
||
| 1. Update the `niryo_moveit/config/params.yml` file's `ROS_IP` parameter to match that of the Niryo One. | ||
|
|
||
| 1. Copy the `niryo_moveit` package to the `catkin_ws` directory on the Niryo One's catkin workspace at `/home/niryo/catkin_ws` and run the `catkin_make` command. | ||
|
|
||
| > Using the SCP command to transfer the `niryo_moveit` package might look something like, `scp -r ~/PATH/TO/niryo_moveit niryo@NIRYO_IP_ADDRESS:/home/niryo/catkin_ws/src` | ||
|
|
||
| # Execution | ||
| 1. Use the `part_4.launch` file to setup the ROS params and start the `server_endpoint` and `sim_real_pnp` scripts. | ||
| 1. Use the `part_4.launch` file to start the `server_endpoint` and `sim_real_pnp` scripts. | ||
| - `roslaunch niryo_moveit part_4.launch` | ||
|
|
||
| > Note: As with previous parts, you can configure this launch file with a custom IP address or port: | ||
| ```bash | ||
| roslaunch niryo_moveit part_4.launch tcp_ip:=127.0.0.1 tcp_port:=10005 | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same as above. should we include the entire command or just the parameters? |
||
| ``` | ||
|
|
||
| 1. Return to the Unity Editor and press Play. Press the UI Button to send the joint configurations to ROS on the Niryo One, and watch the robot arm move simultaneously in simulation and real life! | ||
|
|
||
|
|
||
|
|
||
9 changes: 7 additions & 2 deletions
9
tutorials/pick_and_place/ROS/src/niryo_moveit/launch/part_2.launch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,10 @@ | ||
| <launch> | ||
| <rosparam file="$(find niryo_moveit)/config/params.yaml" command="load"/> | ||
| <node name="server_endpoint" pkg="ros_tcp_endpoint" type="default_server_endpoint.py" args="--wait" output="screen" respawn="true" /> | ||
| <arg name="tcp_ip" default="0.0.0.0"/> | ||
| <arg name="tcp_port" default="10000"/> | ||
|
|
||
| <node name="server_endpoint" pkg="ros_tcp_endpoint" type="default_server_endpoint.py" args="--wait" output="screen" respawn="true"> | ||
| <param name="tcp_ip" type="string" value="$(arg tcp_ip)"/> | ||
| <param name="tcp_port" type="int" value="$(arg tcp_port)"/> | ||
| </node> | ||
| <node name="trajectory_subscriber" pkg="niryo_moveit" type="trajectory_subscriber.py" args="--wait" output="screen"/> | ||
| </launch> |
9 changes: 7 additions & 2 deletions
9
tutorials/pick_and_place/ROS/src/niryo_moveit/launch/part_3.launch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,11 @@ | ||
| <launch> | ||
| <rosparam file="$(find niryo_moveit)/config/params.yaml" command="load"/> | ||
| <node name="server_endpoint" pkg="ros_tcp_endpoint" type="default_server_endpoint.py" args="--wait" output="screen" respawn="true" /> | ||
| <arg name="tcp_ip" default="0.0.0.0"/> | ||
| <arg name="tcp_port" default="10000"/> | ||
|
|
||
| <node name="server_endpoint" pkg="ros_tcp_endpoint" type="default_server_endpoint.py" args="--wait" output="screen" respawn="true"> | ||
| <param name="tcp_ip" type="string" value="$(arg tcp_ip)"/> | ||
| <param name="tcp_port" type="int" value="$(arg tcp_port)"/> | ||
| </node> | ||
| <node name="mover" pkg="niryo_moveit" type="mover.py" args="--wait" output="screen"/> | ||
| <include file="$(find niryo_moveit)/launch/demo.launch" /> | ||
| </launch> |
9 changes: 7 additions & 2 deletions
9
tutorials/pick_and_place/ROS/src/niryo_moveit/launch/part_4.launch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,10 @@ | ||
| <launch> | ||
| <rosparam file="$(find niryo_moveit)/config/params.yaml" command="load"/> | ||
| <node name="server_endpoint" pkg="ros_tcp_endpoint" type="default_server_endpoint.py" args="--wait" output="screen" respawn="true" /> | ||
| <arg name="tcp_ip" default="0.0.0.0"/> | ||
| <arg name="tcp_port" default="10000"/> | ||
|
|
||
| <node name="server_endpoint" pkg="ros_tcp_endpoint" type="default_server_endpoint.py" args="--wait" output="screen" respawn="true"> | ||
| <param name="tcp_ip" type="string" value="$(arg tcp_ip)"/> | ||
| <param name="tcp_port" type="int" value="$(arg tcp_port)"/> | ||
| </node> | ||
| <node name="sim_real_pnp" pkg="niryo_moveit" type="sim_real_pnp.py" args="--wait" output="screen"/> | ||
| </launch> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,5 @@ | ||
| #!/bin/bash | ||
|
|
||
| source /opt/ros/melodic/setup.bash | ||
| echo "ROS_IP: $(hostname -i)" > $ROS_WORKSPACE/src/ros-tcp-endpoint/config/params.yaml | ||
| cd $ROS_WORKSPACE | ||
| catkin_make |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,5 @@ | ||
| #!/bin/bash | ||
|
|
||
| source /opt/ros/$ROS_DISTRO/setup.bash | ||
| echo "ROS_IP: 0.0.0.0" > $ROS_WORKSPACE/src/ros_tcp_endpoint/config/params.yaml | ||
| cd $ROS_WORKSPACE | ||
| catkin_make |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am wondering if this can become a bit misleading for most users that are not going to use custom IP address and they may try running it? How about just mentioning the parameter rather than the full command line? e.g.
As with part 2, you can configure this launch file with a custom IP address or port using the
tcp_ipandtcp_portparameters respectively.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I felt like it was important to show the syntax for passing in the parameters (the := notation is not exactly guessable).