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

Automated ship movement - claiming resources #92

Merged
merged 1 commit into from
Aug 1, 2018

Conversation

miniature-tiger
Copy link
Owner

When ships reach new islands, resources are revealed. This update focuses on automating the decision to claim those resources by computer opponents. A simplified approach has been used initially but scoped to allow updates to more complex strategies.

automove.js

  • Addition of method to check whether there are resources to be claimed by computer opponents and decide whether a resource should be claimed

main.js, resource.js

  • Code for claiming of resource tile moved from main.js to resource.js and updated to work for both human and computer opponent game logic.

movement.js

  • Separate method for post ship transition actions created (conflicts, harbour repair, land discovery). [May convert to async if more transitions actions created].

gameManagement.js

  • Update to separate array console.logs from workflow console.logs and add a new button to the developer panel. Array logs are useful in checking whether all game elements are being captured but not always necessary when checking the flow of game logic.

When ships reach new islands, resources are revealed. This update focuses on automating the decision to claim those resources by computer opponents. A simplified approach has been used initially but scoped to allow updates to more complex strategies.
@miniature-tiger miniature-tiger merged commit f117420 into master Aug 1, 2018
@miniature-tiger miniature-tiger deleted the miniature-tiger-patch-85 branch November 13, 2018 10:40
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