Conversation

lukemanley

@lukemanleylukemanley added ReshapingConcat, Merge/Join, Stack/Unstack, ExplodeArrowpyarrow functionalitylabels Jun 11, 2023
@lukemanleylukemanley added this to the 2.1 milestone Jun 11, 2023
@@ -141,3 +143,25 @@ def test_explode_scalars_can_ignore_index():
result = s.explode(ignore_index=True)
expected = pd.Series([1, 2, 3])
tm.assert_series_equal(result, expected)


@pytest.mark.skipif(pa_version_under7p0, reason="minimum pyarrow not installed")
Copy link
Member

Choose a reason for hiding this comment

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

Nit: Could you just use pa = pytest.importorskip("pyarrow") below?

Copy link
Member Author

Choose a reason for hiding this comment

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

updated



@pytest.mark.skipif(pa_version_under7p0, reason="minimum pyarrow not installed")
def test_explode_pyarrow_list_type():
Copy link
Member

Choose a reason for hiding this comment

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

Could you parameterize on ignore_index as well?

Copy link
Member Author

Choose a reason for hiding this comment

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

updated

Choose a reason for hiding this comment

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

Looks good just a merge conflict

@mroeschkemroeschke merged commit ee43ae0 into pandas-dev:main Jun 13, 2023
@mroeschke

Thanks @lukemanley

@lukemanleylukemanley deleted the series-explode-pyarrow-list-type branch June 13, 2023 23:27
@lukemanleylukemanley mentioned this pull request Jun 30, 2023
3 tasks
Daquisu pushed a commit to Daquisu/pandas that referenced this pull request Jul 8, 2023
…53602)

* ENH: Series.explode to support pyarrow-backed list types

* gh refs

* update test
Sign up for free to join this conversation on . Already have an account? Sign in to comment
Arrowpyarrow functionalityReshapingConcat, Merge/Join, Stack/Unstack, Explode
None yet

Successfully merging this pull request may close these issues.