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

Repo update #220

Merged
merged 5 commits into from
May 24, 2024
Merged

Repo update #220

merged 5 commits into from
May 24, 2024

Conversation

smoya23
Copy link
Contributor

@smoya23 smoya23 commented May 23, 2024

  • Creating a specific launch file for Gen3 Lite robots
  • Fixing and updating the MoveIt configs for Gen3 robots
  • Restructuring documentation for clarity

@smoya23 smoya23 requested a review from martinleroux May 23, 2024 18:14
@smoya23 smoya23 self-assigned this May 23, 2024
Copy link
Collaborator

@martinleroux martinleroux left a comment

Choose a reason for hiding this comment

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

Good stuff, some fixes still required. See pending comments.


* `gripper` : Gripper to use. Possible values for the Gen3 are either `robotiq_2f_85` or `robotiq_2f_140`. For the Gen3 Lite, the only option is `gen3_lite_2f`. Default value is an empty string, which will display the arm without a gripper.

* `dof` : Degrees of freedom of the arm. Possible values for the Gen3 are either `6` or `7`. For the Gen3 Lite, the only option is `6`. Default value is `7`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we have the logic so the argument is ignored for Gen3 lite? To avoid problems with the default value.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point, I removed the argument completely from the launch file and set the value to 6

README.md Outdated

* `fake_sensor_commands` : Enable fake command interfaces for sensors used for simple simulations. Used only if 'use_fake_hardware' parameter is true. Default value is `false`.

* `robot_controller` : Robot controller to start. Default value is `joint_trajectory_controller`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are there other options?

use_internal_bus_gripper_comm:=true
gripper_joint_name:=right_finger_bottom_joint
tf_prefix:=''
robot_ip:=192.168.1.10
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a comment: 192.168.2.10 when using USB to Ethernet interface instead of USB via RNDIS

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added comment in the README arguments explanation

@smoya23 smoya23 force-pushed the pr_gen3-lite-launch-file branch from c3c03e9 to 38bff1f Compare May 23, 2024 19:58
@martinleroux martinleroux merged commit 33ec22c into main May 24, 2024
7 of 10 checks passed
@martinleroux martinleroux deleted the pr_gen3-lite-launch-file branch May 24, 2024 13:45
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