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

First draft of release notes for v3.3 #1835

Merged
merged 4 commits into from
Sep 24, 2024

Conversation

cary-ilm
Copy link
Member

No description provided.

Comment on lines +82 to +83
- The C++ API now uses the OpenEXRCore library underneath.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can summarize why this is important? Is there a perf benefit?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kimball said he's got notes, I left that section for him

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some notes, not exhaustive:

  • The C++ API has started to use the OpenEXRCore library underneath
  • This is a transparent change to the existing API, although the ABI (i.e. structure / class layout) has changed
  • Existing reading of pixel data should be more efficient due to fewer memory allocations / frees during the process of reading. Additionally, some more specialisation of unpacking routines may result in faster unpack times
  • All compression routines are implemented by the C Core layer underneath and no longer duplicated
  • Initial support for "stateless" reading of scanlines has been proposed, allowing multiple threads to read scanlines into different frame buffer objects at the same time. While well tested at the Core level, the C++ api should be considered experimental for this release
  • Thread dispatch for reading different file types has been made more homogeneous, so is simpler and more consistent

CHANGES.md Outdated
file sizes.

- ``exrmanifest`` - Read exr files and print the contents of the
embedded manifest.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"manifest" is going to be new for some people, wonder if we should have a note for learning more about it?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

something like "... of the embedded manifest. The manifest provides a mapping to a human-readable identifiers, see https://github.com/AcademySoftwareFoundation/openexr/blob/25c2d775f889948d48114f382b58e35d313ecad3/website/DeepIDsSpecification.rst for more information."

@cary-ilm cary-ilm merged commit b2d8156 into AcademySoftwareFoundation:main Sep 24, 2024
6 checks passed
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.

4 participants