gazebosim / gazebosim/sdformat

camera.sdf: redundancy between //distortion/center and //lens/intrinsics/c* should be fixed

Open
#1,071 7 comments 2 reactions 1 assignee Claimed by @deepanshubansal01 View on GitHub
enhancement
Dominant language
C++
Stars
216
Forks
125
Avg merge
1d 14h
Merged PRs (30d)
14

Description

## Desired behavior

The [//camera/distortion/center](https://github.com/gazebosim/sdformat/blob/sdformat12_12.5.0/sdf/1.9/camera.sdf#L133-L135) and [//camera/lens/intrinsics/cx & //camera/lens/intrinsics/cy](https://github.com/gazebosim/sdformat/blob/sdformat12_12.5.0/sdf/1.9/camera.sdf#L183-L188) parameters describe the same information about the optical center of an image. This can lead to confusion for users and requires duplication of parameter values in order to maintain consistency.

We could consider deprecating one of these parameters and exclusively using the other.

## Alternatives considered

## Implementation suggestion

## Additional context

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.