geometric-intelligence/TopoBenchmark

Collate function running index

Closed this issue · 2 comments

I suppose there was a bug on lines 84 and 94; the variable for cell_i was always current_number_of_nodes but not current_number_of_cells. Doublecheck that it should be current_number_of_cells.

Image

Hi, any updates on this one?

current_number_of_cells is correct. My question is why do we need lines 90-93? Why are we changing the cells features? I think we should just remove them but maybe I'm not understanding something