ggml-org/whisper.cpp

OpenBLAS hardcoded in Makefile - and should we use it?

Ouverte

#414 ouverte le 16 janv. 2023

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (0 personne assignée)C++ (6 072 forks)batch import
enhancementgood first issue

Métriques du dépôt

Stars
 (52 942 étoiles)
Métriques de merge PR
 (Merge moyen 5j 20h) (36 PRs mergées en 30 j)

Description

the path to openblas libraries is hardcoded as: /usr/local/include/openblas a la:

ifdef WHISPER_OPENBLAS
        CFLAGS  += -DGGML_USE_OPENBLAS -I/usr/local/include/openblas
        LDFLAGS += -lopenblas
endif

As I wish to give whisper.cpp all the help I can with libraries it can use (M1 arm64 MBP13"), I could install macports' version but it would probably be located in /opt/local/include/openblas (I'm assuming).

Should I install that and change the Makefile? Benefits?

Guide contributeur

Direction de recherche
Etudiez comment rendre le chemin OpenBLAS configurable, par exemple en ajoutant une variable d'environnement ou en utilisant pkg config pour detecter le chemin d'installation.
Stack technique
ccpp
Domaine
backendbuild system
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Makefile
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70