Add support for lossy HTJ2K compressor

[palemieux]

• add support for a separate lossy HTJ2K compressor (using a log transfer function as opposed to DWA's log-gamma transfer function)
• adds log MSE computation to exrmetrics (to measure the distortion introduced by lossy compressors)

See https://academysoftwarefdn.slack.com/archives/CMLRW4N73/p1777317753088359

[cary-ilm]

@Vertexwahn, any suggestion to fix the Bazel failure here? I see two errors in the logs:

src/lib/OpenEXRCore/internal_ht.cpp:10:10: fatal error: openjph/ojph_arch.h: No such file or directory
   10 | #include <openjph/ojph_arch.h>

and:

In file included from src/lib/OpenEXRCore/base.c:8:
src/lib/OpenEXRCore/openexr_version.h:11: warning: "OPENEXR_VERSION_MAJOR" redefined
11 | # define OPENEXR_VERSION_MAJOR 4

[cary-ilm]

And @palemieux, can you amend the commits with -s to add the required DCO?

[Vertexwahn]

Short time solution: Change in MODULE.bazel the line bazel_dep(name = "openjph", version = "0.27.3") to bazel_dep(name = "openjph", version = "0.27.0")

[Vertexwahn]

Once bazelbuild/bazel-central-registry#9032 is merge you can change bazel_dep(name = "openjph", version = "0.27... to bazel_dep(name = "openjph", version = "0.27.3.bcr.1") (bcr.1 is a patch that contains the missing include path)

[peterhillman]

It might be better to put the exrmetrics changes into a separate PR (apart from minimal support for modifying the amount of compression). The MSE changes aren't directly related to lossy HTJ2K

[cary-ilm]

Just to confirm, are the changes to the vendored OpenJPH simply from vendoring in 0.27.3? Are there any changes we need to make sure don't get stomped on?

[palemieux]

Just to confirm, are the changes to the vendored OpenJPH simply from vendoring in 0.27.3? Are there any changes we need to make sure don't get stomped on?

Stock 27.3. No additional changes. Thanks for making sure.

[palemieux]

It might be better to put the exrmetrics changes into a separate PR (apart from minimal support for modifying the amount of compression). The MSE changes aren't directly related to lossy HTJ2K

Ok. Will split it off before the lossy J2K branch is merged.

[cary-ilm]

@peterhillman and @palemieux, since this PR is going into the lossy-htj2k feature branch, the exrmetrics changes can stay here and also go into a separate PR for main. That will highlight them more prominently in the commit history and release notes.

[palemieux]

Ok. I can create a PR against main.

[cary-ilm]

@palemieux, I think this is only waiting on @michaeldsmith's comment suggestions?

[palemieux]

See #2448 for a standalone PR adding MSE to exrmetrics.

[cary-ilm]

A few minor suggestions, otherwise it looks ready to go to me.