Assign contiguous index values to structures

dendrograms / astrodendro

Generate a dendrogram from a dataset

https://dendrograms.readthedocs.io/

Other

37 stars 38 forks source link

Assign contiguous index values to structures #19

Closed ChrisBeaumont closed 11 years ago

ChrisBeaumont commented 11 years ago

Currently, the set of Structure.idx values is a sparse subset of range(1, data.size). For the index code I am working on, it would be much simpler if they were just range(1, n_structure).

@astrofrog, is the particular value of Structure.idx used for anything right now? Would anything break if we switched schemes?

astrofrog commented 11 years ago

I don't think it's used for anything so I think you can switch to this scheme if you want!

ChrisBeaumont commented 11 years ago

Thanks, I will try this and see if anything falls over

ChrisBeaumont commented 11 years ago

Ok, I decided to address this in another way, without reassigning the original indexes