Skip to content

Backport PR #55619 on branch 2.1.x (BUG: Groupby not keeping string dtype for empty objects) #55705

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

Merged

Conversation

lithomas1
Copy link
Member

@lithomas1 lithomas1 commented Oct 26, 2023

  • BUG: Groupby not keeping string dtype for empty objects

  • Fix

Backports PR #55619

…5619)

* BUG: Groupby not keeping string dtype for empty objects

* Fix

---------

Co-authored-by: Thomas Li <[email protected]>
(cherry picked from commit 8afd868)
@lithomas1 lithomas1 requested a review from rhshadrach as a code owner October 26, 2023 13:18
@lithomas1 lithomas1 added this to the 2.1.2 milestone Oct 26, 2023
@lithomas1 lithomas1 added Groupby Strings String extension data type and string data labels Oct 26, 2023
@lithomas1
Copy link
Member Author

Just needed to move a test for this one.

@lithomas1
Copy link
Member Author

lithomas1 commented Oct 26, 2023

Ack, looks like there's a segmentation fault here.
(not sure if it's spurious or not, it happens after the tests finish?)

see https://github.com/pandas-dev/pandas/actions/runs/6654698565/job/18083267496?pr=55705#step:9:467.

@phofl I'm gonna bump this off the milestone then.

@lithomas1 lithomas1 modified the milestones: 2.1.2, 2.1.3 Oct 26, 2023
@phofl
Copy link
Member

phofl commented Oct 26, 2023

Yeah no worries

@lithomas1
Copy link
Member Author

Nevermind about the segfault, it seems to be unrelated to this PR (I saw it happen on the 2.1.x branch too, and I made a comment about it on the release issue).
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=812339&view=logs&j=385a3385-ae8b-54d1-9abe-3c614f74cb73&t=d0913101-e169-5874-b37d-533167693e73&l=1989.

I'll send a PR to move the release note.

@lithomas1 lithomas1 closed this Nov 9, 2023
@lithomas1 lithomas1 deleted the auto-backport-of-pr-55619-on-2.1.x branch November 9, 2023 22:35
@lithomas1 lithomas1 restored the auto-backport-of-pr-55619-on-2.1.x branch December 4, 2023 04:49
@lithomas1 lithomas1 reopened this Dec 4, 2023
@lithomas1
Copy link
Member Author

Oops, looks like I accidentally hit close on this for some reason.

(I think I meant to hit merge).

@lithomas1 lithomas1 merged commit ea39f7b into pandas-dev:2.1.x Dec 4, 2023
@lithomas1 lithomas1 deleted the auto-backport-of-pr-55619-on-2.1.x branch December 4, 2023 15:00
@lithomas1
Copy link
Member Author

Merging,

I need to move the release note now.

@lithomas1 lithomas1 mentioned this pull request Dec 4, 2023
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Groupby Strings String extension data type and string data
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants