Add tunable tone/gamut mapping constants
This is sort of the reason why pl_options
was introduced, so I can create micro-tunables like this for experts to play with (or to quickly iterate on design decisions) without expecting downstream clients to expose them all as separate controls.
Consider this a test drive.
Edited by Niklas Haas
Merge request reports
Activity
Filter activity
added 10 commits
- c44b76d9 - gamut_mapping: make hard-coded constants tunable
- 9f7a5c24 - options: fix gamut_mapping options order
- 91b296ba - options: expose pl_color_map_constants options
- 25b3f3a8 - options: don't reject 0.0 floats
- 3f34426f - tests/options: explicitly test setting 0.0
- a9fd5ce3 - tone_mapping: change st2094-10 knee point default to 0.7
- 3fa9a881 - tone_mapping: change gamma default knee to 0.3
- 18125d54 - tone_mapping: refactor tone mapping parameters API
- f1402dc5 - options: map pl_tone_map_constants
- 7b2de5ed - demos/plplay: remove deprecated tone mapping param UI
Toggle commit listTODO:
- expose these controls also in plplay
I thought
pl_options
would make my life easier, but now it turns out I still need to add new options in three places: the code, options.c, and demos/plplay.c. Maybe we could auto-generate advanced options in plplay by enumeratingpl_options
?added 13 commits
-
63c3767e - 1 commit from branch
videolan:master
- 63c3767e...15613d58 - 2 earlier commits
- 72dd716a - options: fix gamut_mapping options order
- 73519283 - options: expose pl_color_map_constants options
- 997020fa - options: don't reject 0.0 floats
- 8a5e97ef - tests/options: explicitly test setting 0.0
- 89685163 - tone_mapping: change st2094-10 knee point default to 0.7
- e44c48a7 - tone_mapping: change gamma default knee to 0.3
- eba01004 - tone_mapping: refactor tone mapping parameters API
- eda5c496 - options: map pl_tone_map_constants
- 5a0d3d35 - demos/plplay: remove deprecated tone mapping param UI
- dbdfb172 - demos/plplay: add advanced constants section
Toggle commit list-
63c3767e - 1 commit from branch
Please register or sign in to reply