this post was submitted on 19 Jun 2023
7 points (100.0% liked)
Café
777 readers
1 users here now
Welcome to our virtual third place, The Café.
Come on in and make a new human connection over a cup of coffee (or Teh Tarik). This is a casual community, do whatever you want, share your oyen pics, your frustrations, and even organize a weekend picnic with the community. The world is your oyster.
Rules are simple, be kind and civil with each other. As with any other café, rude patrons will be kicked out.
founded 1 year ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
view the rest of the comments
Update: I found this issue on magick's github repo, but I can't find something similar so maybe it's missing a pictrs config, or pict-rs itself is having issues by maybe... not supporting -add-orient. :( Pict-rs is a rust service, so it'll probably be harder for me to dig into that as I'm not familiar with it...
I'll probably create a ticket for when I have time, need to finish up some IRL stuff and head to bed soon. ~~Or if anyone wants to create the ticket also can~~
Ahh, interesting. Thanks for the work!
if anyone is familiar with rust (I am not), this should be the relevant code: https://git.asonix.dog/asonix/pict-rs/src/branch/v0.3.x/src/magick.rs (linking to the v0.3.x branch because that's what lemmy-ansible uses).
perhaps if we really want to dig into this issue, it could start with either preventing pict-rs from auto-converting to webp, or upgrading pict-rs to the latest 0.4.0?
I'm not sure if all images are stored as webp, because when I viewed the image in element inspector it showed up as jpeg(it either happens to all image extensions or images are stored on webp but converted to jpeg when displaying).
Thanks for sending the link, I didn't manage to find this last night D: will take a look after work... I hope today no need to OT
just saw that the latest 0.4.0 uses -auto-orient: https://git.asonix.dog/asonix/pict-rs/src/branch/main/src/magick.rs#L126
Oh... So it really is the auto orient then! Thanks for your help, I'll post a GitHub issue later on Lemmy's repo later(maybe the Ansible one since that's where the version is defined?) if no one else has done it yet.