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

[cob4-2] Sensorring with asus camera #349

Merged
merged 3 commits into from
Nov 10, 2015

Conversation

ipa-nhg
Copy link
Member

@ipa-nhg ipa-nhg commented Nov 5, 2015

No description provided.


# camera coordinate frame (image coordinate system with x-axis to the right, y-axis downwards, z-axis into viewing direction of the camera)
# string
camera_frame: cam3d_rgb_optical_frame
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this parameter looks wrong, but I get an error if I set it as sensorring_cam3d_frame...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the error? can you copy-paste it?

@ipa-nhg
Copy link
Member Author

ipa-nhg commented Nov 5, 2015

<group>
<machine name="cob4-2-s1" address="$(arg cob4-2-s1)" env-loader="$(find cob_bringup)/env.sh" default="true" timeout="30"/>
<!-- pc monitor -->
<include file="$(find cob_bringup)/tools/pc_monitor.launch" >
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we probably have to update diagnostics configuration to include cob4-2-s1 pc in the PC diagnostics.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, you are right. I updated the diagnostics configuration

@floweisshardt
Copy link
Member

LGTM

floweisshardt pushed a commit that referenced this pull request Nov 10, 2015
[cob4-2] Sensorring with asus camera
@floweisshardt floweisshardt merged commit 56e9150 into ipa320:indigo_dev Nov 10, 2015
@fmessmer
Copy link
Member

Still, I would like to know what the error was!
See my comment above....

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.

None yet

3 participants