Hello, under what license is this project released under ? Thank-you.

Vertaisarviointi

Projekti kloonattu pe 27.4. kello 22.30. README:ssa olleiden ohjeiden mukaisesti. Ohjelma käynnistyi ohjeita noudattamalla. Käskyllä help tulostui lista komennoista.
Suoritin ohjeiden mukaisen esimerkkiskenaarion

load res/scenario1.json
plan
show plan

Ohjelma tulosti

[Start] -> Go to tree -> Pick apple -> [Goal] (cost: 2, actions: 2)

Kirjoitin step jolloin ei tapahtunut mitään. Ohjelman lopetus exit komennolla toimi.

Muut help-komennolla saatavat käskyt ilmoittivat [käsky: ]Command not found.

Ohjelman dokumentaatio on hyvin laaja. En varsinaisesti saanut mitään kokonaiskuvaa siitä, mitä ohjelmalla on tarkoitus tehtä. Ehdotan että kirjoittaisit vaikkapa README:hen siitä.

Luokat vaikuttavat puhtaalta koodilta. Javadocit voisivat olla kuvaavampia jotta ulkopuolinen ymmärtäisi paremmin mitä kukin luokka ja metodi tekee ja mikä sen tarkoitus on.

Testauskattavuus on erinomainen.

Päivä ja kellonaika: 20.4, ~20

En ole kyllä aivan varma mitä ongelmaa tässä yritetään ratkaista, joten saatan olla väärässä, mutta,

Jos ongelma on rajoittunut booleaneihin dagissa, optimaalisen ratkaisun saa helposti lineaarisessa ajassa pitämällä yllä lopulliseen tavoitteeseen tarvittavia prerequisite-actioneja jotka voi tehdä, ja suorittamalla niitä yksi kerrallaan, samalla tarkistaen onko jokin uusi prerequisite-action nyt mahdollinen.

Yleisemmässä tapauksessa, jos jokin actionien property estää aikaisemman, kuten esim jos action vaatii muuttujan olevan positiivinen, ja jotkin actionit vähentävät tätä muuttujaa, käyttämällä dp:tä kaikkien statejen yli, niin että tilassa säilytetään minimipenalty, voi saavuttaa tyyppiä 2^n aikavaatimuksen n^n sijasta. Käytännössä dp on kyllä varmaan hidas, koska siihen on vaikea saada mitään heuristiikkoja.

HeapAlgorithm:n rivillä 82,
plans.addAll(plansToAdd);
Tämä rivi tekee, jos k on lisättävien määrä, k log(k) työtä. Tekemällä vaan

PriorityQueue<SubPlan> tmp = plans;
plans = plansToAdd;
plansToAdd = tmp;

Saavuttaa saman asian vakiomäärällä työtä, koska tässä kohdassa plans on aina tyhjä. Ei kyllä vaikuta koko algon aikavaativuuteen.

Testit toimivat hyvin.

Pikkujuttuja:

• help-komento ei listaa use-komentoa.
• wikissä sivulla "What is GOAP?" linkit wikin muille sivuille ovat hajalla