[gwenview] [Bug 463132] Gwenview Removed Support for All RAW Files (Regression)
https://bugs.kde.org/show_bug.cgi?id=463132 Martin Kyral changed: What|Removed |Added CC||sine.nom...@centrum.cz --- Comment #7 from Martin Kyral --- I have no idea why Qt Raw plugin is being preferred over KDCraw, but the current implementation bring serious feature regression: while KDCraw implementation extracts insplit second already generated (in camera) preview jpeg from the raw file, the Qt Raw plugin implementation generates the jpeg data from raw, which takes considerable time even on pretty fast HW (on 10 core i7 ~ 7s). The same applies even to the thumbnailing. Imagine a folder containing some 500 raw images and you can do the math yourself. This change renders raw support in gwenview virtually unusable. I'd suggest either fixing the Qt Raw plugin implementation so it utilizes the embedded hi-res preview or preferring kdcraw over qt raw plugin. https://bugs.kde.org/show_bug.cgi?id=454206 -- You are receiving this mail because: You are watching all bug changes.
[gwenview] [Bug 463132] Gwenview Removed Support for All RAW Files (Regression)
https://bugs.kde.org/show_bug.cgi?id=463132 Nate Graham changed: What|Removed |Added Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap |hics/gwenview/commit/a7dc8f |hics/gwenview/commit/c419ec |eefc2147a08526b61bf361b63f7 |4a052b39eae7c17caf2a89d39d7 |8645ea0 |6b43777 --- Comment #6 from Nate Graham --- Git commit c419ec4a052b39eae7c17caf2a89d39d76b43777 by Nate Graham, on behalf of Mirco Miranda. Committed on 10/01/2023 at 14:59. Pushed by ngraham into branch 'cherry-pick-a7dc8fee'. Fixed coexistence between Qt Raw plugin and KDcraw Use the RAW plugin to load raw images if installed: - Previews are always generated from the RAW image at the fastest possible speed. - Images are loaded in the highest possible quality (default plugin raw). - If the raw plugin is not installed everything works as before (use KDcraw). FIXED-IN: 22.12.2 (cherry picked from commit a7dc8feefc2147a08526b61bf361b63f78645ea0) M +2-1lib/document/loadingdocumentimpl.cpp M +7-1lib/thumbnailprovider/thumbnailgenerator.cpp https://invent.kde.org/graphics/gwenview/commit/c419ec4a052b39eae7c17caf2a89d39d76b43777 -- You are receiving this mail because: You are watching all bug changes.
[gwenview] [Bug 463132] Gwenview Removed Support for All RAW Files (Regression)
https://bugs.kde.org/show_bug.cgi?id=463132 Nate Graham changed: What|Removed |Added Status|ASSIGNED|RESOLVED Version Fixed In||22.12.2 Resolution|--- |FIXED Latest Commit||https://invent.kde.org/grap ||hics/gwenview/commit/a7dc8f ||eefc2147a08526b61bf361b63f7 ||8645ea0 --- Comment #5 from Nate Graham --- Git commit a7dc8feefc2147a08526b61bf361b63f78645ea0 by Nate Graham, on behalf of Mirco Miranda. Committed on 10/01/2023 at 14:53. Pushed by ngraham into branch 'master'. Fixed coexistence between Qt Raw plugin and KDcraw Use the RAW plugin to load raw images if installed: - Previews are always generated from the RAW image at the fastest possible speed. - Images are loaded in the highest possible quality (default plugin raw). - If the raw plugin is not installed everything works as before (use KDcraw). FIXED-IN: 22.12.2 M +2-1lib/document/loadingdocumentimpl.cpp M +7-1lib/thumbnailprovider/thumbnailgenerator.cpp https://invent.kde.org/graphics/gwenview/commit/a7dc8feefc2147a08526b61bf361b63f78645ea0 -- You are receiving this mail because: You are watching all bug changes.
[gwenview] [Bug 463132] Gwenview Removed Support for All RAW Files (Regression)
https://bugs.kde.org/show_bug.cgi?id=463132 Nate Graham changed: What|Removed |Added CC||n...@kde.org Status|REPORTED|ASSIGNED Ever confirmed|0 |1 -- You are receiving this mail because: You are watching all bug changes.
[gwenview] [Bug 463132] Gwenview Removed Support for All RAW Files (Regression)
https://bugs.kde.org/show_bug.cgi?id=463132 --- Comment #4 from Mirco Miranda --- This is a possible fix: https://invent.kde.org/graphics/gwenview/-/merge_requests/174 -- You are receiving this mail because: You are watching all bug changes.
[gwenview] [Bug 463132] Gwenview Removed Support for All RAW Files (Regression)
https://bugs.kde.org/show_bug.cgi?id=463132 Marian Kyral changed: What|Removed |Added CC||mky...@email.cz --- Comment #3 from Marian Kyral --- I have the same issue. RAW files works correctly in latest gwenview with kimageformats-5.98. But after update kimageformats to version 5.99+, RAW images fails to load. -- You are receiving this mail because: You are watching all bug changes.
[gwenview] [Bug 463132] Gwenview Removed Support for All RAW Files (Regression)
https://bugs.kde.org/show_bug.cgi?id=463132 Koshika changed: What|Removed |Added CC||koshi...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[gwenview] [Bug 463132] Gwenview Removed Support for All RAW Files (Regression)
https://bugs.kde.org/show_bug.cgi?id=463132 --- Comment #2 from Mirco Miranda --- (In reply to Nicolas Fella from comment #1) > Mirco, you worked a lot on RAW stuff, any idea what's going on here? If I disable libkdcraw support and recompile Gwenview, the raw files are displayed correctly. Gwenview treats RAW files differently from other images assuming there is no plugin capable of reading them. There is probably a logical conflict within the code due to the presence of the plugin. The solutions can be the following: 1- disable KDCraw search from CMakeList of Gwenview (find_package(KF5KDcraw)) 2- patch Gwenview to contemplate the presence of the plug-in Solution 1 makes loading RAW files slow (plugin by default looks for maximum quality). To speed up the loading I added a hack to the plugin: by setting the quality to 1 in QImageReader, it does more or less what KDCraw does and makes loading much faster (sa: https://invent.kde.org/frameworks/kimageformats/-/blob/master/src/imageformats/raw_p.h). So, the solution 2 could be a) check if raw plugin is installed: set quality on raw file loading and bypass the KDRaw code or b) use the current code an fix it to coexist with raw plugin (when installed). I'm not familiar with Gwenview code, so I should do a deep debugging. Solution a) could be solved with a few lines of code. -- You are receiving this mail because: You are watching all bug changes.
[gwenview] [Bug 463132] Gwenview Removed Support for All RAW Files (Regression)
https://bugs.kde.org/show_bug.cgi?id=463132 Nicolas Fella changed: What|Removed |Added CC||mirco...@gmail.com, ||nicolas.fe...@gmx.de --- Comment #1 from Nicolas Fella --- Mirco, you worked a lot on RAW stuff, any idea what's going on here? -- You are receiving this mail because: You are watching all bug changes.
