Skip to content

Conversation

espoirMur
Copy link
Contributor

@espoirMur espoirMur commented Nov 9, 2024

@espoirMur espoirMur changed the title Man group/set on StringDatatype Set __module__ on StringDatatypes Nov 9, 2024
@espoirMur espoirMur changed the title Set __module__ on StringDatatypes ENH: Set __module__ on StringDatatypes Nov 9, 2024
@simonjayhawkins simonjayhawkins added Output-Formatting __repr__ of pandas objects, to_string Sprints Sprint Pull Requests labels Nov 9, 2024
Copy link
Member

@simonjayhawkins simonjayhawkins left a comment

Choose a reason for hiding this comment

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

thanks @espoirMur

a couple of changes needed

# error: Definition of "_concat_same_type" in base class "NDArrayBacked" is
# incompatible with definition in base class "ExtensionArray"


Copy link
Member

Choose a reason for hiding this comment

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

please revert these additions

return new_string_array


@set_module("pandas")
Copy link
Member

Choose a reason for hiding this comment

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

AttributeError: module 'pandas' has no attribute 'BaseStringArray'

@simonjayhawkins simonjayhawkins added this to the 3.0 milestone Nov 9, 2024
Copy link
Member

@simonjayhawkins simonjayhawkins left a comment

Choose a reason for hiding this comment

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

Thanks @espoirMur

@jorisvandenbossche jorisvandenbossche changed the title ENH: Set __module__ on StringDatatypes ENH: Set __module__ on StringDtype Nov 11, 2024
Copy link
Member

@jorisvandenbossche jorisvandenbossche left a comment

Choose a reason for hiding this comment

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

Thanks!

@jorisvandenbossche jorisvandenbossche merged commit d3c595e into pandas-dev:main Nov 11, 2024
14 of 22 checks passed
@espoirMur espoirMur deleted the man-group/set-module-stringdtype branch November 11, 2024 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Output-Formatting __repr__ of pandas objects, to_string Sprints Sprint Pull Requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants