Review Request 118517: New brush stabilizer

Dmitry Kazakov dimula73 at gmail.com
Sun Jun 15 11:26:44 BST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118517/#review60123
-----------------------------------------------------------

Ship it!


Thanks, Juan!

I've just pushed the patch to master!

http://commits.kde.org/calligra/58e452c466505aa44c0208bd649d0c8facf0194f

- Dmitry Kazakov


On June 9, 2014, 10:22 a.m., Juan Luis Boya García wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118517/
> -----------------------------------------------------------
> 
> (Updated June 9, 2014, 10:22 a.m.)
> 
> 
> Review request for Calligra, Dmitry Kazakov, Sven Langkamp, and Boudewijn Rempt.
> 
> 
> Repository: calligra
> 
> 
> Description
> -------
> 
> This commit adds to calligra a new stabilizer feature as an alternative
> to the current "Weigthed smoothing".
> 
> This stabilizer is very simple, calculating an arithmetic mean of the
> last N values (position, pressure and tilt) each millisecond and drawing
> a line from the previous point to there.
> 
> This takes into account both the length of the strokes and the speed
> they are made with.
> 
> Fast and small movements are considered erratic and are likely to be
> ignored yielding a mostly straight line.
> 
> On the other hand, slower and bigger movements are assumed to be
> deliberated and will follow the shape the user draws. The slower and
> bigger, the more accurate.
> 
> This process makes the strokes draw 'slowly'. The user sees how the line
> pursues their cursor. The effect is harder or softer depending on the N
> chosen (sample size), which is user-customizable.
> 
> Starting and end points are forced by the algorithm to be drawn exactly
> where the user began and ended the line.
> 
> 
> Diffs
> -----
> 
>   krita/plugins/tools/defaulttools/kis_tool_brush.h ed2d389 
>   krita/plugins/tools/defaulttools/kis_tool_brush.cc 84fede3 
>   krita/ui/tool/kis_smoothing_options.h 14905eb 
>   krita/ui/tool/kis_tool_freehand.h b6bb0fd 
>   krita/ui/tool/kis_tool_freehand_helper.h ee49086 
>   krita/ui/tool/kis_tool_freehand_helper.cpp a1f8eac 
> 
> Diff: https://git.reviewboard.kde.org/r/118517/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Juan Luis Boya García
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20140615/48a2ba93/attachment.htm>


More information about the calligra-devel mailing list