D9848: Updated the blur method to use the more efficient dual kawase blur algorithm.

[Alex Nemeth]

anemeth added a comment.
Restricted Application edited projects, added Plasma; removed KWin.


  In https://phabricator.kde.org/D9848#191008, @zzag wrote:
  
  > In https://phabricator.kde.org/D9848#190964, @luebking wrote:
  >
  > > I'm not actually smiling ...
  > >
  > > Anyway, instead of using three vectors it's strongly suggest to use one vector (or rather array) of a struct.
  > >  a) you can access the inner values by names
  > >  b) it's far more elegant to define = { {1, 2, 3}, {4, 5, 6}, ...}
  >
  >
  > c) it's way way elegant to calculate required params(like PixiJS does for determining kawase kernels https://github.com/pixijs/pixi-filters/blob/master/filters/kawase-blur/src/KawaseBlurFilter.js#L84-L100)
  
  
  it's a nice concept but these numbers in question can not be calculated

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D9848

To: anemeth, #plasma, #kwin
Cc: luebking, broulik, romangg, zzag, anthonyfieroni, mart, davidedmundson, fredrik, ngraham, plasma-devel, kwin, #kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180114/d5572e2f/attachment.html>