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

PERF/REGR: revert #41785 #42338

Merged
merged 4 commits into from
Jul 8, 2021

Conversation

jbrockmendel
Copy link
Member

@jbrockmendel jbrockmendel commented Jul 1, 2021

@jreback jreback added Constructors Series/DataFrame/Index/pd.array Constructors Performance Memory or execution speed performance labels Jul 1, 2021
@simonjayhawkins simonjayhawkins added this to the 1.3.1 milestone Jul 2, 2021
@simonjayhawkins
Copy link
Member

Thanks @jbrockmendel as this now goes in 1.3.1, will need a release note.

@simonjayhawkins
Copy link
Member

Thanks @jbrockmendel as this now goes in 1.3.1, will need a release note.

#42312 now merged

@jreback
Copy link
Contributor

jreback commented Jul 4, 2021

@jbrockmendel lgtm can you add a release note for 1.3.1

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

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

lgtm merge on green

@jreback jreback merged commit 82eb380 into pandas-dev:master Jul 8, 2021
@jreback
Copy link
Contributor

jreback commented Jul 8, 2021

@meeseeksdev backport 1.3.x

@lumberbot-app
Copy link

lumberbot-app bot commented Jul 8, 2021

Something went wrong ... Please have a look at my logs.

simonjayhawkins pushed a commit that referenced this pull request Jul 8, 2021
Co-authored-by: jbrockmendel <jbrockmendel@gmail.com>
@jbrockmendel jbrockmendel deleted the regr-perf-dict-of-dicts branch July 8, 2021 16:54
@@ -16,6 +16,7 @@ Fixed regressions
~~~~~~~~~~~~~~~~~
- Pandas could not be built on PyPy (:issue:`42355`)
- :class:`DataFrame` constructed with with an older version of pandas could not be unpickled (:issue:`42345`)
- Performance regression in constructing a :class:`DataFrame` from a dictionary of dictionaries (:issue:`42338`)
Copy link
Member

Choose a reason for hiding this comment

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

should have been 42248.

Copy link
Member Author

Choose a reason for hiding this comment

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

thanks, will add this to my next "collected cleanups" branch

Copy link
Member

Choose a reason for hiding this comment

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

would need to backport, but no worries if added with other changes since the release readiness script will show that the release notes aren't in sync and I will sync directly to 1.3.x.

Copy link
Member

Choose a reason for hiding this comment

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

done in #42516

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Constructors Series/DataFrame/Index/pd.array Constructors Performance Memory or execution speed performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PERF: regression in DataFrame construction from nested dict
3 participants