Review Request 116611: Eigen3 support

Cyrille Berger Skott cberger at cberger.net
Thu Mar 13 10:32:24 GMT 2014


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


Have not tested, but reading the diff, it should be ok, except for the following two comments:


krita/plugins/paintops/libbrush/CMakeLists.txt
<https://git.reviewboard.kde.org/r/116611/#comment37227>

    this look strange, I guess it was intented to be EIGEN2_SUPPORT (there is no EIGEN3_SUPPORT). That said, I would recommend not using EIGEN2_SUPPORT anyway, since there is talk of dropping it in the near future. And since the line is likely wrong, I would suggest to try to remove it.



sheets/functions/math.cpp
<https://git.reviewboard.kde.org/r/116611/#comment37228>

    This line is wrong, it should be eMatrixInverse = eMatrix.inverse();
    
    and line 1231 with the FullPivLU can be removed.


- Cyrille Berger Skott


On March 5, 2014, 12:36 p.m., Maximiliano Curia wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116611/
> -----------------------------------------------------------
> 
> (Updated March 5, 2014, 12:36 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Repository: calligra
> 
> 
> Description
> -------
> 
> Hi,
> 
> In Debian eigen2 was removed from the archive, so, this is the patch we are using to support eigen3. The pathch was originaly developed by Anton Gladky, and later modified by the debian calligra maintainers.
> 
> Please consider adding it as part of the official calligra.
> 
> Happy hacking,
> 
> 
> Diffs
> -----
> 
>   krita/plugins/tools/selectiontools/kis_tool_select_magnetic.h d69c6b82845b00bc6e540d5daccafcc9850f4dc5 
>   krita/plugins/tools/selectiontools/kis_tool_select_brush.cc 08cd4efdc071145485abd511885546a63c44390e 
>   krita/plugins/paintops/libbrush/CMakeLists.txt ac9cd72146eebefe58d20896b63c7d221460d288 
>   krita/plugins/extensions/dockers/advancedcolorselector/kis_color_selector_triangle.cpp f2889b866c3d733f8fbc3ac5c93ba94061099b2c 
>   krita/plugins/extensions/dockers/advancedcolorselector/kis_color_selector_ring.cpp 5f88701aea3eddcf4d3deb5059f91d12cb22fd71 
>   krita/image/tests/kis_paint_information_test.cpp 188103c29add9bb706134ff75ae9e260af5b0de0 
>   krita/image/tests/kis_convolution_painter_test.cpp d6562dd3aef886d4173ee4f2c18692fc0ae13f82 
>   krita/image/kis_perspective_math.cpp 74566e205244e92627cf84adc1360204028ddeba 
>   krita/CMakeLists.txt b81e9f41c78291b284e04a613b00ba4d1c8a5191 
>   cmake/modules/FindEigen3.cmake PRE-CREATION 
>   CMakeLists.txt f3a193c20c83fe8df737be838d23d9f5247c997c 
>   sheets/CMakeLists.txt 04f213f72a06c32f790c86091d97e19900a70a91 
>   sheets/functions/CMakeLists.txt ba20613be6ca0c8516f19a6bcb9da861370b098b 
>   sheets/functions/math.cpp 6b475f5dbc4bac22b2f6930ec25e08b302e6abe2 
>   sheets/tests/CMakeLists.txt 0d1d37c09d0415c10133cc5c27dad9b3e9693eb1 
> 
> Diff: https://git.reviewboard.kde.org/r/116611/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Maximiliano Curia
> 
>

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


More information about the calligra-devel mailing list