discussion on improvement for milestone4

[czwcandy]

[Chris Gao]:

• [x] add more description in the READme

[ paolocodina]:

• [x] -filter, make the plots clearer, just add introduction in READme file [Anu]
• [x] -change the font and the shape of the 'Load Data Frame' button [weilin]
• [x] Adding a small section to display the author's name would be beneficial. [weilin]

[Sophia Zhao]

• [x] -terms like 'apparent temperature' and 'sum of shortwave radiation' might be confusing., we can change [Anu]
• [x] (add a link on dashboard to READme file) [Anu]

[John] -no more, already finished 3/5

[Joel]

• [x] -Invert cards to be green background and white text font [weilin]
• [x] -Clearer about terms like "apparent temp" etc. Fix axis titles. [Anu]
• [x] -Style for reload button [Weilin]
• [x] -Remove "Date" from x-axis [Katherine]
• [x] - Make font in parenthesis smaller [weilin]
• [x] - Why is the date popup not close to the widget [weilin]
• [x] -Select only most important temperatures (finish in READme) [Anu]
• [x] - Link to repo, names, at the bottom of the sidebar [Anu]

[Milestone 4]

• [x] Reflection [Katherine]
• [x] Use a binary format for loading data faster [oak]
• [x] Have no redundant filtering/mutating of the data in the app [oak]
• [x] Create a processed data file that is loaded in the app instead (stored in data/processed). [oak]
• [x] Have no excessively slow computation that could be easily sped up, e.g. no row-wise use of apply unless well justified and optimized.[oak]
• [x] Use caching for at least one function [oak]
• [x] Set a favicon and tab title [weilin]
• [x] Update the GIF animation. Use the same path as before and overwrite your previous GIF with a new once that reflects the final stage of your app. [Katherine]
• [x] Update any screenshots and descriptions in your README that are outdated. [Anu]
• [x] Including installation instructions that allow for the teaching team to download and run your app in a reproducible environment (Docker not needed). -finished
• [x] The same for the description and topics in the "About" section of your GitHub repo (add topic like 'calendar' under 'about') [Anu]
• [x] Assess whether it is necessary to have the Altair/Plotly toolbars that show up on hover. If not, remove them (especially if you have many charts). A good reason for keeping them would be if it is important to e.g. download the charts. -keep the tool bars

[czwcandy]

we finished all feedbacks, issue should be closed. Thanks everyone.