Skip to content

Image data pixel order specification #681

Description

@thomassedlmayer

Describe the feature

Currently, it is not possible to define the starting point of the first pixel row and column for camera image data (CameraSensorView>image_data) in the CameraSensorViewConfiguration. It could be top left or bottom right or any other combination but it can not be specified in OSI.

Proposed solution:

Add an enum to CameraSensorViewConfiguration, indicating starting point of pixel ordering (e.g., bottom-left, top-left) or processing/storage order (left-right-top-down, right-left-bottom-up, etc.)

Other considered options with disadvantages:

  • Fixed definition (e.g., only allow left to right, top down);
    Not backwards compatible because more restrictive (probably too restrictive in general)
  • Positive/negative pixel-height/width to represent a coordinate system where 0/0 is the origin and the signed dimension indicates the respective direction to follow when processing the image pixels (this is done similarly in Microsoft bitmaps for example);
    Negative values for the variable "number_of_pixels_horizontal" (which basically signifies the pixel width) may seem confusing because of the wording; would not be backwards compatible as well because of the rule "is_greater_than_or_equal_to: 1"

Describe the backwards compatibility

Breaking changes are not necessary with the proposed solution. So far you could not make any specifications or assumptions concerning pixel order, so this is just additional information, which is necessary to reliably interpret given image data.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    FeatureRequestProposals which enhance the interface or add additional features.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions