Tom Madams
Tom Madams
The TFLite C++ runtime now supports the Coral Edge TPU: https://coral.ai/docs/edgetpu/tflite-cpp/ We should hook that up.
On the one hand, it's more efficient for the CPU to generate NCHW input features. On the other, TensorFlow supports NHWC convolutions on a wider variety of platforms. When I...
`GUARDED_BY` -> `ABSL_GUARDED_BY` `LOCKS_EXCLUDED` -> `ABSL_LOCKS_EXCLUDED` etc
We need to add support for the following required GTP commands: - playsgf - select_position And optionally: - prune_nodes - winrate_evals
That way the tree's "root" is actually the real root of the tree. Also the current implementation of the `undo` GTP command is completely broken I think because it just...
The `cc` directory is getting a little out of hand. It could do with having a few new subdirectories added, e.g. `base`, `bin`. Maybe move things like `symmetries.*` into the...
In order to get multiple models running on the same TPU, we had to add the following hack to work around a bug in TensorFlow: https://github.com/tensorflow/minigo/blob/d5e9697318b364ccc2b977a4805a96f1e72e7abc/dual_net.py#L523 This bug got fixed...
When tree search reporting is enabled (`report_search_interval_ > 0`), the `GtpPlayer` always writes the full search state: visit counts, delta Q, current search, etc. Not all frontends care about all...
The GTP spec says our extension commands should be prefixed with something like `mg-`. Also, let's replace `__GTP_CMD_DONE__` with `mg-gtp-cmd-done` so that it looks more like the `mg-position` and `mg-update`...