Skip to content

Conversation

@scooler
Copy link
Contributor

@scooler scooler commented Mar 16, 2017

Reference Issue

Fixes #8599

What does this implement/fix? Explain your changes.

Fixes what seems to be a typo in a SGD documentation
x_i should be in R^m (as n is the number of features) - better illustrated in the issue ^

x_i should be in R^m (as n is the number of features)
@lesteve
Copy link
Member

lesteve commented Mar 16, 2017

Thanks a lot, merging!

@lesteve lesteve merged commit 306de84 into scikit-learn:master Mar 16, 2017
@Przemo10 Przemo10 mentioned this pull request Mar 17, 2017
herilalaina pushed a commit to herilalaina/scikit-learn that referenced this pull request Mar 26, 2017
x_i should be in R^m (as n is the number of features)
massich pushed a commit to massich/scikit-learn that referenced this pull request Apr 26, 2017
x_i should be in R^m (as n is the number of features)
Sundrique pushed a commit to Sundrique/scikit-learn that referenced this pull request Jun 14, 2017
x_i should be in R^m (as n is the number of features)
NelleV pushed a commit to NelleV/scikit-learn that referenced this pull request Aug 11, 2017
x_i should be in R^m (as n is the number of features)
paulha pushed a commit to paulha/scikit-learn that referenced this pull request Aug 19, 2017
x_i should be in R^m (as n is the number of features)
maskani-moh pushed a commit to maskani-moh/scikit-learn that referenced this pull request Nov 15, 2017
x_i should be in R^m (as n is the number of features)
jwjohnson314 pushed a commit to jwjohnson314/scikit-learn that referenced this pull request Dec 18, 2017
x_i should be in R^m (as n is the number of features)
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.

Error in Documentation on SGD

2 participants