Skip to content

Conversation

vivekk0903
Copy link
Contributor

@vivekk0903 vivekk0903 commented Feb 7, 2017

Changed n_neighbours to n_neighbors in Parameters description

What does this implement/fix? Explain your changes.

Just spelling of parameter n_neighbors is corrected. Previously it was n_neighbours in Parameter description. It was confusion to me, when I wanted to use the params in GridSearch. And used the parameters from the description.

Changed n_neighbours to n_neighbors in Parameters description
@coveralls
Copy link

coveralls commented Feb 7, 2017

Coverage Status

Coverage remained the same at 98.934% when pulling 4f74d24 on vivekk-ezdi:patch-1 into cbfa74c on scikit-learn-contrib:master.

@chkoar chkoar merged commit 0a7fcc4 into scikit-learn-contrib:master Feb 7, 2017
@vivekk0903 vivekk0903 deleted the patch-1 branch February 7, 2017 06:32
glemaitre pushed a commit that referenced this pull request Feb 7, 2017
DOC: Change n_neighbours to n_neighbors in Parameters description
christophe-rannou pushed a commit to christophe-rannou/imbalanced-learn that referenced this pull request Apr 3, 2017
DOC: Change n_neighbours to n_neighbors in Parameters description
glemaitre pushed a commit to glemaitre/imbalanced-learn that referenced this pull request Jun 15, 2017
DOC: Change n_neighbours to n_neighbors in Parameters description
glemaitre pushed a commit to glemaitre/imbalanced-learn that referenced this pull request Jun 15, 2017
DOC: Change n_neighbours to n_neighbors in Parameters description
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants