Skip to content

Conversation

@dsaxton
Copy link
Contributor

@dsaxton dsaxton commented Oct 2, 2020

@simonjayhawkins simonjayhawkins added NA - MaskedArrays Related to pd.NA and nullable extension arrays Output-Formatting __repr__ of pandas objects, to_string labels Oct 2, 2020
assert df == expected


def test_nullable_float_to_string(float_ea_dtype):
Copy link
Contributor

Choose a reason for hiding this comment

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

i think should move these to EA tests and try to do these generically (for all of the numberic types); i guess ok here as well (but group them together int. & float both with and w/o nulls)

Copy link
Contributor

Choose a reason for hiding this comment

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

do we have int nullable tests for formatting?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not as far as I can tell, entirely possible I'm just missing them

Copy link
Contributor

Choose a reason for hiding this comment

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

kk can you add (this PR would be great)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. These tests could probably be cleaned up a bit also, this file is very big and testing lots of different things.

Copy link
Contributor

Choose a reason for hiding this comment

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

oh yes, for sure. if you'd create an issue would be great.

@dsaxton dsaxton marked this pull request as ready for review October 14, 2020 15:01
@jreback jreback added this to the 1.2 milestone Oct 15, 2020
assert df == expected


def test_nullable_float_to_string(float_ea_dtype):
Copy link
Contributor

Choose a reason for hiding this comment

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

do we have int nullable tests for formatting?

@jreback jreback merged commit c2a96c6 into pandas-dev:master Oct 16, 2020
@jreback
Copy link
Contributor

jreback commented Oct 16, 2020

thanks @dsaxton

@dsaxton dsaxton deleted the nullable-float-array-string-format branch October 16, 2020 01:48
@jorisvandenbossche
Copy link
Member

Thanks @dsaxton !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NA - MaskedArrays Related to pd.NA and nullable extension arrays Output-Formatting __repr__ of pandas objects, to_string

4 participants