AcademySoftwareFoundation / OpenShadingLanguage

Advanced shading language for production GI renderers
BSD 3-Clause "New" or "Revised" License
2.07k stars 350 forks source link

RendererServices API for letting get_texture_handle consider colorspace #1641

Closed lgritz closed 1 year ago

lgritz commented 1 year ago

This makes some minor changes to the RS APIs to add an additional optional parameter to get_texture_handle so it takes a TextureOpt* to accommodate a similar change on the OIIO TextureSystem side from https://github.com/OpenImageIO/oiio/pull/3761

The purpose here is to add color management options to texture calls. However, please note that this PR only reserves a public API change we know we'll want, but it does not fully implement the color management. Further work will be needed, as follows:

Signed-off-by: Larry Gritz lg@larrygritz.com

lgritz commented 1 year ago

I will merge this at end of day tomorrow unless somebody has concerns.