search

marekpiotradamczyk / bestiazwroclawia

7 stars 7 forks source link

Add TT implementation. #28

Closed DomiKoPL closed 3 months ago

DomiKoPL commented 4 months ago

I have to add TT to Search. However search currently is just a simple function, so I would need to add it as a parameter. Weird. I have to think if I should do it correctly now or temporarily do it like that.

Gustav2137 commented 4 months ago

TT should be either a global variable or we can wrap search into some class and put TT as a field there.

DomiKoPL commented 4 months ago

Yes, we should. There should be a class that implements Engine like MinMaxEngine or sth.

Gustav2137 commented 4 months ago

Oh that's what the engine class is for. I can refactor code to fit this template then.

Gustav2137 commented 4 months ago

I have just committed changes we were discussing to #25. Feel free to review them. Edit: *Feel free to review them as soon as I fix whatever I broke in the current code.

Gustav2137 commented 4 months ago

Ok done.

Gustav2137 commented 3 months ago

lgtm, merging