search

eishub / tygron

Tygron-EIS Connector
0 stars 5 forks source link

add functionID to building percept #45

Closed dionytadema closed 8 years ago

dionytadema commented 8 years ago

in order to percept if and how much AZC's the municipality constructed we need to be able to retrieve the function id of a building

suggested building percept would than be building(Id, Name, OwnerID, ConstructionYear, Categories, FunctionID, Floors)

MaxGroenenboom commented 8 years ago

Can't a building have multiple functions? Like shops on the ground floor and the upper floors apartments.

dionytadema commented 8 years ago

No a building can only have one function

MaxGroenenboom commented 8 years ago

Seems like you're right, I could've sworn I saw a building in the engine once with multiple functions ^^

MaxGroenenboom commented 8 years ago

The amount of units or floorsize is already used in an indicator I suppose? Why not use that indicator instead of letting the agent find it out itself?

dionytadema commented 8 years ago

that is a custom indicator and those are broken right now, and even if they would work it is still usefull to be able to check a building for it's function.

I already have the changed percept working localy so i can make a pr anytime others agree with this issue

MaxGroenenboom commented 8 years ago

True, I'm in favor of this enhancement anyways.