AcademySoftwareFoundation / OpenColorIO

A color management framework for visual effects and animation.
https://opencolorio.org
BSD 3-Clause "New" or "Revised" License
1.74k stars 430 forks source link

CI Workflows various fixes #1945

Closed remia closed 3 months ago

remia commented 5 months ago

This PR attempt to fix various issues on the GHA workflows:

KelSolaar commented 3 months ago

My only comment is whether we should include the GIL release in this PR? https://github.com/AcademySoftwareFoundation/OpenColorIO/pull/1945/files#diff-6c87f76bfb1f06a48bb762f97632eee604f4d8cd2e852347c3b46209a056086a

Those seems to be beyond pure CI workflow changes.

If people are happy with it then fine.

doug-walker commented 3 months ago

@KelSolaar, given the large delays in getting PRs merged, we all sometimes include small miscellaneous improvements into a larger PR. It's not ideal, but it's a response to the current state of affairs where we don't have a sufficient number of reviewers.

KelSolaar commented 3 months ago

@doug-walker : All good with me and I understand the frustration! It is one of the reason I have been hesitant to move colour to the ASWF and a more formal review process, it certainly hinders the development velocity.

KelSolaar commented 3 months ago

Merging for good measure because the aforementioned note was the only comment I had! Thanks @remia.

lgritz commented 3 months ago

@doug-walker : All good with me and I understand the frustration! It is one of the reason I have been hesitant to move colour to the ASWF and a more formal review process, it certainly hinders the development velocity.

Such rules are made by every project TSC individually to pick the right set of procedures for their project and team. There is no ASWF-mandated policy on how reviews and merges work.

lgritz commented 3 months ago

I'm sorry, I see I was tagged here, but I didn't notice this PR until today. Apologies if you wanted me to look at something here and I failed to reply. (I do "watch" this project, but sometimes when I'm busy, I'm a little eager with the delete key if I don't see a PR title that I think is highly relevant to me.)

remia commented 3 months ago

@KelSolaar Agree the GIL changes could have been separated in a different PR, I included these here because it fixed the WIndows Platform Latest workflow when I switched them to C++23.

@lgritz No worry at all! I tagged you because I made some OSL related fix for the unit tests.