ggml-org/llama.cpp

Refactor: Add more typechecking to GGUFWriter.add_key_value

Open

#9 095 ouverte le 19 août 2024

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)C++ (18 202 forks)batch import
help wantedrefactoring

Métriques du dépôt

Stars
 (110 169 stars)
Métriques de merge PR
 (Merge moyen 5j 11h) (457 PRs mergées en 30 j)

Description

Background Description

As per this discussion https://github.com/ggerganov/llama.cpp/pull/9074#issuecomment-2296799118 write better error messages in case of wrong types.

FYI: This ticket is free for others to approach

Possible Refactor Approaches

N/A

Guide contributeur

Direction de recherche
Le problème nécessite d'améliorer la vérification des types dans GGUFWriter::add key value. Consultez la discussion à l'adresse https://github.com/ggerganov/llama.cpp/pull/9074#issuecomment 2296799118 pour le contexte sur les messages d'erreur souhaités. Les fichiers à modifier se trouvent probablement dans l'implémentation du writer GGUF, peut être dans gguf writer.cpp ou similaire. Implémentez une meilleure validation des types et des messages d'erreur pour les types incompatibles.
Stack technique
cpp
Domaine
backendtooling
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++ basicsfamiliarity with llama.cppunderstanding of GGUF format
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70