Skip to content

Consider removing DisparityImage header field #88

New issue
New issue
Closed
Closed
Consider removing DisparityImage header field#88
Labels
backlog
@tfoote

Description

@tfoote
tfoote
opened on Apr 2, 2020

This is a follow up to the Aggregate Foxy Message API Review

The DisparityImage has a header that's documented as it will likely be removed in a future release.

common_interfaces/stereo_msgs/msg/DisparityImage.msg

Lines 1 to 3 in 519e851

# Separate header for compatibility with current TimeSynchronizer.
# Likely to be removed in a later release, use image.header instead.
std_msgs/Header header

Since that was several years ago are we ready to remove it now? The one question is whether the TimeSynchronizer can support introspecting into the internal image headers.

Metadata

Metadata

Assignees

No one assigned

    Labels

    backlog

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions