While looking into cover art save behavior (PICARD-1696), I noticed that ImageList.__eq__ compares images in sorted order, but _save_images_to_tags() writes APIC frames in insertion order.
2026-03-01 06050, 2026
DeepakTiwari[m]
Would it make sense to sort images before saving them to ID3 tags (by normalized types), so that save order is deterministic?
2026-03-01 06050, 2026
DeepakTiwari[m]
I just want to confirm whether insertion order is intentional before trying a fix.
NOTICE: PICARD-1696 (https://tickets.metabrainz.org/browse/PICARD-1696): No stable sort order for cover art saved to ID3
2026-03-01 06003, 2026
outsidecontext[m joined the channel
2026-03-01 06003, 2026
outsidecontext[m
@[Deepak Tiwari] maybe the comparison and save order should be consistent, yes. I'll need to look deeper into the code to say more. But for this specific issue it needs to be fixed in mutagen first, as the order gets changed by mutagen upon saving. See the mutagen issue linked in our ticket.
