search

ADVRHumanoids / iit-centauro-ros-pkg

2 stars 0 forks source link

Different options for realsense cameras #23

Closed torydebra closed 2 years ago

torydebra commented 2 years ago

Can we have the possibility to add to the model only some realsense camera? With the last development in master now we have : 

<xacro:property name="ADD_D435I_HEAD" value="$(arg realsense)" />
<xacro:property name="ADD_D435I_BASE" value="$(arg realsense)" />
<xacro:property name="ADD_T265_BASE" value="$(arg realsense)" />

With a single option to add them all or no one. I would like to add only the head one, for example

I can make the pull request if you think this is a good option to have @alaurenzi @liesrock

liesrock commented 2 years ago

Yes this would be good to have in my opinion: you can proceed with the PR for me

alaurenzi commented 2 years ago

Yes, please start from branch master

liesrock commented 2 years ago

I believe we can close this,right @torydebra ?

torydebra commented 2 years ago

Yes, thanks for the merge