Summary: | Animation curves: Inserting new keyframe from "insert button" is not working properly | ||
---|---|---|---|
Product: | [Applications] krita | Reporter: | grum999 |
Component: | Animation | Assignee: | Krita Bugs <krita-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | ahab.greybeard, eoinoneill1991 |
Priority: | NOR | ||
Version: | nightly build (please specify the git hash!) | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/graphics/krita/commit/e29bc450163151a57a20fa3b245693a2cd9618ec | Version Fixed In: |
Description
grum999
2021-06-19 15:35:07 UTC
I can confirm this for the Jun 21 5.0.0-prealpha (git 778ddae) appimage on Debian 10. The new keyframe has a value that is forced to the initial parameter value, whatever that may be, not a properly interpolated value on the existing curve. If the user has set up several parameter curves than this is very inconvenient because they all then have to be manually 'corrected'. *** Bug 438342 has been marked as a duplicate of this bug. *** Git commit e29bc450163151a57a20fa3b245693a2cd9618ec by Eoin O'Neill. Committed on 08/07/2021 at 21:14. Pushed by eoinoneill into branch 'master'. New scalar keyframes between two keyframes should now self assign to interpolated value. More work can be done here in the future for getting proper interpolation "handles" based on the previous key's interpolation handle data -- but that will require more work. For now, it at least adds the key to the correct value instead of duplicating last keys value. M +2 -2 plugins/dockers/animation/KisAnimCurvesDocker.cpp M +9 -1 plugins/dockers/animation/KisAnimUtils.cpp https://invent.kde.org/graphics/krita/commit/e29bc450163151a57a20fa3b245693a2cd9618ec Hi When I click on button, Krita crash (krita-5.0.0-prealpha-13f04ca-x86_64.appimage) with message: ASSERT (krita): "hash != KeyTraits::NullHash" in file /home/appimage/workspace/Krita_Nightly_Appimage_Build/krita/libs/image/3rdparty/lock_free_map/leapfrog.h, line 161 Not sure about origin of problem (this commit or stuff made by Dmitry about memory?) but as it crash I reopen bug More information here: https://krita-artists.org/t/krita-slowing-down-as-time-passes-after-upgrading-to-4-4-5/25443/34?u=grum999 Grum999 Grumm, Is this happening on that same test file provided on one of your other bug reports? Or is this a new file where this issue is happening. FWIW my change should not be affecting memory in any major way, but I will investigate some of the commits between the revisions you've mentioned and see if I can isolate a cause. Hi Just test last available appimage (krita-5.0.0-prealpha-627782b-x86_64.appimage) and for me it's ok: - Original problem (wrong values) is fixed - No more crash when "insert button" is clicked Also I can confirm this fix the https://bugs.kde.org/show_bug.cgi?id=438342 I change bug to RESOLVED/FIXED Grum999 |