search

INET-Complexity / Core-ESL

Open-source, distributed, economic simulation libraries for Java and Scala
7 stars 6 forks source link

added agent.MasonScheduledAgent.java #1

Open x0range opened 7 years ago

x0range commented 7 years ago

Hi Paul,

this includes the MasonScheduledAgent class as discussed.

Best wishes

DavoudTaghawiNejad commented 7 years ago

Please rename findNextEvent to scheduleNextEvent, so it is clear what this function does when reading it from the perspective of the agent.

DavoudTaghawiNejad commented 7 years ago

Hi, I integrated this change into the refactored and maven compatible version. This can therefore be closed.

x0range commented 7 years ago

Hi, I added comments consistent with those in other Core ESL modules last week which are not included in your last maven compatible pull request commit. Also, one method is named scheduleNextEvent but should be named getNextEventTime.

DavoudTaghawiNejad commented 7 years ago

OK, I will merge it in then.

DavoudTaghawiNejad commented 7 years ago

Hi,

as I changed the package structure. MasonScheduledAgent is called now SimpleScheduledAgent. In order to facilitate the merge, could you please rename the agent. If you want a different name, rename the agent to SimpleScheduledAgent, and once the branch is merged in you can rename the agent again.

Davoud