Skip to content

choose gainmap application space adaptively basing on input gamuts #336

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

ram-mohan
Copy link
Contributor

during generateGainMap, the color spaces of hdr intent and sdr intent are unified. if a wider gamut space is converted to narrower-gamut this may result in pixel values less than zero and/or greater than 1. choosing offsets to keep the domain of log during gainmap computation is tricky. This can be mitigated by converting narrower gamut space data to wider gamut space. This yields lesser overrange values. current change implements this.

for this to work as intended writing xmp metadata in the bitstream needs to be disabled.

Test: ./ultrahdr_app options

Change-Id: I0155a1dc043f3c3e0493e04e5a6eabe160659476

during generateGainMap, the color spaces of hdr intent and sdr intent
are unified. if a wider gamut space is converted to narrower-gamut this
may result in pixel values less than zero and/or greater than 1.
choosing offsets to keep the domain of log during gainmap computation is
tricky. This can be mitigated by converting narrower gamut space data to
wider gamut space. This yields lesser overrange values. current change
implements this.

for this to work as intended writing xmp metadata in the bitstream needs
to be disabled.

Test: ./ultrahdr_app options

Co-authored-by: Vivek R Jadhav <vivek.jadhav@ittiam.com>
Change-Id: I0155a1dc043f3c3e0493e04e5a6eabe160659476
@DichenZhang1 DichenZhang1 merged commit bd7712f into google:main Dec 5, 2024
20 checks passed
@ram-mohan ram-mohan deleted the adapgmsp branch December 5, 2024 02:07
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