Review Request 117816: Use marching cubes algorithm from Aqsis

[Percy Camilo Triveño Aucahuasi]

> On April 28, 2014, 9:47 a.m., Aleix Pol Gonzalez wrote:
> > analitzaplot/private/backends/implicitsurface.cpp, line 73
> > <https://git.reviewboard.kde.org/r/117816/diff/1/?file=268860#file268860line73>
> >
> >     Is this relevant?

Yes it is, the initial cube is [-6,6]x[-6,6]x[-6,6] so if the user doesn't specify intervals, then we can see more parts of the surface.


- Percy Camilo


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


On April 28, 2014, 6:18 a.m., Percy Camilo Triveño Aucahuasi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117816/
> -----------------------------------------------------------
> 
> (Updated April 28, 2014, 6:18 a.m.)
> 
> 
> Review request for KDE Edu.
> 
> 
> Repository: analitza
> 
> 
> Description
> -------
> 
> Replace our marching cubes algorithm implementation for the one of Aqsis project. The Aqsis implementation is fastest and offers topological guarantees, in other words we'll have better surfaces as result (without missing faces, etc.)
> 
> Finally, we'll have less code to maintain :)
> 
> 
> Diffs
> -----
> 
>   analitzaplot/private/utils/octree.cpp 6106120 
>   analitzaplot/private/utils/octree.h 1cf4b24 
>   analitzaplot/private/utils/marchingcubes.cpp 49331f1 
>   analitzaplot/private/utils/marchingcubes.h ef0b0c4 
>   analitzaplot/private/utils/lookuptable.h PRE-CREATION 
>   analitzaplot/private/backends/implicitsurface.cpp 68c7918 
>   analitzaplot/CMakeLists.txt 9890449 
> 
> Diff: https://git.reviewboard.kde.org/r/117816/diff/
> 
> 
> Testing
> -------
> 
> Builds ok.
> All analitzaplot tests pass.
> Also, there is no licenses issues since the code is GPL >= 2 (same of Analitza)
> 
> 
> Thanks,
> 
> Percy Camilo Triveño Aucahuasi
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20140428/4aaa8a68/attachment-0001.html>