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

[fetcheus] add fetch head box collision #1186

Merged

Conversation

sktometometo
Copy link
Contributor

  • add fetch headbox collision object when initializing fetch-interface

Please merge this pullreq after mergining #1174

@sktometometo sktometometo changed the title [WIP] add fetch head box collision [WIP][fetcheus] add fetch head box collision Nov 23, 2019
@708yamaguchi
Copy link
Member

Regarding to 83295ba#diff-d7eb7619176f5641c713e05e846088d4R203-R205

You can use move-coords method to adapt any posture of fetch.
For example,

(setq *fetch-headbox* (make-cube 100 201 120))
(send *fetch-headbox* :move-coords (send *fetch* :head_pan_link_lk :worldcoords) (send *fetch* :base_link_lk :worldcoords))
(send *fetch-headbox* :translate #f(33.75 0 150) (send *fetch* :head_pan_link_lk :worldcoords))

(You may use slot variable robot instead of *fetch*.)

@sktometometo
Copy link
Contributor Author

@708yamaguchi
I have fixed :add-headbox-collision-objecct method according to your advice.
Thank you.

@knorth55
Copy link
Member

knorth55 commented Aug 5, 2020

@sktometometo is this still wip?

@sktometometo sktometometo changed the title [WIP][fetcheus] add fetch head box collision [fetcheus] add fetch head box collision Aug 5, 2020
@sktometometo
Copy link
Contributor Author

@knorth55 I think it may be not confirmed by experiment that this change does work well.

@knorth55
Copy link
Member

knorth55 commented Aug 6, 2020

please check this PR if it is working properly

@knorth55
Copy link
Member

knorth55 commented Sep 3, 2020

@sktometometo can you also add the latest urdf and meshes?
cc. @708yamaguchi

@knorth55 knorth55 added this to the 1.1.1 milestone Oct 21, 2020
@sktometometo sktometometo self-assigned this Feb 19, 2021
@knorth55 knorth55 force-pushed the feature/add-fetch-head-collision branch from d00faf2 to 6178cc2 Compare September 12, 2021 15:58
@k-okada
Copy link
Member

k-okada commented May 24, 2022

please update labels...

@k-okada k-okada merged commit c57dc82 into jsk-ros-pkg:master May 24, 2022
@sktometometo sktometometo deleted the feature/add-fetch-head-collision branch May 24, 2022 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants