Skip to content

SG-44518: 1362: feature: Pixel Inspector optionally returns raw pixel values - #1363

Open
bsloan666 wants to merge 1 commit into
AcademySoftwareFoundation:mainfrom
bsloan666:feat_pixel_inspector_raw_pixel_values
Open

SG-44518: 1362: feature: Pixel Inspector optionally returns raw pixel values#1363
bsloan666 wants to merge 1 commit into
AcademySoftwareFoundation:mainfrom
bsloan666:feat_pixel_inspector_raw_pixel_values

Conversation

@bsloan666

Copy link
Copy Markdown

This change modifies CommandsModule::sourcePixelValue() so that it returns the viewed file's raw pixel values when "Ignore File Primaries" is checked in RV's color menu. The normal behavior is to convert the linear RGB values in the file to a Rec709 gamut if the color primaries of the input file pixels are known.

This is particularly useful when viewing ACES EXR media and verifying that a sampled pixel value is the same as when sampled in other applications such as Nuke or OpenImageIO

Three source files are changed:

• src/lib/image/TwkFB/TwkFB/Operations.h
• src/lib/image/TwkFB/Operations.cpp
• src/lib/app/RvApp/CommandsModule.cpp

This has been tested on MacOS Sequoia 15.3.1 using the 2024 VFX platform configuration.

@linux-foundation-easycla

linux-foundation-easycla Bot commented Jul 23, 2026

Copy link
Copy Markdown

CLA Missing ID

  • ❌ The email address for the commit (1f54bc8) is not linked to the GitHub account, preventing the EasyCLA check. Consult this Help Article and GitHub Help to resolve. (To view the commit's email address, add .patch at the end of this PR page's URL.) For further assistance with EasyCLA, please visit our EasyCLA portal and chat with our support bot.

…imaries is checked

Signed-off-by: Blake Sloan <bsloan@marvelstudios.com>
@bsloan666
bsloan666 force-pushed the feat_pixel_inspector_raw_pixel_values branch from c774623 to 1f54bc8 Compare July 23, 2026 20:53
@bernie-laberge bernie-laberge changed the title 1362: feature: Pixel Inspector optionally returns raw pixel values SG-44518: 1362: feature: Pixel Inspector optionally returns raw pixel values Jul 24, 2026
@bernie-laberge bernie-laberge added PR: Acknowledged New PR has been acknowledge by the TSC community Contribution from the Open RV Community labels Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Contribution from the Open RV Community PR: Acknowledged New PR has been acknowledge by the TSC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants