[krita] [Bug 411457] Change saturation for rerference images almost freezes Krita
https://bugs.kde.org/show_bug.cgi?id=411457 wolthera changed: What|Removed |Added Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap |hics/krita/commit/63689a3ef |hics/krita/commit/a76799863 |a04a0757030b95c16e6d0c479e3 |44a2e47b4d334d09a923e4a5e44 |b7df|d012 --- Comment #5 from wolthera --- Git commit a7679986344a2e47b4d334d09a923e4a5e44d012 by Wolthera van Hövell tot Westerflier. Committed on 06/08/2020 at 17:36. Pushed by woltherav into branch 'krita/4.3'. Add signal compressor in the reference image tool to prevent freezes. The saturation slider takes up quite a bit of computation time, so we use the techniqu of using a signal compressor to reduce the amount of changes being calculated. M +13 -2 plugins/tools/defaulttool/referenceimagestool/ToolReferenceImagesWidget.cpp M +2-0 plugins/tools/defaulttool/referenceimagestool/ToolReferenceImagesWidget.h https://invent.kde.org/graphics/krita/commit/a7679986344a2e47b4d334d09a923e4a5e44d012 -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 411457] Change saturation for rerference images almost freezes Krita
https://bugs.kde.org/show_bug.cgi?id=411457 wolthera changed: What|Removed |Added Resolution|--- |FIXED Status|CONFIRMED |RESOLVED Latest Commit||https://invent.kde.org/grap ||hics/krita/commit/63689a3ef ||a04a0757030b95c16e6d0c479e3 ||b7df --- Comment #4 from wolthera --- Git commit 63689a3efa04a0757030b95c16e6d0c479e3b7df by Wolthera van Hövell tot Westerflier. Committed on 06/08/2020 at 17:34. Pushed by woltherav into branch 'master'. Add signal compressor in the reference image tool to prevent freezes. The saturation slider takes up quite a bit of computation time, so we use the techniqu of using a signal compressor to reduce the amount of changes being calculated. M +13 -2 plugins/tools/defaulttool/referenceimagestool/ToolReferenceImagesWidget.cpp M +2-0 plugins/tools/defaulttool/referenceimagestool/ToolReferenceImagesWidget.h https://invent.kde.org/graphics/krita/commit/63689a3efa04a0757030b95c16e6d0c479e3b7df -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 411457] Change saturation for rerference images almost freezes Krita
https://bugs.kde.org/show_bug.cgi?id=411457 wolthera changed: What|Removed |Added Ever confirmed|0 |1 Status|REPORTED|CONFIRMED CC||griffinval...@gmail.com --- Comment #3 from wolthera --- I can confirm here with an A4 sized reference image. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 411457] Change saturation for rerference images almost freezes Krita
https://bugs.kde.org/show_bug.cgi?id=411457 Karl Ove Hufthammer changed: What|Removed |Added Status|NEEDSINFO |REPORTED Resolution|WAITINGFORINFO |--- --- Comment #2 from Karl Ove Hufthammer --- The size of the reference image seems to matter most. For small reference images, everything works fine. For large ones (either rescaled in Krita or shown at the original size), Krita slows to a crawl. Here’s a (very large) example image (tested on a 1600 × 1200 document): https://unsplash.com/photos/MN251xZ9mBk/download?force=true Changing the opacity is near instant, but changing the saturation is *extremely* slow. And when you drag the slider left and right a few times (without letting go of the mouse button), Krita seems to want to calculate all intermediate saturation values, not just the final one (?), and basically freezes for a *long* time. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 411457] Change saturation for rerference images almost freezes Krita
https://bugs.kde.org/show_bug.cgi?id=411457 Boudewijn Rempt changed: What|Removed |Added Status|REPORTED|NEEDSINFO Resolution|--- |WAITINGFORINFO CC||b...@valdyas.org --- Comment #1 from Boudewijn Rempt --- Hm... What's the size of the image and of ther eference images? I tried on a 1600x1200 image with three reference images, and everything was pretty much instant. -- You are receiving this mail because: You are watching all bug changes.