Skip to content
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

Flatbuffers: selectable custom LUT files #334

Merged
merged 1 commit into from
Sep 10, 2022

Conversation

awawa-dev
Copy link
Owner

The JSON API for controlling tone mapping has been extended to include an optional LUT file name for FlatBuffers.
This allows to select proper LUT for tone mapping depending on the external video properties.
For example you can have two or more LUT files for selection: one for SDR, one for HDR, one for DV etc.
HyperHDR expects that these files are located in HyperHDR home folder e.g. /home/pi/.hyperhdr

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • [ x] Feature
  • Code style update
  • Refactor
  • Docs
  • Build-related changes
  • Other, please describe:

If changing the UI of web configuration, please provide the before/after screenshot:

obraz

Does this PR introduce a breaking change? (check one)

  • Yes
  • [ x] No

@awawa-dev awawa-dev added the enhancement New feature or request label Sep 5, 2022
@awawa-dev awawa-dev closed this Sep 6, 2022
@awawa-dev awawa-dev force-pushed the Flatbuffers_Custom_LUT_files_API branch from 3488738 to ba42c26 Compare September 6, 2022 14:30
@awawa-dev awawa-dev reopened this Sep 6, 2022
@awawa-dev awawa-dev merged commit f30babe into master Sep 10, 2022
@awawa-dev awawa-dev deleted the Flatbuffers_Custom_LUT_files_API branch September 10, 2022 15:05
chbartsch pushed a commit to chbartsch/HyperHDR that referenced this pull request Nov 29, 2022
@awawa-dev awawa-dev mentioned this pull request Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant