Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

naive implementation #2

Open
wants to merge 1 commit into
base: logging-computation
Choose a base branch
from

Conversation

benderpremier
Copy link
Owner

This is a naive implementation of aimacode#81 . I can see a few issues emerging principally around measuring performance based on the state of an agent.

If I take the vacuum world example, we will probably want a performance measure that take into account the number of actions performed by the vacuum cleaner. When moving or sucking, the vacuum cleaner consumes energy and hence we should somehow associate a state to the agent. The idea would be to be able to measure at each point in time, how much energy has been consumed by the agent and score the performance based on that.

The current Agent API has a run method that returns an updated environment but doesn't update the agent state. I am not sure what's the best way to handle it:

  • override the run method when implementing the agent trait ?
  • add a STATE type parameter to the agent trait. Update this state in the run method and return the updated agent alongside the updated environment. But then it feels that we could move the run method out of the agent altogether and put it in a Simulation trait/case class

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant