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

Fix pr247 conflicts #2

Merged

Conversation

vamsikalagaturu
Copy link

No description provided.

aalmrad and others added 10 commits October 7, 2024 10:03
* Added the empty gripper option for gen3.launch.py

* Format modifications
* Added Gen3 7DoF integration with Husky mobile robot from clearpath

* Applied format modifications

* Set the clearpath_config package to the latest commit

* Finalizing Demo IROS

* Modifications to README file
* Added Gen3 7DoF integration with Husky mobile robot from clearpath

* Applied format modifications

* Set the clearpath_config package to the latest commit

* Finalizing Demo IROS

* Modifications to README file

* Finalized all the modifications needed for the repo (except formatting)

* Formatting modifications

* Final formatting modifications
…control for Gen3_Lite + Fixed bugs (Kinovarobotics#252)

* Separated the arm and gripper controllers for Gen3_Lite in Kortex_bringup

* Modified the Gen3_Lite related description and ros2_control  files

* Modified the Gen3_Lite Moveit Package for arm-gripper control separation
@JafarAbdi
Copy link
Owner

Thanks @vamsikalagaturu, do you mind discarding the unrelated commits?

@vamsikalagaturu
Copy link
Author

but they are the latest commits in the upstream. i rebased to head and resolved conflicts. so, it should be fine for the PR I think right?

@JafarAbdi
Copy link
Owner

They aren't related to our changes, so they will mess up the PR

@vamsikalagaturu
Copy link
Author

vamsikalagaturu commented Nov 20, 2024

can you try/verify? because "moveit_active" commit from the upstream caused the merge conflict that was mentioned in your PR. Or if its easier for you, you can directly rebase your original PR and discard this one.

@JafarAbdi JafarAbdi merged commit 37c6907 into JafarAbdi:include_ros2_control_arg Nov 22, 2024
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.

3 participants