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

Fixes issues #74 and #10 (UPDATE: Also fixes issues #77 and part of #34) #89

Merged
merged 14 commits into from
Mar 19, 2018

Conversation

gehrigweber
Copy link
Contributor

@gehrigweber gehrigweber commented Mar 2, 2018

Issue #74 (Fix Surrender) has been resolved - When the button is pressed once, the current player's turn is skipped. When it is pressed twice in a row, a winner is decided. Now works on CSIL.

Issue #10 (Save and Load Game) has been resolved - Pressing the "Save Game" button saves the current game and all settings (music on/off, sound effects on/off, surrender, win/lose, skip turn, score, board size, etc.). The necessary information is stored in "save.ser". Pressing the "Load Game" button loads the saved game. Only one game can be saved at a time.

Issue #77 (Opening on window sizes incorrectly) has been resolved - This issue was reopened because, although it works on CSIL - on Windows, this was found to be an issue. The windows now open up to their correct size on Windows machines.

Issue #34 (Add another mode for a single player to play with computer) has been partially resolved - the Random AI portion (150 points) was created, not the intelligent AI (500 points). Unfortunately, the player must click for the AI to place their stone, but the AI is the one who decides where the stone goes (randomly). The player may have to click multiple times in order for the AI to choose a valid spot. The player will also have to skip the AI's turn if they click repeatedly and the AI won't do anything because there are no more valid moves for it to make.

@gehrigweber gehrigweber changed the title Fixes issues #74 and #10 Fixes issues #74 and #10 (UPDATE: Also fixes issues #77 and part of #34) Mar 15, 2018
@gehrigweber
Copy link
Contributor Author

I had to add the new issue fixes to this pull request because apparently there can only be one pull request active from one branch to another at one time, and this pull request wasn't accepted yet.

@brianslee brianslee merged commit 98c4508 into ucsb-cs56-projects:master Mar 19, 2018
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.

3 participants