Bug 331192 - using multiple sensors on a value divide the result value by number of active sensors
Summary: using multiple sensors on a value divide the result value by number of active...
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Brush engines (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-16 16:07 UTC by animtim
Modified: 2014-02-17 12:57 UTC (History)
1 user (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 animtim 2014-02-16 16:07:49 UTC
Since the recent multi-sensors fix, now when you activate multiple sensors on a value, the result is always divided by the number of active sensors.

So I guess the solution is to multiply this output value by the number of active sensors.

Reproducible: Always

Steps to Reproduce:
1.take a preset with size = 100px
2.activate Pressure and another sensor (like declination for example..), and set the curves of both to always 100%/max value
3.paint with the preset: the line has 50px width
Comment 1 Halla Rempt 2014-02-17 12:56:12 UTC
well, then we don't have to divide it :-). I'll roll back the commit that added averaging.
Comment 2 Halla Rempt 2014-02-17 12:57:01 UTC
Git commit 13f05d90b1d9db7c1840b933a38b91ee659f507f by Boudewijn Rempt.
Committed on 17/02/2014 at 12:56.
Pushed by rempt into branch 'calligra/2.8'.

M  +0    -4    krita/plugins/paintops/libpaintop/kis_curve_option.cpp

http://commits.kde.org/calligra/13f05d90b1d9db7c1840b933a38b91ee659f507f
Comment 3 Halla Rempt 2014-02-17 12:57:12 UTC
Git commit e7757fb96dea100c996e8fb98bbbe2032acc2492 by Boudewijn Rempt.
Committed on 17/02/2014 at 12:56.
Pushed by rempt into branch 'master'.

M  +0    -4    krita/plugins/paintops/libpaintop/kis_curve_option.cpp

http://commits.kde.org/calligra/e7757fb96dea100c996e8fb98bbbe2032acc2492