Performance trap solver.reset() about connect4 HOT 3 OPEN

Thanks Colin for your feedback, I'll remove the reset line.

You are absolutely right that there is no need to reset the transposition table between each position resolution. The reset was there as my last version was used to generate an opening book (available here https://github.com/PascalPons/connect4/releases/tag/book).
To generate such book I solved all <12 depth position and kept the most complex positions, complexity being measured by the number of explored node. Without reseting the transposition table, the number of explored nodes (and processing time) was highly biased by the previous state of the transposition table.

Second point, be careful that the transposition table is not thread-safe. So make sure to have one instance of solver and transposition table per thread. for a batch resolution, it could be more convenient to run multiple process rather than multi-thread.

from connect4.

I actually call your solver from python via cython, so I have no choice but to make multiple processes :)

from connect4.

Thanks Colin for your feedback, I'll remove the reset line.

You are absolutely right that there is no need to reset the transposition table between each position resolution. The reset was there as my last version was used to generate an opening book (available here https://github.com/PascalPons/connect4/releases/tag/book). To generate such book I solved all <12 depth position and kept the most complex positions, complexity being measured by the number of explored node. Without reseting the transposition table, the number of explored nodes (and processing time) was highly biased by the previous state of the transposition table.

Second point, be careful that the transposition table is not thread-safe. So make sure to have one instance of solver and transposition table per thread. for a batch resolution, it could be more convenient to run multiple process rather than multi-thread.

About generating an opening book, you don't have to reset TT if all your positions set (file) you load, are with a same length.

from connect4.