Skip to content
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.

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

Leela-zero-next doesn't compile on old CPU (without OpenCL). What about to make binary releases? #2643

Open
GD-fix opened this issue Oct 2, 2021 · 9 comments

Comments

@GD-fix
Copy link

GD-fix commented Oct 2, 2021

.../leela-zero-next> cmake -D USE_CPU_ONLY=1 -D USE_BLAS=1 .
...
CMake Error at cmake/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
Could NOT find OpenCL (missing: OpenCL_LIBRARY) (found version "2.2")
Call Stack (most recent call first):
cmake/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE)
cmake/Modules/FindOpenCL.cmake:135 (find_package_handle_standard_args)
CMakeLists.txt:29 (find_package)

-- Configuring incomplete, errors occurred!

After replacement of "find_package(OpenCL REQUIRED)" in CMakeLists.txt on:
if(USE_CPU_ONLY)
message(STATUS "Using CPU based computation.")
else()
message(STATUS "Looking for system OpenCL library.")
find_package(OpenCL REQUIRED)
endif()

got:
...
CMake Warning at CMakeLists.txt:55 (find_package):
By not providing "FindQt5Core.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "Qt5Core", but
CMake did not find one.

Could not find a package configuration file provided by "Qt5Core" with any
of the following names:

Qt5CoreConfig.cmake
qt5core-config.cmake

Add the installation prefix of "Qt5Core" to CMAKE_PREFIX_PATH or set
"Qt5Core_DIR" to a directory containing one of the above files. If
"Qt5Core" provides a separate development package or SDK, be sure it has
been installed.

CMake Warning at CMakeLists.txt:134 (message):
Qt is not found, build for autogtp and validation is disabled

-- Configuring incomplete, errors occurred!

After deletion of the string:
add_subdirectory(gtest EXCLUDE_FROM_ALL) # We don't want to install gtest, exclude it from all

it was compiled (but with error in dependencies of target tests) and run...

Is it possible to make binary releases like KataGo (https://github.com/lightvector/KataGo/releases/latest)?

@GD-fix
Copy link
Author

GD-fix commented Nov 13, 2021

In this case it will be the really inducement to download new version, especially considering that there isn't any game level strengthening compared with last release (https://lifein19x19.com/viewtopic.php?f=18&t=13322&p=269439#p269439).

@teekuningas
Copy link

Did you update the submodules?

@GD-fix
Copy link
Author

GD-fix commented Dec 25, 2021

When? Last commit is still from Aug 24, 2021...

@GD-fix
Copy link
Author

GD-fix commented Apr 15, 2023

In 06.02.23 version all the same...

@OmnipotentEntity
Copy link
Contributor

Leela Zero isn't currently being maintained. But your problem seems to be related to library installation (of Qt5), it is not specific to Leela Zero.

@GD-fix
Copy link
Author

GD-fix commented Apr 22, 2023

Last commit of the LeelaZero "next" branch: "on Feb 6"...

The OS (OpenSUSE Tumbleweed) was completely updated before last LeelaZero "next" branch version compiling, but the problem remains:

add_subdirectory(gtest EXCLUDE_FROM_ALL) # We don't want to install gtest, exclude it from all
After deletion of this string it was compiled (but with error in dependencies of target tests) and run...

So the problem seems to be related to LeelaZero "gtest" module...

@GD-fix
Copy link
Author

GD-fix commented May 13, 2023

This version of non "being maintained" LeelaZero is stronger than official release...

@OmnipotentEntity
Copy link
Contributor

OmnipotentEntity commented May 22, 2023

Last commit of the LeelaZero "next" branch: "on Feb 6"...

That was a link correction. So was the previous commit. The last actual code change was on May 4, 2021 slightly more than 2 years ago at this point. Meaning that the measurement that LeelaZero has somehow gotten stronger is quite simply wrong, and the observed changes are merely an illustration of statistical noise.

Anyway, your error your originally posted about is because you do not have the development libraries for OpenCL and Qt5 installed on your computer, or that they are not in the location that CMake is searching for them in. If the former, simply install the libraries. If the latter tell CMake where to find the files manually. For this using an interactive cmake command such as ccmake might be beneficial.

However, if you have the previously compiled version of the code released in May 2021, then you do not need to recompile the code because absolutely nothing has changed since then.

Hope this helps.

@GD-fix
Copy link
Author

GD-fix commented May 27, 2023

How developers said, there is not any reason to use GPU with my video card. So, I'm using (see the title and the beginning of this issue):cmake -D USE_CPU_ONLY=1
Why in this case should it matter if I have OpenCL libraries or not at all?

Thanks in advance.

P. S. I have both previously and last compiled versions, because there isn't the problem with QT5 on new OS and:

After deletion of the string:
add_subdirectory(gtest EXCLUDE_FROM_ALL) # We don't want to install gtest, exclude it from all
it was compiled (but with error in dependencies of target tests) and run...

The goal of this issue is to ask developers to upload compiled releases (individual for GPU and CPU only support like KataGo, or only one, but with something like --nogpu run for CPU only support)...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants