Connect (m,n,k) game + a few minor changes

[LucasBorboleta]

Hi Paul !

I hope things go well for you.

Being in the doubt ( either 1) you are busy with other stuff and that's fine or 2) you are expecting me to rework my branch) I have reworked my branch "LucasBorboleta" in the form of a new branch "LucasBorboleta2" by taking into account (I hope so) your feedback from my former branch.

Lucas

[pbsinclair42]

Hi Lucas, sorry for the delay in getting to these, I've been super busy the past couple of months I'm afraid. Thanks for sticking with this, I'll get to reviewing this once life is slightly less crazy.

[LucasBorboleta]

Hi Paul ! Good to hear you. I hope that the important things in life are going right for you. Don’t mind for my pull request ; it is hobby. Cheers, Lucas

[kstruempf]

Hey :) Just wanted to check in and see if you have plans to merge these changes @pbsinclair42? Briefly looking them over they seem very reasonable 👍

[LucasBorboleta]

Konstantin,
I am happy that you appreciate my contribution. The intent was to provide a simple, but complete, example of a usage of Paul's MCTS library, which sounds good for discovering the Monte Carlo Tree Search approach.
Unfortunaltey, Paul seems to be busy with other stuff in his life.
Cheers, Lucas

[kstruempf]

@LucasBorboleta since this repo seems to be inactive I decided to fork it and keep working there. I did add this PR as a patch to my code base. I am using this package in another project so it will be nice if it follows conventions...
I plan on making a couple of changes like renaming the function / classes so that they follow the Python Naming Convention, structuring the codebase to multiple packages and add abstract classes for State and Action so that they can be extended more easily by users of the package and also ad a CI/CD pipeline (GitHub Actions).

[kstruempf]

I did get started on my endeavour in this repo and published v2.0.0 pypi package in case anyone is interested :)

[LucasBorboleta]

Konstantin,
Good to know that you are making this MCTS projet alive. Applying common Python naming convention might also help.
In a way of you choice, could you mention, my name "Lucas Borboleta" as associated to the example "Connect(m,n,k)" ? Well, unless it appears automatically from Git historical mechanims. Thanks.

[kstruempf]

Sure :) I added a patch of this PR to the repo so your account is linked with the corresponding commit, but I will mention you in the readme as well as the author of the example :)

[LucasBorboleta]

All right !