achoudh5 / Manipulating_Excel_Sheet

Real world application of Manipulating Excel Sheet using python.
Apache License 2.0
8 stars 14 forks source link

Create a GUI for this application #3

Closed achoudh5 closed 4 years ago

achoudh5 commented 4 years ago

GUI should contain atleast:-

tanya3007 commented 4 years ago

I have prior experience in building GUI with Python. I would like to create and contribute to this task.

Can you assign it to me?

achoudh5 commented 4 years ago

Logic code will be in zip.py, updated zip.py !

tanya3007 commented 4 years ago

Thanks @achoudh5 for assigning this task.

Can you tell me what type of output you want? Do you just want to display the parsed excel sheet on the GUI? or Do you want to save the parsed excel sheet as soon as the user clicks the button?

achoudh5 commented 4 years ago

Good Question @tanya3007 !

tanya3007 commented 4 years ago

Okay Sure. But, I'll be working with tkinter to make the GUI.

achoudh5 commented 4 years ago

okay, yeah I got one with tkinter as well. Let me upload that for you @tanya3007

Just added one of the current tkinter code I am using, it's under GUI folder. Please eliminate errors, I cannot see the button primarily due to version of tkinter on mac. Let me know if you have any questions.

tanya3007 commented 4 years ago

I went through the code. I was writing a code similar to the one you gave. I'll make a PR soon. You can then check it then.

Also, I am not able to add that drag and drop feature. Probably, tkinter currently doesn't provide such feature. Instead of that, I have added a Dialog Box, through which one can browse to the file directory and open it.

I will import the zip.py in my GUI code and will pass the filename received from the dialog box. From there, you can return the columns and rows which you need to display on the screen. So, can you tell me, to which function shall I pass the filename to? and also, in what format will you return the data? Knowing these I can write the code and connect with the zip.py file.

Update :- If the data to be displayed is the one in the last PR I made ( list of dictionaries ), I can display it in two columns and N rows. Will that work?

achoudh5 commented 4 years ago

Some more considerations:-

tanya3007 commented 4 years ago

Okay. Sounds Good. I'll make some changes in the zip,py to make it accept the filepath as a parameter from the GUI.

I'll complete the code and that Input and Output Section.

The User can upload one file at once. As soon as the user selects a file, the script will run and the GUI will be updated. The user can then, again click the button to browse and upload another file. Again as soon as the user selects a file, the script will run and GUI will be updated.

Update 1:- There are two inputs in the zip.py . One is the filepath, another is the Sheet Number. Shall I hardcode the sheet number to be 1?

Update 2:- If the changes are to be made on the same file, is there a need for Download button?

achoudh5 commented 4 years ago

No we have to take both input from user, based on sheet the output file changes. How about you make a box beneath where upload button/drag drop file will be present? In that box will be the input for the sheet.

After drag drop and entry of sheet number in the box, we will cover both cases and then zip.py won't prompt anython. Let me know if you have any question.

Note:- zip.py is constantly updated, make sure you fetch and pull the latest changes.

Update No harm in giving an extra button, imagine the person is noob. We need to cover our tracks and give them as much options as possible :D

tanya3007 commented 4 years ago

I made a PR as per you said.

tkinter module must be installed. There has to be some changes in zip.py. I am mentioning them.

Line 331: def main( y, x=1): Line 341: inp = x Line 342: ss_inp = y

There are some changes in valid_ip.py as well.

Line 7 and Line 9 must be cut and pasted in main at Line 67

Rest, the GUI code is completed. I have tried to cover everything. Can you merge this PR? Issue #19

Update :- I was not able to change as I uploaded the file on browser. So, was having some issues. That's why mentioned the changes.

achoudh5 commented 4 years ago

reviewing...

achoudh5 commented 4 years ago

@tanya3007 I made some comment on your PR!