Skip to content

Add explicit llama.cpp provider and benchmark tooling - #751

Closed
Abhishekpandey123-git wants to merge 1 commit into
fireform-core:502-ollama-model-installfrom
Abhishekpandey123-git:feat/llama-cpp-explicit-provider
Closed

Abhishekpandey123-git wants to merge 1 commit into
fireform-core:502-ollama-model-installfrom
Abhishekpandey123-git:feat/llama-cpp-explicit-provider

Conversation

@Abhishekpandey123-git

Copy link
Copy Markdown

Related to #702.

Adds an explicit llama.cpp provider configuration, registry coverage, and benchmark tooling for reproducible Ollama-versus-llama.cpp comparisons.

This PR is stacked on #733 and targets 502-ollama-model-install so it can be reviewed with the provider/model configuration work it depends on.

Note: comparative benchmark runs and the final performance recommendation still need to be captured before #702 is complete.

Copy link
Copy Markdown
Author

Benchmark limitation: hardware unavailable

The implementation and benchmark tooling are ready for review, and the local backend validation passed (420 tests; Ruff passed). However, I cannot run a fair Ollama-versus-llama.cpp comparison on my current hardware due to resource constraints.

I have not measured or claimed a performance improvement. Latency, throughput, CPU, and memory comparisons remain unmeasured, so the comparison report should be treated as pending rather than as evidence that either backend is faster.

Would a maintainer or contributor with suitable hardware be able to run the documented benchmark workloads, or advise on an available CI/benchmark runner? I can help interpret the output and update the report once genuine measurements are available.

@marcvergees

Copy link
Copy Markdown
Member

Hi @Abhishekpandey123-git, I'm a little bit out this week, lots of midterms in uni and other stuff. I'll be back with this asap. Thanks for you comprehension.
M

@jansans04
jansans04 deleted the branch fireform-core:502-ollama-model-install October 30, 2026 20:26
@jansans04 jansans04 closed this Oct 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants