Hierarchical clustering: Allow more than 20 clusters in Top n spin #6887
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
Fixes #6883.
Description of changes
The spin box expects some maximum value; Qt's default is 99. I set it to 1000, but the widget will show a warning if the value is above 20.
The warning checks the final number of clusters. It won't be shown if Top N setting is too high, but there are not enough data instances for so many clusters. And the warning is also shown when >20 clusters are obtained in some other way, either manually or via treshold.
Includes