Skip to content

Closes #480#481

Open
ryan-deak-zefr wants to merge 31 commits intodask:mainfrom
ZEFR-INC:master
Open

Closes #480#481
ryan-deak-zefr wants to merge 31 commits intodask:mainfrom
ZEFR-INC:master

Conversation

@ryan-deak-zefr
Copy link
Copy Markdown
Contributor

@ryan-deak-zefr ryan-deak-zefr commented Mar 11, 2019

Reference Issues/PRs

Fixes #480. See also:

What does this implement/fix? Explain your changes.

This PR adds sample_weight to metrics in cross validation routines by propagating the sample_weight parameter passed to fit_params. It does not provide the ability to use different sample_weight values in validation.

Any other comments?

A scikit-learn version of this PR has been created at scikit-learn/scikit-learn#13432

For more detailed information on the motivation for this PR, see http://deaktator.github.io/2019/03/10/the-error-in-the-comparator/

@TomAugspurger
Copy link
Copy Markdown
Member

Can you ping me here when the scikit-learn version is merged / closed? I don't have time to follow it at the moment.

Base automatically changed from master to main February 2, 2021 03:43
@adrinjalali
Copy link
Copy Markdown

this might be of interest here: scikit-learn/scikit-learn#26896 (and other metadata routing work we're doing now)

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.

sample_weight is not passed to metrics in cross validation routines

5 participants