New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add performence tests / performance awareness #25

Open

Bonifatius94 opened this issue Nov 19, 2020 · 1 comment

Labels

Owner

Bonifatius94 commented Nov 19, 2020

Issue:

performance is currently not properly measured
if the performance is unknown, it's really hard to make improvements
high performance is the key feature of this project (!!!), needs to perform better than naive implementations

Suggested Solution:

think of useful ways to measure the performance (e.g. code profiling tools showing elapsed times of each function call during a high-performance test)
think of performance constraints when releasing the package (critical functions may not take significantly longer than on previous releases)

Bonifatius94 added the enhancement label

Owner Author

Bonifatius94 commented Feb 16, 2022

The performance issues might relate to lots of Python C-API calls which seem to be rather slow. Therefore think of ways to save API calls:

just expose a function predict_best_draw(chessboard), keep the rest local
add a transposition table for caching already evaluated positions
implement the AI part with TensorFlow in C/C++ rather than in Python

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment