For results maybe do a side by side plot and for table results try doing visualization with a plot. It's hard scrolling up and down to read it.

yaz-saleh commented 3 years ago

Group discussion notes:

[ ] Adjust sizes of plots to lay them down side by side with the table (if possible and looks good)

lephanthuymai commented 3 years ago

Hi @ericlee0920 , My understanding is that there are 2 requirements for this task:

Wrap figures inside paragraphs so that the reader does not need to scroll down to compare the results with the figures
Instead of tables, try visualizing them with charts when possible

Please confirm, thank you very much.

lephanthuymai commented 3 years ago

Regarding #2, after reading the report carefully, I think it is easier to review in the table format instead of charts as it is a standard format when we reviewing models and view classification report, hence no change is made regarding this point.

Regarding #1, I tried wrapping figures inside paragraphs, but it does not look better than arranging figures and tables side by side to reduce the need of scrolling up and down, so I've changed the report to work that way instead.

Please review the generated report again once the PR is merged to the main repo and let me know if you still see any issues.

UBC-MDS / DSCI_522_group_31

For results maybe do a side by side plot and for table results try doing visualization with a plot. It's hard scrolling up and down to read it. #55