dynamo-release icon indicating copy to clipboard operation
dynamo-release copied to clipboard

support setting a resolution for vectorfield clustering

Open chansigit opened this issue 2 years ago • 1 comments

Changes:

  1. add **kwargs for the leiden(), louvain(), infomap() function calls in dyn.vf.cluster_field(), otherwise the function can only use default resolution settings

  2. add a resolution parameter explicitly so users would not bother passing it via kwargs

After this change, tools.clustering.leiden(), tools.clustering.louvain(), tools.clustering.infomap() automatically get resolution`` by its caller vf.cluster_field()

tools.clustering.cluster_community() automatically gets resolution from **kwargs passed by its caller tools.clustering.leiden(), tools.clustering.louvain(), tools.clustering.infomap()

tools.clustering.cluster_community_from_graph() automatically gets resolution from **kwargs passed by its caller tools.clustering.cluster_community()

chansigit avatar May 28 '22 15:05 chansigit

thank you so much @chansigit. feel free to provide your solution as you suggested. I can review your changes later and merge it into dynamo.

Xiaojieqiu avatar May 30 '22 15:05 Xiaojieqiu

thanks. the changes look good1

Xiaojieqiu avatar Mar 10 '23 19:03 Xiaojieqiu