jstkdng / ueberzugpp

Drop in replacement for ueberzug written in C++
GNU General Public License v3.0
753 stars 26 forks source link

[bug] some images fail to render #97

Closed justchokingaround closed 1 year ago

justchokingaround commented 1 year ago

no matter which output method is chosen, some images fail to render, example: image

for reference, here is the original file: lain_red

jstkdng commented 1 year ago

does it show correctly without opencv?

justchokingaround commented 1 year ago

no, it has the same behavior for no-opencv: false and no-opencv: true

jstkdng commented 1 year ago

works on my machine with X11 and sixel, I'll test on wayland later. Which terminal are you testing sixel with?

justchokingaround commented 1 year ago

wezterm, but the same happens in a patched alacritty and in foot

jstkdng commented 1 year ago

ok, justed on wayland and it works fine. Try deleting the cache, maybe the resized image didn't save correctly.

justchokingaround commented 1 year ago

nope, still happening, sometimes it shows this message: image

jstkdng commented 1 year ago

damn, an opencl error. I'd say delete the cache and disable opencv.

justchokingaround commented 1 year ago

disabling opencv and clearing the cache does make it work, however it is still annoying that the errors pop up on the screen, since they displace the rest of what's being displayed, for instance if you're using lf, it will misplace everything

here are some of the error messages i was able to capture, if that helps:

image image

jstkdng commented 1 year ago

yup, that's an issue with arch's libvips package, it is buidling a debug package. Install libvips-git, I had to modify it so that it builds a release package, those messages should be gone with that.

justchokingaround commented 1 year ago

yep, works nicely now, thank you yet again!