Open jonekeat opened 1 year ago
Here is my feedback:
There is a repetition of code. This can be improved by using reactive value for this part https://github.com/clementtcy/R-Shiny-Training-Assessment/blob/main/app.R#L51-L77
The table output can be improved by adding column name & row name (i.e. ''Loss Year" & "Development Year")
The plot output can be improved by formatting the y-axis so that development year is not in decimal
Hi Clement, thanks for attempting the shiny assessment, please find my feedbacks below:
The interface fully fulfilled all requirements, user is able to input some data, calculate and visualize result all in the app, well done!
The app is also able to handle dynamic number of loss year & development year, which is very nice!
There are a lot of empty spaces on the top right corner, but user need to scroll up & down to check input & output. Maybe the UI can be further improve?
For the tail factor input, we should specified the
min
,max
andstep
arguments, maybe should put from 1 to 2, and the each step should increase/decrease by 0.1 instead of 1 (the default) https://github.com/clementtcy/R-Shiny-Training-Assessment/blob/91832bb9f57fa1cb701d1a4449dbc1983ddcbe00/app.R#L5Since the output number are currency values, we can format it using accounting format (put thousand separator), especially y-axis of graph & table cells
The table formatting is quite good, but we can enhance the table cell formatting, like putting different cell/background color to differentiate which cell is input values and which is projected.