-
Notifications
You must be signed in to change notification settings - Fork 98
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
New config with thermal camera: MARBLE_HUSKY_SENSOR_CONFIG_3 and MARBLE_HUSKY_SENSOR_CONFIG_4 #779
Conversation
Signed-off-by: Ashton Larkin <[email protected]>
Build finished. 21 tests run, 0 skipped, 1 failed. |
Signed-off-by: Ian Chen <[email protected]>
Build finished. 21 tests run, 0 skipped, 1 failed. |
Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Ian Chen <[email protected]>
Build finished. 21 tests run, 0 skipped, 1 failed. |
Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Ian Chen <[email protected]>
Build finished. 21 tests run, 0 skipped, 1 failed. |
@iche033 I believe we should add this code snippet to the fuel model's <!-- thermal camera -->
<visual name="thermal_camera_visual">
<pose >0.485 0 0.169 0 0 0</pose>
<geometry>
<box>
<size>0.02 0.025 0.025</size>
</box>
</geometry>
</visual>
<sensor name="thermal_camera" type="thermal">
<pose >0.485 0 0.169 0 0 0</pose>
<camera name="thermal_camera">
<horizontal_fov>0.4922</horizontal_fov>
<lens>
<intrinsics>
<!-- fx = fy = width / ( 2 * tan (hfov / 2 ) ) -->
<fx>764.35630</fx>
<fy>764.35630</fy>
<!-- cx = ( width + 1 ) / 2 -->
<cx>192.5</cx>
<!-- cy = ( height + 1 ) / 2 -->
<cy>144.5</cy>
<s>0</s>
</intrinsics>
</lens>
<image>
<width>384</width>
<height>288</height>
</image>
<clip>
<near>0.1</near>
<far>100</far>
</clip>
</camera>
<always_on>1</always_on>
<update_rate>25</update_rate>
</sensor> |
Build finished. 21 tests run, 0 skipped, 1 failed. |
@iche033 I just tested configurations 3 and 4. Everything seems to be working for me! I can echo data from the topics, and I can visualize the thermal camera in rviz. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
works for me as well
The Fuel models have been updated to use:
|
The thermocam got lost from Fuel... #821. |
The thermal camera used for this model is the same as the one in #773. The only difference between the thermal camera for this model and the one in #773 should be the the camera's location.
sensor config 3 = sensor config 1 + thermal camera
, andsensor config 4 = sensor config 2 + thermal camera
Here is where I have placed the thermal camera (right above the lower LIDAR sensor):
Signed-off-by: Ashton Larkin [email protected]