Skip to content

ENH: Do unique check before computing indexer in reindex #51304

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

Merged
merged 2 commits into from
Feb 14, 2023

Conversation

phofl
Copy link
Member

@phofl phofl commented Feb 10, 2023

@mroeschke mroeschke added Indexing Related to indexing on series/frames, not to indexes themselves Performance Memory or execution speed performance labels Feb 10, 2023
@mroeschke
Copy link
Member

Looks good. Could use a performance whatsnew note

@phofl
Copy link
Member Author

phofl commented Feb 14, 2023

I'd feel a bit weird advertising that we are raising the error faster now :) But ok to add, if you feel strongly about it

@mroeschke mroeschke added this to the 2.0 milestone Feb 14, 2023
@mroeschke
Copy link
Member

Guess that's fair. Thanks @phofl

@mroeschke mroeschke merged commit b196a09 into pandas-dev:main Feb 14, 2023
@phofl phofl deleted the 51287 branch February 14, 2023 20:59
@Alexia-I
Copy link

Hello, @phofl @mroeschke I was wondering if this pull request has been backported? According to the backporting guidelines, it seems that PRs with a milestone should be backported. However, I couldn't find a corresponding backport PR linked to this one. Can you please clarify if a backport has been created or if there's a different situation here?

@phofl
Copy link
Member Author

phofl commented Feb 25, 2024

This pr was released in 2.0, which is what the milestone tells you.

You’ll have to clarify your question. To which branch should we have backported this pr?

@Alexia-I
Copy link

Thank you for your response!

Sorry for any confusion earlier, I'm not familiar with the open-source management mechanisms on GitHub. I took another look at the documentation and realized indeed when a milestone corresponds to the master version, a backport isn't necessary.

My question is actually about the process of fixing bugs: when you address a bug, do you first fix it in the master version and then backport that fix to the milestone, which is the target version? Does this mean that other versions won't receive the fix? For example, if the master is at 2.0 and the milestone is set for 1.3.x, does that mean only these two versions will be corrected on GitHub, and other minor versions, such as 1.4.x, 1.5.x, won't be fixed? Additionally, regarding PyPI releases, if a version on GitHub is backported, will the content in the PyPI package be updated with the fix?

@mroeschke
Copy link
Member

pandas only maintains 2 branches during active development: The main branch which reflects the next upcoming minor/major release, e.g. 3.0, and the latest minor release branch, e.g. 2.2.x. A backported PR will apply to main and the latest minor release branch. The latest minor release branch usually only gets 3 or 4 packages on PyPI and conda e.g. 2.2.1, 2.2.2, 2.2.3.

@Alexia-I
Copy link

Thank you for your response! Now I get it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Indexing Related to indexing on series/frames, not to indexes themselves Performance Memory or execution speed performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: pandas halted in a busy loop when assigning new column with non-unique index
3 participants