eishub / tygron

Tygron-EIS Connector
0 stars 5 forks source link

Percept suggestion - zone/6 #9

Closed nskartoredjo closed 8 years ago

nskartoredjo commented 8 years ago

zone/6 - zone(<zone_ID>,<name>,<size>,<max_floors>, <allowed_catagories>, <function_ID>)

<zone_ID> - ID given to the zone <name> - Name of the zone <max_floor> - The maximum allowed amount of floors to be set for the zone <size> - The Amount of squared meters of zone ground <allowed_catagories> - The allowed catagories of the zone <function_ID> - The function id of the zone

General info about the different zones in the map.

NickTUD commented 8 years ago
jannelie commented 8 years ago

wat wordt er bedoeld met <function_ID>, is dit een lijst met ID's voor elke functie die mogelijk is voor die zone, of wordt er iets anders mee bedoeld?

TomBrunner commented 8 years ago

Het lijkt erop dat categories al beschrijft wat er wel/niet gebouwt mag worden. En functies zijn alleen de functies voor het bouwen van dingen. Dus daarom lijkt het me dat functies weggelaten zou moeten worden.

jannelie commented 8 years ago

Ik denk dat dit een goed idee is, omdat je niet onnodige informatie wilt hebben.

MaxGroenenboom commented 8 years ago

Hoe weet mijn agent in welke categorie een functie zit?

jannelie commented 8 years ago

Een functie heeft een lijst met categorieen waartoe die fubctie behoort

MaxGroenenboom commented 8 years ago

Maar voor functies is geen percept toch?

jannelie commented 8 years ago

Ja die stond er al in

MaxGroenenboom commented 8 years ago

oh my bad, ik gebruik nu de issue tracker als overzicht van de percepts, omdat sommige percepts opnieuw gedefinieerd zijn

MaxGroenenboom commented 8 years ago

Volgens mij zijn we dinsdag tot de conclusie gekomen though dat we functions los meesturen om overhead in de agent te verminderen (correct me if I'm wrong)

jannelie commented 8 years ago

Dat zou ik moeten navragen, maar wij kregen het idee dat functies opnieuw meesturen juist overhead gaf.

MaxGroenenboom commented 8 years ago

ja in een database klopt dat, maar hier maakt memory overhead minder uit, dus was het efficienter om dingen er meerdere keren in te hebben. Ik vond het zelf ook beter om alleen categories te doen, maar waarschijnlijk is het met functies erbij wel sneller voor de agent.

jannelie commented 8 years ago

oke, dan zullen we proberen om die functie ids weer erin te stoppen.

jannelie commented 8 years ago

We hebben er nog eens goed naar gekeken,, maar het toevoegen van functie Id's levert weinig voordeel. Je kunt heel gemakkelijk met de lijst van categories de juiste functies vinden door het percept van functies. Dit haal je één keer binnen, en anders geef je elke keer van de zone alle functies mee die je kunt doen. Terwijl je met categories dezelfde informatie hebt. Daarom hebben we het percept veranderd naar zone/5 en de lijst met funcite ID's weggelaten - Crash Test Dummies

MaxGroenenboom commented 8 years ago

Ik vind het prima, dat was in eerste instantie al hoe ik het wilde. Dinsdag hebben we besloten het op de oude manier te doen dus vandaar dat het er zo in stond ;) Ik zou ook zorgen dat de beschrijving hier nog klopt, en misschien een performance test doen om te zien welke methode sneller is? Ik weet niet of dat haalbaar is in goal hoor, maar het is een poging waard.