Bug 393336 - Brush rotation using tablet pen's rotation capped at 180 degrees.
Summary: Brush rotation using tablet pen's rotation capped at 180 degrees.
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Brush Engine/Shape (show other bugs)
Version: 4.2.7-beta1
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Dmitry Kazakov
URL:
Keywords: needs_verification, triaged
Depends on:
Blocks:
 
Reported: 2018-04-20 12:44 UTC by rebuilderster
Modified: 2020-03-30 21:34 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description rebuilderster 2018-04-20 12:44:51 UTC
I'm using a Cintiq 24HD with an Art Pen.

Since 4.0.1, brush rotation has broken a bit. I have  brush with the rotation controlled by the pen's rotation. Using an asymmetrical brush, I notice that: If I try to rotate the pen 180 degrees clockwise, the rotation stops at 180 degrees and the brush orientation is reset to 0 degrees. If I try to rotate the brush past 0 degrees counterclockwise, the brush does not rotate.

This seems to be new behaviour as the brush I tested works in 4.0
Comment 1 Dmitry Kazakov 2018-09-21 07:57:17 UTC
Hi, rebuildester!

I have tested the thing on Intuos5 tablet with ArtPen and it seems to work. You have this trouble on Linux, right?
Comment 2 Halla Rempt 2019-05-20 13:10:05 UTC
Setting to needsinfo
Comment 3 Bug Janitor Service 2019-06-04 04:33:11 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 4 fbampaloukas 2019-06-16 14:09:17 UTC
Closing as Worksforme due to inactivity for more than 15 days as per:

https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging#Policies

Fanis
Comment 5 Halla Rempt 2019-06-16 19:30:14 UTC
Fanis, 

You don't need to do that manually, we've got the Bug Janitor script for that.
Comment 6 rebuilderster 2019-09-16 13:58:24 UTC
I'm sorry to say this bug is back as of 4.2.6. I'm now on Windows 10, and have the same issue I originally described. On 4.2.5 there is no problem.
Comment 7 Oscar Franco 2019-09-20 03:14:31 UTC
I can confirm the bug on 4.2.6, using cintiq HD13 and intuos4 small with a Wacom Art Pen on Windows.
Comment 8 Oscar Franco 2019-09-28 14:38:05 UTC
pen rotation now works on 4.2.7.beta1, but only for rotation, if I'd want to map pen rotation to other attributes it won't work. Tested it with size and opacity.
Comment 9 Dmitry Kazakov 2019-12-09 13:42:43 UTC
Discussion of the bug at krita-artists:
https://krita-artists.org/t/weird-problem-with-pen-rotation/1682
Comment 10 Dmitry Kazakov 2020-03-10 17:49:35 UTC
Yes, I can reproduce
Comment 11 Dmitry Kazakov 2020-03-13 17:50:33 UTC
The problem is going to be fixed in this MR:
https://invent.kde.org/kde/krita/-/merge_requests/265
Comment 12 Dmitry Kazakov 2020-03-20 07:02:19 UTC
Git commit 9675ed2c2077b73fb7fd049fbc264920f84aa24c by Dmitry Kazakov.
Committed on 19/03/2020 at 21:29.
Pushed by rempt into branch 'krita/4.2'.

Make rotation sensor be "additive"

With new curve scaling methods, using rotation
as a "scaling sensor" doesn't work anymore.

M  +6    -1    plugins/paintops/libpaintop/sensors/kis_dynamic_sensors.h

https://invent.kde.org/kde/krita/commit/9675ed2c2077b73fb7fd049fbc264920f84aa24c
Comment 13 Dmitry Kazakov 2020-03-30 21:34:38 UTC
Git commit 22f0f0e9da077459814ee953e85ed5009d32f03c by Dmitry Kazakov.
Committed on 10/03/2020 at 21:01.
Pushed by dkazakov into branch 'master'.

Make rotation sensor be "additive"

With new curve scaling methods, using rotation
as a "scaling sensor" doesn't work anymore.

M  +6    -1    plugins/paintops/libpaintop/sensors/kis_dynamic_sensors.h

https://invent.kde.org/kde/krita/commit/22f0f0e9da077459814ee953e85ed5009d32f03c