-
Notifications
You must be signed in to change notification settings - Fork 459
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
Create the 2.4.1 release #2107
Merged
doug-walker
merged 16 commits into
AcademySoftwareFoundation:RB-2.4
from
autodesk-forks:walker-rb-241
Dec 12, 2024
Merged
Create the 2.4.1 release #2107
doug-walker
merged 16 commits into
AcademySoftwareFoundation:RB-2.4
from
autodesk-forks:walker-rb-241
Dec 12, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ySoftwareFoundation#2077) Signed-off-by: Eric Renaud-Houde <[email protected]> (cherry picked from commit 59d6a86) Signed-off-by: Doug Walker <[email protected]>
…reFoundation#2072) By default Visual Studio projects do not compile source files in parallel. Add "/MP" flag to enable that. From scratch OCIO build on Ryzen 5950X, VS2022: 580sec -> 208sec. Most of remaining single-threaded time is cloning external libraries or building external libraries (most of them don't do parallel compilation either). But within building OCIO itself, CPU usage goes close to 100% instead of 5%. Signed-off-by: Aras Pranckevicius <[email protected]> Co-authored-by: Doug Walker <[email protected]> (cherry picked from commit 707734d) Signed-off-by: Doug Walker <[email protected]>
Instead of creating temporary LUT files for GPU tests, read them from the testdata directory instead. This removes the need to clean them up after each run. Signed-off-by: Ananth Bhaskararaman <[email protected]> (cherry picked from commit b3900df) Signed-off-by: Doug Walker <[email protected]>
* ocio gm in photoshop Signed-off-by: Dave Sawyer <[email protected]> * Update supported_apps.yml Signed-off-by: Dave Sawyer <[email protected]> * ps and fnordware text Signed-off-by: Dave Sawyer <[email protected]> --------- Signed-off-by: Dave Sawyer <[email protected]> (cherry picked from commit 81c07fd) Signed-off-by: Doug Walker <[email protected]>
* Update doc building instructions Signed-off-by: Craig Zerouni <[email protected]> * Remove stray comment line Signed-off-by: Craig Zerouni <[email protected]> --------- Signed-off-by: Craig Zerouni <[email protected]> (cherry picked from commit 800efd8) Signed-off-by: Doug Walker <[email protected]>
…tion#2088) Signed-off-by: Chris Nash <[email protected]> Co-authored-by: zachlewis <[email protected]> (cherry picked from commit 9bc5b8e) Signed-off-by: Doug Walker <[email protected]>
…emySoftwareFoundation#2095) (cherry picked from commit f37ca55) Signed-off-by: Doug Walker <[email protected]>
* Fix broken Linux and Mac CI workflow Signed-off-by: Doug Walker <[email protected]> * Update job name Signed-off-by: Doug Walker <[email protected]> --------- Signed-off-by: Doug Walker <[email protected]> (cherry picked from commit c7ad2a9) Signed-off-by: Doug Walker <[email protected]>
…oundation#2074) * Iridas .cube and other text format parsing optimizations Primarily driven by a wish to increase performance of parsing .cube files. But the functions that are added or changed are used across parsing of all/most of text based formats. With these changes, parsing "Khronos PBR Neutral" .cube file (5.4MB) on Ryzen 5950X / VS2022 Release build: 167ms -> 123ms. - Add locale independent IsSpace(char). Somewhat similar to 3aab90d, whitespace trimming perhaps should not be locale dependent. - Add IsEmptyOrWhiteSpace() and use that inside ParseUtils::nextline, instead of doing Trim(line).empty(). - Add StringUtils::StartsWith(char) and use that in various parsers that were constructing whole std::string object just to check for a single character. - When building for C++17 or later, NumberUtils can use standard <charconv> from_chars functions (except on Apple platforms, where those are not implemented for floating point types as of Xcode 15). This has advantage of not having to deal with errno or locales. Saves some thread local storage accesses and function calls (e.g. on Windows errno is actually a function call). - There's a CMake setup change that adds /Zc:__cplusplus flag for MSVC; for backwards compat reasons it does not report proper C++ version detection defines otherwise. Signed-off-by: Aras Pranckevicius <[email protected]> * Fix test failures (char can be signed, doh) Signed-off-by: Aras Pranckevicius <[email protected]> * Tests: add unit test coverage for NumberUtils::from_chars directly Currently it was only tested indirectly via XMLReaderUtils_tests and file format tests Signed-off-by: Aras Pranckevicius <[email protected]> * Fix from_chars in C++17 code path to understand hex prefix (0x) and skip optional whitespace To match the pre-C++17 behavior that was there before Signed-off-by: Aras Pranckevicius <[email protected]> * Fix detection of <charconv> float from_chars availability Signed-off-by: Aras Pranckevicius <[email protected]> * Tests: Fix missing <limits> include for gcc Signed-off-by: Aras Pranckevicius <[email protected]> * Tests: fix uninitialized variable warning-as-error on gcc Signed-off-by: Aras Pranckevicius <[email protected]> --------- Signed-off-by: Aras Pranckevicius <[email protected]> Co-authored-by: Rémi Achard <[email protected]> Co-authored-by: Doug Walker <[email protected]> (cherry picked from commit f2dc147) Signed-off-by: Doug Walker <[email protected]>
…reFoundation#2068) * fix views/display submenus Signed-off-by: Hannah <[email protected]> * fix global variable name Signed-off-by: Hannah <[email protected]> --------- Signed-off-by: Hannah <[email protected]> Co-authored-by: Doug Walker <[email protected]> (cherry picked from commit 5c2fa57) Signed-off-by: Doug Walker <[email protected]>
…ation#2090) Signed-off-by: Doug Walker <[email protected]> (cherry picked from commit dafefe6) Signed-off-by: Doug Walker <[email protected]>
…areFoundation#2096) * Error found: ocio_to_colorlookup_all.py Signed-off-by: Seif Ashraf <[email protected]> * Update colorlookup_to_spi1d.py Signed-off-by: Seif Ashraf <[email protected]> * Update ocio_to_colorlookup_rgb.py Signed-off-by: Seif Ashraf <[email protected]> --------- Signed-off-by: Seif Ashraf <[email protected]> Co-authored-by: Doug Walker <[email protected]> (cherry picked from commit 0f06f04) Signed-off-by: Doug Walker <[email protected]>
…del 5.0 (SM_5_0). (AcademySoftwareFoundation#2078) Signed-off-by: Eric Renaud-Houde <[email protected]> Co-authored-by: Doug Walker <[email protected]> (cherry picked from commit 3132579) Signed-off-by: Doug Walker <[email protected]>
… multiple op pairs (AcademySoftwareFoundation#2104) * Fix opt combine issue Signed-off-by: Doug Walker <[email protected]> * Improve gamma test Signed-off-by: Doug Walker <[email protected]> * Add check for DisplayP3-HDR built-in Signed-off-by: Doug Walker <[email protected]> --------- Signed-off-by: Doug Walker <[email protected]> (cherry picked from commit 1890b7d) Signed-off-by: Doug Walker <[email protected]>
…ATH opt-in (AcademySoftwareFoundation#2106) Signed-off-by: Rémi Achard <[email protected]> Co-authored-by: Doug Walker <[email protected]> (cherry picked from commit 6fa40a4) Signed-off-by: Doug Walker <[email protected]>
Signed-off-by: Doug Walker <[email protected]>
cozdas
approved these changes
Dec 12, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
KelSolaar
approved these changes
Dec 12, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's ship it!
KelSolaar
approved these changes
Dec 12, 2024
remia
approved these changes
Dec 12, 2024
doug-walker
merged commit Dec 12, 2024
ebd076a
into
AcademySoftwareFoundation:RB-2.4
63 checks passed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR cherry-picks the commits from main since 2.4.0 that will go in the 2.4.1 release. (This is all of them except the work on making an Actions workflow to automatically update the website.)
An additional commit is added to update the library version.