open-discussion > Possible bug in landmark drawing
Showing 1-3 of 3 posts
Display:
Results per page:
Oct 13, 2011  07:10 AM | Carlos Becker
Possible bug in landmark drawing
I've just realized that if I load a set of landmarks that are not spherical ones, V3D still draws them as spherical landmarks.
I think that the source of the problem is in v3d_main.cpp, namely at:


switch(tmpLocation.howUseful())

{

case pxLocaUseful:

curColor = (b_color==1) ? QColor(255, 0, 0, alpha) : QColor(255, 255, 255, alpha);

curShape = pxSphere; //pxCircle;

b_draw=1;

break;


case pxLocaNotUseful:

curColor = (b_color==1) ? QColor(0, 255, 0, alpha) : QColor(255, 255, 255, alpha);

curShape = pxCube; //pxRect;

b_draw=1;

break;


case pxLocaUnsure:

curColor = (b_color==1) ? QColor(0, 0, 255, alpha) : QColor(255, 255, 255, alpha);

curShape = pxTriangle;

b_draw=1;

break;


case pxTemp: //080405

curColor = (b_color==1) ? QColor(155, 155, 0, alpha) : QColor(255, 255, 255, alpha);

curShape = pxDot;

b_draw=1;

break;


default:

b_draw=0;

break;

}


Should this be removed and instead do curShape = tmpLocation.shape?
If not, what is the overall meaning?

Thanks.
Oct 13, 2011  12:10 PM | Hanchuan Peng
RE: Possible bug in landmark drawing
It seems you are right on it. It may be an unfixed temporary test case. -hanchuan

Originally posted by Carlos Becker:
I've just realized that if I load a set of landmarks that are not spherical ones, V3D still draws them as spherical landmarks.
I think that the source of the problem is in v3d_main.cpp, namely at:


switch(tmpLocation.howUseful())

{

case pxLocaUseful:

curColor = (b_color==1) ? QColor(255, 0, 0, alpha) : QColor(255, 255, 255, alpha);

curShape = pxSphere; //pxCircle;

b_draw=1;

break;

case pxLocaNotUseful:

curColor = (b_color==1) ? QColor(0, 255, 0, alpha) : QColor(255, 255, 255, alpha);

curShape = pxCube; //pxRect;

b_draw=1;

break;

case pxLocaUnsure:

curColor = (b_color==1) ? QColor(0, 0, 255, alpha) : QColor(255, 255, 255, alpha);

curShape = pxTriangle;

b_draw=1;

break;

case pxTemp: //080405

curColor = (b_color==1) ? QColor(155, 155, 0, alpha) : QColor(255, 255, 255, alpha);

curShape = pxDot;

b_draw=1;

break;

default:

b_draw=0;

break;

}


Should this be removed and instead do curShape = tmpLocation.shape?
If not, what is the overall meaning?

Thanks.
Oct 17, 2011  04:10 PM | Hanchuan Peng
RE: Possible bug in landmark drawing
changed as of the version v2.703.