Skip to content
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

Move config checking code from the ociocheck app into the core library #1949

Open
doug-walker opened this issue Feb 13, 2024 · 0 comments
Open
Labels
Feature Request New addition to OCIO functionality. Help Wanted Issues that the TSC has decided are worth implementing, but don't currently have the dev resources.

Comments

@doug-walker
Copy link
Collaborator

The ociocheck command-line app currently provides the most thorough validation of a config. Some of these checks would be quite useful for a variety of apps that use OCIO (and ocioview would be a prime example). But in order to make use of them, the checks should move from the ociocheck app itself into the core library where they are accessible from the API.

The current validate method on the Config class is very useful as a quick method of checking a config, but it does not try things such as resolving and loading FileTransforms (ociocheck loads all FileTransforms). It would be best to keep validate a quick check since that is already widely used by client apps. So any time-consuming tests or tests that require loading external files should be put in a separate function/method.

@doug-walker doug-walker added Feature Request New addition to OCIO functionality. Help Wanted Issues that the TSC has decided are worth implementing, but don't currently have the dev resources. labels Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request New addition to OCIO functionality. Help Wanted Issues that the TSC has decided are worth implementing, but don't currently have the dev resources.
Projects
None yet
Development

No branches or pull requests

1 participant