mhonert/velvet-chess

:chess_pawn: Velvet Chess Engine - written in Rust

velvet-chess's Issues

Build Issue: thread '<unknown>' has overflowed its stack

This happens on my Ubuntu box, and on a WSL Ubuntu image. I have attempted to increase the stack size, by cd'ing into the engine dir and building via cargo rustc --release --bin velvet -- -C link-args=-Wl,-zstack-size=16777216. This did not work.

Analog Hors suggested the following code change to spawn_engine_thread(), which worked for him.

    std::thread::Builder::new().stack_size(20_000_000).spawn(move || {
        let mut engine = Engine::new(rx);
        engine.start_loop();
    }).unwrap();

A portion of the conversation that was had:

Ponder and/or FRC?

Hi, great to see progress on the engine! It's almost "TCEC Ready" with the NNUE strength. Do you have plans to implement ponder and/or FRC support?

Velvet in Ipmanchess list

Hi..

Don't know if you are on Discord..but i'm already a time testing your engine Velvet !

Here you can see your progress in this list: https://ipmanchess.yolasite.com/i7-11800h.php
scroll also more down for older versions

Sorry that i put it here as a issue ,it was just a way to contact you..

Kind regards,
Ipman.

Windows Defender on Windows 11 ARM64 detect Velvet v3.0.0 popcount sse41.exe as a Trojan!

Martin I know that it is a false positive, but I must allow it to play games, please fix it :-)

https://www.chess.com/forum/view/game-analysis/freeware-against-commercial-chess-software-arena-3-5-1-vs-chessbase-fritz-17

Android compatibility

When cross compiling to Android with the following command
cargo build --release --bin velvet --target=aarch64-linux-android
I get the following error messages (version 5.0.0 still worked):

warning: In file included from fathom/src/tbprobe.c:50:
warning: /usr/include/pthread.h:681:6: error: 'regparm' is not valid on this platform
warning: __cleanup_fct_attribute;
warning: ^~~~~~~~~~~~~~~~~~~~~~~
warning: /usr/include/bits/pthreadtypes.h:255:50: note: expanded from macro '__cleanup_fct_attribute'
warning: # define __cleanup_fct_attribute attribute ((regparm (1)))
warning: ^ ~
warning: In file included from fathom/src/tbprobe.c:50:
warning: /usr/include/pthread.h:693:3: error: 'regparm' is not valid on this platform
warning: __cleanup_fct_attribute;
warning: ^~~~~~~~~~~~~~~~~~~~~~~
warning: /usr/include/bits/pthreadtypes.h:255:50: note: expanded from macro '__cleanup_fct_attribute'
warning: # define __cleanup_fct_attribute attribute ((regparm (1)))
warning: ^ ~
warning: In file included from fathom/src/tbprobe.c:50:
warning: /usr/include/pthread.h:734:6: error: 'regparm' is not valid on this platform
warning: __cleanup_fct_attribute attribute ((noreturn))
warning: ^~~~~~~~~~~~~~~~~~~~~~~
warning: /usr/include/bits/pthreadtypes.h:255:50: note: expanded from macro '__cleanup_fct_attribute'
warning: # define __cleanup_fct_attribute attribute ((regparm (1)))
warning: ^ ~
warning: fathom/src/tbprobe.c:247:10: warning: value size does not match register size specified by the constraint and modifier [-Wasm-operand-widths]
warning: return bswap64(input);
warning: ^
warning: fathom/src/stdendian.h:86:33: note: expanded from macro 'bswap64'
warning: #define bswap64(x) bswap_64(x)
warning: ^
warning: /usr/include/byteswap.h:37:21: note: expanded from macro 'bswap_64'
warning: #define bswap_64(x) __bswap_64 (x)
warning: ^
warning: /usr/include/bits/byteswap.h:132:20: note: expanded from macro '__bswap_64'
warning: __r.__l[0] = __bswap_32 (__w.__l[1]);
warning: ^
warning: /usr/include/bits/byteswap.h:76:16: note: expanded from macro '__bswap_32'
warning: : "=r" (__v)
warning: ^
warning: fathom/src/tbprobe.c:247:10: note: use constraint modifier "w"
warning: fathom/src/stdendian.h:86:33: note: expanded from macro 'bswap64'
warning: #define bswap64(x) bswap_64(x)
warning: ^
warning: /usr/include/byteswap.h:37:21: note: expanded from macro 'bswap_64'
warning: #define bswap_64(x) __bswap_64 (x)
warning: ^
warning: /usr/include/bits/byteswap.h:132:20: note: expanded from macro '__bswap_64'
warning: __r.__l[0] = __bswap_32 (__w.__l[1]);
warning: ^
warning: /usr/include/bits/byteswap.h:74:19: note: expanded from macro '__bswap_32'
warning: "rorl $16, %0;"
warning: ^
warning: fathom/src/tbprobe.c:247:10: warning: value size does not match register size specified by the constraint and modifier [-Wasm-operand-widths]
warning: return bswap64(input);
warning: ^
warning: fathom/src/stdendian.h:86:33: note: expanded from macro 'bswap64'
warning: #define bswap64(x) bswap_64(x)
warning: ^
warning: /usr/include/byteswap.h:37:21: note: expanded from macro 'bswap_64'
warning: #define bswap_64(x) __bswap_64 (x)
warning: ^
warning: /usr/include/bits/byteswap.h:133:20: note: expanded from macro '__bswap_64'
warning: __r.__l[1] = __bswap_32 (__w.__l[0]);
warning: ^
warning: /usr/include/bits/byteswap.h:76:16: note: expanded from macro '__bswap_32'
warning: : "=r" (__v)
warning: ^
warning: fathom/src/tbprobe.c:247:10: note: use constraint modifier "w"
warning: fathom/src/stdendian.h:86:33: note: expanded from macro 'bswap64'
warning: #define bswap64(x) bswap_64(x)
warning: ^
warning: /usr/include/byteswap.h:37:21: note: expanded from macro 'bswap_64'
warning: #define bswap_64(x) __bswap_64 (x)
warning: ^
warning: /usr/include/bits/byteswap.h:133:20: note: expanded from macro '__bswap_64'
warning: __r.__l[1] = __bswap_32 (__w.__l[0]);
warning: ^
warning: /usr/include/bits/byteswap.h:74:19: note: expanded from macro '__bswap_32'
warning: "rorl $16, %0;"
warning: ^
warning: fathom/src/tbprobe.c:251:10: warning: value size does not match register size specified by the constraint and modifier [-Wasm-operand-widths]
warning: return bswap32(input);
warning: ^
warning: fathom/src/stdendian.h:85:33: note: expanded from macro 'bswap32'
warning: #define bswap32(x) bswap_32(x)
warning: ^
warning: /usr/include/byteswap.h:34:21: note: expanded from macro 'bswap_32'
warning: #define bswap_32(x) __bswap_32 (x)
warning: ^
warning: /usr/include/bits/byteswap.h:76:16: note: expanded from macro '__bswap_32'
warning: : "=r" (__v)
warning: ^
warning: fathom/src/tbprobe.c:251:10: note: use constraint modifier "w"
warning: fathom/src/stdendian.h:85:33: note: expanded from macro 'bswap32'
warning: #define bswap32(x) bswap_32(x)
warning: ^
warning: /usr/include/byteswap.h:34:21: note: expanded from macro 'bswap_32'
warning: #define bswap_32(x) __bswap_32 (x)
warning: ^
warning: /usr/include/bits/byteswap.h:74:19: note: expanded from macro '__bswap_32'
warning: "rorl $16, %0;"
warning: ^
warning: fathom/src/tbprobe.c:1605:12: warning: cast to 'uint8_t *' (aka 'unsigned char *') from smaller integer type 'unsigned int' [-Wint-to-pointer-cast]
warning: data = (uint8_t *)(((uintptr_t)data + 0x3f) & ~0x3f);
warning: ^
warning: fathom/src/tbprobe.c:1609:14: warning: cast to 'uint8_t *' (aka 'unsigned char *') from smaller integer type 'unsigned int' [-Wint-to-pointer-cast]
warning: data = (uint8_t *)(((uintptr_t)data + 0x3f) & ~0x3f);
warning: ^
warning: fathom/src/tbprobe.c:2459:41: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
warning: if (popcount(pos->white | pos->black) <= cardinalityDTM) {
warning: ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~
warning: 6 warnings and 3 errors generated.