Investigate different approaches to binding to ggml
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 30/100
- Type d'issue
- Refactorisation
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Domaine
- backend, build-system, devtools
Piste de recherche
Examinez le binding ctypes existant dans ggml/ggml.py ainsi que la configuration de build de CMakeLists.txt. Examinez les branches de test pour nanobind et SWIG afin de comprendre les tentatives d’intégration. L’objectif est d’évaluer les systèmes de binding selon les exigences indiquées : performances, maintenabilité, extensibilité et compatibilité avec le système de build de ggml. Déterminez les compromis de chaque approche sans implémenter de solution complète.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Current Approach
Use scikit-build-core and a CMakeLists file in the root directory to compile ggml as a shared library and install it relative to the ggml/ggml.py file so that it can be loaded by ctypes.CDLL.
Issues with Current Approach
#define's are not accessible from the shared library, this is an issue for values that are computed on the host system at compile time.- Platform specific functions such as those for CUDA, OpenCL, and Metal are also awkward to conditionally support with the ctypes approach
- Assuming the relative location of files in an installed package causes issues, especially when installing in ie. editable mode for local development.
- Some arguments such as ctypes Array's require awkward typing workarounds.
Alternative Binding Systems
- Cython
- Pybind11
- Nanobind (WIP)
- SWIG (WIP)
- CFFI(?)
- Stick with ctypes
Requirements
- Should be at least as fast as ctypes approach (large overhead is not acceptable for this application)
- Should be simple to maintain (ggml changes often, should be at least as simple as ctypes to add new function definitions / types / etc)
- Should be easy for others to extend or modify ggml
- Should use ggml's existing build system and flags when compiling, should not limit which platforms / optimizations can be used
- Langage dominant
- Python
- Étoiles
- 152
- Forks
- 17
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de abetlen/ggml-python
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
abetlen/ggml-python#91 ·
-
Import errorOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
abetlen/ggml-python#90 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
abetlen/ggml-python#93 · 1 commentaire ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
abetlen/ggml-python#92 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 30/100
abetlen/ggml-python#21 ·
Toutes les issues de abetlen/ggml-python
Issues similaires
-
bug severity:low
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 2 jours
-
curriculum documentation quality
Difficulté 2/5 1-3 heures Accessibilité débutants 77/100
githubnext/gh-aw-workshop#5367 ·
Les mainteneurs répondent en général sous 3 jours
-
OAuth documentation for GET /api/v2/users/me/ appears inconsistent with required global read scopeOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
Create Noise Latents ignores randomize_seed and produces the same noise every runPeut-être pris @griptapeops l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
griptape-ai/griptape-nodes-library-diffusers#127 ·
Les mainteneurs répondent en général sous 1 jour
-
Jenkins capacity report: metrics history is lost when archiving a month failsPeut-être pris @RaphaelFakhri l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
adoptium/infrastructure#4568 ·
Les mainteneurs répondent en général sous 1 jour