Skip to content

[MRG] TST Fixes test for California housing#16389

Merged
rth merged 1 commit intoscikit-learn:masterfrom
thomasjpfan:fix_test_fetch_asframe
Feb 5, 2020
Merged

[MRG] TST Fixes test for California housing#16389
rth merged 1 commit intoscikit-learn:masterfrom
thomasjpfan:fix_test_fetch_asframe

Conversation

@thomasjpfan
Copy link
Copy Markdown
Member

Test fails on master because target is a Series.

@rth
Copy link
Copy Markdown
Member

rth commented Feb 5, 2020

So this test is not actually run until #16348 is merged, right?

Copy link
Copy Markdown
Member

@rth rth left a comment

Choose a reason for hiding this comment

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

LGTM.

@rth rth merged commit 87a5930 into scikit-learn:master Feb 5, 2020
thomasjpfan added a commit to thomasjpfan/scikit-learn that referenced this pull request Feb 22, 2020
panpiort8 pushed a commit to panpiort8/scikit-learn that referenced this pull request Mar 3, 2020
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