feat: 🎸 basic excel importer #112
Merged
Conversation
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 Excel importer is very rudimentary but it gets the job done in most cases. It assumes that the data to be imported lives in one of the sheets in a "reasonable" position (i.e. headers in Row 1 and first column is A1). If a file has multiple sheet, the user can change the sheet to import. No explicit range selection in the sheet itself is possible.
DeanEby
reviewed
Apr 2, 2025
Collaborator
DeanEby
left a comment
DeanEby
left a comment
There was a problem hiding this comment.
The code looks good to me. One small thing I noticed is that the "modify_session" method could be named something like "change_import_sheet" or just "change_sheet" to be more precise.
DeanEby
added a commit
that referenced
this pull request
Apr 8, 2025
* Analysisnamingcopy (#111) * factor: update analysis naming screen copy * factor: update analysis naming screen copy style * Prtestbuild [ENH] Add a GitHub action to test .exe upon pull requests #113 (#118) * refactor: separate building exe and creating new release * upload artifacts only if triggered from release * debug: handle workflow inputs * remove precommit test * test windows exe run on PR * debug: fix executable filepath * debug: fix filepath to updated name * debug: path / to \ * refactor: implement run checks for macos * debug: is_release == false instead of ! * Issue #110 new dev workflow documentation (#116) * Add devops contributor workflow documentation * Update release section * feat: 🎸 basic excel importer (#112) * feat: 🎸 basic excel importer This Excel importer is very rudimentary but it gets the job done in most cases. It assumes that the data to be imported lives in one of the sheets in a "reasonable" position (i.e. headers in Row 1 and first column is A1). If a file has multiple sheet, the user can change the sheet to import. No explicit range selection in the sheet itself is possible. * run isort for styling --------- Co-authored-by: DeanEby <ebyd21@gmail.com> * Opendirectoryscreen copyupdate (#115) * factor: remove initial output analysis * refactor: context dependent post-analysis menu * remove debug print statement * refactor: slight copy changes * debug: remove export outputs debug print * test: 💍 expose CsvConfig for CsvTestData (#114) * test: 💍 expose CsvConfig for CsvTestData The CsvConfig class encodes how csv files can be loaded for testing, but it was never exposed or wired up in the CsvTestData file. This commit corrects this, and adds an example of how to use it to customize the way test CSV files are loaded for testing. * style: 💄 isort && black --------- Co-authored-by: soul-codes <40335030+soul-codes@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
This Excel importer is very rudimentary but it gets the job done in most cases. It assumes that the data to be imported lives in one of the sheets in a "reasonable" position (i.e. headers in Row 1 and first column is A1). If a file has multiple sheet, the user can change the sheet to import. No explicit range selection in the sheet itself is possible.