ppigazzini

Results 323 comments of ppigazzini

DEV updated, modifying the Priority/Throughput as normal user the server flashes the error message, but the run is modified. ![image](https://github.com/official-stockfish/fishtest/assets/15718418/15a6c1cd-36ca-4a9b-821f-340f43d9fef7) The PR needs some works: 1. the check introduced by...

@Mugundanmcw if you have msys2 installed on your Windows ARM system, please report the output of `cat /proc/cpuinfo` My test on GutHub to update `get_native_properties.sh` for Windows ARM failed. This...

Mmm, coreinfo get the correct CPU name and returns the CPU flags like msys2... Click to view ``` Coreinfo v3.6 - Dump information on system CPU and memory topology Copyright...

Other weird output (x86_64 everywhere) using msys2 on the github runner: ``` uname -a MINGW64_NT-10.0-22631-ARM64 fv-az1341-421 3.5.7-2644508f.x86_64 2025-02-06 19:32 UTC x86_64 Msys uname -s MINGW64_NT-10.0-22631-ARM64 uname -m x86_64 uname -r...

It was my hypothesis, Sysinternals for ARM64 has ARM binaries. In that case `Coreinfo64a.exe` runs natively and don't find any flags because the program itself was not updated to find...

In July 2023 we explored the compiler option as well, here is an old copy of the script, that depends on the compiler installed. At example this script does not...

msys2 installer is only x86_64, I got the same flags also using other msys2 programs (eg `awk`). It's hard to investigate without a PC with ARM CPU.

master vs PR ![image](https://github.com/user-attachments/assets/c962fd3b-cd57-4c20-ab13-fd02d974d088)

@Disservin the workers look good with the PR. As highlighted by @vondele, merging the PR will change the TC for system with high concurrency, with a jump in the PT.

New commit with: simplification with `concurrent.futures`, proper exception management, bench for SMP use case. MNps master vs PR for both normal test (thread=1) and SMP test (threads=8). As expected with...