Sort Excel Sheet Hacktoberfest_Inputt.xls#20
Open
Ishita-Mehta wants to merge 2 commits intoachoudh5:mainfrom
Open
Sort Excel Sheet Hacktoberfest_Inputt.xls#20Ishita-Mehta wants to merge 2 commits intoachoudh5:mainfrom
Ishita-Mehta wants to merge 2 commits intoachoudh5:mainfrom
Conversation
…cel file, sorted output excel file.
Owner
|
Can you import this as a module in |
achoudh5
requested changes
Oct 8, 2020
…it__.py so that the module can be imported. Imported the module in zip.py and called it in the main() function.
Author
|
Hi @achoudh5 Sorry for the delay. Had a busy week at work. I've committed the following changes:
Please let me know if anything else is to be done. Thanks! |
achoudh5
requested changes
Oct 10, 2020
Owner
There was a problem hiding this comment.
@Ishita-Mehta Most of the changes done are good but if I glance at the first column where I am guessing those are the original sheet's row numbers it's a bit wonky. I would request the following changes:-
- Put a heading in row 1, column 1 . tell the user what these numbers mean?
- Check the column 1 entries, use more test cases. All the rows with no cource and destination color should be a the bottom (in terms of priority/sorting). So, as per your script i could see that the first row has nothing but
abcd textin source column. You should put this rown in bottom just because it didn't come up with a source and dest color. - Once you are done with priority/sorting of rows with no source and destination colors, take a look at the mapping of your original row numbers to the newly sorted excel file. The
abcd textrow which we move to the bottom should still show the mapping to be 1 cz it was row 1 in the original sheet and similarly for others too (ex, Yellow to red row should show line 8 not 6 after sorting too) - Can we do it all at one place,
zip.pykinda uses the Inputt.xlsx and do the desired changes inplace. Rather than creating a newsorted.xlsxcan you do the changes within the original xlsx.
Let me know if you have more questions :)
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Addressing Issue #11
Added a folder 'Sort Excel File' with excel_sorter.py
This script reads input excel file, sorts according to Source and Destination column and then outputs a new excel file with sorted rows. This is achieved using pandas library.