Bug 331192

Summary:	using multiple sensors on a value divide the result value by number of active sensors
Product:	[Applications] krita	Reporter:	animtim
Component:	Brush engines	Assignee:	Krita Bugs <krita-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	halla
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:	http://commits.kde.org/calligra/e7757fb96dea100c996e8fb98bbbe2032acc2492	Version Fixed In:
Sentry Crash Report:

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