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

Adsk Contrib - Various CLF writer improvements #1020

Conversation

hodoulp
Copy link
Member

@hodoulp hodoulp commented May 21, 2020

Signed-off-by: Patrick Hodoul [email protected]

The pull request contains several fixes to the CLF/CTF writer:

  • The right precision for the double serialization was missing for some Ops (such as the Log one),
  • The layout of the LUT values is improved to better align them on the fly,
  • The writer performance is improved by at least 2 for the LUT serialization.

Note: The ociomakeclf app has now a new argument to measure the performance of the write step only. The app was used to validate the write performance improvement.

@hodoulp hodoulp added the v2.0 label May 21, 2020
@hodoulp hodoulp merged commit 42dbc96 into AcademySoftwareFoundation:master Jun 1, 2020
@hodoulp hodoulp deleted the adsk_contrib/log_precision branch June 1, 2020 18:20
meimchu pushed a commit to meimchu/OpenColorIO that referenced this pull request Jun 2, 2020
scoopxyz pushed a commit to scoopxyz/OpenColorIO that referenced this pull request Jul 1, 2020
scoopxyz pushed a commit to scoopxyz/OpenColorIO that referenced this pull request Jul 12, 2020
michdolan pushed a commit to michdolan/OpenColorIO that referenced this pull request Jul 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants