Ohjelma toimii hyvin. Käyttöliittymä oli alkuun hieman epäintuitiivinen, mutta hetken käytön jälkeen sujui kivasti. Ohjelman perusteella Markovin ketju toimii oikealla tavalla, pystyin selvästi huomaamaan parannuksen ketjun asteen kohotessa. Hyvä te.
Ehkä ainoa selvä parannus olisi midi tiedoston automaattinen nimeäminen (ellei muuten ole määritelty). Olisi kiva hetken aikaa luoda midi tiedostoja erilaisilla säädöillä ennen kuin lähtee vertailemaan niitä toistimessa. Tällä hetkellä oletuksena on, että vanha (saman niminen) tulee ylikirjoitetuksi, ellei aina erikseen vaihda polku muuttujaa. Omasta näkökulmasta olisi kätevää, jos ohjelma lisäisi, vaikka aikaleiman nimen perään, jolloin erilaisia tiedostoja olisi nopeaa ja kätevää tehdä. Pikku juttu sinänsä.
Toinen käytettävyyttä parantava tekijä olisi käyttää ohte:sta tuttua poetry riippuvuuksienhallintaa ja invoke komentojärjestelmää.
Koodi
Koodi näyttää pääosin erittäin hyvältä. Annan kuitenkin muutaman pienen parannus ehdotuksen.
Trie solmun voisi laittaa omaan tiedostoon.
Turhia hastag kommentteja esim. trie tiedostossa voisi poistaa ja pysyä ainoastaan docstringissä.
Markovin ketjun aste on rajoitettu alaspäin, mutta ei ylöspäin. Esim. arvoksi voi laittaa miljoonan mikä ei ole teoriassakaan mahdollista toteuttaa näin pienellä opetusdatalla.
Testaaminen
Ensimmmäinen testi yritys johti seuraavaan virheeseen:
FileNotFoundError: [Errno 2] No such file or directory: 'src/tests/testiMidi2.mid'
Oman nopea päätelmä on, että testi luo testaus midin nimellä "testiMidi", mutta testiohjelman toinen testikomponentti etsii väärää nimeä; "testiMidi2"
Kokeilin seuraavaksi muuttaa testiMidi tiedoston nimen testiMidi2 siinä toivossa, että testit menisivät läpi. Eteen tuli kuitenkin toisenlainen virhe:
FAILED src/tests/midi_kasittelija_test.py::TestMidiKasittelija::test_molli_ja_transponointi - IndexError: list index out of range
Jätin testit sikseen.
Loppusanat
Ohjelman ydinkomponentit on totetettu onistuneesti ja muutamaa 'epäkohtaa' lukuunottamatta ohjelma on todella hienossa vaiheessa! Löytämäni 'virheet' ovat muutenkin enemmän kosmeettisia kuin ydinongelmia, joka sekin kertoo jo paljon. Uuden aiheen oppiminen ja sen soveltaminen ei ole helppo juttu, olette kuitenkin projektissanne osoittaneet erinomaisuutta tällä saralla! Kannattaa muistaa nauttia näistä pienistä voitoista.
Koodikatselmointi 2
viikko 5 projekti ladattu: 15.10.2022 17:13
Sisällysluettelo
Käyttö
Ohjelma toimii hyvin. Käyttöliittymä oli alkuun hieman epäintuitiivinen, mutta hetken käytön jälkeen sujui kivasti. Ohjelman perusteella Markovin ketju toimii oikealla tavalla, pystyin selvästi huomaamaan parannuksen ketjun asteen kohotessa. Hyvä te.
Ehkä ainoa selvä parannus olisi midi tiedoston automaattinen nimeäminen (ellei muuten ole määritelty). Olisi kiva hetken aikaa luoda midi tiedostoja erilaisilla säädöillä ennen kuin lähtee vertailemaan niitä toistimessa. Tällä hetkellä oletuksena on, että vanha (saman niminen) tulee ylikirjoitetuksi, ellei aina erikseen vaihda polku muuttujaa. Omasta näkökulmasta olisi kätevää, jos ohjelma lisäisi, vaikka aikaleiman nimen perään, jolloin erilaisia tiedostoja olisi nopeaa ja kätevää tehdä. Pikku juttu sinänsä.
Toinen käytettävyyttä parantava tekijä olisi käyttää ohte:sta tuttua poetry riippuvuuksienhallintaa ja invoke komentojärjestelmää.
Koodi
Koodi näyttää pääosin erittäin hyvältä. Annan kuitenkin muutaman pienen parannus ehdotuksen.
Testaaminen
Ensimmmäinen testi yritys johti seuraavaan virheeseen:
Oman nopea päätelmä on, että testi luo testaus midin nimellä "testiMidi", mutta testiohjelman toinen testikomponentti etsii väärää nimeä; "testiMidi2"
Kokeilin seuraavaksi muuttaa testiMidi tiedoston nimen testiMidi2 siinä toivossa, että testit menisivät läpi. Eteen tuli kuitenkin toisenlainen virhe:
Jätin testit sikseen.
Loppusanat
Ohjelman ydinkomponentit on totetettu onistuneesti ja muutamaa 'epäkohtaa' lukuunottamatta ohjelma on todella hienossa vaiheessa! Löytämäni 'virheet' ovat muutenkin enemmän kosmeettisia kuin ydinongelmia, joka sekin kertoo jo paljon. Uuden aiheen oppiminen ja sen soveltaminen ei ole helppo juttu, olette kuitenkin projektissanne osoittaneet erinomaisuutta tällä saralla! Kannattaa muistaa nauttia näistä pienistä voitoista.
Hyvää syksyä!