Rémi Achard
Rémi Achard
Pushed an update with my current work in progress, this mainly split the new FixedFunctionOps into 3 for JMh conversion, Tonescale / ChromaCompress and GamutCompress. The code still follows the...
@KelSolaar I added more (if not all) transforms in my latest commit and started a [spreadsheet](https://docs.google.com/spreadsheets/d/1BxfwNGeL90tWj-ZI-lytHmw3b_cwYcCnHRJc3kkzQTw/edit?gid=0#gid=0) to list everything I have so far. I'm not opposed to changing the list...
Thanks for the feedback! > It might be useful to reorder the D60 tab in the spreadsheet to match the D65 one so that it's easier to compare one to...
Hi @KelSolaar, The assignment of ViewTransform to CTLs should be available in the [spreadsheet](https://docs.google.com/spreadsheets/d/1BxfwNGeL90tWj-ZI-lytHmw3b_cwYcCnHRJc3kkzQTw/edit?gid=0#gid=0) shared earlier. I agree the names are not directly matching in some cases, for example all...
I pushed an update that should fix the HLSL shader compilation issue by extracting the hue component from the texture into a constant array, as suggested by @KevinJW. Also included...
> @remia: As of https://github.com/AcademySoftwareFoundation/OpenColorIO/commit/6585f18c889d68f236548ae536811a0b27571458, I can only see the 31 builtins https://github.com/AcademySoftwareFoundation/OpenColorIO/pull/1983#issuecomment-2295180317. Is this expected or should I have the 54 from the spreadsheet? I haven't had time to...
Good catch, I fixed that now, hopefully should work!
I did some profiling on the CPU path and GPU timings and didn't find regression since earlier results.
Good to merge, I updated the branch against main, feel free to do it when the tests are completed!
> I'm extremely curious to hear if @cozdas feels like the copilot review comments are helpful and correct. Do we have an idea on the associated cost?