Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

delete kruskal with limit size, use largest size to limit it(uncontrolled group ... #19

Closed
todo bot opened this issue Jan 19, 2020 · 1 comment
Closed

Comments

@todo
Copy link

todo bot commented Jan 19, 2020

// todo: delete kruskal with limit size, use largest size to limit it(uncontrolled group number but better), need discuss.
/**
*
* @param matrix


This issue was generated by todo based on a todo comment in 24f2609 when #17 was merged. cc @Kanaries.
@todo todo bot added the todo 🗒️ label Jan 19, 2020
@ObservedObserver
Copy link
Member

It seems not logical to limit a group size if members in it are highly correlated with each other. Now, we are using a threshold of correlation and group number.

The original reason for this issue is seeking a solution to limit the number of the field in one visualization. However, a large group of fields can be made into a dashboard or story instead of a single vis.

AntoineYANG pushed a commit that referenced this issue Aug 1, 2023
fix(permission): upgrade text & env check
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant