-
Notifications
You must be signed in to change notification settings - Fork 121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP: Colorspace example #162
Commits on Apr 29, 2024
-
A set of OpenColorIO-related ImageEffect clip properties which allow hosts to pass information about their OCIO configuration into a plug-in. This allows plug-ins which display images in their own UI to match the colours used in the host, and is also useful in effects which are sensitive to the working colourspace of the input clip. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 2a7543f - Browse repository at this point
Copy the full SHA 2a7543fView commit details -
Made all OCIO properties read only
Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 7e8c002 - Browse repository at this point
Copy the full SHA 7e8c002View commit details -
OCIO properties now offer two-way communication, allowing both plug-in and host to specify a preferred colourspace, and setting the colourspace on output clips. Properties related to display are now set at an image effect rather than clip level, as they don’t change per clip and this makes them usable by generators. Overall use of the OCIO properties is now controlled by kOfxImageEffectPropSupportsOCIO to avoid additional work by plug-ins or hosts when OCIO is not available. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for e1beee2 - Browse repository at this point
Copy the full SHA e1beee2View commit details -
Revised Preferred Colourspace + doc improvement
kOfxImageClipPropOCIOPreferredColourspace is now specified by plug-ins during kOfxImageEffectActionGetClipPreferences, and by hosts on an output clip instance. Also clarified that plug-ins should not expect the display-related properties to be set during a render event. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for e2800d3 - Browse repository at this point
Copy the full SHA e2800d3View commit details -
Reworked the OCIO extension to support ACES
The OCIO API has been extended to be a broader colour management API, although still using OCIO as the foundation. This commit adds the ACESCG and ACES colour management styles, which only use OCIO as a source of colourspace names and could be implemented using other colour management pipelines. OCIO-specific properties are separated and form a superset of the functionality offered by the ACES styles. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 8cbc93e - Browse repository at this point
Copy the full SHA 8cbc93eView commit details -
Moved the colourspace API to its own header
Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for b07f15f - Browse repository at this point
Copy the full SHA b07f15fView commit details -
Colourspace defs based on OCIO ACES Studio config
The Python script scripts/genColour uses OCIO to process the config and write out a C header file containing preprocessor definitions for various names and attributes related to the colourspaces and roles in the config. ofxColourspaceList.h is the result of running this script on the ACES Studio config, which is the proposed common language the OFX colour support as it can be implemented without actually using OCIO. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 3a8e7ee - Browse repository at this point
Copy the full SHA 3a8e7eeView commit details -
Improve the readability of roles
Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 559e0ba - Browse repository at this point
Copy the full SHA 559e0baView commit details -
Allow a list of preferred colourspaces
Also removed the ACESCG style and renamed ACES to Native. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for caf96af - Browse repository at this point
Copy the full SHA caf96afView commit details -
Added missing colourspace name definitions
Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 132d027 - Browse repository at this point
Copy the full SHA 132d027View commit details -
Added the concept of core colourspaces and display names
The colourspace list has been updated to studio-config-v2.1.0_aces-v1.3_ocio-v2.3. The Studio and CG configs are compared to add a new attribute “IsCore”. This is true if the colourspae is present in the CG config, and defines a smaller set of colourspaces. Also, for most colourspaces, the first alias is a better short name than the actual colourspace name. The name is now used as the “DisplayName” attribute and the first alias is used as the base name instead. genColour is now hard-coded to specific OCIO configs so it’s easier to do the Studio vs CG comparisons. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for f69bfec - Browse repository at this point
Copy the full SHA f69bfecView commit details -
Workaround for missing srgb_tx encoding
This is an error in the studio config, workaround it until it’s fixed upstream by manually setting the encoding. See AcademySoftwareFoundation/OpenColorIO-Config-ACES#123. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for ff9b225 - Browse repository at this point
Copy the full SHA ff9b225View commit details -
Fix core colourspace labelling logic
The core colourspace list now contains more items, better matching the CG config. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 122c534 - Browse repository at this point
Copy the full SHA 122c534View commit details
Commits on May 9, 2024
-
Multiple improvements from code review
1. The OCIO Look property has been removed as this information is part of a View. 2. The Native mode header has been simplified, removing unnecessary aliases and the role mappings. DisplayName has been changed to Label. 3. The definition of core colourspaces has been changed to no longer depend on the OCIO cg config. genColour has a list of core colourspaces, now including HDR colourspaces as requested by @SonyDennisAdams. 4. New roles have been added for sdr_video and hdr_video. These are arbitrarily mapped but the intention is that plug-ins use them to request any sdr/hdr video colourspace. 5. The colourspace header now includes docs for roles, explaining their meaning. 6. The new genOCIOConfig script produces an OpenFX OCIO config, which includes the above changes. Hosts should ship this config so that even with running in Native mode, they can supply it to a plug-in that uses OCIO, via kOfxImageEffectPropOCIOConfig. This config is now used to produce the colourspace header. 7. A config generated by the above script is included as well - I put it in include for now but it could go elsewhere. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for e0cca3c - Browse repository at this point
Copy the full SHA e0cca3cView commit details
Commits on May 10, 2024
-
Reorganised core/non-core colourspaces
In ofxColourspaceList.h, colourspaces are now grouped into core and non-core, rather than display and scene. The previous genColour hacked around the fact that display colourspaces were marked inactive in the ACES config. Apparently these colourspaces will become active upstream in future, so preempt that by making them active in our config. This simplifies the code in genColour. Also corrected the filename name of our OCIO config and the invocation examples for genOCIOConfig and genColour. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 0c8339c - Browse repository at this point
Copy the full SHA 0c8339cView commit details
Commits on Jun 4, 2024
-
OfxImageClipPropColourspace and OfxImageClipPropPreferredColourspaces now accept special strings of the form “OfxColourspace_<clipname>. This allows for cross-referencing between different clips to ensure consistency across multiple inputs and from input to output. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 8e06771 - Browse repository at this point
Copy the full SHA 8e06771View commit details -
Full, Basic and Core colour management styles
To simplify development for plug-ins which have simple colour management requirements, this commit introduces Basic (roles only) and Core (roles + core colourspaces) colour management styles. Native has been renamed to Full and now the term native is used to refer to all three of these styles. Signed-off-by: John-Paul Smith <[email protected]>
3Configuration menu - View commit details
-
Copy full SHA for ce4acf1 - Browse repository at this point
Copy the full SHA ce4acf1View commit details -
Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for a768098 - Browse repository at this point
Copy the full SHA a768098View commit details
Commits on Jul 1, 2024
-
Added basic colourspaces instead of roles
Basic colour management mode no longer refers to roles and instead uses a small set of OFX-specific colourspace names which should cover most scenarios. These are defined in OCIO using roles for compatibility purposes, but in OFX, the expectation is that these might correspond to any colourspace that has the same encoding. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 01a3ca8 - Browse repository at this point
Copy the full SHA 01a3ca8View commit details
Commits on Jul 2, 2024
-
Output colourspace negotiation action
A new action, OfxImageEffectActionGetOutputColourspace, allows the host to specify its preferred colourspaces for the plug-in’s output clip, and for the plug-in to specify the colourspace it selected. OfxImageEffectActionGetClipPreferences now mentions that the plug-in can specify input clip colourspace preferences in this action. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 88f35d0 - Browse repository at this point
Copy the full SHA 88f35d0View commit details
Commits on Jul 10, 2024
-
Improvements to the docs and OCIO config
To aid with versioning, the OFX OCIO config has been renamed, now including the OpenFX version number instead of the version number of the upstream OCIO config. Basic colourspaces have been removed from the reference OCIO config, allowing hosts and plug-ins to freely choose any colourspace with the correct attributes. Two colourspaces were missing from the core style and have been added. Various doc improvements to fix typos and make the intended usage clearer. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for af85e80 - Browse repository at this point
Copy the full SHA af85e80View commit details
Commits on Jul 11, 2024
-
Support versioning native styles
Although the API should remain stable, colourspaces will change over time, and it’s likely hosts and plug-ins will support multiple versions of the spec for backwards compatibility. To address this, new properties have been introduced to allow negotiation of the config to use for native mode. This is somewhat similar to the use of built-in configs in OCIO. Now that the native mode config negotiation is handled separately, the OCIO config property is reserved for use only in the OCIO style. Hosts are now responsible for selecting the colour management style and config, and must set these in image effect properties. Anticipating support for multiple versions, ofxColourspaceList.h has been renamed to a versioned name and is referred to as “the config header”. For now, it could be used as a drop-in replacement for ofxColourspaceList.h, but as soon as a second version is added, developers will need a strategy to manage these. Additionally, the basic colourspaces have been renamed to include whether they are scene or display referred, and a descriptive label has been added. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 9f0533b - Browse repository at this point
Copy the full SHA 9f0533bView commit details
Commits on Jul 12, 2024
-
Allow use of basic colourspaces in OCIO mode
It makse sense to allow the use of basic colourspaces in OCIO mode, but in order to allow that, it’s necessary for negotiate the native-mode config even in OCIO mode, to allow for future changes to the set of basic colourspaces. Also some minor doc improvements. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for b3caf78 - Browse repository at this point
Copy the full SHA b3caf78View commit details -
Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for be3de0e - Browse repository at this point
Copy the full SHA be3de0eView commit details -
Doc clarifications for OCIO mode
Clarifying the use of basic colourspaces. Signed-off-by: John-Paul Smith <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 9a7eec4 - Browse repository at this point
Copy the full SHA 9a7eec4View commit details -
Signed-off-by: Gary Oberbrunner <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 3316c2e - Browse repository at this point
Copy the full SHA 3316c2eView commit details -
WIP: colorspace example updates
Signed-off-by: Gary Oberbrunner <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 7f5598f - Browse repository at this point
Copy the full SHA 7f5598fView commit details -
WIP: colourspace example now draws text with image info & colourspaces
Signed-off-by: Gary Oberbrunner <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 396fe61 - Browse repository at this point
Copy the full SHA 396fe61View commit details -
Add params to set preferred colourspaces
Also remove a bunch of old unused code from the example I copied this from. Signed-off-by: Gary Oberbrunner <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for edce85d - Browse repository at this point
Copy the full SHA edce85dView commit details -
Update colorspace example: fixes from Phil Barrett
- Fix negative rowbytes - Add unspecified preferred input space - Set input preferred space correctly Signed-off-by: Gary Oberbrunner <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 9a9748c - Browse repository at this point
Copy the full SHA 9a9748cView commit details -
Rename ColorSpace to ColourSpace for consistency
- Example dir name - Example source file name Signed-off-by: Gary Oberbrunner <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for c3ecca0 - Browse repository at this point
Copy the full SHA c3ecca0View commit details -
Fix a couple of Info.plist CFBundleExecutable strings
Signed-off-by: Gary Oberbrunner <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for e23ef6f - Browse repository at this point
Copy the full SHA e23ef6fView commit details -
Add Emacs .dir-locals.el to set our C/C++ indentation standard
Signed-off-by: Gary Oberbrunner <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 6e95184 - Browse repository at this point
Copy the full SHA 6e95184View commit details -
Signed-off-by: Gary Oberbrunner <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 63d3227 - Browse repository at this point
Copy the full SHA 63d3227View commit details