Skip to content

Conversation

@agodbug
Copy link

@agodbug agodbug commented Aug 17, 2022

fix a bug using at to setitem will raise InvalidIndexError: df.at[slice, item] = array_or_other if item existed

fix a bug of at indexer: when using df.at[slice, item] = array for existed item it will raise the pandas.errors.InvalidIndexError
Copy link
Member

@datapythonista datapythonista left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @agodbug, some comments:

  • Can you provide a useful title so we understand what this is about please
  • Can you leave the template of the PR with the bullet points there, and check the points that you completed (and complete them all please)
  • Can you link to an issue if what you're addressing here has an issue. If it doesn't, can you add one with a minimal code examples that shows the bug
  • Why is this PR is against 1.4.x branch and not to main?
  • Can you add a note to the release notes about what you're fixing
  • Can you add a test that shows the bug that you're fixing (so it fails before your fix, and passes after your fix)

Thank you!

@datapythonista datapythonista added Bug Needs Info Clarification about behavior needed to assess issue Needs Tests Unit test(s) needed to prevent regressions labels Aug 17, 2022
@simonjayhawkins
Copy link
Member

  • Why is this PR is against 1.4.x branch and not to main?

I don't think you can change that after the PR is opened? so closing this one.

@agodbug please reopen PR against main. (after considering the points raised in #48120 (review))

@simonjayhawkins simonjayhawkins mentioned this pull request Aug 24, 2022
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Needs Info Clarification about behavior needed to assess issue Needs Tests Unit test(s) needed to prevent regressions

3 participants