Add JPEG XL support to image processing.

[veluca93]

As discussed in #2421.

Sanity check:

• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Code changes

• Are you doing the PR on the next branch?

If the change is a new feature or adding to/changing an existing one:

• Have you created/updated the relevant documentation page(s)?

[Keats]

Can you add some tests? They are in imageproc/tests

[veluca93]

Can you add some tests? They are in imageproc/tests

Absolutely - what do you think I should test? For now I just added a PNG to JXL test.

[Keats]

Maybe a JXL output with a JXL input? The CI failures look legit though

[veluca93]

Maybe a JXL output with a JXL input? The CI failures look legit though

Ah, JXL reading is not supported :D do you think I should add that?

As for the CI... I believe I need to add the system dependency, but I have no idea how to configure that - any pointers?

[Keats]

Ah, JXL reading is not supported :D do you think I should add that?

It depends whether it can be added or not. If so then yes

As for the CI... I believe I need to add the system dependency, but I have no idea how to configure that - any pointers?

It looks like there is a vendored feature for statically linking it

[veluca93]

Ah, JXL reading is not supported :D do you think I should add that?

It depends whether it can be added or not. If so then yes

As for the CI... I believe I need to add the system dependency, but I have no idea how to configure that - any pointers?

It looks like there is a vendored feature for statically linking it

Did both of those. Also, my pr to jpegxl-rs was accepted and a release should be coming soon, so perhaps waiting a day or two and switching to the next version of jpegxl-rs would be the ideal solution.

In case you are wondering why I did not use the "image" feature of jpegxl-rs: it depends on a different version of "image", and trying to upgrade the dependency for zola to 0.25 produced some dependency incompatibility errors.

[Keats]

it depends on a different version of "image", and trying to upgrade the dependency for zola to 0.25 produced some dependency incompatibility errors.

I will update to 0.25 soonish, but I haven't looked at the image feature of jpeg-xl-rs. If that's better, I can update it now and you rebase your PR on it?

[veluca93]

it depends on a different version of "image", and trying to upgrade the dependency for zola to 0.25 produced some dependency incompatibility errors.

I will update to 0.25 soonish, but I haven't looked at the image feature of jpeg-xl-rs. If that's better, I can update it now and you rebase your PR on it?

If you can do that, it'd be great!

Looks like there are some build issues with jpegxl-rs, so it'll be a while before this PR can be merged anyway :-)

[Keats]

next has been updated with image 0.25

[veluca93]

Thanks! I updated the PR accordingly (also squashing the commits, since I was going to rebase anyway...)

[Keats]

Can you rebase? CI is fixed minus windows which i'll look at later

[veluca93]

Can you rebase? CI is fixed minus windows which i'll look at later

I made a mess with rebasing, but it should be done now :-)

[Keats]

CI failures seem legit

[veluca93]

CI failures seem legit

Yep, they are, I am hoping they will be resolved on the jpegxl_rs side (inflation/jpegxl-rs#50) and jxl side (inflation/jpegxl-rs#51) as soon as I can get to it :-)