The patch looks good, just some very small details to correct before it can get merged:
Coding style: inconsistent format. See https://developer.blender.org/docs/handbook/tooling/clangformat/ for how to format your code automatically to match
Thanks for the patch! I will test later, but the code looks fine :)
Regarding the blend file you should create a PR in the test repository to update the test files, see https://projects.blender…
@Dario-San I think it always makes sense to extend the automated testing framework with more test cases, so your contribution would definitely be useful. I am willing to help and review your…
I think these are also expected. If you render with F12 both CPU and GPU results are consistent with 4.1. The behavior of crop is not much different than regular scaling, where the size of the new…
Thanks for the report! :)
This is indeed expected, because the compositor CPU backend changed in 4.2. You should see the same contour when you click on the viewer node. But when you hit F12…
Hi, thanks for the patch :) Now that blender/blender#119603 is merged, we can move forward with this patch.
We should really use a proper enum now that we are converting from one enum of sampling to another.
I think using a proper struct for the scale node makes sense, but I think versioning might be needed. Have you checked if node trees from older files can be loaded and executed properly?
Is it possible to have gizmos in image editor as well? Would be amazing for those who don't use backdrop
I think it makes sense and I'm interested in implementing that also, but I have no…