ggml-org/llama.cpp

Refactor: Add more typechecking to GGUFWriter.add_key_value

Open

#9.095 aberto em 19 de ago. de 2024

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)C++ (18.202 forks)batch import
help wantedrefactoring

Métricas do repositório

Stars
 (110.169 stars)
Métricas de merge de PR
 (Mesclagem média 5d 11h) (457 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
A issue requer a melhoria da verificação de tipos em GGUFWriter::add key value. Revise a discussão em https://github.com/ggerganov/llama.cpp/pull/9074#issuecomment 2296799118 para o contexto sobre as mensagens de erro desejadas. Os arquivos a serem modificados provavelmente estão na implementação do escritor GGUF, possivelmente em gguf writer.cpp ou similar. Implemente melhor validação de tipo e mensagens de erro para tipos incompatíveis.
Pilha de tecnologia
cpp
Domain
backendtooling
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++ basicsfamiliarity with llama.cppunderstanding of GGUF format
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70