ggml-org/llama.cpp

Llava functions compiled as extern "C" throw exceptions

Geschlossen

#7.073 geöffnet am 04.05.2024

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (21.825 Forks)batch import
buggood first issue

Repository-Metriken

Stars
 (124.401 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 6T 8h) (389 gemergte PRs in 30 T)

Beschreibung

Basically this: llama.cpp\examples\llava\clip.cpp(1277,13): warning : 'clip_model_load' has a non-throwing exception specification but can still throw [-Wexceptions] llama.cpp\examples\llava\clip.cpp(2075,5): warning : 'clip_n_mmproj_embd' has a non-throwing exception specification but can still throw [-Wexceptions]

As these are library exported functions and wrapped in extern "C", they should not allow exceptions to cross the boundary. C language has no idea what to do with them.

Compiled with clang-cl in windows.

Contributor Guide

Research-Richtung
Das Problem liegt in clip.cpp in den Zeilen 1277 und 2075, wo Funktionen, die mit extern "C" deklariert sind (clip model load und clip n mmproj embd), keine Ausnahmen werfende Exception Spezifikationen haben, aber dennoch Ausnahmen werfen können. Zur Behebung sollten die Funktionskörper in try catch Blöcke eingeschlossen werden, die Fehlerindikatoren (z.B. 1 oder nullptr) zurückgeben, anstatt die Ausnahmen weiterzuleiten. Untersuchen Sie das vorhandene Fehlerbehandlungsmuster in der Datei und stellen Sie sicher, dass alle extern sichtbaren Funktionen dieser Regel folgen. Es gibt derzeit noch keine verknüpften PRs oder Zuweisungen.
Tech Stack
cppc
Domain
backend
Issue Type
Fehler
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.
Meist klar
Voraussetzungen
C++ exception handlingextern "C" linkage
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40