[go: up one dir, main page]

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

TST (string dtype): change any_string_dtype fixture to use actual dtype instances #59345

Merged

Conversation

jorisvandenbossche
Copy link
Member
@jorisvandenbossche jorisvandenbossche commented Jul 29, 2024

(for now on top of #59330, so only look at second commit)

This will make it easier to add the object-dtype based variant of the dtype with NaN sentinel

xref #54792

@jorisvandenbossche jorisvandenbossche added Testing pandas testing functions or related to the test suite Strings String extension data type and string data labels Jul 29, 2024
@jorisvandenbossche jorisvandenbossche marked this pull request as ready for review July 30, 2024 06:38
@jorisvandenbossche
Copy link
Member Author

OK, this seems to be passing now. And it's also fixing some additional failures on main (a newly added test that should have had an xfail)

@mroeschke mroeschke added this to the 3.0 milestone Jul 31, 2024
@mroeschke mroeschke merged commit 89c8d7a into pandas-dev:main Jul 31, 2024
39 of 45 checks passed
@mroeschke
Copy link
Member

Thanks @jorisvandenbossche

@jorisvandenbossche jorisvandenbossche deleted the string-dtype-tests-fixtures branch July 31, 2024 17:39
WillAyd pushed a commit that referenced this pull request Aug 13, 2024
…pe instances (#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
WillAyd pushed a commit to WillAyd/pandas that referenced this pull request Aug 14, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
WillAyd pushed a commit to WillAyd/pandas that referenced this pull request Aug 15, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
WillAyd pushed a commit to WillAyd/pandas that referenced this pull request Aug 15, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
WillAyd pushed a commit to WillAyd/pandas that referenced this pull request Aug 15, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
@jorisvandenbossche jorisvandenbossche modified the milestones: 3.0, 2.3 Aug 20, 2024
WillAyd pushed a commit to WillAyd/pandas that referenced this pull request Aug 21, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
WillAyd pushed a commit to WillAyd/pandas that referenced this pull request Aug 22, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
WillAyd pushed a commit to WillAyd/pandas that referenced this pull request Aug 22, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
WillAyd pushed a commit to WillAyd/pandas that referenced this pull request Aug 27, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
WillAyd pushed a commit to WillAyd/pandas that referenced this pull request Sep 20, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
jorisvandenbossche added a commit to WillAyd/pandas that referenced this pull request Oct 2, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
jorisvandenbossche added a commit to WillAyd/pandas that referenced this pull request Oct 2, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
jorisvandenbossche added a commit to WillAyd/pandas that referenced this pull request Oct 3, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
jorisvandenbossche added a commit to WillAyd/pandas that referenced this pull request Oct 7, 2024
…pe instances (pandas-dev#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
jorisvandenbossche added a commit that referenced this pull request Oct 9, 2024
…pe instances (#59345)

* TST (string dtype): change any_string_dtype fixture to use actual dtype instances

* avoid pyarrow import error during test collection

* fix dtype equality in case pyarrow is not installed

* keep using mode.string_storage as default for NA variant + more xfails

* fix test_series_string_inference_storage_definition

* remove no longer necessary xfails

---------

Co-authored-by: Matthew Roeschke <10647082+mroeschke@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported Strings String extension data type and string data Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants