search

inflation / jpegxl-rs

GNU General Public License v3.0
56 stars 11 forks source link

does this support animated jxl? #78

Open BPplays opened 3 weeks ago

BPplays commented 3 weeks ago

does this wrapper support reading animated jxl files?

inflation commented 3 weeks ago

Not currently, since almost no one demands it.

If I'm going to implement it, would you like to receive the image frame-by-frame or all the frames at once?

BPplays commented 3 weeks ago

If I'm going to implement it, would you like to receive the image frame-by-frame or all the frames at once?

@inflation i don't really know which one would be better, i made a PR TagStudioDev/TagStudio#357 using pillow-jpegxl-plugin which is using this wrapper. adding better support for animated images but qt only seems to support animated webp and gif so i would have to transcode it to webp still.

inflation commented 2 weeks ago

Hmm. I'm not familiar with QT, but there should probably be a plugin for reading and writing jxl files for it.

BPplays commented 2 weeks ago

@inflation since i wrote that i got transcoding working. i just wanted to give the info on what i was working on if that might affect a decision on the type of implement you would go with. also i just added on to the existing implementation using QMovie but that only seems to support animated webp and gif and i just wanted to use the format support of Pillow to support more formats.