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.

Sign up for GitHub

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

Jump to bottom

Add basic Region actor behavior #26

Merged
merged 5 commits into from
Nov 10, 2018
Merged

Add basic Region actor behavior #26

merged 5 commits into from
Nov 10, 2018

Conversation

NotBad4U
Copy link
Member

@NotBad4U NotBad4U commented Nov 7, 2018 •
edited
Loading

Part of the work for #16

This PR add the skeleton for the Region actor:

  • calculate the AOI of a player
  • Define a ID
  • Store the player and energy in a Set.
  • Create the initial state with unique ID for each entities: we use the actorRef as unique ID
  • Initiate a new game stage

The Arbitrator should use the new Region interface.

@NotBad4U NotBad4U changed the title [WIP] Add basic Region actor behavior Add basic Region actor behavior Nov 9, 2018
@NotBad4U
Copy link
Member Author

NotBad4U commented Nov 9, 2018

Can you start a review @d-plaindoux ?

@NotBad4U NotBad4U requested a review from d-plaindoux November 10, 2018 00:02
@NotBad4U NotBad4U self-assigned this Nov 10, 2018
@NotBad4U NotBad4U added this to the first version milestone Nov 10, 2018
d-plaindoux
d-plaindoux approved these changes Nov 10, 2018
View reviewed changes
Copy link
Member

@d-plaindoux d-plaindoux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since the 'GetEntitiesAOISet' is stashed during the initialisation stage this arbitrator can send its 'GetEntitiesAOISet' to start the game at any moment. That's fine.

@d-plaindoux d-plaindoux merged commit dd04f14 into master Nov 10, 2018
@NotBad4U NotBad4U deleted the region-actor branch November 10, 2018 18:25
This was referenced Nov 10, 2018
Open
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@d-plaindoux d-plaindoux d-plaindoux approved these changes

Assignees

@NotBad4U NotBad4U

Labels
None yet
Projects
None yet
Milestone
first version
Development

Successfully merging this pull request may close these issues.
2 participants
@NotBad4U @d-plaindoux