Skip to content

[MRG] Fix SVC and NuSVC coef_ shape in the docstring [Fixes #9196]#11660

Merged
jnothman merged 1 commit intoscikit-learn:masterfrom
adrinjalali:docs/svc-nusvc
Jul 24, 2018
Merged

[MRG] Fix SVC and NuSVC coef_ shape in the docstring [Fixes #9196]#11660
jnothman merged 1 commit intoscikit-learn:masterfrom
adrinjalali:docs/svc-nusvc

Conversation

@adrinjalali
Copy link
Copy Markdown
Member

@adrinjalali adrinjalali commented Jul 23, 2018

Fixes #9196

Fixes the shape of coef_ in the docstrings of NuSVC and SVC.

@amueller
Copy link
Copy Markdown
Member

why wip? Looks good?

@amueller
Copy link
Copy Markdown
Member

If you say "Fixes #9196" it'll automatically close that issue.

@adrinjalali adrinjalali changed the title [WIP] Fix SVC and NuSVC coef_ shape in the docstring [See #9196] [WIP] Fix SVC and NuSVC coef_ shape in the docstring [Fixes #9196] Jul 23, 2018
@adrinjalali
Copy link
Copy Markdown
Member Author

why wip? Looks good?

I assume it's WIP until all tests are passed. Cause I never know.

If you say "Fixes #9196" it'll automatically close that issue.

Done!

@adrinjalali adrinjalali changed the title [WIP] Fix SVC and NuSVC coef_ shape in the docstring [Fixes #9196] [MRG] Fix SVC and NuSVC coef_ shape in the docstring [Fixes #9196] Jul 23, 2018
@jnothman
Copy link
Copy Markdown
Member

jnothman commented Jul 24, 2018 via email

@jnothman jnothman merged commit 0395b32 into scikit-learn:master Jul 24, 2018
@jnothman
Copy link
Copy Markdown
Member

Thanks @adrinjalali

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