Määrittelydokumentti oli selkeä ja tyhjentävä, ainakin kun tietää vähän ennestään käsiteltävistä asioista.
Koodia ei ole vielä hirveästi arvioitavaksi - kannattaa ehkä vähän kiirehtiä. Käytännöt ovat hiukan sekalaiset, esim algorithm.hpp:ssä sisennetään perusnamespace, image.hpp:ssä ei.
algorithms.hpp:ssä on sekalainen seura funktioita, mitään muista noista en kutsuisi algoritmiksi kuin FFT:tä. Jos jotain noista funktioista ei tarvitse muuhun kuin FFT:n toteutukseen, ne kannattaisi ehkä määritellä vain suoraan cpp-filussa (paitsi jos haluat erikseen testata ne). Muut kannattaisi siirtää joihinkin utils.hpp:hen tai johonkin, algoritmeista ei ihmiset osaa niitä etsiä.
Imagen loadausfunktio on aika pitkä ja vähän rikki - mitä tapahtuu jos size ei olekaan width*height? Tai miksi käytetään raskasta std::polar-funktiota kun kulma on 0, kun voisi antaa suoraan reaaliosana ja imaginaariosana?
Dynaamisen muistinvarauksen hoitaminen käsin ei pidemmän päälle tule olemaan mukavaa, suosittelen toteuttamaan esimerkiksi kompleksilukutaulukot ihan luokkina, jotka hoitaa muistinhallinnan automaattisesti.
FFT:hen toivoisi vähän enemmän testejä. Mutta ymmärrän että on vielä kesken.
Kyllähän tässä vaiheessa olisi melko helppoa lähteä jatkokehittämään, vaikka kaikista osista testit eivät ihan vakuuta.
Nyt kun katsoo uutta koodiasi niin aika moni näistä on jo korjattu, mutta kun käskettiin arvioida viime torstain versio niin arvioidaan viime torstain versio. ;D