X-Git-Url: http://users.mur.at/ms/git/gitweb/?p=OpenColorIO-Configs.git;a=blobdiff_plain;f=aces_1.0.0%2Fpython%2Faces_ocio%2Fcreate_aces_config.py;h=a18ba48fdd8e56aae339234713b774be738571ab;hp=e97d5d71a5e835f6caad5c7e3295ad6691621162;hb=6eac5f132b3888eee16a8306d91b9405346f3ac2;hpb=48d67eb8a5804cd9cabf0a22c3f421502b983b24 diff --git a/aces_1.0.0/python/aces_ocio/create_aces_config.py b/aces_1.0.0/python/aces_ocio/create_aces_config.py index e97d5d7..a18ba48 100755 --- a/aces_1.0.0/python/aces_ocio/create_aces_config.py +++ b/aces_1.0.0/python/aces_ocio/create_aces_config.py @@ -8,6 +8,7 @@ Defines objects creating the *ACES* configuration. import math import numpy import os +import pprint import shutil import string import sys @@ -19,13 +20,16 @@ sys.path.append(os.path.join(os.path.dirname(__file__), '..')) import PyOpenColorIO as OCIO -import aces_ocio.createARRIColorSpaces as arri -import aces_ocio.createCanonColorSpaces as canon -import aces_ocio.createREDColorSpaces as red -import aces_ocio.createSonyColorSpaces as sony -import aces_ocio.generateLUT as genlut +import aces_ocio.create_arri_colorspaces as arri +import aces_ocio.create_canon_colorspaces as canon +import aces_ocio.create_red_colorspaces as red +import aces_ocio.create_sony_colorspaces as sony +from aces_ocio.generate_lut import ( + generate_1d_LUT_from_CTL, + generate_3d_LUT_from_CTL, + write_SPI_1d) from aces_ocio.process import Process -from aces_ocio.util import ColorSpace, mat44FromMat33 +from aces_ocio.utilities import ColorSpace, mat44_from_mat33 __author__ = 'ACES Developers' __copyright__ = 'Copyright (C) 2014 - 2015 - ACES Developers' @@ -36,33 +40,33 @@ __status__ = 'Production' __all__ = ['ACES_OCIO_CTL_DIRECTORY_ENVIRON', 'ACES_OCIO_CONFIGURATION_DIRECTORY_ENVIRON', - 'setConfigDefaultRoles', - 'writeConfig', - 'generateOCIOTransform', - 'createConfig', - 'generateLUTs', - 'generateBakedLUTs', - 'createConfigDir', - 'getTransformInfo', - 'getODTInfo', - 'getLMTInfo', - 'createACESConfig', + 'set_config_default_roles', + 'write_config', + 'generate_OCIO_transform', + 'create_config', + 'generate_LUTs', + 'generate_baked_LUTs', + 'create_config_dir', + 'get_transform_info', + 'get_ODT_info', + 'get_LMT_info', + 'create_ACES_config', 'main'] ACES_OCIO_CTL_DIRECTORY_ENVIRON = 'ACES_OCIO_CTL_DIRECTORY' ACES_OCIO_CONFIGURATION_DIRECTORY_ENVIRON = 'ACES_OCIO_CONFIGURATION_DIRECTORY' -def setConfigDefaultRoles(config, - color_picking="", - color_timing="", - compositing_log="", - data="", - default="", - matte_paint="", - reference="", - scene_linear="", - texture_paint=""): +def set_config_default_roles(config, + color_picking="", + color_timing="", + compositing_log="", + data="", + default="", + matte_paint="", + reference="", + scene_linear="", + texture_paint=""): """ Sets given *OCIO* configuration default roles. @@ -117,7 +121,7 @@ def setConfigDefaultRoles(config, return True -def writeConfig(config, configPath, sanityCheck=True): +def write_config(config, config_path, sanity_check=True): """ Writes the configuration to given path. @@ -132,7 +136,7 @@ def writeConfig(config, configPath, sanityCheck=True): Return value description. """ - if sanityCheck: + if sanity_check: try: config.sanityCheck() except Exception, e: @@ -141,12 +145,12 @@ def writeConfig(config, configPath, sanityCheck=True): return # sys.exit() - fileHandle = open(configPath, mode='w') - fileHandle.write(config.serialize()) - fileHandle.close() + file_handle = open(config_path, mode='w') + file_handle.write(config.serialize()) + file_handle.close() -def generateOCIOTransform(transforms): +def generate_OCIO_transform(transforms): """ Object description. @@ -163,67 +167,68 @@ def generateOCIOTransform(transforms): # print("Generating transforms") - interpolationOptions = { + interpolation_options = { 'linear': OCIO.Constants.INTERP_LINEAR, 'nearest': OCIO.Constants.INTERP_NEAREST, 'tetrahedral': OCIO.Constants.INTERP_TETRAHEDRAL } - directionOptions = { + direction_options = { 'forward': OCIO.Constants.TRANSFORM_DIR_FORWARD, 'inverse': OCIO.Constants.TRANSFORM_DIR_INVERSE } - ocioTransforms = [] + ocio_transforms = [] for transform in transforms: if transform['type'] == 'lutFile': - ocioTransform = OCIO.FileTransform( + ocio_transform = OCIO.FileTransform( src=transform['path'], - interpolation=interpolationOptions[transform['interpolation']], - direction=directionOptions[transform['direction']]) - ocioTransforms.append(ocioTransform) + interpolation=interpolation_options[ + transform['interpolation']], + direction=direction_options[transform['direction']]) + ocio_transforms.append(ocio_transform) elif transform['type'] == 'matrix': - ocioTransform = OCIO.MatrixTransform() + ocio_transform = OCIO.MatrixTransform() # MatrixTransform member variables can't be initialized directly. # Each must be set individually. - ocioTransform.setMatrix(transform['matrix']) + ocio_transform.setMatrix(transform['matrix']) if 'offset' in transform: - ocioTransform.setOffset(transform['offset']) + ocio_transform.setOffset(transform['offset']) if 'direction' in transform: - ocioTransform.setDirection( - directionOptions[transform['direction']]) + ocio_transform.setDirection( + direction_options[transform['direction']]) - ocioTransforms.append(ocioTransform) + ocio_transforms.append(ocio_transform) elif transform['type'] == 'exponent': - ocioTransform = OCIO.ExponentTransform() - ocioTransform.setValue(transform['value']) - ocioTransforms.append(ocioTransform) + ocio_transform = OCIO.ExponentTransform() + ocio_transform.setValue(transform['value']) + ocio_transforms.append(ocio_transform) elif transform['type'] == 'log': - ocioTransform = OCIO.LogTransform( + ocio_transform = OCIO.LogTransform( base=transform['base'], - direction=directionOptions[transform['direction']]) + direction=direction_options[transform['direction']]) - ocioTransforms.append(ocioTransform) + ocio_transforms.append(ocio_transform) else: print("Ignoring unknown transform type : %s" % transform['type']) # Build a group transform if necessary - if len(ocioTransforms) > 1: - transformG = OCIO.GroupTransform() - for transform in ocioTransforms: - transformG.push_back(transform) - transform = transformG + if len(ocio_transforms) > 1: + transform_G = OCIO.GroupTransform() + for transform in ocio_transforms: + transform_G.push_back(transform) + transform = transform_G # Or take the first transform from the list else: - transform = ocioTransforms[0] + transform = ocio_transforms[0] return transform -def createConfig(configData, nuke=False): +def create_config(config_data, nuke=False): """ Object description. @@ -250,19 +255,19 @@ def createConfig(configData, nuke=False): # # Define the reference color space # - referenceData = configData['referenceColorSpace'] - print("Adding the reference color space : %s" % referenceData.name) + reference_data = config_data['referenceColorSpace'] + print("Adding the reference color space : %s" % reference_data.name) # Create a color space reference = OCIO.ColorSpace( - name=referenceData.name, - bitDepth=referenceData.bitDepth, - description=referenceData.description, - equalityGroup=referenceData.equalityGroup, - family=referenceData.family, - isData=referenceData.isData, - allocation=referenceData.allocationType, - allocationVars=referenceData.allocationVars) + name=reference_data.name, + bitDepth=reference_data.bit_depth, + description=reference_data.description, + equalityGroup=reference_data.equality_group, + family=reference_data.family, + isData=reference_data.is_data, + allocation=reference_data.allocation_type, + allocationVars=reference_data.allocation_vars) # Add to config config.addColorSpace(reference) @@ -270,36 +275,36 @@ def createConfig(configData, nuke=False): # # Create the rest of the color spaces # - for colorspace in sorted(configData['colorSpaces']): + for colorspace in sorted(config_data['colorSpaces']): print("Creating new color space : %s" % colorspace.name) - ocioColorspace = OCIO.ColorSpace( + ocio_colorspace = OCIO.ColorSpace( name=colorspace.name, - bitDepth=colorspace.bitDepth, + bitDepth=colorspace.bit_depth, description=colorspace.description, - equalityGroup=colorspace.equalityGroup, + equalityGroup=colorspace.equality_group, family=colorspace.family, - isData=colorspace.isData, - allocation=colorspace.allocationType, - allocationVars=colorspace.allocationVars) + isData=colorspace.is_data, + allocation=colorspace.allocation_type, + allocationVars=colorspace.allocation_vars) - if colorspace.toReferenceTransforms != []: + if colorspace.to_reference_transforms != []: print("Generating To-Reference transforms") - ocioTransform = generateOCIOTransform( - colorspace.toReferenceTransforms) - ocioColorspace.setTransform( - ocioTransform, + ocio_transform = generate_OCIO_transform( + colorspace.to_reference_transforms) + ocio_colorspace.setTransform( + ocio_transform, OCIO.Constants.COLORSPACE_DIR_TO_REFERENCE) - if colorspace.fromReferenceTransforms != []: + if colorspace.from_reference_transforms != []: print("Generating From-Reference transforms") - ocioTransform = generateOCIOTransform( - colorspace.fromReferenceTransforms) - ocioColorspace.setTransform( - ocioTransform, + ocio_transform = generate_OCIO_transform( + colorspace.from_reference_transforms) + ocio_colorspace.setTransform( + ocio_transform, OCIO.Constants.COLORSPACE_DIR_FROM_REFERENCE) - config.addColorSpace(ocioColorspace) + config.addColorSpace(ocio_colorspace) print("") @@ -311,11 +316,11 @@ def createConfig(configData, nuke=False): # Generic display and view setup if not nuke: - for display, viewList in configData['displays'].iteritems(): - for viewName, colorspace in viewList.iteritems(): - config.addDisplay(display, viewName, colorspace.name) - if not (viewName in views): - views.append(viewName) + for display, view_list in config_data['displays'].iteritems(): + for view_name, colorspace in view_list.iteritems(): + config.addDisplay(display, view_name, colorspace.name) + if not (view_name in views): + views.append(view_name) displays.append(display) # A Nuke specific set of views and displays # @@ -324,13 +329,13 @@ def createConfig(configData, nuke=False): # Would be better to automate. # else: - for display, viewList in configData['displays'].iteritems(): - for viewName, colorspace in viewList.iteritems(): - if (viewName == 'Output Transform'): - viewName = 'View' - config.addDisplay(display, viewName, colorspace.name) - if not (viewName in views): - views.append(viewName) + for display, view_list in config_data['displays'].iteritems(): + for view_name, colorspace in view_list.iteritems(): + if (view_name == 'Output Transform'): + view_name = 'View' + config.addDisplay(display, view_name, colorspace.name) + if not (view_name in views): + views.append(view_name) displays.append(display) config.addDisplay('linear', 'View', 'ACES2065-1') @@ -347,16 +352,17 @@ def createConfig(configData, nuke=False): # # Add Default Roles - setConfigDefaultRoles(config, - color_picking=reference.getName(), - color_timing=reference.getName(), - compositing_log=reference.getName(), - data=reference.getName(), - default=reference.getName(), - matte_paint=reference.getName(), - reference=reference.getName(), - scene_linear=reference.getName(), - texture_paint=reference.getName()) + set_config_default_roles( + config, + color_picking=reference.getName(), + color_timing=reference.getName(), + compositing_log=reference.getName(), + data=reference.getName(), + default=reference.getName(), + matte_paint=reference.getName(), + reference=reference.getName(), + scene_linear=reference.getName(), + texture_paint=reference.getName()) # Check to make sure we didn't screw something up config.sanityCheck() @@ -364,14 +370,14 @@ def createConfig(configData, nuke=False): return config -def generateLUTs(odtInfo, - lmtInfo, - shaperName, - acesCTLReleaseDir, - lutDir, - lutResolution1d=4096, - lutResolution3d=64, - cleanup=True): +def generate_LUTs(odt_info, + lmt_info, + shaper_name, + aces_CTL_directory, + lut_directory, + lut_resolution_1d=4096, + lut_resolution_3d=64, + cleanup=True): """ Object description. @@ -388,7 +394,7 @@ def generateLUTs(odtInfo, """ print("generateLUTs - begin") - configData = {} + config_data = {} # # Define the reference color space @@ -396,74 +402,74 @@ def generateLUTs(odtInfo, ACES = ColorSpace('ACES2065-1') ACES.description = ( 'The Academy Color Encoding System reference color space') - ACES.equalityGroup = '' + ACES.equality_group = '' ACES.family = 'ACES' - ACES.isData = False - ACES.allocationType = OCIO.Constants.ALLOCATION_LG2 - ACES.allocationVars = [-15, 6] + ACES.is_data = False + ACES.allocation_type = OCIO.Constants.ALLOCATION_LG2 + ACES.allocation_vars = [-15, 6] - configData['referenceColorSpace'] = ACES + config_data['referenceColorSpace'] = ACES # # Define the displays # - configData['displays'] = {} + config_data['displays'] = {} # # Define the other color spaces # - configData['colorSpaces'] = [] + config_data['colorSpaces'] = [] # Matrix converting ACES AP1 primaries to AP0 - acesAP1toAP0 = [0.6954522414, 0.1406786965, 0.1638690622, - 0.0447945634, 0.8596711185, 0.0955343182, - -0.0055258826, 0.0040252103, 1.0015006723] + ACES_AP1_to_AP0 = [0.6954522414, 0.1406786965, 0.1638690622, + 0.0447945634, 0.8596711185, 0.0955343182, + -0.0055258826, 0.0040252103, 1.0015006723] # Matrix converting ACES AP0 primaries to XYZ - acesAP0toXYZ = [0.9525523959, 0.0000000000, 0.0000936786, - 0.3439664498, 0.7281660966, -0.0721325464, - 0.0000000000, 0.0000000000, 1.0088251844] + ACES_AP0_to_XYZ = [0.9525523959, 0.0000000000, 0.0000936786, + 0.3439664498, 0.7281660966, -0.0721325464, + 0.0000000000, 0.0000000000, 1.0088251844] # # ACEScc # - def createACEScc(name='ACEScc', - minValue=0.0, - maxValue=1.0, - inputScale=1.0): + def create_ACEScc(name='ACEScc', + min_value=0.0, + max_value=1.0, + input_scale=1.0): cs = ColorSpace(name) cs.description = "The %s color space" % name - cs.equalityGroup = '' + cs.equality_group = '' cs.family = 'ACES' - cs.isData = False + cs.is_data = False ctls = [ - '%s/ACEScc/ACEScsc.ACEScc_to_ACES.a1.0.0.ctl' % acesCTLReleaseDir, + '%s/ACEScc/ACEScsc.ACEScc_to_ACES.a1.0.0.ctl' % aces_CTL_directory, # This transform gets back to the AP1 primaries # Useful as the 1d LUT is only covering the transfer function # The primaries switch is covered by the matrix below - '%s/ACEScg/ACEScsc.ACES_to_ACEScg.a1.0.0.ctl' % acesCTLReleaseDir + '%s/ACEScg/ACEScsc.ACES_to_ACEScg.a1.0.0.ctl' % aces_CTL_directory ] lut = "%s_to_ACES.spi1d" % name # Remove spaces and parentheses lut = lut.replace(' ', '_').replace(')', '_').replace('(', '_') - genlut.generate1dLUTFromCTL( - lutDir + "/" + lut, + generate_1d_LUT_from_CTL( + lut_directory + "/" + lut, ctls, - lutResolution1d, + lut_resolution_1d, 'float', - inputScale, + input_scale, 1.0, {}, cleanup, - acesCTLReleaseDir, - minValue, - maxValue) + aces_CTL_directory, + min_value, + max_value) - cs.toReferenceTransforms = [] - cs.toReferenceTransforms.append({ + cs.to_reference_transforms = [] + cs.to_reference_transforms.append({ 'type': 'lutFile', 'path': lut, 'interpolation': 'linear', @@ -471,54 +477,54 @@ def generateLUTs(odtInfo, }) # AP1 primaries to AP0 primaries - cs.toReferenceTransforms.append({ + cs.to_reference_transforms.append({ 'type': 'matrix', - 'matrix': mat44FromMat33(acesAP1toAP0), + 'matrix': mat44_from_mat33(ACES_AP1_to_AP0), 'direction': 'forward' }) - cs.fromReferenceTransforms = [] + cs.from_reference_transforms = [] return cs - ACEScc = createACEScc() - configData['colorSpaces'].append(ACEScc) + ACEScc = create_ACEScc() + config_data['colorSpaces'].append(ACEScc) # # ACESproxy # - def createACESProxy(name='ACESproxy'): + def create_ACESproxy(name='ACESproxy'): cs = ColorSpace(name) cs.description = "The %s color space" % name - cs.equalityGroup = '' + cs.equality_group = '' cs.family = 'ACES' - cs.isData = False + cs.is_data = False ctls = [ '%s/ACESproxy/ACEScsc.ACESproxy10i_to_ACES.a1.0.0.ctl' % ( - acesCTLReleaseDir), + aces_CTL_directory), # This transform gets back to the AP1 primaries # Useful as the 1d LUT is only covering the transfer function # The primaries switch is covered by the matrix below - '%s/ACEScg/ACEScsc.ACES_to_ACEScg.a1.0.0.ctl' % acesCTLReleaseDir + '%s/ACEScg/ACEScsc.ACES_to_ACEScg.a1.0.0.ctl' % aces_CTL_directory ] lut = "%s_to_aces.spi1d" % name # Remove spaces and parentheses lut = lut.replace(' ', '_').replace(')', '_').replace('(', '_') - genlut.generate1dLUTFromCTL( - lutDir + "/" + lut, + generate_1d_LUT_from_CTL( + lut_directory + "/" + lut, ctls, - lutResolution1d, + lut_resolution_1d, 'uint16', 64.0, 1.0, {}, cleanup, - acesCTLReleaseDir) + aces_CTL_directory) - cs.toReferenceTransforms = [] - cs.toReferenceTransforms.append({ + cs.to_reference_transforms = [] + cs.to_reference_transforms.append({ 'type': 'lutFile', 'path': lut, 'interpolation': 'linear', @@ -526,63 +532,63 @@ def generateLUTs(odtInfo, }) # AP1 primaries to AP0 primaries - cs.toReferenceTransforms.append({ + cs.to_reference_transforms.append({ 'type': 'matrix', - 'matrix': mat44FromMat33(acesAP1toAP0), + 'matrix': mat44_from_mat33(ACES_AP1_to_AP0), 'direction': 'forward' }) - cs.fromReferenceTransforms = [] + cs.from_reference_transforms = [] return cs - ACESproxy = createACESProxy() - configData['colorSpaces'].append(ACESproxy) + ACESproxy = create_ACESproxy() + config_data['colorSpaces'].append(ACESproxy) # # ACEScg # - def createACEScg(name='ACEScg'): + def create_ACEScg(name='ACEScg'): cs = ColorSpace(name) cs.description = "The %s color space" % name - cs.equalityGroup = '' + cs.equality_group = '' cs.family = 'ACES' - cs.isData = False + cs.is_data = False - cs.toReferenceTransforms = [] + cs.to_reference_transforms = [] # AP1 primaries to AP0 primaries - cs.toReferenceTransforms.append({ + cs.to_reference_transforms.append({ 'type': 'matrix', - 'matrix': mat44FromMat33(acesAP1toAP0), + 'matrix': mat44_from_mat33(ACES_AP1_to_AP0), 'direction': 'forward' }) - cs.fromReferenceTransforms = [] + cs.from_reference_transforms = [] return cs - ACEScg = createACEScg() - configData['colorSpaces'].append(ACEScg) + ACEScg = create_ACEScg() + config_data['colorSpaces'].append(ACEScg) # # ADX # - def createADX(bitdepth=10, name='ADX'): - name = "%s%s" % (name, bitdepth) + def create_ADX(bit_depth=10, name='ADX'): + name = "%s%s" % (name, bit_depth) cs = ColorSpace(name) cs.description = "%s color space - used for film scans" % name - cs.equalityGroup = '' + cs.equality_group = '' cs.family = 'ADX' - cs.isData = False + cs.is_data = False - if bitdepth == 10: - cs.bitDepth = bitDepth = OCIO.Constants.BIT_DEPTH_UINT10 + if bit_depth == 10: + cs.bit_depth = bit_depth = OCIO.Constants.BIT_DEPTH_UINT10 adx_to_cdd = [1023.0 / 500.0, 0.0, 0.0, 0.0, 0.0, 1023.0 / 500.0, 0.0, 0.0, 0.0, 0.0, 1023.0 / 500.0, 0.0, 0.0, 0.0, 0.0, 1.0] offset = [-95.0 / 500.0, -95.0 / 500.0, -95.0 / 500.0, 0.0] - elif bitdepth == 16: - cs.bitDepth = bitDepth = OCIO.Constants.BIT_DEPTH_UINT16 + elif bit_depth == 16: + cs.bit_depth = bit_depth = OCIO.Constants.BIT_DEPTH_UINT16 adx_to_cdd = [65535.0 / 8000.0, 0.0, 0.0, 0.0, 0.0, 65535.0 / 8000.0, 0.0, 0.0, 0.0, 0.0, 65535.0 / 8000.0, 0.0, @@ -590,10 +596,10 @@ def generateLUTs(odtInfo, offset = [-1520.0 / 8000.0, -1520.0 / 8000.0, -1520.0 / 8000.0, 0.0] - cs.toReferenceTransforms = [] + cs.to_reference_transforms = [] # Convert from ADX to Channel-Dependent Density - cs.toReferenceTransforms.append({ + cs.to_reference_transforms.append({ 'type': 'matrix', 'matrix': adx_to_cdd, 'offset': offset, @@ -601,7 +607,7 @@ def generateLUTs(odtInfo, }) # Convert from Channel-Dependent Density to Channel-Independent Density - cs.toReferenceTransforms.append({ + cs.to_reference_transforms.append({ 'type': 'matrix', 'matrix': [0.75573, 0.22197, 0.02230, 0, 0.05901, 0.96928, -0.02829, 0, @@ -611,8 +617,8 @@ def generateLUTs(odtInfo, }) # Copied from Alex Fry's adx_cid_to_rle.py - def createCIDtoRLELUT(): - def interpolate1D(x, xp, fp): + def create_CID_to_RLE_LUT(): + def interpolate_1D(x, xp, fp): return numpy.interp(x, xp, fp) LUT_1D_xp = [-0.190000000000000, @@ -644,33 +650,34 @@ def generateLUTs(odtInfo, def cid_to_rle(x): if x <= 0.6: - return interpolate1D(x, LUT_1D_xp, LUT_1D_fp) + return interpolate_1D(x, LUT_1D_xp, LUT_1D_fp) return (100.0 / 55.0) * x - REF_PT - def Fit(value, fromMin, fromMax, toMin, toMax): - if fromMin == fromMax: - raise ValueError("fromMin == fromMax") - return (value - fromMin) / (fromMax - fromMin) * ( - toMax - toMin) + toMin + def fit(value, from_min, from_max, to_min, to_max): + if from_min == from_max: + raise ValueError("from_min == from_max") + return (value - from_min) / (from_max - from_min) * ( + to_max - to_min) + to_min NUM_SAMPLES = 2 ** 12 RANGE = (-0.19, 3.0) data = [] for i in xrange(NUM_SAMPLES): x = i / (NUM_SAMPLES - 1.0) - x = Fit(x, 0.0, 1.0, RANGE[0], RANGE[1]) + x = fit(x, 0.0, 1.0, RANGE[0], RANGE[1]) data.append(cid_to_rle(x)) lut = 'ADX_CID_to_RLE.spi1d' - genlut.writeSPI1D(lutDir + "/" + lut, RANGE[0], RANGE[1], data, - NUM_SAMPLES, 1) + write_SPI_1d(lut_directory + "/" + lut, RANGE[0], RANGE[1], + data, + NUM_SAMPLES, 1) return lut # Convert Channel Independent Density values to Relative Log Exposure # values. - lut = createCIDtoRLELUT() - cs.toReferenceTransforms.append({ + lut = create_CID_to_RLE_LUT() + cs.to_reference_transforms.append({ 'type': 'lutFile', 'path': lut, 'interpolation': 'linear', @@ -678,14 +685,14 @@ def generateLUTs(odtInfo, }) # Convert Relative Log Exposure values to Relative Exposure values - cs.toReferenceTransforms.append({ + cs.to_reference_transforms.append({ 'type': 'log', 'base': 10, 'direction': 'inverse' }) # Convert Relative Exposure values to ACES values - cs.toReferenceTransforms.append({ + cs.to_reference_transforms.append({ 'type': 'matrix', 'matrix': [0.72286, 0.12630, 0.15084, 0, 0.11923, 0.76418, 0.11659, 0, @@ -694,145 +701,143 @@ def generateLUTs(odtInfo, 'direction': 'forward' }) - cs.fromReferenceTransforms = [] + cs.from_reference_transforms = [] return cs - ADX10 = createADX(bitdepth=10) - configData['colorSpaces'].append(ADX10) + ADX10 = create_ADX(bit_depth=10) + config_data['colorSpaces'].append(ADX10) - ADX16 = createADX(bitdepth=16) - configData['colorSpaces'].append(ADX16) + ADX16 = create_ADX(bit_depth=16) + config_data['colorSpaces'].append(ADX16) # # Camera Input Transforms # # RED color spaces to ACES - redColorSpaces = red.createColorSpaces(lutDir, lutResolution1d) - for cs in redColorSpaces: - configData['colorSpaces'].append(cs) + red_colorspaces = red.create_colorspaces(lut_directory, lut_resolution_1d) + for cs in red_colorspaces: + config_data['colorSpaces'].append(cs) # Canon-Log to ACES - canonColorSpaces = canon.createColorSpaces(lutDir, lutResolution1d) - for cs in canonColorSpaces: - configData['colorSpaces'].append(cs) - - # SLog to ACES - sonyColorSpaces = sony.createColorSpaces(lutDir, lutResolution1d) - for cs in sonyColorSpaces: - configData['colorSpaces'].append(cs) - - # LogC to ACES - arriColorSpaces = arri.createColorSpaces(lutDir, lutResolution1d) - for cs in arriColorSpaces: - configData['colorSpaces'].append(cs) + canon_colorspaces = canon.create_colorspaces(lut_directory, + lut_resolution_1d) + for cs in canon_colorspaces: + config_data['colorSpaces'].append(cs) + + # S-Log to ACES + sony_colorSpaces = sony.create_colorspaces(lut_directory, + lut_resolution_1d) + for cs in sony_colorSpaces: + config_data['colorSpaces'].append(cs) + + # Log-C to ACES + arri_colorSpaces = arri.create_colorspaces(lut_directory, + lut_resolution_1d) + for cs in arri_colorSpaces: + config_data['colorSpaces'].append(cs) # # Generic log transform # - def createGenericLog(name='log', - minValue=0.0, - maxValue=1.0, - inputScale=1.0, - middleGrey=0.18, - minExposure=-6.0, - maxExposure=6.5, - lutResolution1d=lutResolution1d): + def create_generic_log(name='log', + min_value=0.0, + max_value=1.0, + input_scale=1.0, + middle_grey=0.18, + min_exposure=-6.0, + max_exposure=6.5, + lut_resolution_1d=lut_resolution_1d): cs = ColorSpace(name) cs.description = "The %s color space" % name - cs.equalityGroup = name + cs.equality_group = name cs.family = 'Utility' - cs.isData = False + cs.is_data = False ctls = [ '%s/utilities/ACESlib.OCIO_shaper_log2_to_lin_param.a1.0.0.ctl' % ( - acesCTLReleaseDir) - ] + aces_CTL_directory)] lut = "%s_to_aces.spi1d" % name # Remove spaces and parentheses lut = lut.replace(' ', '_').replace(')', '_').replace('(', '_') - genlut.generate1dLUTFromCTL( - lutDir + "/" + lut, + generate_1d_LUT_from_CTL( + lut_directory + "/" + lut, ctls, - lutResolution1d, + lut_resolution_1d, 'float', - inputScale, + input_scale, 1.0, { - 'middleGrey': middleGrey, - 'minExposure': minExposure, - 'maxExposure': maxExposure + 'middleGrey': middle_grey, + 'minExposure': min_exposure, + 'maxExposure': max_exposure }, cleanup, - acesCTLReleaseDir, - minValue, - maxValue) + aces_CTL_directory, + min_value, + max_value) - cs.toReferenceTransforms = [] - cs.toReferenceTransforms.append({ + cs.to_reference_transforms = [] + cs.to_reference_transforms.append({ 'type': 'lutFile', 'path': lut, 'interpolation': 'linear', 'direction': 'forward' }) - cs.fromReferenceTransforms = [] + cs.from_reference_transforms = [] return cs # # ACES LMTs # - def createACESLMT(lmtName, - lmtValues, - shaperInfo, - lutResolution1d=1024, - lutResolution3d=64, - cleanup=True): - cs = ColorSpace("%s" % lmtName) - cs.description = "The ACES Look Transform: %s" % lmtName - cs.equalityGroup = '' + def create_ACES_LMT(lmt_name, + lmt_values, + shaper_info, + lut_resolution_1d=1024, + lut_resolution_3d=64, + cleanup=True): + cs = ColorSpace("%s" % lmt_name) + cs.description = "The ACES Look Transform: %s" % lmt_name + cs.equality_group = '' cs.family = 'Look' - cs.isData = False + cs.is_data = False - import pprint - - pprint.pprint(lmtValues) + pprint.pprint(lmt_values) # # Generate the shaper transform # - (shaperName, - shaperToACESCTL, - shaperFromACESCTL, - shaperInputScale, - shaperParams) = shaperInfo - - shaperLut = "%s_to_aces.spi1d" % shaperName - if (not os.path.exists(lutDir + "/" + shaperLut)): - ctls = [ - shaperToACESCTL % acesCTLReleaseDir - ] + (shaper_name, + shaper_to_ACES_CTL, + shaper_from_ACES_CTL, + shaper_input_scale, + shaper_params) = shaper_info + + shaper_lut = "%s_to_aces.spi1d" % shaper_name + if (not os.path.exists(lut_directory + "/" + shaper_lut)): + ctls = [shaper_to_ACES_CTL % aces_CTL_directory] # Remove spaces and parentheses - shaperLut = shaperLut.replace( + shaper_lut = shaper_lut.replace( ' ', '_').replace(')', '_').replace('(', '_') - genlut.generate1dLUTFromCTL( - lutDir + "/" + shaperLut, + generate_1d_LUT_from_CTL( + lut_directory + "/" + shaper_lut, ctls, - lutResolution1d, + lut_resolution_1d, 'float', - 1.0 / shaperInputScale, + 1.0 / shaper_input_scale, 1.0, - shaperParams, + shaper_params, cleanup, - acesCTLReleaseDir) + aces_CTL_directory) - shaperOCIOTransform = { + shaper_OCIO_transform = { 'type': 'lutFile', - 'path': shaperLut, + 'path': shaper_lut, 'interpolation': 'linear', 'direction': 'inverse' } @@ -840,31 +845,31 @@ def generateLUTs(odtInfo, # # Generate the forward transform # - cs.fromReferenceTransforms = [] + cs.from_reference_transforms = [] - if 'transformCTL' in lmtValues: + if 'transformCTL' in lmt_values: ctls = [ - shaperToACESCTL % acesCTLReleaseDir, - '%s/%s' % (acesCTLReleaseDir, lmtValues['transformCTL']) + shaper_to_ACES_CTL % aces_CTL_directory, + '%s/%s' % (aces_CTL_directory, lmt_values['transformCTL']) ] - lut = "%s.%s.spi3d" % (shaperName, lmtName) + lut = "%s.%s.spi3d" % (shaper_name, lmt_name) # Remove spaces and parentheses lut = lut.replace(' ', '_').replace(')', '_').replace('(', '_') - genlut.generate3dLUTFromCTL( - lutDir + "/" + lut, + generate_3d_LUT_from_CTL( + lut_directory + "/" + lut, ctls, - lutResolution3d, + lut_resolution_3d, 'float', - 1.0 / shaperInputScale, + 1.0 / shaper_input_scale, 1.0, - shaperParams, + shaper_params, cleanup, - acesCTLReleaseDir) + aces_CTL_directory) - cs.fromReferenceTransforms.append(shaperOCIOTransform) - cs.fromReferenceTransforms.append({ + cs.from_reference_transforms.append(shaper_OCIO_transform) + cs.from_reference_transforms.append({ 'type': 'lutFile', 'path': lut, 'interpolation': 'tetrahedral', @@ -874,40 +879,40 @@ def generateLUTs(odtInfo, # # Generate the inverse transform # - cs.toReferenceTransforms = [] + cs.to_reference_transforms = [] - if 'transformCTLInverse' in lmtValues: + if 'transformCTLInverse' in lmt_values: ctls = [ '%s/%s' % ( - acesCTLReleaseDir, odtValues['transformCTLInverse']), - shaperFromACESCTL % acesCTLReleaseDir + aces_CTL_directory, odt_values['transformCTLInverse']), + shaper_from_ACES_CTL % aces_CTL_directory ] - lut = "Inverse.%s.%s.spi3d" % (odtName, shaperName) + lut = "Inverse.%s.%s.spi3d" % (odt_name, shaper_name) # Remove spaces and parentheses lut = lut.replace(' ', '_').replace(')', '_').replace('(', '_') - genlut.generate3dLUTFromCTL( - lutDir + "/" + lut, + generate_3d_LUT_from_CTL( + lut_directory + "/" + lut, ctls, - lutResolution3d, + lut_resolution_3d, 'half', 1.0, - shaperInputScale, - shaperParams, + shaper_input_scale, + shaper_params, cleanup, - acesCTLReleaseDir) + aces_CTL_directory) - cs.toReferenceTransforms.append({ + cs.to_reference_transforms.append({ 'type': 'lutFile', 'path': lut, 'interpolation': 'tetrahedral', 'direction': 'forward' }) - shaperInverse = shaperOCIOTransform.copy() - shaperInverse['direction'] = 'forward' - cs.toReferenceTransforms.append(shaperInverse) + shaper_inverse = shaper_OCIO_transform.copy() + shaper_inverse['direction'] = 'forward' + cs.to_reference_transforms.append(shaper_inverse) return cs @@ -915,106 +920,102 @@ def generateLUTs(odtInfo, # LMT Shaper # - lmtLutResolution1d = max(4096, lutResolution1d) - lmtLutResolution3d = max(65, lutResolution3d) + lmt_lut_resolution_1d = max(4096, lut_resolution_1d) + lmt_lut_resolution_3d = max(65, lut_resolution_3d) # Log 2 shaper - lmtShaperName = 'LMT Shaper' - lmtParams = { + lmt_shaper_name = 'LMT Shaper' + lmt_params = { 'middleGrey': 0.18, 'minExposure': -10.0, 'maxExposure': 6.5 } - lmtShaper = createGenericLog(name=lmtShaperName, - middleGrey=lmtParams['middleGrey'], - minExposure=lmtParams['minExposure'], - maxExposure=lmtParams['maxExposure'], - lutResolution1d=lmtLutResolution1d) - configData['colorSpaces'].append(lmtShaper) + lmt_shaper = create_generic_log(name=lmt_shaper_name, + middle_grey=lmt_params['middleGrey'], + min_exposure=lmt_params['minExposure'], + max_exposure=lmt_params['maxExposure'], + lut_resolution_1d=lmt_lut_resolution_1d) + config_data['colorSpaces'].append(lmt_shaper) - shaperInputScale_genericLog2 = 1.0 + shaper_input_scale_generic_log2 = 1.0 # Log 2 shaper name and CTL transforms bundled up - lmtShaperData = [ - lmtShaperName, + lmt_shaper_data = [ + lmt_shaper_name, '%s/utilities/ACESlib.OCIO_shaper_log2_to_lin_param.a1.0.0.ctl', '%s/utilities/ACESlib.OCIO_shaper_lin_to_log2_param.a1.0.0.ctl', - shaperInputScale_genericLog2, - lmtParams + shaper_input_scale_generic_log2, + lmt_params ] - sortedLMTs = sorted(lmtInfo.iteritems(), key=lambda x: x[1]) - print(sortedLMTs) - for lmt in sortedLMTs: - (lmtName, lmtValues) = lmt - cs = createACESLMT( - lmtValues['transformUserName'], - lmtValues, - lmtShaperData, - lmtLutResolution1d, - lmtLutResolution3d, + sorted_LMTs = sorted(lmt_info.iteritems(), key=lambda x: x[1]) + print(sorted_LMTs) + for lmt in sorted_LMTs: + (lmt_name, lmt_values) = lmt + cs = create_ACES_LMT( + lmt_values['transformUserName'], + lmt_values, + lmt_shaper_data, + lmt_lut_resolution_1d, + lmt_lut_resolution_3d, cleanup) - configData['colorSpaces'].append(cs) + config_data['colorSpaces'].append(cs) # # ACES RRT with the supplied ODT # - def createACESRRTplusODT(odtName, - odtValues, - shaperInfo, - lutResolution1d=1024, - lutResolution3d=64, - cleanup=True): - cs = ColorSpace("%s" % odtName) + def create_ACES_RRT_plus_ODT(odt_name, + odt_values, + shaper_info, + lut_resolution_1d=1024, + lut_resolution_3d=64, + cleanup=True): + cs = ColorSpace("%s" % odt_name) cs.description = "%s - %s Output Transform" % ( - odtValues['transformUserNamePrefix'], odtName) - cs.equalityGroup = '' + odt_values['transformUserNamePrefix'], odt_name) + cs.equality_group = '' cs.family = 'Output' - cs.isData = False - - import pprint + cs.is_data = False - pprint.pprint(odtValues) + pprint.pprint(odt_values) # # Generate the shaper transform # # if 'shaperCTL' in odtValues: - (shaperName, - shaperToACESCTL, - shaperFromACESCTL, - shaperInputScale, - shaperParams) = shaperInfo - - if 'legalRange' in odtValues: - shaperParams['legalRange'] = odtValues['legalRange'] + (shaper_name, + shaper_to_ACES_CTL, + shaper_from_ACES_CTL, + shaper_input_scale, + shaper_params) = shaper_info + + if 'legalRange' in odt_values: + shaper_params['legalRange'] = odt_values['legalRange'] else: - shaperParams['legalRange'] = 0 + shaper_params['legalRange'] = 0 - shaperLut = "%s_to_aces.spi1d" % shaperName - if (not os.path.exists(lutDir + "/" + shaperLut)): - ctls = [ - shaperToACESCTL % acesCTLReleaseDir - ] + shaper_lut = "%s_to_aces.spi1d" % shaper_name + if (not os.path.exists(lut_directory + "/" + shaper_lut)): + ctls = [shaper_to_ACES_CTL % aces_CTL_directory] # Remove spaces and parentheses - shaperLut = shaperLut.replace( + shaper_lut = shaper_lut.replace( ' ', '_').replace(')', '_').replace('(', '_') - genlut.generate1dLUTFromCTL( - lutDir + "/" + shaperLut, + generate_1d_LUT_from_CTL( + lut_directory + "/" + shaper_lut, ctls, - lutResolution1d, + lut_resolution_1d, 'float', - 1.0 / shaperInputScale, + 1.0 / shaper_input_scale, 1.0, - shaperParams, + shaper_params, cleanup, - acesCTLReleaseDir) + aces_CTL_directory) - shaperOCIOTransform = { + shaper_OCIO_transform = { 'type': 'lutFile', - 'path': shaperLut, + 'path': shaper_lut, 'interpolation': 'linear', 'direction': 'inverse' } @@ -1022,47 +1023,48 @@ def generateLUTs(odtInfo, # # Generate the forward transform # - cs.fromReferenceTransforms = [] + cs.from_reference_transforms = [] - if 'transformLUT' in odtValues: + if 'transformLUT' in odt_values: # Copy into the lut dir - transformLUTFileName = os.path.basename(odtValues['transformLUT']) - lut = lutDir + "/" + transformLUTFileName - shutil.copy(odtValues['transformLUT'], lut) + transform_LUT_file_name = os.path.basename( + odt_values['transformLUT']) + lut = lut_directory + "/" + transform_LUT_file_name + shutil.copy(odt_values['transformLUT'], lut) - cs.fromReferenceTransforms.append(shaperOCIOTransform) - cs.fromReferenceTransforms.append({ + cs.from_reference_transforms.append(shaper_OCIO_transform) + cs.from_reference_transforms.append({ 'type': 'lutFile', - 'path': transformLUTFileName, + 'path': transform_LUT_file_name, 'interpolation': 'tetrahedral', 'direction': 'forward' }) - elif 'transformCTL' in odtValues: + elif 'transformCTL' in odt_values: # shaperLut ctls = [ - shaperToACESCTL % acesCTLReleaseDir, - '%s/rrt/RRT.a1.0.0.ctl' % acesCTLReleaseDir, - '%s/odt/%s' % (acesCTLReleaseDir, odtValues['transformCTL']) + shaper_to_ACES_CTL % aces_CTL_directory, + '%s/rrt/RRT.a1.0.0.ctl' % aces_CTL_directory, + '%s/odt/%s' % (aces_CTL_directory, odt_values['transformCTL']) ] - lut = "%s.RRT.a1.0.0.%s.spi3d" % (shaperName, odtName) + lut = "%s.RRT.a1.0.0.%s.spi3d" % (shaper_name, odt_name) # Remove spaces and parentheses lut = lut.replace(' ', '_').replace(')', '_').replace('(', '_') - genlut.generate3dLUTFromCTL(lutDir + "/" + lut, - # shaperLUT, - ctls, - lutResolution3d, - 'float', - 1.0 / shaperInputScale, - 1.0, - shaperParams, - cleanup, - acesCTLReleaseDir) - - cs.fromReferenceTransforms.append(shaperOCIOTransform) - cs.fromReferenceTransforms.append({ + generate_3d_LUT_from_CTL(lut_directory + "/" + lut, + # shaperLUT, + ctls, + lut_resolution_3d, + 'float', + 1.0 / shaper_input_scale, + 1.0, + shaper_params, + cleanup, + aces_CTL_directory) + + cs.from_reference_transforms.append(shaper_OCIO_transform) + cs.from_reference_transforms.append({ 'type': 'lutFile', 'path': lut, 'interpolation': 'tetrahedral', @@ -1072,259 +1074,266 @@ def generateLUTs(odtInfo, # # Generate the inverse transform # - cs.toReferenceTransforms = [] + cs.to_reference_transforms = [] - if 'transformLUTInverse' in odtValues: + if 'transformLUTInverse' in odt_values: # Copy into the lut dir - transformLUTInverseFileName = os.path.basename( - odtValues['transformLUTInverse']) - lut = lutDir + "/" + transformLUTInverseFileName - shutil.copy(odtValues['transformLUTInverse'], lut) + transform_LUT_inverse_file_name = os.path.basename( + odt_values['transformLUTInverse']) + lut = lut_directory + "/" + transform_LUT_inverse_file_name + shutil.copy(odt_values['transformLUTInverse'], lut) - cs.toReferenceTransforms.append({ + cs.to_reference_transforms.append({ 'type': 'lutFile', - 'path': transformLUTInverseFileName, + 'path': transform_LUT_inverse_file_name, 'interpolation': 'tetrahedral', 'direction': 'forward' }) - shaperInverse = shaperOCIOTransform.copy() - shaperInverse['direction'] = 'forward' - cs.toReferenceTransforms.append(shaperInverse) - elif 'transformCTLInverse' in odtValues: + shaper_inverse = shaper_OCIO_transform.copy() + shaper_inverse['direction'] = 'forward' + cs.to_reference_transforms.append(shaper_inverse) + elif 'transformCTLInverse' in odt_values: ctls = [ '%s/odt/%s' % ( - acesCTLReleaseDir, odtValues['transformCTLInverse']), - '%s/rrt/InvRRT.a1.0.0.ctl' % acesCTLReleaseDir, - shaperFromACESCTL % acesCTLReleaseDir + aces_CTL_directory, odt_values['transformCTLInverse']), + '%s/rrt/InvRRT.a1.0.0.ctl' % aces_CTL_directory, + shaper_from_ACES_CTL % aces_CTL_directory ] - lut = "InvRRT.a1.0.0.%s.%s.spi3d" % (odtName, shaperName) + lut = "InvRRT.a1.0.0.%s.%s.spi3d" % (odt_name, shaper_name) # Remove spaces and parentheses lut = lut.replace(' ', '_').replace(')', '_').replace('(', '_') - genlut.generate3dLUTFromCTL( - lutDir + "/" + lut, + generate_3d_LUT_from_CTL( + lut_directory + "/" + lut, # None, ctls, - lutResolution3d, + lut_resolution_3d, 'half', 1.0, - shaperInputScale, - shaperParams, + shaper_input_scale, + shaper_params, cleanup, - acesCTLReleaseDir) + aces_CTL_directory) - cs.toReferenceTransforms.append({ + cs.to_reference_transforms.append({ 'type': 'lutFile', 'path': lut, 'interpolation': 'tetrahedral', 'direction': 'forward' }) - shaperInverse = shaperOCIOTransform.copy() - shaperInverse['direction'] = 'forward' - cs.toReferenceTransforms.append(shaperInverse) + shaper_inverse = shaper_OCIO_transform.copy() + shaper_inverse['direction'] = 'forward' + cs.to_reference_transforms.append(shaper_inverse) return cs # # RRT/ODT shaper options # - shaperData = {} + shaper_data = {} # Log 2 shaper - log2ShaperName = shaperName - log2Params = { + log2_shaper_name = shaper_name + log2_params = { 'middleGrey': 0.18, 'minExposure': -6.0, 'maxExposure': 6.5 } - log2Shaper = createGenericLog(name=log2ShaperName, - middleGrey=log2Params['middleGrey'], - minExposure=log2Params['minExposure'], - maxExposure=log2Params['maxExposure']) - configData['colorSpaces'].append(log2Shaper) + log2_shaper = create_generic_log( + name=log2_shaper_name, + middle_grey=log2_params['middleGrey'], + min_exposure=log2_params['minExposure'], + max_exposure=log2_params['maxExposure']) + config_data['colorSpaces'].append(log2_shaper) - shaperInputScale_genericLog2 = 1.0 + shaper_input_scale_generic_log2 = 1.0 # Log 2 shaper name and CTL transforms bundled up - log2ShaperData = [ - log2ShaperName, + log2_shaper_data = [ + log2_shaper_name, '%s/utilities/ACESlib.OCIO_shaper_log2_to_lin_param.a1.0.0.ctl', '%s/utilities/ACESlib.OCIO_shaper_lin_to_log2_param.a1.0.0.ctl', - shaperInputScale_genericLog2, - log2Params + shaper_input_scale_generic_log2, + log2_params ] - shaperData[log2ShaperName] = log2ShaperData + shaper_data[log2_shaper_name] = log2_shaper_data # # Shaper that also includes the AP1 primaries # - Needed for some LUT baking steps # - log2ShaperAP1 = createGenericLog(name=log2ShaperName, - middleGrey=log2Params['middleGrey'], - minExposure=log2Params['minExposure'], - maxExposure=log2Params['maxExposure']) - log2ShaperAP1.name = "%s - AP1" % log2ShaperAP1.name + log2_shaper_AP1 = create_generic_log( + name=log2_shaper_name, + middle_grey=log2_params['middleGrey'], + min_exposure=log2_params['minExposure'], + max_exposure=log2_params['maxExposure']) + log2_shaper_AP1.name = "%s - AP1" % log2_shaper_AP1.name # AP1 primaries to AP0 primaries - log2ShaperAP1.toReferenceTransforms.append({ + log2_shaper_AP1.to_reference_transforms.append({ 'type': 'matrix', - 'matrix': mat44FromMat33(acesAP1toAP0), + 'matrix': mat44_from_mat33(ACES_AP1_to_AP0), 'direction': 'forward' }) - configData['colorSpaces'].append(log2ShaperAP1) + config_data['colorSpaces'].append(log2_shaper_AP1) # # Choose your shaper # - rrtShaperName = log2ShaperName - rrtShaper = log2ShaperData + rrt_shaper_name = log2_shaper_name + rrt_shaper = log2_shaper_data # # RRT + ODT Combinations # - sortedOdts = sorted(odtInfo.iteritems(), key=lambda x: x[1]) - print(sortedOdts) - for odt in sortedOdts: - (odtName, odtValues) = odt + sorted_odts = sorted(odt_info.iteritems(), key=lambda x: x[1]) + print(sorted_odts) + for odt in sorted_odts: + (odt_name, odt_values) = odt # Have to handle ODTs that can generate either legal or full output - if odtName in ['Academy.Rec2020_100nits_dim.a1.0.0', - 'Academy.Rec709_100nits_dim.a1.0.0', - 'Academy.Rec709_D60sim_100nits_dim.a1.0.0']: - odtNameLegal = '%s - Legal' % odtValues['transformUserName'] + if odt_name in ['Academy.Rec2020_100nits_dim.a1.0.0', + 'Academy.Rec709_100nits_dim.a1.0.0', + 'Academy.Rec709_D60sim_100nits_dim.a1.0.0']: + odt_name_legal = '%s - Legal' % odt_values['transformUserName'] else: - odtNameLegal = odtValues['transformUserName'] + odt_name_legal = odt_values['transformUserName'] - odtLegal = odtValues.copy() - odtLegal['legalRange'] = 1 + odt_legal = odt_values.copy() + odt_legal['legalRange'] = 1 - cs = createACESRRTplusODT( - odtNameLegal, - odtLegal, - rrtShaper, - lutResolution1d, - lutResolution3d, + cs = create_ACES_RRT_plus_ODT( + odt_name_legal, + odt_legal, + rrt_shaper, + lut_resolution_1d, + lut_resolution_3d, cleanup) - configData['colorSpaces'].append(cs) + config_data['colorSpaces'].append(cs) # Create a display entry using this color space - configData['displays'][odtNameLegal] = { + config_data['displays'][odt_name_legal] = { 'Linear': ACES, 'Log': ACEScc, 'Output Transform': cs} - if odtName in ['Academy.Rec2020_100nits_dim.a1.0.0', - 'Academy.Rec709_100nits_dim.a1.0.0', - 'Academy.Rec709_D60sim_100nits_dim.a1.0.0']: - print("Generating full range ODT for %s" % odtName) - - odtNameFull = "%s - Full" % odtValues['transformUserName'] - odtFull = odtValues.copy() - odtFull['legalRange'] = 0 - - csFull = createACESRRTplusODT( - odtNameFull, - odtFull, - rrtShaper, - lutResolution1d, - lutResolution3d, + if odt_name in ['Academy.Rec2020_100nits_dim.a1.0.0', + 'Academy.Rec709_100nits_dim.a1.0.0', + 'Academy.Rec709_D60sim_100nits_dim.a1.0.0']: + print("Generating full range ODT for %s" % odt_name) + + odt_name_full = "%s - Full" % odt_values['transformUserName'] + odt_full = odt_values.copy() + odt_full['legalRange'] = 0 + + cs_full = create_ACES_RRT_plus_ODT( + odt_name_full, + odt_full, + rrt_shaper, + lut_resolution_1d, + lut_resolution_3d, cleanup) - configData['colorSpaces'].append(csFull) + config_data['colorSpaces'].append(cs_full) # Create a display entry using this color space - configData['displays'][odtNameFull] = { + config_data['displays'][odt_name_full] = { 'Linear': ACES, 'Log': ACEScc, - 'Output Transform': csFull} + 'Output Transform': cs_full} # # Generic Matrix transform # - def createGenericMatrix(name='matrix', - fromReferenceValues=[], - toReferenceValues=[]): + def create_generic_matrix(name='matrix', + from_reference_values=[], + to_reference_values=[]): cs = ColorSpace(name) cs.description = "The %s color space" % name - cs.equalityGroup = name + cs.equality_group = name cs.family = 'Utility' - cs.isData = False + cs.is_data = False - cs.toReferenceTransforms = [] - if toReferenceValues != []: - for matrix in toReferenceValues: - cs.toReferenceTransforms.append({ + cs.to_reference_transforms = [] + if to_reference_values != []: + for matrix in to_reference_values: + cs.to_reference_transforms.append({ 'type': 'matrix', - 'matrix': mat44FromMat33(matrix), + 'matrix': mat44_from_mat33(matrix), 'direction': 'forward' }) - cs.fromReferenceTransforms = [] - if fromReferenceValues != []: - for matrix in fromReferenceValues: - cs.fromReferenceTransforms.append({ + cs.from_reference_transforms = [] + if from_reference_values != []: + for matrix in from_reference_values: + cs.from_reference_transforms.append({ 'type': 'matrix', - 'matrix': mat44FromMat33(matrix), + 'matrix': mat44_from_mat33(matrix), 'direction': 'forward' }) return cs - cs = createGenericMatrix('XYZ', fromReferenceValues=[acesAP0toXYZ]) - configData['colorSpaces'].append(cs) + cs = create_generic_matrix('XYZ', from_reference_values=[ACES_AP0_to_XYZ]) + config_data['colorSpaces'].append(cs) - cs = createGenericMatrix('Linear - AP1', toReferenceValues=[acesAP1toAP0]) - configData['colorSpaces'].append(cs) + cs = create_generic_matrix( + 'Linear - AP1', to_reference_values=[ACES_AP1_to_AP0]) + config_data['colorSpaces'].append(cs) # ACES to Linear, P3D60 primaries - xyzToP3D60 = [2.4027414142, -0.8974841639, -0.3880533700, - -0.8325796487, 1.7692317536, 0.0237127115, - 0.0388233815, -0.0824996856, 1.0363685997] + XYZ_to_P3D60 = [2.4027414142, -0.8974841639, -0.3880533700, + -0.8325796487, 1.7692317536, 0.0237127115, + 0.0388233815, -0.0824996856, 1.0363685997] - cs = createGenericMatrix('Linear - P3-D60', - fromReferenceValues=[acesAP0toXYZ, xyzToP3D60]) - configData['colorSpaces'].append(cs) + cs = create_generic_matrix( + 'Linear - P3-D60', + from_reference_values=[ACES_AP0_to_XYZ, XYZ_to_P3D60]) + config_data['colorSpaces'].append(cs) # ACES to Linear, P3D60 primaries - xyzToP3DCI = [2.7253940305, -1.0180030062, -0.4401631952, - -0.7951680258, 1.6897320548, 0.0226471906, - 0.0412418914, -0.0876390192, 1.1009293786] + XYZ_to_P3DCI = [2.7253940305, -1.0180030062, -0.4401631952, + -0.7951680258, 1.6897320548, 0.0226471906, + 0.0412418914, -0.0876390192, 1.1009293786] - cs = createGenericMatrix('Linear - P3-DCI', - fromReferenceValues=[acesAP0toXYZ, xyzToP3DCI]) - configData['colorSpaces'].append(cs) + cs = create_generic_matrix( + 'Linear - P3-DCI', + from_reference_values=[ACES_AP0_to_XYZ, XYZ_to_P3DCI]) + config_data['colorSpaces'].append(cs) # ACES to Linear, Rec 709 primaries - xyzToRec709 = [3.2409699419, -1.5373831776, -0.4986107603, - -0.9692436363, 1.8759675015, 0.0415550574, - 0.0556300797, -0.2039769589, 1.0569715142] + XYZ_to_Rec709 = [3.2409699419, -1.5373831776, -0.4986107603, + -0.9692436363, 1.8759675015, 0.0415550574, + 0.0556300797, -0.2039769589, 1.0569715142] - cs = createGenericMatrix('Linear - Rec.709', - fromReferenceValues=[acesAP0toXYZ, xyzToRec709]) - configData['colorSpaces'].append(cs) + cs = create_generic_matrix( + 'Linear - Rec.709', + from_reference_values=[ACES_AP0_to_XYZ, XYZ_to_Rec709]) + config_data['colorSpaces'].append(cs) # ACES to Linear, Rec 2020 primaries - xyzToRec2020 = [1.7166511880, -0.3556707838, -0.2533662814, - -0.6666843518, 1.6164812366, 0.0157685458, - 0.0176398574, -0.0427706133, 0.9421031212] + XYZ_to_Rec2020 = [1.7166511880, -0.3556707838, -0.2533662814, + -0.6666843518, 1.6164812366, 0.0157685458, + 0.0176398574, -0.0427706133, 0.9421031212] - cs = createGenericMatrix('Linear - Rec.2020', - fromReferenceValues=[acesAP0toXYZ, xyzToRec2020]) - configData['colorSpaces'].append(cs) + cs = create_generic_matrix( + 'Linear - Rec.2020', + from_reference_values=[ACES_AP0_to_XYZ, XYZ_to_Rec2020]) + config_data['colorSpaces'].append(cs) print("generateLUTs - end") - return configData + return config_data -def generateBakedLUTs(odtInfo, - shaperName, - bakedDir, - configPath, - lutResolution1d, - lutResolution3d, - lutResolutionShaper=1024): +def generate_baked_LUTs(odt_info, + shaper_name, + baked_directory, + config_path, + lut_resolution_1d, + lut_resolution_3d, + lut_resolution_shaper=1024): """ Object description. @@ -1340,95 +1349,113 @@ def generateBakedLUTs(odtInfo, """ # Add the legal and full variations into this list - odtInfoC = dict(odtInfo) - for odtCTLName, odtValues in odtInfo.iteritems(): - if odtCTLName in ['Academy.Rec2020_100nits_dim.a1.0.0', - 'Academy.Rec709_100nits_dim.a1.0.0', - 'Academy.Rec709_D60sim_100nits_dim.a1.0.0']: - odtName = odtValues["transformUserName"] + odt_info_C = dict(odt_info) + for odt_CTL_name, odt_values in odt_info.iteritems(): + if odt_CTL_name in ['Academy.Rec2020_100nits_dim.a1.0.0', + 'Academy.Rec709_100nits_dim.a1.0.0', + 'Academy.Rec709_D60sim_100nits_dim.a1.0.0']: + odt_name = odt_values["transformUserName"] - odtValuesLegal = dict(odtValues) - odtValuesLegal["transformUserName"] = "%s - Legal" % odtName - odtInfoC["%s - Legal" % odtCTLName] = odtValuesLegal + odt_values_legal = dict(odt_values) + odt_values_legal["transformUserName"] = "%s - Legal" % odt_name + odt_info_C["%s - Legal" % odt_CTL_name] = odt_values_legal - odtValuesFull = dict(odtValues) - odtValuesFull["transformUserName"] = "%s - Full" % odtName - odtInfoC["%s - Full" % odtCTLName] = odtValuesFull + odt_values_full = dict(odt_values) + odt_values_full["transformUserName"] = "%s - Full" % odt_name + odt_info_C["%s - Full" % odt_CTL_name] = odt_values_full - del (odtInfoC[odtCTLName]) + del (odt_info_C[odt_CTL_name]) - for odtCTLName, odtValues in odtInfoC.iteritems(): - odtPrefix = odtValues["transformUserNamePrefix"] - odtName = odtValues["transformUserName"] + for odt_CTL_name, odt_values in odt_info_C.iteritems(): + odt_prefix = odt_values["transformUserNamePrefix"] + odt_name = odt_values["transformUserName"] # For Photoshop - for inputspace in ["ACEScc", "ACESproxy"]: - args = ["--iconfig", configPath, "-v", "--inputspace", inputspace] - args += ["--outputspace", "%s" % odtName] + for input_space in ["ACEScc", "ACESproxy"]: + args = ["--iconfig", config_path, + "-v", + "--inputspace", input_space] + args += ["--outputspace", "%s" % odt_name] args += ["--description", - "%s - %s for %s data" % (odtPrefix, odtName, inputspace)] - args += ["--shaperspace", shaperName, "--shapersize", - str(lutResolutionShaper)] - args += ["--cubesize", str(lutResolution3d)] - args += ["--format", "icc", "%s/photoshop/%s for %s.icc" % ( - bakedDir, odtName, inputspace)] - - bakeLUT = Process(description="bake a LUT", cmd="ociobakelut", - args=args) - bakeLUT.execute() - - # For Flame, Lustre - for inputspace in ["ACEScc", "ACESproxy"]: - args = ["--iconfig", configPath, "-v", "--inputspace", inputspace] - args += ["--outputspace", "%s" % odtName] + "%s - %s for %s data" % (odt_prefix, + odt_name, + input_space)] + args += ["--shaperspace", shaper_name, + "--shapersize", str(lut_resolution_shaper)] + args += ["--cubesize", str(lut_resolution_3d)] + args += ["--format", + "icc", + "%s/photoshop/%s for %s.icc" % (baked_directory, + odt_name, + input_space)] + + bake_LUT = Process(description="bake a LUT", + cmd="ociobakelut", + args=args) + bake_LUT.execute() + + # For Flame, Lustre + for input_space in ["ACEScc", "ACESproxy"]: + args = ["--iconfig", config_path, + "-v", + "--inputspace", input_space] + args += ["--outputspace", "%s" % odt_name] args += ["--description", - "%s - %s for %s data" % (odtPrefix, odtName, inputspace)] - args += ["--shaperspace", shaperName, "--shapersize", - str(lutResolutionShaper)] - args += ["--cubesize", str(lutResolution3d)] + "%s - %s for %s data" % ( + odt_prefix, odt_name, input_space)] + args += ["--shaperspace", shaper_name, + "--shapersize", str(lut_resolution_shaper)] + args += ["--cubesize", str(lut_resolution_3d)] fargs = ["--format", "flame", "%s/flame/%s for %s Flame.3dl" % ( - bakedDir, odtName, inputspace)] - bakeLUT = Process(description="bake a LUT", cmd="ociobakelut", - args=(args + fargs)) - bakeLUT.execute() + baked_directory, odt_name, input_space)] + bake_LUT = Process(description="bake a LUT", + cmd="ociobakelut", + args=(args + fargs)) + bake_LUT.execute() largs = ["--format", "lustre", "%s/lustre/%s for %s Lustre.3dl" % ( - bakedDir, odtName, inputspace)] - bakeLUT = Process(description="bake a LUT", cmd="ociobakelut", - args=(args + largs)) - bakeLUT.execute() + baked_directory, odt_name, input_space)] + bake_LUT = Process(description="bake a LUT", + cmd="ociobakelut", + args=(args + largs)) + bake_LUT.execute() # For Maya, Houdini - for inputspace in ["ACEScg", "ACES2065-1"]: - args = ["--iconfig", configPath, "-v", "--inputspace", inputspace] - args += ["--outputspace", "%s" % odtName] + for input_space in ["ACEScg", "ACES2065-1"]: + args = ["--iconfig", config_path, + "-v", + "--inputspace", input_space] + args += ["--outputspace", "%s" % odt_name] args += ["--description", - "%s - %s for %s data" % (odtPrefix, odtName, inputspace)] - if inputspace == 'ACEScg': - linShaperName = "%s - AP1" % shaperName + "%s - %s for %s data" % ( + odt_prefix, odt_name, input_space)] + if input_space == 'ACEScg': + lin_shaper_name = "%s - AP1" % shaper_name else: - linShaperName = shaperName - args += ["--shaperspace", linShaperName, "--shapersize", - str(lutResolutionShaper)] + lin_shaper_name = shaper_name + args += ["--shaperspace", lin_shaper_name, + "--shapersize", str(lut_resolution_shaper)] - args += ["--cubesize", str(lutResolution3d)] + args += ["--cubesize", str(lut_resolution_3d)] margs = ["--format", "cinespace", "%s/maya/%s for %s Maya.csp" % ( - bakedDir, odtName, inputspace)] - bakeLUT = Process(description="bake a LUT", cmd="ociobakelut", - args=(args + margs)) - bakeLUT.execute() + baked_directory, odt_name, input_space)] + bake_LUT = Process(description="bake a LUT", + cmd="ociobakelut", + args=(args + margs)) + bake_LUT.execute() hargs = ["--format", "houdini", "%s/houdini/%s for %s Houdini.lut" % ( - bakedDir, odtName, inputspace)] - bakeLUT = Process(description="bake a LUT", cmd="ociobakelut", - args=(args + hargs)) - bakeLUT.execute() + baked_directory, odt_name, input_space)] + bake_LUT = Process(description="bake a LUT", + cmd="ociobakelut", + args=(args + hargs)) + bake_LUT.execute() -def createConfigDir(configDir, bakeSecondaryLUTs): +def create_config_dir(config_directory, bake_secondary_LUTs): """ Object description. @@ -1443,21 +1470,20 @@ def createConfigDir(configDir, bakeSecondaryLUTs): Return value description. """ - dirs = [configDir, "%s/luts" % configDir] - if bakeSecondaryLUTs: - dirs.extend(["%s/baked" % configDir, - "%s/baked/flame" % configDir, - "%s/baked/photoshop" % configDir, - "%s/baked/houdini" % configDir, - "%s/baked/lustre" % configDir, - "%s/baked/maya" % configDir]) + dirs = [config_directory, "%s/luts" % config_directory] + if bake_secondary_LUTs: + dirs.extend(["%s/baked" % config_directory, + "%s/baked/flame" % config_directory, + "%s/baked/photoshop" % config_directory, + "%s/baked/houdini" % config_directory, + "%s/baked/lustre" % config_directory, + "%s/baked/maya" % config_directory]) for d in dirs: - if not os.path.exists(d): - os.mkdir(d) + not os.path.exists(d) and os.mkdir(d) -def getTransformInfo(ctlTransform): +def get_transform_info(ctl_transform): """ Object description. @@ -1472,27 +1498,26 @@ def getTransformInfo(ctlTransform): Return value description. """ - fp = open(ctlTransform, 'rb') + # TODO: Use *with* statement. + fp = open(ctl_transform, 'rb') # Read lines lines = fp.readlines() # Grab transform ID and User Name - transformID = lines[1][3:].split('<')[1].split('>')[1].lstrip().rstrip() + transform_ID = lines[1][3:].split('<')[1].split('>')[1].strip() # print(transformID) - transformUserName = '-'.join( - lines[2][3:].split('<')[1].split('>')[1].split('-')[ - 1:]).lstrip().rstrip() - transformUserNamePrefix = \ - lines[2][3:].split('<')[1].split('>')[1].split('-')[ - 0].lstrip().rstrip() + transform_user_name = '-'.join( + lines[2][3:].split('<')[1].split('>')[1].split('-')[1:]).strip() + transform_user_name_prefix = ( + lines[2][3:].split('<')[1].split('>')[1].split('-')[0].strip()) # print(transformUserName) fp.close() - return transformID, transformUserName, transformUserNamePrefix + return transform_ID, transform_user_name, transform_user_name_prefix -def getODTInfo(acesCTLReleaseDir): +def get_ODT_info(aces_CTL_directory): """ Object description. @@ -1509,70 +1534,72 @@ def getODTInfo(acesCTLReleaseDir): Return value description. """ + # TODO: Investigate usage of *files_walker* definition here. # Credit to Alex Fry for the original approach here - odtDir = os.path.join(acesCTLReleaseDir, "odt") - allodt = [] - for dirName, subdirList, fileList in os.walk(odtDir): - for fname in fileList: - allodt.append((os.path.join(dirName, fname))) + odt_dir = os.path.join(aces_CTL_directory, "odt") + all_odt = [] + for dir_name, subdir_list, file_list in os.walk(odt_dir): + for fname in file_list: + all_odt.append((os.path.join(dir_name, fname))) - odtCTLs = [x for x in allodt if - ("InvODT" not in x) and (os.path.split(x)[-1][0] != '.')] + odt_CTLs = [x for x in all_odt if + ("InvODT" not in x) and (os.path.split(x)[-1][0] != '.')] # print odtCTLs odts = {} - for odtCTL in odtCTLs: - odtTokens = os.path.split(odtCTL) + for odt_CTL in odt_CTLs: + odt_tokens = os.path.split(odt_CTL) # print(odtTokens) # Handle nested directories - odtPathTokens = os.path.split(odtTokens[-2]) - odtDir = odtPathTokens[-1] - while odtPathTokens[-2][-3:] != 'odt': - odtPathTokens = os.path.split(odtPathTokens[-2]) - odtDir = os.path.join(odtPathTokens[-1], odtDir) + odt_path_tokens = os.path.split(odt_tokens[-2]) + odt_dir = odt_path_tokens[-1] + while odt_path_tokens[-2][-3:] != 'odt': + odt_path_tokens = os.path.split(odt_path_tokens[-2]) + odt_dir = os.path.join(odt_path_tokens[-1], odt_dir) # Build full name # print("odtDir : %s" % odtDir) - transformCTL = odtTokens[-1] + transform_CTL = odt_tokens[-1] # print(transformCTL) - odtName = string.join(transformCTL.split('.')[1:-1], '.') + odt_name = string.join(transform_CTL.split('.')[1:-1], '.') # print(odtName) # Find id, user name and user name prefix - (transformID, transformUserName, - transformUserNamePrefix) = getTransformInfo( - "%s/odt/%s/%s" % (acesCTLReleaseDir, odtDir, transformCTL)) + (transform_ID, + transform_user_name, + transform_user_name_prefix) = get_transform_info( + "%s/odt/%s/%s" % (aces_CTL_directory, odt_dir, transform_CTL)) # Find inverse - transformCTLInverse = "InvODT.%s.ctl" % odtName + transform_CTL_inverse = "InvODT.%s.ctl" % odt_name if not os.path.exists( - os.path.join(odtTokens[-2], transformCTLInverse)): - transformCTLInverse = None + os.path.join(odt_tokens[-2], transform_CTL_inverse)): + transform_CTL_inverse = None # print(transformCTLInverse) # Add to list of ODTs - odts[odtName] = {} - odts[odtName]['transformCTL'] = os.path.join(odtDir, transformCTL) - if transformCTLInverse != None: - odts[odtName]['transformCTLInverse'] = os.path.join( - odtDir, transformCTLInverse) - - odts[odtName]['transformID'] = transformID - odts[odtName]['transformUserNamePrefix'] = transformUserNamePrefix - odts[odtName]['transformUserName'] = transformUserName - - print("ODT : %s" % odtName) - print("\tTransform ID : %s" % transformID) - print("\tTransform User Name Prefix : %s" % transformUserNamePrefix) - print("\tTransform User Name : %s" % transformUserName) + odts[odt_name] = {} + odts[odt_name]['transformCTL'] = os.path.join(odt_dir, transform_CTL) + if transform_CTL_inverse != None: + odts[odt_name]['transformCTLInverse'] = os.path.join( + odt_dir, transform_CTL_inverse) + + odts[odt_name]['transformID'] = transform_ID + odts[odt_name]['transformUserNamePrefix'] = transform_user_name_prefix + odts[odt_name]['transformUserName'] = transform_user_name + + print("ODT : %s" % odt_name) + print("\tTransform ID : %s" % transform_ID) + print("\tTransform User Name Prefix : %s" % transform_user_name_prefix) + print("\tTransform User Name : %s" % transform_user_name) print("\tForward ctl : %s" % ( - odts[odtName]['transformCTL'])) - if 'transformCTLInverse' in odts[odtName]: + odts[odt_name]['transformCTL'])) + if 'transformCTLInverse' in odts[odt_name]: print("\tInverse ctl : %s" % ( - odts[odtName]['transformCTLInverse'])) + odts[odt_name]['transformCTLInverse'])) else: print("\tInverse ctl : %s" % "None") @@ -1581,7 +1608,7 @@ def getODTInfo(acesCTLReleaseDir): return odts -def getLMTInfo(acesCTLReleaseDir): +def get_LMT_info(aces_CTL_directory): """ Object description. @@ -1598,71 +1625,74 @@ def getLMTInfo(acesCTLReleaseDir): Return value description. """ + # TODO: Investigate refactoring with previous definition. + # Credit to Alex Fry for the original approach here - lmtDir = os.path.join(acesCTLReleaseDir, "lmt") - alllmt = [] - for dirName, subdirList, fileList in os.walk(lmtDir): - for fname in fileList: - alllmt.append((os.path.join(dirName, fname))) + lmt_dir = os.path.join(aces_CTL_directory, "lmt") + all_lmt = [] + for dir_name, subdir_list, file_list in os.walk(lmt_dir): + for fname in file_list: + all_lmt.append((os.path.join(dir_name, fname))) - lmtCTLs = [x for x in alllmt if - ("InvLMT" not in x) and ("README" not in x) and ( - os.path.split(x)[-1][0] != '.')] + lmt_CTLs = [x for x in all_lmt if + ("InvLMT" not in x) and ("README" not in x) and ( + os.path.split(x)[-1][0] != '.')] # print lmtCTLs lmts = {} - for lmtCTL in lmtCTLs: - lmtTokens = os.path.split(lmtCTL) + for lmt_CTL in lmt_CTLs: + lmt_tokens = os.path.split(lmt_CTL) # print(lmtTokens) # Handle nested directories - lmtPathTokens = os.path.split(lmtTokens[-2]) - lmtDir = lmtPathTokens[-1] - while lmtPathTokens[-2][-3:] != 'ctl': - lmtPathTokens = os.path.split(lmtPathTokens[-2]) - lmtDir = os.path.join(lmtPathTokens[-1], lmtDir) + lmt_path_tokens = os.path.split(lmt_tokens[-2]) + lmt_dir = lmt_path_tokens[-1] + while lmt_path_tokens[-2][-3:] != 'ctl': + lmt_path_tokens = os.path.split(lmt_path_tokens[-2]) + lmt_dir = os.path.join(lmt_path_tokens[-1], lmt_dir) # Build full name # print("lmtDir : %s" % lmtDir) - transformCTL = lmtTokens[-1] + transform_CTL = lmt_tokens[-1] # print(transformCTL) - lmtName = string.join(transformCTL.split('.')[1:-1], '.') + lmt_name = string.join(transform_CTL.split('.')[1:-1], '.') # print(lmtName) # Find id, user name and user name prefix - (transformID, transformUserName, - transformUserNamePrefix) = getTransformInfo( - "%s/%s/%s" % (acesCTLReleaseDir, lmtDir, transformCTL)) + (transform_ID, + transform_user_name, + transform_user_name_prefix) = get_transform_info( + "%s/%s/%s" % (aces_CTL_directory, lmt_dir, transform_CTL)) # Find inverse - transformCTLInverse = "InvLMT.%s.ctl" % lmtName + transform_CTL_inverse = "InvLMT.%s.ctl" % lmt_name if not os.path.exists( - os.path.join(lmtTokens[-2], transformCTLInverse)): - transformCTLInverse = None + os.path.join(lmt_tokens[-2], transform_CTL_inverse)): + transform_CTL_inverse = None # print(transformCTLInverse) # Add to list of LMTs - lmts[lmtName] = {} - lmts[lmtName]['transformCTL'] = os.path.join(lmtDir, transformCTL) - if transformCTLInverse != None: - # TODO: Check unresolved *odtName* referemce. - lmts[odtName]['transformCTLInverse'] = os.path.join( - lmtDir, transformCTLInverse) - - lmts[lmtName]['transformID'] = transformID - lmts[lmtName]['transformUserNamePrefix'] = transformUserNamePrefix - lmts[lmtName]['transformUserName'] = transformUserName - - print("LMT : %s" % lmtName) - print("\tTransform ID : %s" % transformID) - print("\tTransform User Name Prefix : %s" % transformUserNamePrefix) - print("\tTransform User Name : %s" % transformUserName) - print("\t Forward ctl : %s" % lmts[lmtName]['transformCTL']) - if 'transformCTLInverse' in lmts[lmtName]: + lmts[lmt_name] = {} + lmts[lmt_name]['transformCTL'] = os.path.join(lmt_dir, transform_CTL) + if transform_CTL_inverse != None: + # TODO: Check unresolved *odt_name* referemce. + lmts[odt_name]['transformCTLInverse'] = os.path.join( + lmt_dir, transform_CTL_inverse) + + lmts[lmt_name]['transformID'] = transform_ID + lmts[lmt_name]['transformUserNamePrefix'] = transform_user_name_prefix + lmts[lmt_name]['transformUserName'] = transform_user_name + + print("LMT : %s" % lmt_name) + print("\tTransform ID : %s" % transform_ID) + print("\tTransform User Name Prefix : %s" % transform_user_name_prefix) + print("\tTransform User Name : %s" % transform_user_name) + print("\t Forward ctl : %s" % lmts[lmt_name]['transformCTL']) + if 'transformCTLInverse' in lmts[lmt_name]: print("\t Inverse ctl : %s" % ( - lmts[lmtName]['transformCTLInverse'])) + lmts[lmt_name]['transformCTLInverse'])) else: print("\t Inverse ctl : %s" % "None") @@ -1671,12 +1701,12 @@ def getLMTInfo(acesCTLReleaseDir): return lmts -def createACESConfig(acesCTLReleaseDir, - configDir, - lutResolution1d=4096, - lutResolution3d=64, - bakeSecondaryLUTs=True, - cleanup=True): +def create_ACES_config(aces_CTL_directory, + config_directory, + lut_resolution_1d=4096, + lut_resolution_3d=64, + bake_secondary_LUTs=True, + cleanup=True): """ Creates the ACES configuration. @@ -1692,52 +1722,52 @@ def createACESConfig(acesCTLReleaseDir, """ # Get ODT names and CTL paths - odtInfo = getODTInfo(acesCTLReleaseDir) + odt_info = get_ODT_info(aces_CTL_directory) # Get ODT names and CTL paths - lmtInfo = getLMTInfo(acesCTLReleaseDir) + lmt_info = get_LMT_info(aces_CTL_directory) # Create config dir - createConfigDir(configDir, bakeSecondaryLUTs) + create_config_dir(config_directory, bake_secondary_LUTs) # Generate config data and LUTs for different transforms - lutDir = "%s/luts" % configDir - shaperName = 'Output Shaper' - configData = generateLUTs(odtInfo, - lmtInfo, - shaperName, - acesCTLReleaseDir, - lutDir, - lutResolution1d, - lutResolution3d, - cleanup) + lut_directory = "%s/luts" % config_directory + shaper_name = 'Output Shaper' + config_data = generate_LUTs(odt_info, + lmt_info, + shaper_name, + aces_CTL_directory, + lut_directory, + lut_resolution_1d, + lut_resolution_3d, + cleanup) # Create the config using the generated LUTs print("Creating generic config") - config = createConfig(configData) + config = create_config(config_data) print("\n\n\n") # Write the config to disk - writeConfig(config, "%s/config.ocio" % configDir) + write_config(config, "%s/config.ocio" % config_directory) # Create a config that will work well with Nuke using the previously # generated LUTs. print("Creating Nuke-specific config") - nuke_config = createConfig(configData, nuke=True) + nuke_config = create_config(config_data, nuke=True) print("\n\n\n") # Write the config to disk - writeConfig(nuke_config, "%s/nuke_config.ocio" % configDir) + write_config(nuke_config, "%s/nuke_config.ocio" % config_directory) # Bake secondary LUTs using the config - if bakeSecondaryLUTs: - generateBakedLUTs(odtInfo, - shaperName, - "%s/baked" % configDir, - "%s/config.ocio" % configDir, - lutResolution1d, - lutResolution3d, - lutResolution1d) + if bake_secondary_LUTs: + generate_baked_LUTs(odt_info, + shaper_name, + "%s/baked" % config_directory, + "%s/config.ocio" % config_directory, + lut_resolution_1d, + lut_resolution_3d, + lut_resolution_1d) return True @@ -1777,38 +1807,38 @@ def main(): # # Get options # - acesCTLDir = options.acesCTLDir - configDir = options.configDir - lutResolution1d = int(options.lutResolution1d) - lutResolution3d = int(options.lutResolution3d) - bakeSecondaryLUTs = not (options.dontBakeSecondaryLUTs) - cleanupTempImages = not (options.keepTempImages) + aces_CTL_directory = options.acesCTLDir + config_directory = options.configDir + lut_resolution_1d = int(options.lutResolution1d) + lut_resolution_3d = int(options.lutResolution3d) + bake_secondary_LUTs = not (options.dontBakeSecondaryLUTs) + cleanup_temp_images = not (options.keepTempImages) try: - argsStart = sys.argv.index('--') + 1 - args = sys.argv[argsStart:] + args_start = sys.argv.index('--') + 1 + args = sys.argv[args_start:] except: - argsStart = len(sys.argv) + 1 + args_start = len(sys.argv) + 1 args = [] print("command line : \n%s\n" % " ".join(sys.argv)) # TODO: Use assertion and mention environment variables. - if not acesCTLDir: + if not aces_CTL_directory: print("process: No ACES CTL directory specified") return - if not configDir: + if not config_directory: print("process: No configuration directory specified") return # # Generate the configuration # - return createACESConfig(acesCTLDir, - configDir, - lutResolution1d, - lutResolution3d, - bakeSecondaryLUTs, - cleanupTempImages) + return create_ACES_config(aces_CTL_directory, + config_directory, + lut_resolution_1d, + lut_resolution_3d, + bake_secondary_LUTs, + cleanup_temp_images) if __name__ == '__main__':