Skip to content

GH1654 Numeric changes for 3.0#1667

Open
loicdiridollou wants to merge 8 commits intopandas-dev:mainfrom
loicdiridollou:gh1654_numeric
Open

GH1654 Numeric changes for 3.0#1667
loicdiridollou wants to merge 8 commits intopandas-dev:mainfrom
loicdiridollou:gh1654_numeric

Conversation

@loicdiridollou
Copy link
Member

@loicdiridollou loicdiridollou commented Feb 8, 2026

Copy link
Contributor

@cmp0xff cmp0xff left a comment

Choose a reason for hiding this comment

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

Look good in general.

method: Literal["pearson", "kendall", "spearman"] = "pearson",
numeric_only: _bool = False,
min_periods: int | None = None,
) -> Series: ...
Copy link
Contributor

Choose a reason for hiding this comment

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

Can it actually be always float?

Suggested change
) -> Series: ...
) -> Series[float]: ...

self,
arg: Callable[[S1], S2 | NAType] | Mapping[S1, S2] | Series[S2],
arg: Callable[Concatenate[S1, ...], S2 | NAType],
na_action: Literal["ignore"] = ...,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
na_action: Literal["ignore"] = ...,
na_action: Literal["ignore"] = "ignore",

Copy link
Member Author

Choose a reason for hiding this comment

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

The default is None so I will remove the ... and add the default where it needs to

Copy link
Member Author

Choose a reason for hiding this comment

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

Technically the default is None but that would break a couple of things, let me take a deeper look here, will mark it as draft for a bit.

@loicdiridollou loicdiridollou marked this pull request as draft February 9, 2026 15:33
@loicdiridollou loicdiridollou marked this pull request as ready for review February 13, 2026 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants