Fix board spherical

[dcoeurjo]

Bugfix in quad visualization in BoardD3D and Viewer3D

[coveralls]

Coverage decreased (-0.02%) to 90.35% when pulling 0e53264 on dcoeurjo:FixBoardSpherical into ac65f35 on DGtal-team:master.

[kerautret]

@dcoeurjo From just a test it looks that exported normals are not always near surface exterior.
I tested it from the 3DCurvatureViewer PR of Jeremy with Al.vol...

[dcoeurjo]

Sorry but I don't understand what you mean

[kerautret]

I don't know if it is related to your edit but in the export of voxel primitive the face order should be in inverted in some direction see:

[dcoeurjo]

which tool generates this vector field ?

[kerautret]

meshlab...

[dcoeurjo]

well.. I don't know.. elementary normal vectors to quad is something highly dependent on how you create the quads.. Can you please provide an example with a single cube ?

[kerautret]

Yes here is what we obtain from obj of a single voxel centered in 0: (the sequence is perhaps not good)

OBJ format

generated from Board3D from the DGTal library

mtllib test.mtl

o 0
usemtl Mat_0
v -0.5 -0.5 0.5
v 0.5 -0.5 0.5
v -0.5 -0.5 -0.5
v 0.5 -0.5 -0.5
v -0.5 0.5 0.5
v 0.5 0.5 0.5
v -0.5 0.5 -0.5
v 0.5 0.5 -0.5
vn -1 0 0
vn 0 0 -1
vn 0 1 0
vn 0 0 1
vn 0 -1 0
vn 1 0 0
f -8//-6 -7//-6 -5//-6 -6//-6
f -8//-5 -6//-5 -2//-5 -4//-5
f -8//-4 -7//-4 -3//-4 -4//-4
f -7//-3 -5//-3 -1//-3 -3//-3
f -6//-2 -5//-2 -1//-2 -2//-2
f -4//-1 -3//-1 -1//-1 -2//-1

[dcoeurjo]

Did you send a cube or a digital surface surfel set to the board3d ?

[kerautret]

a digital point with mode paving:
aBoard << SetMode3D("PointVector", "Paving");
aBoard << Point(0,0,0);
aBoard.saveOBJ("test.obj");

[dcoeurjo]

Ok so it has nothing to do with my PR (I only change things in Quad drawing)

[kerautret]

Yes, the pb looks to be present in the master, we perhaps have miss it in previous PR...

[kerautret]

I add an issue on that.

[kerautret]

could you update from master for merging ?

[dcoeurjo]

Done

[kerautret]

thanks , I add an issue, sorry for the false bug, waiting for travis and merging

[kerautret]

travis looks too long for some config, merging