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

Face expressions on cer in gazebo #17

Closed
vvasco opened this issue May 17, 2019 · 9 comments
Closed

Face expressions on cer in gazebo #17

vvasco opened this issue May 17, 2019 · 9 comments

Comments

@vvasco
Copy link

vvasco commented May 17, 2019

The new branch feat/face-expressions extends the cer.sdf to include face expressions on the robot in gazebo, by attaching the videoTexture plugin to the head_link_visual of the cer model.
The module faceExpressionImage has to be run and /faceExpressionImage/image:o needs to be connected to /SIM_CER_ROBOT::head_link::head_link_visual.

Two issues were encountered:

  • the sim_cer_head.dae had to be changed (thanks @aerydna), otherwise face expressions were not mapped correctly on the head;
  • it seems that there is a conflict between the plugins of the cameras and the videoTexture plugin, as they open the same port /SIM_CER_ROBOT::head_link::head_link_visual. For this reason, I created a new model without the cameras.

cc @aerydna @fiorisi

@aerydna
Copy link
Contributor

aerydna commented May 17, 2019

there is a meeting scheduled on May 30th with @fiorisi for integrating the cer head mesh mapping fix i've done in master. for the name conflict can you push the fix line i suggested yesterday @vvasco? does it worked?

@vvasco
Copy link
Author

vvasco commented May 17, 2019

hi @aerydna, actually it did not work: with the fix line, the name parameter is used, but still two ports with the same name are opened.

@aerydna
Copy link
Contributor

aerydna commented May 18, 2019

ok i'll check it out asap

@fiorisi
Copy link
Member

fiorisi commented May 20, 2019

cc @miggia

@vvasco
Copy link
Author

vvasco commented Sep 2, 2019

Hi @aerydna @fiorisi, what's the status of this issue?

@pattacini
Copy link
Member

@aerydna @fiorisi we need to progress on this; it'getting quite important.

@fiorisi
Copy link
Member

fiorisi commented Sep 23, 2019

As discussed with @aerydna, we agreed to simply merge the modification from your branch into the master branch.
Before merging, it would be nice if you can please perform a test. A screenshot of the simulation is enough to validate the fix.

For future modifications, it is important to remember that the models (URDF or SDF) are the product of a generation process, which was implemented in this script (now not working).

@vvasco
Copy link
Author

vvasco commented Sep 26, 2019

Hi @fiorisi @aerydna, it seems that the second problem mentioned initially is not there anymore (I'm trying to trace back which commit solved this):

it seems that there is a conflict between the plugins of the cameras and the videoTexture plugin, as they open the same port /SIM_CER_ROBOT::head_link::head_link_visual. For this reason, I created a new model without the cameras.

Therefore, I'm doing some modifications in my branch in order to remove the cer_no_cameras model (not needed anymore) and include the video texture in the cer model.

@pattacini
Copy link
Member

@vvasco opened up the corresponding PR in #18.

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

No branches or pull requests

4 participants