Skip to content

Commit

Permalink
Corrected role property names
Browse files Browse the repository at this point in the history
Signed-off-by: John-Paul Smith <[email protected]>
  • Loading branch information
john-paulsmith committed Oct 2, 2024
1 parent 7372ac2 commit 5d469ed
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 21 deletions.
36 changes: 18 additions & 18 deletions include/ofx-native-v1.5_aces-v1.3_ocio-v2.3.h
Original file line number Diff line number Diff line change
Expand Up @@ -770,64 +770,64 @@ Any scene-referred colourspace with a log transfer function.
Guaranteed to be ACES2065-1.
*/
#define kOfxColourspaceRoleAcesInterchange "aces_interchange"
#define kOfxColourspaceAcesInterchangeIsBasic false
#define kOfxColourspaceAcesInterchangeIsCore true
#define kOfxColourspaceRoleAcesInterchangeIsBasic false
#define kOfxColourspaceRoleAcesInterchangeIsCore true

/** @brief cie_xyz_d65_interchange
CIE XYZ colorimetry with the neutral axis at D65.
*/
#define kOfxColourspaceRoleCieXyzD65Interchange "cie_xyz_d65_interchange"
#define kOfxColourspaceCieXyzD65InterchangeIsBasic false
#define kOfxColourspaceCieXyzD65InterchangeIsCore true
#define kOfxColourspaceRoleCieXyzD65InterchangeIsBasic false
#define kOfxColourspaceRoleCieXyzD65InterchangeIsCore true

/** @brief color_picking
The colourspace to use for colour pickers, typically a display colourspace.
*/
#define kOfxColourspaceRoleColorPicking "color_picking"
#define kOfxColourspaceColorPickingIsBasic false
#define kOfxColourspaceColorPickingIsCore true
#define kOfxColourspaceRoleColorPickingIsBasic false
#define kOfxColourspaceRoleColorPickingIsCore true

/** @brief color_timing
A colourspace suitable for colour grading, typically a log colourspace.
*/
#define kOfxColourspaceRoleColorTiming "color_timing"
#define kOfxColourspaceColorTimingIsBasic false
#define kOfxColourspaceColorTimingIsCore true
#define kOfxColourspaceRoleColorTimingIsBasic false
#define kOfxColourspaceRoleColorTimingIsCore true

/** @brief compositing_log
Any scene-referred colourspace with a log transfer function.
*/
#define kOfxColourspaceRoleCompositingLog "compositing_log"
#define kOfxColourspaceCompositingLogIsBasic false
#define kOfxColourspaceCompositingLogIsCore true
#define kOfxColourspaceRoleCompositingLogIsBasic false
#define kOfxColourspaceRoleCompositingLogIsCore true

/** @brief data
Image values should not be treated as colour, e.g. motion vectors or masks. Mapped to the raw colourspace.
*/
#define kOfxColourspaceRoleData "data"
#define kOfxColourspaceDataIsBasic false
#define kOfxColourspaceDataIsCore true
#define kOfxColourspaceRoleDataIsBasic false
#define kOfxColourspaceRoleDataIsCore true

/** @brief matte_paint
A colourspace suitable for matte painting.
*/
#define kOfxColourspaceRoleMattePaint "matte_paint"
#define kOfxColourspaceMattePaintIsBasic false
#define kOfxColourspaceMattePaintIsCore true
#define kOfxColourspaceRoleMattePaintIsBasic false
#define kOfxColourspaceRoleMattePaintIsCore true

/** @brief scene_linear
Any scene-referred linear colourspace.
*/
#define kOfxColourspaceRoleSceneLinear "scene_linear"
#define kOfxColourspaceSceneLinearIsBasic false
#define kOfxColourspaceSceneLinearIsCore true
#define kOfxColourspaceRoleSceneLinearIsBasic false
#define kOfxColourspaceRoleSceneLinearIsCore true

/** @brief texture_paint
A colourspace suitable for texture painting, typically sRGB.
*/
#define kOfxColourspaceRoleTexturePaint "texture_paint"
#define kOfxColourspaceTexturePaintIsBasic false
#define kOfxColourspaceTexturePaintIsCore true
#define kOfxColourspaceRoleTexturePaintIsBasic false
#define kOfxColourspaceRoleTexturePaintIsCore true

#ifdef __cplusplus
}
Expand Down
7 changes: 4 additions & 3 deletions scripts/genColour
Original file line number Diff line number Diff line change
Expand Up @@ -98,11 +98,12 @@ def print_basic_colourspace(role):

def print_role(role):
role_name = role[0]
role_prefix = f'Role{camel_name(role_name)}'
print(f'\n/** @brief {role_name}\n{role_docs[role_name]}\n*/')
print(f'{define_base}Role{camel_name(role_name)} "{role_name}"')
print(f'{define_base}{role_prefix} "{role_name}"')
# non-basic roles are included in the core style
print_bool_property(role_name, 'Basic', False)
print_bool_property(role_name, 'Core', True)
print_bool_property(role_prefix, 'Basic', False)
print_bool_property(role_prefix, 'Core', True)

def print_string_property(name, propname, value):
print(f'{define_base}{camel_name(name)}{propname} "{value}"')
Expand Down

0 comments on commit 5d469ed

Please sign in to comment.