Skip to content

Distinguish packed depth precision from grayscale visualization - #10189

Merged
Blyron merged 2 commits into
carla-simulator:ue58-devfrom
Afloat16:Afloat16/depth-grayscale-precision
Nov 9, 2026
Merged

Blyron merged 2 commits into
carla-simulator:ue58-devfrom
Afloat16:Afloat16/depth-grayscale-precision

Conversation

@Afloat16

@Afloat16 Afloat16 commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Description

The sensor reference currently calls both grayscale converters millimetric. Explain where the packed measurement is retained and quantify what linear grayscale can resolve before users discard their original data.

Where has this been tested?

Linux, CPython 3.13.17. Documentation for the Unreal Engine 5.8 branch at fe0178f. No simulator was needed for the documentation checks.

Tests

Checked the 24-bit normalization against LibCarla ColorConverter::Depth and compared packed and 8-bit steps over the documented 1000 m range. Ran the full documentation build with python -m mkdocs build --strict and checked the generated page.

CI workflows

None.

Possible Drawbacks

The change is confined to the documentation and its example.


This change is Reviewable

@Afloat16
Afloat16 requested a review from a team as a code owner November 6, 2026 19:13
@update-docs

update-docs Bot commented Nov 6, 2026

Copy link
Copy Markdown

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would update our CHANGELOG.md based on your changes.

@Blyron
Blyron merged commit 2692f38 into carla-simulator:ue58-dev Nov 9, 2026
Afloat16 added a commit to Afloat16/carla that referenced this pull request Nov 9, 2026
Retain the existing documentation patch and the depth-precision entry from carla-simulator#10189. Place the two changelog entries at separate section anchors so they merge independently.
Afloat16 added a commit to Afloat16/carla that referenced this pull request Nov 9, 2026
Retain the existing documentation patch and the depth-precision entry from carla-simulator#10189. Place the two changelog entries at separate section anchors so they merge independently.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants