Summary: | border appears after applying filter | ||
---|---|---|---|
Product: | [Applications] krita | Reporter: | mako_matt <matt.coudert> |
Component: | Filters | Assignee: | Dmitry Kazakov <dimula73> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | ahab.greybeard, dimula73 |
Priority: | NOR | Keywords: | regression |
Version: | nightly build (please specify the git hash!) | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Microsoft Windows | ||
Latest Commit: | https://invent.kde.org/graphics/krita/commit/9a9fe379855679ebe5c6ef7b36a59152640f1347 | Version Fixed In: | |
Sentry Crash Report: | |||
Attachments: |
before filter
After filter test-image |
Description
mako_matt
2021-06-18 09:59:43 UTC
Created attachment 139471 [details]
before filter
Created attachment 139472 [details]
After filter
Testing with the Jun 17 5.0.0-prealpha (git 76b3db3) appimage on Debian 10, I don't see that white line appear when Sharpen is used. There is a 1 px wide effect on the right and bottom edge that gives a very slight increase in brightness which is only noticeable with a very dark image, not with a pure black image. (In reply to Ahab Greybeard from comment #3) > Testing with the Jun 17 5.0.0-prealpha (git 76b3db3) appimage on Debian 10, > I don't see that white line appear when Sharpen is used. > > There is a 1 px wide effect on the right and bottom edge that gives a very > slight increase in brightness which is only noticeable with a very dark > image, not with a pure black image. i add an image to test the issue with. in krita 5 a border appear, in 4.4.5 no borber. I've tested with blur and edge detection filter and there is a border on the bottom and right. Maybe the area for the filter use one pixel out of the canvas. Created attachment 139564 [details]
test-image
Setting to confirmed. With the Sharpen filter, the effect and its visibility depends on the lightness on the image pixels at the edge. At first, I tested with an image that was quite dark at the edges. The effect is greater for lighter coloured pixels. With the Gaussian Blur filter, there is a different effect where the right and bottom edge have a reduction of opacity. How far this extends into the image is proportional to the radius of the blur used. Again, this depends on the lightness of the image and the effect is not noticeable with dark images. This does not happen with 4.4.5, hence regression. Git commit 9a9fe379855679ebe5c6ef7b36a59152640f1347 by Dmitry Kazakov. Committed on 27/07/2021 at 10:18. Pushed by dkazakov into branch 'master'. Fix multiple issues in the new implementation of the filter stroke 1) Apply rect should not be bigger than the size of the image, otherwise bug 438852 appears. 2) When cancelling the filtering to apply a different filter configuration, we should should inhibit the updates to avoid flickering. It became broken after porting to undo-based stretegy (because this strategy uses mutated jobs). 3) When silently cancelling the stroke we should save the updates somewhere externally, so that the next stroke can update this area. Previously, this problem was solved by ExtraCleanUpUpdates, which became broken during the refactoring. Now there is a shared data structure ExternalCancelUpdatesStorage, which passes proper update rects between the strokes. M +8 -3 libs/image/kis_stroke_strategy_undo_command_based.cpp M +1 -1 libs/image/kis_stroke_strategy_undo_command_based.h M +11 -39 libs/ui/kis_filter_manager.cc M +93 -49 libs/ui/tool/strokes/kis_filter_stroke_strategy.cpp M +10 -25 libs/ui/tool/strokes/kis_filter_stroke_strategy.h https://invent.kde.org/graphics/krita/commit/9a9fe379855679ebe5c6ef7b36a59152640f1347 |