Skip to content

Potential regression induced by PR #56921 #57641

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

Open
DeaMariaLeon opened this issue Feb 27, 2024 · 2 comments
Open

Potential regression induced by PR #56921 #57641

DeaMariaLeon opened this issue Feb 27, 2024 · 2 comments
Assignees
Labels
Performance Memory or execution speed performance Regression Functionality that used to work in a prior pandas version Warnings Warnings that appear or should be added to pandas
Milestone

Comments

@DeaMariaLeon
Copy link
Member

PR #56921
Benchmarks:
join_merge.Join.time_join_dataframes_cross (Python) with sort=True
timeseries.ResetIndex.time_reset_datetimeindex (Python) with t=None
timeseries.ResetIndex.time_reset_datetimeindex (Python) with t='US/Eastern'
join_merge.Join.time_join_dataframes_cross (Python) with sort=False

@rhshadrach
Screenshot 2024-02-27 at 09 58 18

@rhshadrach
Copy link
Member

rhshadrach commented Feb 27, 2024

This is hitting the added check here:

if (
_get_option("performance_warnings")
and sum(not block.is_extension for block in self.blocks) > 100
):

It is hit twice for the entire benchmark; i.e. not in a tight loop. The regression is 0.2ms; this is expected. Closing.

@rhshadrach rhshadrach reopened this Mar 4, 2024
@rhshadrach rhshadrach self-assigned this Mar 4, 2024
@rhshadrach rhshadrach added Performance Memory or execution speed performance Regression Functionality that used to work in a prior pandas version Warnings Warnings that appear or should be added to pandas labels Mar 4, 2024
@rhshadrach rhshadrach added this to the 3.0 milestone Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Performance Memory or execution speed performance Regression Functionality that used to work in a prior pandas version Warnings Warnings that appear or should be added to pandas
Projects
None yet
Development

No branches or pull requests

2 participants