Rémi Achard

Results 122 comments of 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?