Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update 02_end_to_end_machine_learning_project.ipynb #61

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Giovanniko
Copy link

Bonjour @ageron, your book is helping a lot, I hope this helps you: Changed on line 1208 housing.corr() to housing.corr(numeric_only=True) to remove deprecated warning for future versions.

Bonjour @ageron, your book is helping a lot, I hope this helps you:
Changed on line 1208 housing.corr() to housing.corr(numeric_only=True) to remove deprecated warning for future versions.
@voacado
Copy link

voacado commented May 7, 2023

Starting as of Pandas v2.0.0+, the default param value of numeric_only=False.

I was encountering a ValueError when creating a correlation matrix in the Chapter 2 code: corr_matrix = housing.corr() about converting a string to a float and it was fixed by manually defining numeric_only=True.

@frederickjjoubert
Copy link

This fixes #86

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