pwyq / Fill-Game

Two Player (Adversarial) Version of Fillomino | (Depth-First) Proof Number Search, Monte Carlo Tree Search, Minimax, Negamax, Alpha-beta pruning
GNU General Public License v3.0
2 stars 0 forks source link

feat: implement MCTS #70

Closed Samarium150 closed 1 year ago

Samarium150 commented 1 year ago

Close #46

Samarium150 commented 1 year ago

which part is the rollout of MCTS

simulate

pwyq commented 1 year ago
You may not use a using-directive to make all names from a namespace available.

// Forbidden -- This pollutes the namespace.
using namespace foo;

https://google.github.io/styleguide/cppguide.html#Namespaces

I am changing every using namespace solver::helper;

Samarium150 commented 1 year ago
You may not use a using-directive to make all names from a namespace available.

// Forbidden -- This pollutes the namespace.
using namespace foo;

https://google.github.io/styleguide/cppguide.html#Namespaces

I am changing every using namespace solver::helper;

Did I use it somewhere? I totally forgot.

pwyq commented 1 year ago

How did you run MCTS

I am doing

      MCTS* mcts = new MCTS(game);
      res_move   = mcts->search();
      delete mcts;

I got

terminate called after throwing an instance of 'std::invalid_argument'
  what():  Cannot play at 2 2 with value 2, which is already filled. Full game = 22.*131*..2
Samarium150 commented 1 year ago
terminate called after throwing an instance of 'std::invalid_argument'
  what():  Cannot play at 2 2 with value 2, which is already filled. Full game = 22.*131*..2

Probably something wrong with simulate

pwyq commented 1 year ago

PR #71 is ready. It will marge into this 46-mcts branch