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

DEPR: deprecate Index.is_object #50227

Merged
merged 11 commits into from
Jan 27, 2023
Merged

Conversation

bang128
Copy link
Contributor

@bang128 bang128 commented Dec 13, 2022

@bang128
Copy link
Contributor Author

bang128 commented Dec 15, 2022

@topper-123 Do you know what I got wrong? I have looked at some other pull requests, but they have both common and different files to fix. Actually, I am not sure how to detect which file I should change is_object to is_object_dtype. Do you have any suggestion for me?

@topper-123
Copy link
Contributor

topper-123 commented Dec 15, 2022

Hi @bang128 I think you're getting quite close. The issues that I see are:

1: doc/source/whatsnew/v2.0.0.rst
There are some artifacts in doc/source/whatsnew/v2.0.0.rst likely from a git rebase. Can you fix those.

2: failures
In general you need to go through the test logs above (click the "details" button) to see which errors have been raised. I can see these, which need to be fixed in one test log:

FAILED pandas/tests/series/methods/test_describe.py::TestSeriesDescribe::test_numeric_result_dtype[complex] - TypeError: a must be an array of real numbers
FAILED pandas/tests/series/methods/test_describe.py::TestSeriesDescribe::test_numeric_result_dtype[complex64] - TypeError: a must be an array of real numbers
FAILED pandas/tests/series/methods/test_describe.py::TestSeriesDescribe::test_numeric_result_dtype[complex128] - TypeError: a must be an array of real numbers

Looks like something related to the treatment of complex numbers. I'm not able to run these tests right now, but if you could attempt to solve them, I can take a closer look tonight and help you fix them, if you haven't been able to before.

There may be others, I haven't checked all test failures.

@topper-123
Copy link
Contributor

Hi @bang128 can you try to fix the issues in doc/source/whatsnew/v2.0.0.rst and push again. I think that may be enough to fix this.

Regarding how to find the instances to correct, you should run the pandas test suite locally and fix errors that show up. Depending on your coding proficiency, this may not be entirely trivial, but it'll be a good learning exercise in any case :-)

@bang128
Copy link
Contributor Author

bang128 commented Dec 15, 2022

Okay, let me try to fix the doc issue first

@github-actions
Copy link
Contributor

This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this.

@topper-123
Copy link
Contributor

topper-123 commented Jan 16, 2023

Hi @bang128. What's the status on your PR?

It look good to me, but needs a rebase.

@topper-123 topper-123 closed this Jan 16, 2023
@topper-123 topper-123 reopened this Jan 16, 2023
@topper-123
Copy link
Contributor

Can you rebase.

@topper-123
Copy link
Contributor

Ping. Should I rebase for you?

@topper-123 topper-123 merged commit 01693d6 into pandas-dev:main Jan 27, 2023
@topper-123
Copy link
Contributor

Hey @bang128 , the rebase was a bit difficult, so I’ve rebased & merged this. Thanks a lot for yoyr contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants