Skip to content

Adsk Contrib - Add DCDM displays and improve HLG OOTF implementation#2051

Merged
doug-walker merged 6 commits into
AcademySoftwareFoundation:mainfrom
autodesk-forks:walker_displays
Sep 26, 2024
Merged

Adsk Contrib - Add DCDM displays and improve HLG OOTF implementation#2051
doug-walker merged 6 commits into
AcademySoftwareFoundation:mainfrom
autodesk-forks:walker_displays

Conversation

@doug-walker
Copy link
Copy Markdown
Collaborator

Add SDR and HDR DCDM built-in displays, which are in the set of Output Transforms recommended by ACES 2.

Improve the Rec.2100 surround algorithm used for the Rec.2100-HLG OOTF and EOTF:

  • Fix an issue with the inverse
  • Mirror the result around the origin for negative values

Signed-off-by: Doug Walker <doug.walker@autodesk.com>
Signed-off-by: Doug Walker <doug.walker@autodesk.com>
Comment thread src/OpenColorIO/transforms/builtins/Displays.cpp Outdated
Comment thread src/OpenColorIO/transforms/builtins/Displays.cpp Outdated
Signed-off-by: Doug Walker <doug.walker@autodesk.com>
Signed-off-by: Doug Walker <doug.walker@autodesk.com>
Copy link
Copy Markdown
Collaborator

@zachlewis zachlewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

El gee tee em!
Nice work, Doug!

Copy link
Copy Markdown
Collaborator

@remia remia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only other display I see missing is the Display P3 / Gamma 2.2 variant, but maybe we already decided not to include. LGTM in any case!

Signed-off-by: Doug Walker <doug.walker@autodesk.com>
@doug-walker doug-walker merged commit 5c80c0b into AcademySoftwareFoundation:main Sep 26, 2024
chrisnashdisguise pushed a commit to chrisnashdisguise/OpenColorIO that referenced this pull request Oct 30, 2024
…cademySoftwareFoundation#2051)

* Add DCDM displays, improve Rec.2100 surround

Signed-off-by: Doug Walker <doug.walker@autodesk.com>

* Fix Linux test

Signed-off-by: Doug Walker <doug.walker@autodesk.com>

* Remove 48 nit scaling

Signed-off-by: Doug Walker <doug.walker@autodesk.com>

* Remove D60 DCDM built-ins

Signed-off-by: Doug Walker <doug.walker@autodesk.com>

* Add version check

Signed-off-by: Doug Walker <doug.walker@autodesk.com>

---------

Signed-off-by: Doug Walker <doug.walker@autodesk.com>
Signed-off-by: Chris Nash <chris.nash@disguise.one>
@doug-walker doug-walker deleted the walker_displays branch April 18, 2026 21:36
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.

3 participants