Skip to content

Conversation

@qinhanmin2014
Copy link
Member

Reference Issue

What does this implement/fix? Explain your changes.

It should be included in bug fix, not enhancement.

Any other comments?

@jnothman Sorry for the silly mistake.

Copy link
Member

@jnothman jnothman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No worries! Thanks for checking. I find quality assurance for what's new is pretty hard... I think that splitting it into more files might help, but I'm not sure how granular to go with that.

doc/whats_new/0.19/fix/decomposition-9108.rst might be excessive.

@jnothman jnothman merged commit 8f53f70 into scikit-learn:master Aug 6, 2017
@qinhanmin2014 qinhanmin2014 deleted the my-feature-1 branch August 6, 2017 05:55
jnothman pushed a commit to jnothman/scikit-learn that referenced this pull request Aug 6, 2017
@qinhanmin2014 qinhanmin2014 restored the my-feature-1 branch August 7, 2017 08:50
@qinhanmin2014 qinhanmin2014 deleted the my-feature-1 branch August 7, 2017 08:52
dmohns pushed a commit to dmohns/scikit-learn that referenced this pull request Aug 7, 2017
dmohns pushed a commit to dmohns/scikit-learn that referenced this pull request Aug 7, 2017
paulha pushed a commit to paulha/scikit-learn that referenced this pull request Aug 19, 2017
AishwaryaRK pushed a commit to AishwaryaRK/scikit-learn that referenced this pull request Aug 29, 2017
maskani-moh pushed a commit to maskani-moh/scikit-learn that referenced this pull request Nov 15, 2017
jwjohnson314 pushed a commit to jwjohnson314/scikit-learn that referenced this pull request Dec 18, 2017
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.

2 participants