Skip to content

edit: patch .values warning in time_series.ipynb #1576

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

Closed
wants to merge 4 commits into from

Conversation

JaehyunAhn
Copy link

@JaehyunAhn JaehyunAhn commented May 18, 2020

Edited time_series.ipynb. In some recent stable Pandas versions, warning occurs when trying to call .values method with pandas.core.series.Series object. In this reason, edited .values method to .to_numpy() method to make pandas.core.series.Series object to convert numpy.ndarray object, which pandas official documents recommended (https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.values.html). There is a rationale behind (pandas-dev/pandas#19954). Another way that convert pandas.core.series.Series object with index to numpy.ndarray object is call .array method, which works similar with .to_numpy(). In that case, Series object convert into pandas.core.arrays.numpy_.PandasArray object.

@googlebot googlebot added the cla: yes CLA has been signed label May 18, 2020
@JaehyunAhn JaehyunAhn changed the title fix: patch reshape error in time_series.ipynb fix: patch .values warning in time_series.ipynb May 19, 2020
@JaehyunAhn JaehyunAhn changed the title fix: patch .values warning in time_series.ipynb edit: patch .values warning in time_series.ipynb May 19, 2020
@MarkDaoust
Copy link
Member

Thanks for the PR. This entire notebook got rewritten. If this is still a problem please remake the PR from the master branch.

Sorry.

@MarkDaoust MarkDaoust closed this Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes CLA has been signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants