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

Code refactoring #40

Merged
merged 4 commits into from
Apr 26, 2019
Merged

Code refactoring #40

merged 4 commits into from
Apr 26, 2019

Conversation

emwestin
Copy link
Collaborator

  • Makes use of the Dumb7Fill algorithm (does not include Erins' bug fix, add again if needed).

  • Added Negamax with alpha-beta pruning replacing Minimax. Should allow for easier parallelization [Parallelized AlphaBeta #29]

  • Added iterative deepening [Iterative Deepening #31]

  • Added a rough evaluation function. Far from good [Evaluation function #3]

  • Simplified how the "main" file plays the game. Should be much simpler now.

* Makes use of the Dumb7Fill algorithm (does not include Erins' bug fix, add again if needed).

* Added Negamax with alpha-beta pruning replacing Minimax. Should allow for easier parallelization [#29]

* Added iterative deepening [#31]

* Added a rough evaluation function. Far from good [#3]

* Simplified how the "main" file plays the game. Should be much simpler now.
Copy link
Owner

@ErinvanderVeen ErinvanderVeen left a comment

Choose a reason for hiding this comment

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

I'm not going to merge this into master. Please first merge master into this branch before creating a PR.

src/ai.h Outdated Show resolved Hide resolved
src/ai.h Outdated Show resolved Hide resolved
src/ai.h Outdated Show resolved Hide resolved
src/state_t.h Outdated Show resolved Hide resolved
@ErinvanderVeen
Copy link
Owner

I have approved the changes. You can merge when you want to! @emwestin

@emwestin emwestin merged commit 4e4bd96 into master Apr 26, 2019
@ErinvanderVeen
Copy link
Owner

I noticed you switched from 1 tab to 4 spaces. Was this on purpose?

@emwestin
Copy link
Collaborator Author

Oops, that was caused by my IDE not cooperating. Did not notice before pushing.

@ErinvanderVeen
Copy link
Owner

I'll create a new PR to rectify this.

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.

None yet

2 participants