Ship Workbench editing and verified LUT previews - #28
Merged
Merged
Conversation
# Conflicts: # frontend/src/App.tsx # src/lut_builder/static/assets/index-BnjL_c7J.js # src/lut_builder/static/assets/index-Bw25AxTc.js # src/lut_builder/static/assets/index-CXv223nh.js # src/lut_builder/static/index.html
# Conflicts: # frontend/src/App.tsx # frontend/src/editor.ts # frontend/tests/editor.test.mjs # src/lut_builder/static/assets/index-B78VTtJZ.js # src/lut_builder/static/assets/index-CXv223nh.js # src/lut_builder/static/assets/index-DVnEJIcj.js # src/lut_builder/static/index.html
…face # Conflicts: # frontend/src/App.tsx # src/lut_builder/static/index.html
# Conflicts: # frontend/src/App.tsx # src/lut_builder/static/assets/index-DDJOtsRR.js # src/lut_builder/static/index.html
# Conflicts: # frontend/src/App.tsx # frontend/tests/exposure-graph.test.mjs
# Conflicts: # src/lut_builder/static/index.html
# Conflicts: # src/lut_builder/static/index.html # src/lut_builder/web.py
# Conflicts: # README.md
Today20092
changed the title
Today20092
marked this pull request as ready for review
September 7, 2026 18:02
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The LUT Builder now has the chosen Workbench layout, precise band editing with undo, reusable custom palettes, and a docked, expandable preview. Controls share consistent sizing, the graph stays prominent, warning labels remain contained, and saved palettes survive application restarts at the stable local address
http://127.0.0.1:8765/.Demonstration and camera-matched verification are explicitly separate. Verification processes preserved camera samples through the actual serialized cube, exposes numerical pixel inspection and the exact checked artifact, and invalidates stale results. Local video selection uses decoded presentation timestamps and confirmed recording settings. The existing LUT mathematics and setup workflow are preserved.
This implements the approved Workbench specification #33 and carries forward the previously prepared UV/Ruff/ty workflow, typing fixes, and Python CI. The original Astral work is preserved in the merge history.
Validation completed locally on Windows:
Initial support is SDR Rec.709/Rec.2020 viewing; camera stills are untagged 16-bit RGB PNG or float32 RGB PFM. Supported video is limited to 1080p, 256 MiB, and two minutes. The interface documents formats and limits; camera verification displays the unblended result because no neutral source viewing transform has been established.
Closes #7
Closes #8
Closes #21
Closes #22
Closes #24
Closes #26
Closes #27
Completed implementation slices: #34, #25, #35, #36, #37, #38. Parent planning issues remain available for user review.