search

DanielPerezJensen / mapc-uva

The UVA's entry into the Multi Agent Programming Contest
https://multiagentcontest.org/
1 stars 0 forks source link

move _manhattan_distance(coords1, coords2) from builder to agent #51

Closed YassinAbdelrahman closed 4 years ago

YassinAbdelrahman commented 4 years ago

Could this function be moved from builder.py to agent.py? I think it is a basic function every role could use.

DanielPerezJensen commented 4 years ago

While this may be a good idea for this case, if we allow each midly useful function to go into agent it may bloat it too much and move to far away from the idea we have about what exactly Agent is. Namely a dummy agent that can perform all primitive functions.

YassinAbdelrahman commented 4 years ago

Sounds fair, will just put it in the roles that need it.