Skip to content

server: fix report progress for loading spec models, add "stages" list - #24870

Merged
ngxson merged 4 commits into
ggml-org:masterfrom
ngxson:xsn/server_loading_progress_2
Jun 21, 2026
Merged

server: fix report progress for loading spec models, add "stages" list#24870
ngxson merged 4 commits into
ggml-org:masterfrom
ngxson:xsn/server_loading_progress_2

Conversation

@ngxson

@ngxson ngxson commented Jun 21, 2026

Copy link
Copy Markdown
Collaborator

Overview

cont #24828

loading event now look like this:

{
  "model": "...",
  "event": "model_status",
  "data": {
    "status": "loading",
    "progress": {
      "stages": ["text_model", "spec_model", "mmproj_model"],
      "current": "text_model",
      "value": 0.5
    }
  }
}

cc @ServeurpersoCom

Requirements

@ngxson
ngxson requested a review from a team as a code owner June 21, 2026 12:40
@ngxson
ngxson merged commit 7c082bc into ggml-org:master Jun 21, 2026
25 checks passed
// optionally reserve VRAM for the draft / MTP context before fitting the target model
if (params_base.fit_params) {
if (callback_state) {
callback_state(SERVER_STATE_LOADING, {{"stage", "fit_params"}});

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just curious why fit_params was removed from SSE?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • the time it takes is negligible
  • it doesn't actually report %, removed to make sure everything can now report as %

papamoose pushed a commit to papamoose/llama.cpp that referenced this pull request Jun 27, 2026
ggml-org#24870)

* server: fix report progress for loading spec models, add "stages" list

* improve

* nits

* nits 2
adrianhoehne pushed a commit to adrianhoehne/llama.cpp that referenced this pull request Jul 5, 2026
ggml-org#24870)

* server: fix report progress for loading spec models, add "stages" list

* improve

* nits

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants