-
- Notifications
You must be signed in to change notification settings - Fork 19.4k
Openpyxl engine for reading excel files #25092
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
Changes from 2 commits
Commits
Show all changes
87 commits Select commit Hold shift + click to select a range
e29b4c0 prepare testing reading excel files with multiple engines
tdamsma e0199a8 add openpyxl tests
tdamsma ce4eb01 implement first version of openpyxl reader
tdamsma b25877e pep8 issues
tdamsma 821fa4d suppress openpyxl warnings
tdamsma 4694668 add code for all edge cases that are tested for. Unfortunately got pr…
tdamsma 712f1ef formatting
tdamsma 1d49a0e Merge commit '683c7b55f5195fdf4f524239066cbf6f1301f0e7' into openpyxl…
tdamsma 1473c0e improve docstring
tdamsma 6e8ffba also test openpyxl reader for .xlsm files
tdamsma d57dfc1 explicitly use 64bit floats and ints
tdamsma e984f6b Merge commit '6359bbc4c9ce6dd05bc8b422641cda74871cde43' into openpyxl…
tdamsma 44f7af2 formatting
tdamsma 98d3865 skip TestOpenpyxlReader when openpyxl is not installed
tdamsma d0188ba Attempt to generalize _XlrdReader __init__ and move it to _BaseExcelR…
tdamsma 205d52b Merge commit 'f4568fd76e864d8aee3d23f5a81302262d6e0dcb' into openpyxl…
tdamsma 7b550bf register openpyxl writer engine, fix imports
tdamsma 875de8d import type_error explicitly
tdamsma 12ad6d8 Merge branch 'master' into openpyxl-reader
tdamsma dfd6a36 Merge branch 'master' into openpyxl-reader
tdamsma fef7233 Merge branch 'master' into openpyxl-reader
tdamsma eaafd5f get rid of some py2 compatibility legacy
tdamsma 8d2db02 Merge branch 'master' into openpyxl-reader
tdamsma 13e7793 fix some type chcking
tdamsma b053cce linting
tdamsma fe4dd73 see if this works on linux
tdamsma 64e5f2d run isort on _openpyxl.py
tdamsma 99b2cad Merge remote-tracking branch 'upstream/master' into openpyxl-reader
tdamsma ce5ac05 Merge remote-tracking branch 'upstream/master' into openpyxl-reader
tdamsma c7895ea Merge remote-tracking branch 'pandas/master' into openpyxl-reader
tdamsma 2ca9368 refactor handling of sheet_name keyword
tdamsma 5fb1aef extract code to parse a single sheet to a method
tdamsma 537dd0c extract handling of header keywords
tdamsma 44cddc5 extract handling of convert_float keyword to method
tdamsma e4c8f23 extract handling of index_col to method
tdamsma daff364 extract handling of usecols keyword to method
tdamsma 1224918 remove redundant code
tdamsma 1bfc030 Merge remote-tracking branch 'upstream/master' into excel-read-shared…
tdamsma 747311e Merge branch 'master' into excel-read-shared-init-to-baseclass
tdamsma a77a4c7 implement suggestions @WillAyd
tdamsma ddcaad8 Merge remote-tracking branch 'upstream/master' into excel-read-shared…
tdamsma 757235d Merge branch 'excel-read-shared-init-to-baseclass' into openpyxl-reader
tdamsma cdd627f remove _engine keyword altogether
tdamsma 0b58109 Merge branch 'excel-read-shared-init-to-baseclass' into openpyxl-reader
tdamsma 45f21f8 Clean up __init__
tdamsma e97d029 Implement work around for Linux py35_compat import error
tdamsma 1edae5e fix regression for reading s3 files
tdamsma a69e104 Merge branch 'excel-read-shared-init-to-baseclass' into openpyxl-reader
tdamsma f5f40e4 expand code highlighting the weirdness of a failing/skipped test.
tdamsma 22e24bb remove _engine keyword altogether
tdamsma 903b188 fix regression for reading s3 files
tdamsma 1b3ae99 Merge branch 'excel-read-shared-init-to-baseclass' into openpyxl-reader
tdamsma 02e19a8 Merge remote-tracking branch 'upstream/master' into openpyxl-reader
tdamsma 3e18f97 Merge remote-tracking branch 'upstream/master' into openpyxl-reader
tdamsma d11956c remove accidental commit
tdamsma 61d7a3f ditch some code
tdamsma 13d41b2 Merge remote-tracking branch 'upstream/master' into openpyxl-reader
tdamsma 97c85f5 remove skips for openpyxl for tests that should pass
tdamsma 614d972 Add `by_blocks=True` to failing `assert_frame_equal` tests, as per @W…
tdamsma d87d9c0 Merge remote-tracking branch 'upstream/master' into openpyxl-reader
WillAyd 7348b0c Updated import machinery
WillAyd c1a1792 Cleaned up nan replacement
WillAyd d72ca5a Simplified introspection
WillAyd 0bba345 Used common renaming method
WillAyd 8dd8bf6 Reverted some test changes
WillAyd eaaa680 Reset yield statement
WillAyd 6bf5183 Better missing label handling
WillAyd a06bf9b Aligned implementation with base
WillAyd f43e90f Fix bool handling
WillAyd 8fabe0a Fixed 0 handling
WillAyd 0ff5ce3 Aligned float handling with xlrd
WillAyd fb73692 xfailed overflow test
WillAyd 17b1d73 lint and isort fixup
WillAyd 3d248ed Removed by_blocks
WillAyd c369fd8 Revert "Reverted some test changes"
tdamsma 70b15a4 use readonly mode. Should be more performant and also this ignores Me…
tdamsma a3a3bca formatting issues
tdamsma fcd43f0 handle datetime cells explicitly for openpyxl < 2.5.0 compatibility
tdamsma d9c1fa6 type fixup
WillAyd 3c239a4 whatsnew
WillAyd 4a25a5a Removed np.nan from Scalar
WillAyd 6258e59 revert test_reader changes again. Not needed anymore because of using…
tdamsma 00f34b1 more types and whitespace cleanup
WillAyd a1fba90 Added config for excel reader. Not sure how to test this
tdamsma 88ee325 whatsnew
WillAyd 837ce26 Merge remote-tracking branch 'upstream/master' into openpyxl-reader
WillAyd dddc8c5 Regenerated test1 files
WillAyd File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
woops, this should not have been committed