Screen size panel trigger a bug with current PNG picture.

mriale / PyDPainter

A usable pixel art paint program written in Python

GNU General Public License v3.0

545 stars 26 forks source link

Screen size panel trigger a bug with current PNG picture. #179

Open Sleuthgame opened 4 days ago

Sleuthgame commented 4 days ago

Load this PNG https://www.mobygames.com/game/7515/project-x/screenshots/amiga/118158/
Look at Screen Size to check - the png is hires and the pic is intended to be med-res, but it's ok.
Cancel the Screen Size panel. Picture disapears or something weird happens with palette and size.

Checked it in 2.0.1 and dev with other PNG. Won't happen with IFF.

mriale commented 3 days ago

That was weird! I think I fixed it in the develop branch.

Sleuthgame commented 3 days ago

Yes. Thanks you, I'll have a look.

Sleuthgame commented 3 days ago

Okay, can you check the cycling ball on this file ? It's fine on 2.0.1. and there is a anomaly on cycling now on last dev. https://www.randelshofer.ch/animations/anims/karl_bihlmeier/ABCK.ilbm.html

mriale commented 2 days ago

I got it working again it by reverting a change by @hsk to make color conversion faster by caching colors. Looks like it broke pictures where the color palette has duplicate entries. It's on develop now.

Thanks for finding that, @Sleuthgame!