Make learner work on top of stockfish's thread pool instead of MultiThink. #200
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a rewrite (restructuring) of the learner code so that it uses the newly added feature that allows execution of arbitrary tasks on stockfish's thread pool. In particular it:
Entropy
struct to make passing and summing all the entropy statistics around the code easierAs this is a more or less a complete rewrite the diff is kinda hard to read, best to compare whole files side by side. If you think this should be split further I can try, but in my opinion looking at this in its final stage is easier to compare than looking at intermediate steps in isolation.
As a side effect it may have [partially] fixed #164.