SJTU-IPADS / PowerInfer

High-speed Large Language Model Serving on PCs with Consumer-grade GPUs
MIT License
7.96k stars 412 forks source link

CMake 3.17 or higher is required. The repository asks for version 3.13.4 #144

Closed Rubiel1 closed 9 months ago

Rubiel1 commented 9 months ago

Prerequisites

Before submitting your issue, please ensure the following:

Expected Behavior

Build PowerInfer

Please provide a detailed written description of what you were trying to do, and what you expected PowerInfer to do.

1.- I run the line """PowerInfer]$ cmake -S . -B build -DLLAMA_CUBLAS=ON""" and I was expected PowerInfer to be build.

Current Behavior

Please provide a detailed written description of what PowerInfer did, instead.

I received the output: """CMake Error at CMakeLists.txt:252 (cmake_minimum_required): CMake 3.17 or higher is required. You are running version 3.13.4 """

But the github repository asks for CMake 3.13+

Failure Information (for bugs)

Please help provide information about the failure / bug.

It asks me for CMake 3.17 when the github repository asks for CMake 3.13.

Steps to Reproduce

Please provide detailed steps for reproducing the issue. We are not sitting in front of your screen, so the more detail the better.

  1. step 1 Open CMakeList.txt
  2. step 2 Go to line 252
  3. step 3 "if (LLAMA_CUBLAS) cmake_minimum_required(VERSION 3.17)"
hodlen commented 9 months ago

Thank you for bringing this discrepancy to our attention. We will update our documentation to match the correct CMake version requirement ASAP.