| Summary: | Canvas projection often isn't updated properly after undoing/redoing | ||
|---|---|---|---|
| Product: | [Applications] krita | Reporter: | tomtomtomreportingin |
| Component: | Layer Stack | Assignee: | Dmitry Kazakov <dimula73> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | halla |
| Priority: | NOR | Keywords: | regression |
| Version: | nightly build (please specify the git hash!) | ||
| Target Milestone: | --- | ||
| Platform: | Appimage | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/graphics/krita/commit/0c85656e28a97b7113733b042e7a26dea12f5614 | Version Fixed In: | |
| Attachments: | Cutoff strokes are undoing/redoing | ||
Still occurs in 5.1 git 82766ef. Sometimes occurs even without the use of undo/redo. Marking as a regression. I don't think it actually is a regression, though... Can't say I ever recall noticing this before I started testing 5.1. On my end, it happens basically every time I draw, and sometimes messes up my perception of the artwork, leading me to undo stuff I shouldn't have. Git commit 0617bb063a777e05aaf8e506f287eafa133a897c by Dmitry Kazakov. Committed on 11/07/2022 at 14:23. Pushed by dkazakov into branch 'master'. Fix random artifacts when undoing brush strokes It looks like making indirect painting merge multithreaded revealed a very old race condition in the tiles data manager. The extent of the memento should be calculated in a synchronous way. M +16 -5 libs/image/tiles3/kis_memento.h M +6 -4 libs/image/tiles3/kis_memento_manager.cc M +1 -0 libs/image/tiles3/kis_memento_manager.h https://invent.kde.org/graphics/krita/commit/0617bb063a777e05aaf8e506f287eafa133a897c Git commit 0c85656e28a97b7113733b042e7a26dea12f5614 by Dmitry Kazakov. Committed on 11/07/2022 at 14:23. Pushed by dkazakov into branch 'krita/5.1'. Fix random artifacts when undoing brush strokes It looks like making indirect painting merge multithreaded revealed a very old race condition in the tiles data manager. The extent of the memento should be calculated in a synchronous way. M +16 -5 libs/image/tiles3/kis_memento.h M +6 -4 libs/image/tiles3/kis_memento_manager.cc M +1 -0 libs/image/tiles3/kis_memento_manager.h https://invent.kde.org/graphics/krita/commit/0c85656e28a97b7113733b042e7a26dea12f5614 Hi, Tomtomtom! I think I have fixed the issue. The fix should be present in this build I just kicked off. Please check and report if it is really fixed :) https://binary-factory.kde.org/job/Krita_Stable_Appimage_Build/1551/ Seems like it's been resolved, the issue no longer occurs from what I can tell. Feels nice to be able to draw again without the various artifacts, thanks for the fix. You are welcome! :) |
Created attachment 148482 [details] Cutoff strokes are undoing/redoing git f866b153a4 SUMMARY In Krita 5.1, the canvas visual projection often isn't updated properly after undoing or redoing, leaving cutoff strokes. See attachment. This happens regardless of using the OpenGL canvas. This is not always reproducible. STEPS TO REPRODUCE 1. Draw a series of lines around the same spot, in 100% zoom 2. Undo lines OBSERVED RESULT The projection of the lines are often cutoff. EXPECTED RESULT The projection should accurately reflect the strokes after undo/redo. SOFTWARE/OS VERSIONS Linux/KDE Plasma: Debian sid KDE Plasma Version: 5.24.4 KDE Frameworks Version: 5.90.0 Qt Version: 5.12.12 (appimage)