SimoKorkolainen / ImageRecognition

0 stars 0 forks source link

ImageRecognition - Vertaisarviointi #2

Closed ghost closed 8 years ago

ghost commented 8 years ago

Koodi on varsin kattavaa, ja yksikkötestaus on toteutettu varsin hyvin muun kodin ohella. Koodin toimintaan en sen enmpää osaa ottaa kantaa. Yksi syy tähän on, että koodi näyttää olevan lähes kauttaaltaan kommentoimatonta, mikä tekee koodin tutkimisesta ulkopuolisen toimesta (ml. allekirjoittanut) paikoitellen todella haasteellista. Mm. metodien toiminnan tarkoitusta ei ole millään lailla tuoda esille. Poikkeuksena ovat functions.activation-paketissa olevat luokat. Jok'ikistä luokkaa en projektista lähtenyt tutkimaan, mutta suuren osan kävin läpi.

Projektin muu dokumentaatio on toteutettu erittäin mallikkaasti ja erityisen informatiivisesti. Tästä peukut.

Terveisin, Markus

SimoKorkolainen commented 8 years ago

Kiitos palautteestasi. Olet oikeassa siitä, että koodin kommentoinnissa olisi parantamisen varaa. Unohdinkin kommentoida koodia viime viikolla.

Terveisin

Simo