Skip to content

[MRG+1] Add hyperlink and example for kappa#7214

Merged
ogrisel merged 4 commits intoscikit-learn:masterfrom
zhengruifeng:kappa_doc
Sep 10, 2016
Merged

[MRG+1] Add hyperlink and example for kappa#7214
ogrisel merged 4 commits intoscikit-learn:masterfrom
zhengruifeng:kappa_doc

Conversation

@zhengruifeng
Copy link
Copy Markdown
Contributor

What does this implement/fix? Explain your changes.

1, include kappa in 3.3.2. Classification metrics
2, add a link to definition on wiki
3, add a brief example

.. autosummary::
:template: function.rst

cohen_kappa_score
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Any reason to put it on top? Any specific order?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

alphabetical? ;)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I find it slightly confusing here because it is not a classification metric.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

for me it's often used as a classif metric.

I've seen it used on Kaggle

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@agramfort agramfort changed the title Add hyperlink and example for kappa [MRG+1] Add hyperlink and example for kappa Aug 29, 2016
@agramfort
Copy link
Copy Markdown
Member

travis is not happy

@zhengruifeng
Copy link
Copy Markdown
Contributor Author

updated the result in example

@agramfort
Copy link
Copy Markdown
Member

@amueller ok for you?

@TomDLT
Copy link
Copy Markdown
Member

TomDLT commented Sep 2, 2016

+1

@agramfort
Copy link
Copy Markdown
Member

me + @TomDLT makes +2.

@amueller no objection?

@ogrisel
Copy link
Copy Markdown
Member

ogrisel commented Sep 10, 2016

LGTM, squash merging.

@ogrisel ogrisel merged commit 105d033 into scikit-learn:master Sep 10, 2016
@zhengruifeng zhengruifeng deleted the kappa_doc branch September 11, 2016 07:11
rsmith54 pushed a commit to rsmith54/scikit-learn that referenced this pull request Sep 14, 2016
TomDLT pushed a commit to TomDLT/scikit-learn that referenced this pull request Oct 3, 2016
Sundrique pushed a commit to Sundrique/scikit-learn that referenced this pull request Jun 14, 2017
paulha pushed a commit to paulha/scikit-learn that referenced this pull request Aug 19, 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.

5 participants