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

Improve description for nb parameter on head / tail method #42

Open
mthh opened this issue May 7, 2024 · 0 comments
Open

Improve description for nb parameter on head / tail method #42

mthh opened this issue May 7, 2024 · 0 comments

Comments

@mthh
Copy link
Member

mthh commented May 7, 2024

The head/tail break method was contributed with the following description for the optional nb parameter : Optional number of class. If nb < natural breaks results, upper classes are merged. If nb > natural breaks results, nb is not used and the natural breaks results is returned..

But from git blame it looks like it's been overwritten by commit ee4ffb3 to replace it with "Number of classes desired". I think we should go back to the old description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant