gazebosim / gazebosim/gz-rendering
Use ign-common Image pixelformats
- Dominant language
- C++
- Stars
- 81
- Forks
- 90
- Avg merge
- 1d 13h
- Merged PRs (30d)
- 10
Description
**Original report ([archived issue](https://osrf-migration.github.io/ignition-gh-pages/#!/ignitionrobotics/ign-rendering/issues/10)) by Nate Koenig (Bitbucket: [Nathan Koenig](https://bitbucket.org/%7Bc862cdd9-fcc8-4419-9fc7-e20db14b8fcb%7D/)).**
----------------------------------------
PixelFormats are defined both in `ignition::common::Image` and `ignition::rendering::PixelFormats`.
For consistency, it would be nice to use the version in ignition common. This will also help ignition::msgs.
Contributor guide
Research direction
Start by locating the definitions of PixelFormats in ignition::common::Image and ignition::rendering::PixelFormats, then inspect their users and the ignition::msgs integration mentioned in the issue. Confirm which APIs depend on each definition and update the rendering side to use the common version consistently. Done means duplicate pixel-format definitions are no longer needed and the affected build and tests still pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- computer-graphics
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100