official-pikafish / Pikafish

UCI xiangqi engine
http://pikafish.org
GNU General Public License v3.0
867 stars 162 forks source link

merge master to wasm branch #35

Closed ousc closed 1 year ago

ousc commented 1 year ago

I wrote some tests using Karma, but I'm not sure whether these tests can be submitted to pikafish's project. After testing, the single-thread and multi-thread versions of the merged code can be built and work normally.

PikaCat-OuO commented 1 year ago

Emmm, this PR does not contain [ x ] merge master to wasm branch until 5, March?

ousc commented 1 year ago

Emmm, this PR does not contain [ x ] merge master to wasm branch until 5, March?

I just committed them, but no push, now it's ok

PikaCat-OuO commented 1 year ago

I wrote some tests using Karma, but I'm not sure whether these tests can be submitted to pikafish's project. After testing, the single-thread and multi-thread versions of the merged code can be built and work normally.

I think this is the right place to place the tests: https://github.com/official-pikafish/Pikafish/tree/wasm/tests Anyway, it looks good to me, so I just merge it now.

ousc commented 1 year ago

In fact, I'm not sure whether the existing test can provide tests for wasm.