RawTherapee icon indicating copy to clipboard operation
RawTherapee copied to clipboard

JPG (not RAW) image not updated in the editor when switching to another JPG

Open PeterMacej opened this issue 2 years ago • 4 comments

Steps to reproduce Let's have a folder with only JPG source images, not RAW images.

  1. Open the first img1.jpg in the editor. It displays fine. You can do any adjustments, e.g. rotate.
  2. Navigate to the next img2.jpg, whether with SHIFT+F4 or clicking on the film strip.
  3. The window title shows the correct new name img2.jpg, the histogram and other processing parameter will change as well.
  4. But the displayed picture in the editor doesn't change, it keeps displaying the previous one.

Expected behavior The editor should display the next JPG file.

It works fine with RAW images. It worked in RT 5.8 stable.

Additional information Windows 11 Version: 5.8-3105-g262d00bf1 Branch: dev Commit: 262d00bf1 Commit date: 2022-07-23 Compiler: cc 12.1.0 Processor: generic x86 System: Windows Bit depth: 64 bits Gtkmm: V3.24.6 Lensfun: V0.3.3.0 Build type: release Build flags: -std=c++11 -fno-tree-loop-vectorize -mtune=generic -Werror=unused-label -Werror=delete-incomplete -fno-math-errno -Wno-attributes -Wall -Wuninitialized -Wcast-qual -Wno-deprecated-declarations -Wno-unused-result -Wunused-macros -fopenmp -Werror=unknown-pragmas -O3 -DNDEBUG -ftree-vectorize Link flags: -mtune=generic OpenMP support: ON MMAP support: ON Build OS: MINGW64_NT-10.0-20348 3.3.5-341.x86_64 x86_64 Build date: Sat, 23 Jul 2022 23:05:43 +0000 UTC Build epoch: 1658617543 Build UUID: 1652971e-17e9-4847-8485-8a739c963e69

PeterMacej avatar Aug 12 '22 20:08 PeterMacej

Additional note. As soon as I do any adjustment to the next image, e.g. rotation, the image is correctly updated in the editor.

PeterMacej avatar Aug 12 '22 20:08 PeterMacej

I can reproduce. I think I also sometimes see this happening with raw images.

However, I just built https://github.com/Beep6581/RawTherapee/commit/be67261d0bdeb21ffe15fe968dd5bc16edd407e0 on an up-to-date MSYS2 build system. This commit is the oldest fully working commit closest to the 5.8 release. I also see the issue there. This makes me think that it is actually not caused by RawTherapee directly, but by some external component, maybe in GTK or GDK.

This requires some testing with older components and/or Linux.

Thanatomanic avatar Aug 13 '22 20:08 Thanatomanic

This requires some testing with older components and/or Linux.

Haven't updated to the latest Debian (11 Bullseye) yet, so:

I just tried replicating this with both 5.8 (official, stable appimage) and 5.8-3172/g2409506c5 (latest self build) on Linux (Debian Buster/10.12) and am not able to reproduce this.


EDIT - Some extra info:

AboutThisBuild info: Version: 5.8-3172-g2409506c5 Branch: dev Commit: 2409506c5 Commit date: 2022-07-25 Compiler: gcc 8.3.0 Processor: x86_64 System: Linux Bit depth: 64 bits Gtkmm: V3.24.0 Lensfun: V0.3.2.0 Build type: release Build flags: -Wno-maybe-uninitialized -std=c++11 -march=native -Werror=unused-label -Werror=delete-incomplete -fno-math-errno -Wno-attributes -flto -Wall -Wuninitialized -Wcast-qual -Wno-deprecated-declarations -Wno-unused-result -Wunused-macros -fopenmp -Werror=unknown-pragmas -O3 -DNDEBUG -ftree-vectorize Link flags: -march=native -flto OpenMP support: ON MMAP support: ON Build OS: Linux 5.9.0-0.bpo.5-amd64 x86_64 Build date: Mon, 25 Jul 2022 13:09:40 +0000 UTC Build epoch: 1658754580

Cmake : 3.13.4 Glib : 2.58.3-2 Gtk+ : 3.24.5-1 Exiv2 : 1.0.0.9 Lensfun : 0.3.2-4

jade-nl avatar Aug 14 '22 07:08 jade-nl

I have notice this some time ago and I can reproduce this on Windows 10.

I can not reproduce this on KDE neon

  • Version: 5.8-3107-gd92f5f7ca
  • Branch: dev
  • Commit: d92f5f7ca
  • Commit date: 2022-07-25
  • Compiler: cc 7.5.0
  • Processor: generic x86
  • System: Linux
  • Bit depth: 64 bits
  • Gtkmm: V3.22.2
  • Lensfun: V0.3.2.0
  • Build type: release
  • Build flags: -std=c++11 -mtune=generic -Werror=unused-label -Werror=delete-incomplete -fno-math-errno -Wno-attributes -Wall -Wuninitialized -Wcast-qual -Wno-deprecated-declarations -Wno-unused-result -Wunused-macros -fopenmp -Werror=unknown-pragmas -O3 -DNDEBUG -ftree-vectorize
  • Link flags: -mtune=generic
  • OpenMP support: ON
  • MMAP support: ON
  • Build OS: Linux 5.4.0-1086-azure x86_64
  • Build date: Sat, 30 Jul 2022 17:28:44 +0000 UTC
  • Build epoch: 1659202124
  • Build UUID: 3b24d80b-b9c3-4296-87d6-59783e6c5a2c

The simple solution: Single click the image in the view window. This 'loads' the new image.

rebio avatar Aug 14 '22 08:08 rebio