it seems the Airbrush isnt recognised and the wheel is mapped to the rotation sensor instead of the tangential pressure sensor. Reproducible: Always
Created attachment 90801 [details] diff file for kis_tablet_support_x11.cpp the if block on line 633 doesnt seem to get executed and even if i make it run you get the behaivior explained in the above link. you cant just assign the rotation value to the tangentialPressure value as the tangential pressure curve expects a range from 0.0 - 1.0 and not a range from 0 - 360. added a workaround and converted it to a 0.0 - 1.0 range.
about the airbrush detection, it might not be a problem with krita directly the kde tablet settings gui and a qt tablet example i tried didnt recognise it eighter. the gnome Wacom Tablet settings gui and gimp recognise it.
Created attachment 90849 [details] Patch for updating Tangential Pressure in Hover Mode
Hi, varspoolmail! Should we apply and use both patches, right?
yes works fine for me so far
Ok, I found out how to recognize this Airbrush. Will make a patch tomorrow.
Git commit 142b953c4d0f6b197435cfdcf424f88796e57c4c by Dmitry Kazakov. Committed on 03/02/2015 at 10:18. Pushed by dkazakov into branch 'calligra/2.9'. This patch should probably fix Wacom Airbrush devices This patch adds recognition of the device type on Linux, and tries to fix the tangential pressure parameter. If you happen to have an Airbrush device, please do test whether tangential pressure starts working correctly. Most probable bug: the range of the tangential pressure might be wrong, since I don't know in which limits it is linked in the driver. If you see any problems with it, please generate an event log file for moving the button in all directions slowly. Here is a manual how to do that: https://answers.launchpad.net/krita-ru/+faq/2495 Thanks arturg for providing initial patches! CCMAIL:kimageshop@kde.org M +61 -9 krita/ui/input/wintab/kis_tablet_support_x11.cpp M +1 -1 krita/ui/tool/kis_painting_information_builder.cpp http://commits.kde.org/calligra/142b953c4d0f6b197435cfdcf424f88796e57c4c
Git commit b6fc8a7ff9d9941ad811710f8139bb51b0d1edf6 by Dmitry Kazakov. Committed on 03/02/2015 at 13:00. Pushed by dkazakov into branch 'calligra/2.9'. Fix recognizing Bamboo stylus I'm wondering whether these magic values were ever documented by anyone... M +29 -22 krita/ui/input/wintab/kis_tablet_support_x11.cpp http://commits.kde.org/calligra/b6fc8a7ff9d9941ad811710f8139bb51b0d1edf6
Created attachment 90907 [details] krita tablet debuger log the Airbrush detection works but the normalization doesnt i tried that before i submited my patch, it results in the same behavior as explained in the forum post but now on the tp value. check the attached log Rotation: -900 899 im rotating the wheel from the neutral position 0.5 to the max position 0.5 begining on line: 1608 neutral position: 0.5 line: 2204 midwheel position: 0.0 line: 2732 maxwheel position: 0.5 then the reverse on line: 3474 maxwheel postion: 0.5 line: 4335 midwheel position: 0.0 line: 5099 neutralwheel position: 0.5
Created attachment 90908 [details] patched log replacing it with the following produces the attached log if (deviceType == QTabletEvent::RotationStylus) { rotation = std::fmod(qreal(tablet->savedAxesData.rotation() - tablet->minRotation) / (tablet->maxRotation - tablet->minRotation) + 0.5, 1.0) * 360; } else if (deviceType == QTabletEvent::Airbrush) { tangentialPressure = std::fmod(qreal(tablet->savedAxesData.rotation() - tablet->minRotation) / (tablet->maxRotation - tablet->minRotation) , 2.0); } else { rotation = 0; } while doing the same rotating from neutral to max position and then the reverse
Git commit d2d88be988483c7db437bee072458ba0c73ed9db by Dmitry Kazakov. Committed on 06/02/2015 at 06:47. Pushed by dkazakov into branch 'calligra/2.9'. Fix the formula for calculation of tangential pressure for airbrush stylus Thanks arturg for provided patch! (I changed the patch a bit. The rotation is *not* zero'ed for all non-RotationStylus devices, because our stylus type recognition code is not mature enough, or, more precisely, Wacom device ids are not documented enough) M +6 -6 krita/ui/input/wintab/kis_tablet_support_x11.cpp http://commits.kde.org/calligra/d2d88be988483c7db437bee072458ba0c73ed9db
Hi, var.spool.mail700! Could you please test updated version? I applied your patch almost exactly, with exception that we don't reset rotation to zero for other stylus types, since the stylus type recognition code is a bit weak, since device ids are not documented enough.
works fine for me thanks.
Git commit 75c0e838fafb76e3f3b1ffe6435af9e93c9f19e7 by Boudewijn Rempt, on behalf of Dmitry Kazakov. Committed on 06/02/2015 at 06:47. Pushed by rempt into branch 'master'. Fix the formula for calculation of tangential pressure for airbrush stylus Thanks arturg for provided patch! (I changed the patch a bit. The rotation is *not* zero'ed for all non-RotationStylus devices, because our stylus type recognition code is not mature enough, or, more precisely, Wacom device ids are not documented enough) M +6 -6 krita/ui/input/wintab/kis_tablet_support_x11.cpp http://commits.kde.org/calligra/75c0e838fafb76e3f3b1ffe6435af9e93c9f19e7