royallgroup / TCC

The Topological Cluster Classification algorithm
https://royallgroup.github.io/TCC/
GNU General Public License v3.0
12 stars 5 forks source link

Consider removing net clusters #32

Closed merrygoat closed 6 years ago

merrygoat commented 6 years ago

This bit of code is horribly implemented. This is something that can and is done as a post analysis method in python.

Removing (or fixing) net clusters will also allow testing do_clust on memset in setup.c to improve performance.

merrygoat commented 6 years ago

Consider adding analysis tools to the TCC folder. These would be post analysis tools separate from the source.

Before this is removed from the C the tools folder an alternative should be set up and checked with tests.

The readme should include a hierarchy of cluster types for commonly analysed systems (KA, Wahnstrom, HS at least).

merrygoat commented 6 years ago

Done in commit f49833fe384653cbbe8d4743e158cf52bf77f9e6