Skip to content

ENH: usecols takes input order for read_csv implementation review #61967

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
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

eicchen
Copy link
Contributor

@eicchen eicchen commented Jul 26, 2025

This is just the implementation for using usecols order for read_csv that I wanted to have people look at before moving to apply it to other places like read_excel and read_clipboard. If it all looks good, I'll go back and add all necessary documentation about future deprecation along with a popup when using usecols. This is mainly just for checking that the implementation doesn't have any glaring issues.

I ran the entire test suite just to be safe and it all looks good. The only thing the errored were some datetime tests that had nothing to do with the changes that I could find.

Oh it is also worth noting that pyarrow already uses the usecols order by default so that's probably worth adding to the documentation regardless

@eicchen eicchen changed the title Enh #61386 usecols ENH: usecols takes input order for read_csv implementation review Jul 26, 2025
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.

ENH: read_csv with usecols shouldn't change column order
1 participant