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

Provide YAML default mechanism to write fewer values #37

Open
jeremyselan opened this issue Nov 24, 2010 · 3 comments
Open

Provide YAML default mechanism to write fewer values #37

jeremyselan opened this issue Nov 24, 2010 · 3 comments
Labels
Feature Request New addition to OCIO functionality.

Comments

@jeremyselan
Copy link
Contributor

Right now, yaml serialization often spits out all values regardless of whether or not they are 'set'. While this is explicit and easy to understand, it makes for lengthy files. Perhaps in some cases it is more appropriate to omit the value unless a non-default is specified?

@doug-walker
Copy link
Collaborator

This issue was discussed during the ACES config working group meeting today. The group agreed that we would prefer that attribute values that are default should not be serialized to the yaml. The ColorSpace bitdepth and allocationvars being common examples. The interpolation value of FileTransforms being another example.

We will make that change as part of the v2 work.

@KelSolaar
Copy link
Contributor

That would be great!

To give an example, if we were to remove default equality_group, is_data, allocation and allocationvars from the current ACES config, it would remove over 1100 lines from 5000, more than a fifth!!!

@KevinJW
Copy link
Contributor

KevinJW commented Sep 9, 2020

As pointed out in the meeting, the equality_group not being specified could lead to a bunch of No-Ops, so we may need to clean up the behavioural differences between, not specifying the option, specifying the option but it being empty and specifying the option as an empty string, etc.

Will need to consider if this may break backwards compatibility in v1 configs, etc.

@carolalynn carolalynn added the To Close temp label to help triage closable issues for review label Feb 7, 2024
@doug-walker doug-walker removed the To Close temp label to help triage closable issues for review label May 15, 2024
@doug-walker doug-walker reopened this May 15, 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.
Projects
None yet
Development

No branches or pull requests

5 participants