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

BUG: Correct read_stata #251

Merged
merged 12 commits into from
Sep 27, 2022
Merged

BUG: Correct read_stata #251

merged 12 commits into from
Sep 27, 2022

Conversation

bashtage
Copy link
Contributor

@bashtage bashtage commented Sep 2, 2022

Update and silence for future warnings from pandas/main

@twoertwein twoertwein mentioned this pull request Sep 2, 2022
2 tasks
Copy link
Member

@twoertwein twoertwein left a comment

Choose a reason for hiding this comment

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

set_index isn't ideal, but letting it always return a DataFrame is consistent with how it will be in the near future.

@Dr-Irv
Copy link
Collaborator

Dr-Irv commented Sep 6, 2022

I'm holding off on merging this until 1.5 is released.

@twoertwein
Copy link
Member

Definitely good to wait for 1.5 - especially since the set_index behavior might get reverted before 1.5 releases pandas-dev/pandas#48141 (comment)

@twoertwein
Copy link
Member

Since pandas-dev/pandas#48417 is merged, we can remove the set_index changes and probably merge this PR before 1.5 releases as it hopefully doesn't disrupt 1.4.x.

@bashtage bashtage changed the title Silence future warnings BUG: Correct read_stata Sep 27, 2022
@bashtage
Copy link
Contributor Author

Doesn't do much any more, but does fix a bug in read_stata

@Dr-Irv Dr-Irv merged commit 6d013b4 into pandas-dev:main Sep 27, 2022
@Dr-Irv
Copy link
Collaborator

Dr-Irv commented Sep 27, 2022

thanks @bashtage

@bashtage bashtage deleted the silence-warnings branch September 28, 2022 21:45
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.

3 participants