Skip to content

Support rgba64 in lift_gamma_gain#1177

Merged
ddennedy merged 3 commits into
masterfrom
lift_gamma_gain-rgba64
Nov 29, 2025
Merged

Support rgba64 in lift_gamma_gain#1177
ddennedy merged 3 commits into
masterfrom
lift_gamma_gain-rgba64

Conversation

@ddennedy

Copy link
Copy Markdown
Member

No description provided.

@ddennedy ddennedy requested a review from Copilot November 28, 2025 19:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for 16-bit RGBA (rgba64) image format to the lift_gamma_gain filter. The implementation converts fixed-size 8-bit lookup tables (LUTs) to dynamically allocated memory that supports both 8-bit and 16-bit color depths, enabling higher precision color grading operations.

Key Changes:

  • Converted static 8-bit LUTs to dynamically allocated pointers supporting both 8-bit and 16-bit formats
  • Added format-specific LUT generation with proper memory management and error handling
  • Implemented rgba64 processing path in the sliced processor

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
filter_lift_gamma_gain.yml Bumped filter version from 1 to 2 to reflect the new rgba64 support
filter_lift_gamma_gain.c Refactored LUT storage from fixed arrays to dynamic allocation, added 16-bit LUT generation logic, implemented rgba64 case in image processor, and updated cleanup/initialization code

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/modules/plus/filter_lift_gamma_gain.c Outdated
Comment thread src/modules/plus/filter_lift_gamma_gain.c
Comment thread src/modules/plus/filter_lift_gamma_gain.c
Comment thread src/modules/plus/filter_lift_gamma_gain.c
@ddennedy ddennedy marked this pull request as ready for review November 28, 2025 20:36
@ddennedy ddennedy requested a review from bmatherly November 28, 2025 20:36
Comment thread src/modules/plus/filter_lift_gamma_gain.c
@ddennedy ddennedy merged commit 0cddd13 into master Nov 29, 2025
21 checks passed
@ddennedy ddennedy deleted the lift_gamma_gain-rgba64 branch November 29, 2025 02:13
@ddennedy ddennedy added this to the v7.36.0 milestone Nov 29, 2025
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