Skip to content

Latest commit

 

History

History
35 lines (24 loc) · 1.95 KB

Suggestions.md

File metadata and controls

35 lines (24 loc) · 1.95 KB

Suggestions

Report remarks

About reports

  • Do not put code in your report
  • Next time use as an email topic which follows this pattern: [TUM][FAF Labs][WP] Name Surname
  • Next time name your report file as - WP FAF 111 Name Surname Lab#0 (replace your name, surname and lab number)
  • Next time send only report in PDF format
  • Inside your report - add a link to your lab repository (I know where it is, but it will be easier for me to locate your repository)
  • Better use a report template provided in repository
  • You said nothing about PSG (At least choose one existent that matches your style. If it has any differences - mention them.)
  • Spellcheck your report/documentation/code (even if I don't do it always)

About repository

  • Rename your folder names in your repository to follow lab#0 pattern
  • For each laboratory work add one README file (in its own folder)
  • Keep your finished code in master branch. It is awesome that you get how to work with branches, but proposed work flow is a bit different: use master branch for submission ready code, and use other branches for development and testing. For details look at submission process page
  • It seems that you compiled your project before adding .gitignore. Please remove unnecessary files from repository (foldes bin and obj) and next time check your staged files before making a commit.

About code

  • Your project code is padded using tabs and spaces (it is not consistent and looks strange from an editor with different tab spacing (mine has 1 tab = 2 spaces)). Try to setup your project/IDE in such an order that it will be consistent.

General things

  • Please make an effort and count your points
  • Please next time count your points using provided table (there is a table in report template)
  • Take in account that your next deadline is on 8th of march (even if it is holiday)

Gained points

todo