ggml-org/llama.cpp

Refactor: Add more typechecking to GGUFWriter.add_key_value

Open

#9.095 geöffnet am 19. Aug. 2024

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (18.202 Forks)batch import
help wantedrefactoring

Repository-Metriken

Stars
 (110.169 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 5T 11h) (457 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Issue erfordert eine Verbesserung der Typüberprüfung in GGUFWriter::add key value. Überprüfen Sie die Diskussion unter https://github.com/ggerganov/llama.cpp/pull/9074#issuecomment 2296799118 für den Kontext zu den gewünschten Fehlermeldungen. Die zu ändernden Dateien befinden sich wahrscheinlich in der GGUF Writer Implementierung, möglicherweise in gguf writer.cpp oder ähnlichem. Implementieren Sie eine bessere Typvalidierung und Fehlermeldungen für nicht übereinstimmende Typen.
Tech Stack
cpp
Domain
backendtooling
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++ basicsfamiliarity with llama.cppunderstanding of GGUF format
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70