half edge neighbor is incorrect

[godpenguin7]

when i iterate through a site's halfedges and look at their neighbors, they do not correspond to actual neighbors. and even if i look at the half edge's corresponding edge, its two sites are not neighboring in the diagram.

[godpenguin7]

int i = rand() % myCells.size();

myCells[ i ].selected = true;

const jcv_site* site = &jcv_diagram_get_sites( &diagram )[ i ];

const jcv_graphedge* edge = site->edges;

while ( edge != nullptr )
{
	const jcv_site* neighbor;

	if ( edge->edge->sites[ 0 ] == site )
	neighbor = edge->edge->sites[ 1 ];
	else
	neighbor = edge->edge->sites[ 0 ];

	if ( neighbor != nullptr )
	{
		myCells[ neighbor->index ].selected = true;
	}

	edge = edge->next;
}

[JCash]

Hi! The array is sorted with respect to the points, so i != site->index Try “myCells[ site->index ].selected = true;” and see if that makes a difference

…

Sent from my iPhone

On 24 Dec 2018, at 06:30, godpenguin7 ***@***.***> wrote: int i = rand() % myCells.size(); myCells[ i ].selected = true; const jcv_site* site = &jcv_diagram_get_sites( &diagram )[ i ]; const jcv_graphedge* edge = site->edges; while ( edge != nullptr ) { const jcv_site* neighbor; if ( edge->edge->sites[ 0 ] == site ) neighbor = edge->edge->sites[ 1 ]; else neighbor = edge->edge->sites[ 0 ]; if ( neighbor != nullptr ) { myCells[ neighbor->index ].selected = true; } edge = edge->next; } — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

[godpenguin7]

you were right totally my fault, using site->index fixed it. thanks a lot, i was dreading switching back to this other voronoi library because yours is much faster.