36-650-Fall-2020 / fp-unit-test-assignment-YixuanLuo98

fp-unit-test-assignment-YixuanLuo98 created by GitHub Classroom
1 stars 0 forks source link

main.py function wrap up #2

Open alexazhu opened 3 years ago

alexazhu commented 3 years ago

The code is clear and concise. It is an option to wrap up the csv.reading and data preprocessing into functions for reading purposes, also the user would be able to import and call for one function for example covid_info() to run the interface instead of running the whole file. Thanks.

YixuanLuo98 commented 3 years ago

Thanks for bringing it up! I was actually thinking of the same thing when I read your code. I figured it would be nice to make the read() functions separately and then use it for displaying purposes in the main file! Then I could define how to process the data in another file! This way the main section looks a lot cleaner. I also liked it that you mentioned covid_info() which I forgot to write something similar to make sure this code can be re-runned as well. Thanks!!