Skip to content

chat.md says forgelm chat accepts a GGUF file via llama.cpp; no such loading path exists (CR-A13-006) #449

Closed

Description

@cemililik

Wave 1 · theme D1-8: Docs describing features that do not exist (doc side of code findings) · highest severity High · 1 finding(s) · effort trivial

Summary

An operator follows the manual to chat with the exported GGUF and gets a traceback on the training-error exit code.

Primary finding

CR-A13-006 — chat.md says forgelm chat accepts a GGUF file via llama.cpp; no such loading path exists

severity High · reachability default · intent contract-violation · confidence confirmed · independently confirmed · reproduced end to end

What happens. run_chat calls inference.load_model, which only uses transformers/unsloth from_pretrained with no gguf_file or llama.cpp backend (inference.py:41-117); a .gguf path fails with a config-JSON error and exit 2, contrary to chat.md:56 (EN/TR).

Where.

Evidence.

  • forgelm chat t.gguf < /dev/null -> raw traceback ending 'OSError: It looks like the config file at 't.gguf' is not a valid JSON file.', exit 2.
  • fpy -m forgelm chat ./model.gguf -> exit 2, 'Chat session failed: It looks like the config file at './model.gguf' is not a valid JSON file.'

Recommended fix. Remove the GGUF bullet (point to Ollama or llama.cpp instead), or detect a .gguf suffix and fail with an actionable config error (exit 1).

Acceptance criteria

  • A regression test reproduces the failure scenario and asserts the corrected behaviour.
  • Documentation listed above is corrected in English and in its Turkish mirror in the same PR.
  • Every finding in this issue is resolved or explicitly closed with a reason.

Found by the October 2026 ForgeLM code and documentation review against commit cab2947. Code links point at that commit; line numbers may have moved since. Finding IDs (CR-…, DOC-…, TECH-…) are stable references for this backlog.

Activity

  1. added
    bugSomething isn't working
    severity: highWrong behaviour or a false claim users rely on
    source: review-2026-10Found by the October 2026 code and documentation review
    wave: 1Roadmap wave 1: high severity, small effort
    on Oct 9, 2026
  2. cemililik commented on Oct 22, 2026

    @cemililik
    CollaboratorAuthor

    Fixed on development by 735a02f (#1050). docs/usermanuals/en/deployment/chat.md (and the Turkish mirror) no longer says that forgelm chat loads GGUF files. It now says that the command loads Hugging Face checkpoints only, and points GGUF users to Ollama or llama.cpp. That is the issue's first recommended fix. No other public page makes the claim. Closing as completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    area: post-trainingCode area: post-trainingbugSomething isn't workingseverity: highWrong behaviour or a false claim users rely onsource: review-2026-10Found by the October 2026 code and documentation reviewwave: 1Roadmap wave 1: high severity, small effort

    Type

    No type

    Projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions