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

Small changes on the color recognition and readme #182

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

vncprado
Copy link

I just found a small typo on the Python script for color recognition, and I also included instructions in the readme to clone https://github.com/JenniferBuehler/general-message-pkgs which is required by the example.

@vimior
Copy link
Contributor

vimior commented Apr 20, 2023

@vncprado
Thank you very much for your support and suggestions, but the structure of the gripper of Lite6 is different from that of xarm, and it does not support moveit control for the time being, so this example is not applicable to Lite6.

@vimior
Copy link
Contributor

vimior commented Apr 20, 2023

@vncprado
In addition, cv2.boxPoints is only available in opencv3.0. It was cv2.cv.BoxPoints before. Here it is judged according to the python version. Later we will decide which function to use according to the cv version.

@vncprado
Copy link
Author

Sounds good! Thanks for the feedback.

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