GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fix typo in default model path by ott2 · Pull Request #1366 · ggml-org/llama.cpp

Conversation

@ott2

@ott2 ott2 commented May 8, 2023

Copy link
Copy Markdown
Contributor

Elsewhere in the code the default model path is models/llama-7B/ggml-model.bin; that string appears in

examples/embedding/embedding.cpp
examples/save-load-state/save-load-state.cpp
examples/perplexity/perplexity.cpp
examples/main/main.cpp

This change makes ./main -h print the correct default path.

It might make sense to refactor the code to reference the default path only once. This change makes that refactoring easier to do.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@ott2
fix typo in default model path
afa160d
prusnak
prusnak suggested changes May 8, 2023
View reviewed changes
Comment thread examples/common.h Outdated
float mirostat_eta = 0.10f; // learning rate

std::string model = "models/lamma-7B/ggml-model.bin"; // model path
std::string model = "models/llama-7B/ggml-model.bin"; // model path

@prusnak prusnak May 8, 2023 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Readme mentions ./models/7B/ggml-model-q4_0.bin and I think we should use this everywhere for unification.

Suggested change
std::string model = "models/llama-7B/ggml-model.bin"; // model path
std::string model = "models/7B/ggml-model-q4_0.bin"; // model path

Can you also update this PR to remove params.model = "models/llama-7B/ggml-model.bin"; from files you mention? It doesn't make sense to override the value if it is the same as the default, no?

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

DannyDaemonic and ott2 reacted with thumbs up emoji
All reactions
  • 👍 2 reactions

@ott2 ott2 May 9, 2023

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Should we use models/7B or models/llama-7B as the canonical path? I used to have just 7B but with all the new models supported by llama.cpp I'm now tending to prefer llama-7B for disambiguation.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

@prusnak prusnak May 9, 2023 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

I prefer to keep 7B because that's how the directory is named when you download the model.

Also the instructiond should work no matter what model is located in the 7B directory (Alpaca, Vicuna, etc.)

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

@ott2 ott2 May 16, 2023

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

I've implemented your suggested changes. The default model path is now defined in examples/common.h and this is used elsewhere implicitly, instead of repeating the string in several places.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
ott2 and others added 2 commits May 16, 2023 16:13
@ott2
Merge branch 'ggerganov:master' into master
4e09c4c
@ott2
implement suggestion by @prusnak to define default model once
bbe2205
prusnak
prusnak approved these changes May 16, 2023
View reviewed changes
@prusnak
prusnak merged commit 9560655 into ggml-org:master May 16, 2023
ggerganov pushed a commit to JohannesGaessler/llama.cpp that referenced this pull request May 20, 2023
@ott2 @ggerganov
define default model path once, sync path with readme (ggml-org#1366)
09d8251
ggerganov added a commit that referenced this pull request May 20, 2023
@JohannesGaessler @ott2
@ilyakurdyukov @TheBloke rankaiyx @sw @DannyDaemonic @Green-Sky @ggerganov @n1trane @data-angel @ejones @maximegmd @github-actions @zenixls2
cuda : loading models directly into VRAM, norm calculation on GPU, br…
affc76e
…oadcasting for ggml_mul (#1483)

* Broadcasting for ggml_mul

* CUDA kernel for ggml_mul, norms in VRAM

* GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* define default model path once, sync path with readme (#1366)

* ~7% faster Q5_1 AVX2 code (#1477)

* convert.py: Support models which are stored in a single pytorch_model.bin (#1469)

* Support models in a single pytorch_model.bin

* Remove spurious line with typo

* benchmark-matmul: Print the average of the test results (#1490)

* Remove unused n_parts parameter (#1509)

* Fixes #1511 lambda issue for w64devkit (mingw) (#1513)

* Fix for w64devkit and mingw

* make kv_f16 the default for api users (#1517)

* minor : fix compile warnings

* readme : adds WizardLM to the list of supported models (#1485)

* main : make reverse prompt option act as a stop token in non-interactive mode (#1032)

* Make reverse prompt option act as a stop token in non-interactive scenarios

* Making requested review changes

* Update gpt_params_parse and fix a merge error

* Revert "Update gpt_params_parse and fix a merge error"

This reverts commit 2bb2ff1.

* Update gpt_params_parse and fix a merge error take 2

* examples : add persistent chat (#1495)

* examples : add persistent chat

* examples : fix whitespace

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* tests : add missing header

* ggml : use F16 instead of F32 in Q4_0, Q4_1, Q8_0 (#1508)

* ggml : use F16 instead of F32 in Q4_0, Q4_1 and Q8_0

* llama : bump LLAMA_FILE_VERSION to 3

* cuda : update Q4 and Q8 dequantize kernels

* ggml : fix AVX dot products

* readme : update performance table + hot topics

* ggml : fix scalar implementation of Q4_1 dot

* llama : fix compile warnings in llama_set_state_data()

* llama : fix name shadowing and C4146 (#1526)

* Fix name shadowing and C4146

* Fix if macros not using defined when required

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Code style

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Georgi Gerganov <[email protected]>

* Fix for mingw (#1462)

* llama : add llama_init_backend() API (close #1527)

* feature : add blis and other BLAS implementation support (#1502)

* feature: add blis support

* feature: allow all BLA_VENDOR to be assigned in cmake arguments. align with whisper.cpp pr 927

* fix: version detection for BLA_SIZEOF_INTEGER, recover min version of cmake

* Fix typo in INTEGER

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* Revert "feature : add blis and other BLAS implementation support (#1502)"

This reverts commit 07e9ace.

* GPU weights not in RAM, direct loading with cuFile

* llama : code style fixes + progress print fix

* ggml : ggml_mul better broadcast support

* cmake : workarounds for cufile when CMake version < 3.25

* gg rebase fixup

* Loop in llama.cpp, fixed progress callback

* Attempt clang-tidy fix

* llama : fix vram size computation

* Add forgotten fclose()

---------

Co-authored-by: András Salamon <[email protected]>
Co-authored-by: Ilya Kurdyukov <[email protected]>
Co-authored-by: Tom Jobbins <[email protected]>
Co-authored-by: rankaiyx <[email protected]>
Co-authored-by: Stephan Walter <[email protected]>
Co-authored-by: DannyDaemonic <[email protected]>
Co-authored-by: Erik Scholz <[email protected]>
Co-authored-by: Georgi Gerganov <[email protected]>
Co-authored-by: David Kennedy <[email protected]>
Co-authored-by: Jason McCartney <[email protected]>
Co-authored-by: Evan Jones <[email protected]>
Co-authored-by: Maxime <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Zenix <[email protected]>
@Bearsaerker Bearsaerker mentioned this pull request Mar 12, 2025
Eval bug: Gemma 3 extremly slow prompt processing when using quantized kv cache. #12352
Closed
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
@ott2
define default model path once, sync path with readme (ggml-org#1366)
f0a3be7
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
@JohannesGaessler @ott2
@ilyakurdyukov @TheBloke rankaiyx @sw @DannyDaemonic @Green-Sky @ggerganov @n1trane @data-angel @ejones @maximegmd @github-actions @zenixls2
cuda : loading models directly into VRAM, norm calculation on GPU, br…
dfdc3ef
…oadcasting for ggml_mul (ggml-org#1483)

* Broadcasting for ggml_mul

* CUDA kernel for ggml_mul, norms in VRAM

* GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* define default model path once, sync path with readme (ggml-org#1366)

* ~7% faster Q5_1 AVX2 code (ggml-org#1477)

* convert.py: Support models which are stored in a single pytorch_model.bin (ggml-org#1469)

* Support models in a single pytorch_model.bin

* Remove spurious line with typo

* benchmark-matmul: Print the average of the test results (ggml-org#1490)

* Remove unused n_parts parameter (ggml-org#1509)

* Fixes ggml-org#1511 lambda issue for w64devkit (mingw) (ggml-org#1513)

* Fix for w64devkit and mingw

* make kv_f16 the default for api users (ggml-org#1517)

* minor : fix compile warnings

* readme : adds WizardLM to the list of supported models (ggml-org#1485)

* main : make reverse prompt option act as a stop token in non-interactive mode (ggml-org#1032)

* Make reverse prompt option act as a stop token in non-interactive scenarios

* Making requested review changes

* Update gpt_params_parse and fix a merge error

* Revert "Update gpt_params_parse and fix a merge error"

This reverts commit 2bb2ff1.

* Update gpt_params_parse and fix a merge error take 2

* examples : add persistent chat (ggml-org#1495)

* examples : add persistent chat

* examples : fix whitespace

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* tests : add missing header

* ggml : use F16 instead of F32 in Q4_0, Q4_1, Q8_0 (ggml-org#1508)

* ggml : use F16 instead of F32 in Q4_0, Q4_1 and Q8_0

* llama : bump LLAMA_FILE_VERSION to 3

* cuda : update Q4 and Q8 dequantize kernels

* ggml : fix AVX dot products

* readme : update performance table + hot topics

* ggml : fix scalar implementation of Q4_1 dot

* llama : fix compile warnings in llama_set_state_data()

* llama : fix name shadowing and C4146 (ggml-org#1526)

* Fix name shadowing and C4146

* Fix if macros not using defined when required

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Code style

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Georgi Gerganov <[email protected]>

* Fix for mingw (ggml-org#1462)

* llama : add llama_init_backend() API (close ggml-org#1527)

* feature : add blis and other BLAS implementation support (ggml-org#1502)

* feature: add blis support

* feature: allow all BLA_VENDOR to be assigned in cmake arguments. align with whisper.cpp pr 927

* fix: version detection for BLA_SIZEOF_INTEGER, recover min version of cmake

* Fix typo in INTEGER

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* Revert "feature : add blis and other BLAS implementation support (ggml-org#1502)"

This reverts commit 07e9ace.

* GPU weights not in RAM, direct loading with cuFile

* llama : code style fixes + progress print fix

* ggml : ggml_mul better broadcast support

* cmake : workarounds for cufile when CMake version < 3.25

* gg rebase fixup

* Loop in llama.cpp, fixed progress callback

* Attempt clang-tidy fix

* llama : fix vram size computation

* Add forgotten fclose()

---------

Co-authored-by: András Salamon <[email protected]>
Co-authored-by: Ilya Kurdyukov <[email protected]>
Co-authored-by: Tom Jobbins <[email protected]>
Co-authored-by: rankaiyx <[email protected]>
Co-authored-by: Stephan Walter <[email protected]>
Co-authored-by: DannyDaemonic <[email protected]>
Co-authored-by: Erik Scholz <[email protected]>
Co-authored-by: Georgi Gerganov <[email protected]>
Co-authored-by: David Kennedy <[email protected]>
Co-authored-by: Jason McCartney <[email protected]>
Co-authored-by: Evan Jones <[email protected]>
Co-authored-by: Maxime <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Zenix <[email protected]>
phuongncn pushed a commit to phuongncn/llama.cpp-gx10-dgx-sparks-deepseekv4 that referenced this pull request Apr 28, 2026
@ott2
define default model path once, sync path with readme (ggml-org#1366)
25ea9e7
phuongncn pushed a commit to phuongncn/llama.cpp-gx10-dgx-sparks-deepseekv4 that referenced this pull request Apr 28, 2026
@JohannesGaessler @ott2
@ilyakurdyukov @TheBloke rankaiyx @sw @DannyDaemonic @Green-Sky @ggerganov @n1trane @data-angel @ejones @maximegmd @github-actions @zenixls2
cuda : loading models directly into VRAM, norm calculation on GPU, br…
4d3789c
…oadcasting for ggml_mul (ggml-org#1483)

* Broadcasting for ggml_mul

* CUDA kernel for ggml_mul, norms in VRAM

* GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* define default model path once, sync path with readme (ggml-org#1366)

* ~7% faster Q5_1 AVX2 code (ggml-org#1477)

* convert.py: Support models which are stored in a single pytorch_model.bin (ggml-org#1469)

* Support models in a single pytorch_model.bin

* Remove spurious line with typo

* benchmark-matmul: Print the average of the test results (ggml-org#1490)

* Remove unused n_parts parameter (ggml-org#1509)

* Fixes ggml-org#1511 lambda issue for w64devkit (mingw) (ggml-org#1513)

* Fix for w64devkit and mingw

* make kv_f16 the default for api users (ggml-org#1517)

* minor : fix compile warnings

* readme : adds WizardLM to the list of supported models (ggml-org#1485)

* main : make reverse prompt option act as a stop token in non-interactive mode (ggml-org#1032)

* Make reverse prompt option act as a stop token in non-interactive scenarios

* Making requested review changes

* Update gpt_params_parse and fix a merge error

* Revert "Update gpt_params_parse and fix a merge error"

This reverts commit 2bb2ff1.

* Update gpt_params_parse and fix a merge error take 2

* examples : add persistent chat (ggml-org#1495)

* examples : add persistent chat

* examples : fix whitespace

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* tests : add missing header

* ggml : use F16 instead of F32 in Q4_0, Q4_1, Q8_0 (ggml-org#1508)

* ggml : use F16 instead of F32 in Q4_0, Q4_1 and Q8_0

* llama : bump LLAMA_FILE_VERSION to 3

* cuda : update Q4 and Q8 dequantize kernels

* ggml : fix AVX dot products

* readme : update performance table + hot topics

* ggml : fix scalar implementation of Q4_1 dot

* llama : fix compile warnings in llama_set_state_data()

* llama : fix name shadowing and C4146 (ggml-org#1526)

* Fix name shadowing and C4146

* Fix if macros not using defined when required

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Code style

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Georgi Gerganov <[email protected]>

* Fix for mingw (ggml-org#1462)

* llama : add llama_init_backend() API (close ggml-org#1527)

* feature : add blis and other BLAS implementation support (ggml-org#1502)

* feature: add blis support

* feature: allow all BLA_VENDOR to be assigned in cmake arguments. align with whisper.cpp pr 927

* fix: version detection for BLA_SIZEOF_INTEGER, recover min version of cmake

* Fix typo in INTEGER

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* Revert "feature : add blis and other BLAS implementation support (ggml-org#1502)"

This reverts commit 07e9ace.

* GPU weights not in RAM, direct loading with cuFile

* llama : code style fixes + progress print fix

* ggml : ggml_mul better broadcast support

* cmake : workarounds for cufile when CMake version < 3.25

* gg rebase fixup

* Loop in llama.cpp, fixed progress callback

* Attempt clang-tidy fix

* llama : fix vram size computation

* Add forgotten fclose()

---------

Co-authored-by: András Salamon <[email protected]>
Co-authored-by: Ilya Kurdyukov <[email protected]>
Co-authored-by: Tom Jobbins <[email protected]>
Co-authored-by: rankaiyx <[email protected]>
Co-authored-by: Stephan Walter <[email protected]>
Co-authored-by: DannyDaemonic <[email protected]>
Co-authored-by: Erik Scholz <[email protected]>
Co-authored-by: Georgi Gerganov <[email protected]>
Co-authored-by: David Kennedy <[email protected]>
Co-authored-by: Jason McCartney <[email protected]>
Co-authored-by: Evan Jones <[email protected]>
Co-authored-by: Maxime <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Zenix <[email protected]>
phuongncn pushed a commit to phuongncn/llama.cpp-gx10-dgx-sparks-deepseekv4 that referenced this pull request Apr 28, 2026
@ikawrakow
Disable split mode graph for recurrent/hybrid models when tensor over…
1ef4b5e
…rides (ggml-org#1366)
ljubomirj pushed a commit to ljubomirj/llama.cpp that referenced this pull request May 6, 2026
@ott2
define default model path once, sync path with readme (ggml-org#1366)
9989cda
ljubomirj pushed a commit to ljubomirj/llama.cpp that referenced this pull request May 6, 2026
@JohannesGaessler @ott2
@ilyakurdyukov @TheBloke rankaiyx @sw @DannyDaemonic @Green-Sky @ggerganov @n1trane @data-angel @ejones @maximegmd @github-actions @zenixls2
cuda : loading models directly into VRAM, norm calculation on GPU, br…
58160ca
…oadcasting for ggml_mul (ggml-org#1483)

* Broadcasting for ggml_mul

* CUDA kernel for ggml_mul, norms in VRAM

* GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* define default model path once, sync path with readme (ggml-org#1366)

* ~7% faster Q5_1 AVX2 code (ggml-org#1477)

* convert.py: Support models which are stored in a single pytorch_model.bin (ggml-org#1469)

* Support models in a single pytorch_model.bin

* Remove spurious line with typo

* benchmark-matmul: Print the average of the test results (ggml-org#1490)

* Remove unused n_parts parameter (ggml-org#1509)

* Fixes ggml-org#1511 lambda issue for w64devkit (mingw) (ggml-org#1513)

* Fix for w64devkit and mingw

* make kv_f16 the default for api users (ggml-org#1517)

* minor : fix compile warnings

* readme : adds WizardLM to the list of supported models (ggml-org#1485)

* main : make reverse prompt option act as a stop token in non-interactive mode (ggml-org#1032)

* Make reverse prompt option act as a stop token in non-interactive scenarios

* Making requested review changes

* Update gpt_params_parse and fix a merge error

* Revert "Update gpt_params_parse and fix a merge error"

This reverts commit 2bb2ff1.

* Update gpt_params_parse and fix a merge error take 2

* examples : add persistent chat (ggml-org#1495)

* examples : add persistent chat

* examples : fix whitespace

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* tests : add missing header

* ggml : use F16 instead of F32 in Q4_0, Q4_1, Q8_0 (ggml-org#1508)

* ggml : use F16 instead of F32 in Q4_0, Q4_1 and Q8_0

* llama : bump LLAMA_FILE_VERSION to 3

* cuda : update Q4 and Q8 dequantize kernels

* ggml : fix AVX dot products

* readme : update performance table + hot topics

* ggml : fix scalar implementation of Q4_1 dot

* llama : fix compile warnings in llama_set_state_data()

* llama : fix name shadowing and C4146 (ggml-org#1526)

* Fix name shadowing and C4146

* Fix if macros not using defined when required

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Code style

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Georgi Gerganov <[email protected]>

* Fix for mingw (ggml-org#1462)

* llama : add llama_init_backend() API (close ggml-org#1527)

* feature : add blis and other BLAS implementation support (ggml-org#1502)

* feature: add blis support

* feature: allow all BLA_VENDOR to be assigned in cmake arguments. align with whisper.cpp pr 927

* fix: version detection for BLA_SIZEOF_INTEGER, recover min version of cmake

* Fix typo in INTEGER

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* Revert "feature : add blis and other BLAS implementation support (ggml-org#1502)"

This reverts commit 3516762.

* GPU weights not in RAM, direct loading with cuFile

* llama : code style fixes + progress print fix

* ggml : ggml_mul better broadcast support

* cmake : workarounds for cufile when CMake version < 3.25

* gg rebase fixup

* Loop in llama.cpp, fixed progress callback

* Attempt clang-tidy fix

* llama : fix vram size computation

* Add forgotten fclose()

---------

Co-authored-by: András Salamon <[email protected]>
Co-authored-by: Ilya Kurdyukov <[email protected]>
Co-authored-by: Tom Jobbins <[email protected]>
Co-authored-by: rankaiyx <[email protected]>
Co-authored-by: Stephan Walter <[email protected]>
Co-authored-by: DannyDaemonic <[email protected]>
Co-authored-by: Erik Scholz <[email protected]>
Co-authored-by: Georgi Gerganov <[email protected]>
Co-authored-by: David Kennedy <[email protected]>
Co-authored-by: Jason McCartney <[email protected]>
Co-authored-by: Evan Jones <[email protected]>
Co-authored-by: Maxime <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Zenix <[email protected]>
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
@ott2
define default model path once, sync path with readme (ggml-org#1366)
1f794c6
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
@JohannesGaessler @ott2
@ilyakurdyukov @TheBloke rankaiyx @sw @DannyDaemonic @Green-Sky @ggerganov @n1trane @data-angel @ejones @maximegmd @github-actions @zenixls2
cuda : loading models directly into VRAM, norm calculation on GPU, br…
28fb0d1
…oadcasting for ggml_mul (ggml-org#1483)

* Broadcasting for ggml_mul

* CUDA kernel for ggml_mul, norms in VRAM

* GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* define default model path once, sync path with readme (ggml-org#1366)

* ~7% faster Q5_1 AVX2 code (ggml-org#1477)

* convert.py: Support models which are stored in a single pytorch_model.bin (ggml-org#1469)

* Support models in a single pytorch_model.bin

* Remove spurious line with typo

* benchmark-matmul: Print the average of the test results (ggml-org#1490)

* Remove unused n_parts parameter (ggml-org#1509)

* Fixes ggml-org#1511 lambda issue for w64devkit (mingw) (ggml-org#1513)

* Fix for w64devkit and mingw

* make kv_f16 the default for api users (ggml-org#1517)

* minor : fix compile warnings

* readme : adds WizardLM to the list of supported models (ggml-org#1485)

* main : make reverse prompt option act as a stop token in non-interactive mode (ggml-org#1032)

* Make reverse prompt option act as a stop token in non-interactive scenarios

* Making requested review changes

* Update gpt_params_parse and fix a merge error

* Revert "Update gpt_params_parse and fix a merge error"

This reverts commit 2bb2ff1.

* Update gpt_params_parse and fix a merge error take 2

* examples : add persistent chat (ggml-org#1495)

* examples : add persistent chat

* examples : fix whitespace

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* tests : add missing header

* ggml : use F16 instead of F32 in Q4_0, Q4_1, Q8_0 (ggml-org#1508)

* ggml : use F16 instead of F32 in Q4_0, Q4_1 and Q8_0

* llama : bump LLAMA_FILE_VERSION to 3

* cuda : update Q4 and Q8 dequantize kernels

* ggml : fix AVX dot products

* readme : update performance table + hot topics

* ggml : fix scalar implementation of Q4_1 dot

* llama : fix compile warnings in llama_set_state_data()

* llama : fix name shadowing and C4146 (ggml-org#1526)

* Fix name shadowing and C4146

* Fix if macros not using defined when required

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Code style

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Georgi Gerganov <[email protected]>

* Fix for mingw (ggml-org#1462)

* llama : add llama_init_backend() API (close ggml-org#1527)

* feature : add blis and other BLAS implementation support (ggml-org#1502)

* feature: add blis support

* feature: allow all BLA_VENDOR to be assigned in cmake arguments. align with whisper.cpp pr 927

* fix: version detection for BLA_SIZEOF_INTEGER, recover min version of cmake

* Fix typo in INTEGER

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* Revert "feature : add blis and other BLAS implementation support (ggml-org#1502)"

This reverts commit 16cdcea.

* GPU weights not in RAM, direct loading with cuFile

* llama : code style fixes + progress print fix

* ggml : ggml_mul better broadcast support

* cmake : workarounds for cufile when CMake version < 3.25

* gg rebase fixup

* Loop in llama.cpp, fixed progress callback

* Attempt clang-tidy fix

* llama : fix vram size computation

* Add forgotten fclose()

---------

Co-authored-by: András Salamon <[email protected]>
Co-authored-by: Ilya Kurdyukov <[email protected]>
Co-authored-by: Tom Jobbins <[email protected]>
Co-authored-by: rankaiyx <[email protected]>
Co-authored-by: Stephan Walter <[email protected]>
Co-authored-by: DannyDaemonic <[email protected]>
Co-authored-by: Erik Scholz <[email protected]>
Co-authored-by: Georgi Gerganov <[email protected]>
Co-authored-by: David Kennedy <[email protected]>
Co-authored-by: Jason McCartney <[email protected]>
Co-authored-by: Evan Jones <[email protected]>
Co-authored-by: Maxime <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Zenix <[email protected]>
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
@ott2
define default model path once, sync path with readme (ggml-org#1366)
cc42ea8
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
@JohannesGaessler @ott2
@ilyakurdyukov @TheBloke rankaiyx @sw @DannyDaemonic @Green-Sky @ggerganov @n1trane @data-angel @ejones @maximegmd @github-actions @zenixls2
cuda : loading models directly into VRAM, norm calculation on GPU, br…
2fe532a
…oadcasting for ggml_mul (ggml-org#1483)

* Broadcasting for ggml_mul

* CUDA kernel for ggml_mul, norms in VRAM

* GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* fixup! GPU weights not in RAM, direct loading with cuFile

* define default model path once, sync path with readme (ggml-org#1366)

* ~7% faster Q5_1 AVX2 code (ggml-org#1477)

* convert.py: Support models which are stored in a single pytorch_model.bin (ggml-org#1469)

* Support models in a single pytorch_model.bin

* Remove spurious line with typo

* benchmark-matmul: Print the average of the test results (ggml-org#1490)

* Remove unused n_parts parameter (ggml-org#1509)

* Fixes ggml-org#1511 lambda issue for w64devkit (mingw) (ggml-org#1513)

* Fix for w64devkit and mingw

* make kv_f16 the default for api users (ggml-org#1517)

* minor : fix compile warnings

* readme : adds WizardLM to the list of supported models (ggml-org#1485)

* main : make reverse prompt option act as a stop token in non-interactive mode (ggml-org#1032)

* Make reverse prompt option act as a stop token in non-interactive scenarios

* Making requested review changes

* Update gpt_params_parse and fix a merge error

* Revert "Update gpt_params_parse and fix a merge error"

This reverts commit 2bb2ff1.

* Update gpt_params_parse and fix a merge error take 2

* examples : add persistent chat (ggml-org#1495)

* examples : add persistent chat

* examples : fix whitespace

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* tests : add missing header

* ggml : use F16 instead of F32 in Q4_0, Q4_1, Q8_0 (ggml-org#1508)

* ggml : use F16 instead of F32 in Q4_0, Q4_1 and Q8_0

* llama : bump LLAMA_FILE_VERSION to 3

* cuda : update Q4 and Q8 dequantize kernels

* ggml : fix AVX dot products

* readme : update performance table + hot topics

* ggml : fix scalar implementation of Q4_1 dot

* llama : fix compile warnings in llama_set_state_data()

* llama : fix name shadowing and C4146 (ggml-org#1526)

* Fix name shadowing and C4146

* Fix if macros not using defined when required

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update llama-util.h

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Code style

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Georgi Gerganov <[email protected]>

* Fix for mingw (ggml-org#1462)

* llama : add llama_init_backend() API (close ggml-org#1527)

* feature : add blis and other BLAS implementation support (ggml-org#1502)

* feature: add blis support

* feature: allow all BLA_VENDOR to be assigned in cmake arguments. align with whisper.cpp pr 927

* fix: version detection for BLA_SIZEOF_INTEGER, recover min version of cmake

* Fix typo in INTEGER

Co-authored-by: Georgi Gerganov <[email protected]>

---------

Co-authored-by: Georgi Gerganov <[email protected]>

* Revert "feature : add blis and other BLAS implementation support (ggml-org#1502)"

This reverts commit 65c6ec3.

* GPU weights not in RAM, direct loading with cuFile

* llama : code style fixes + progress print fix

* ggml : ggml_mul better broadcast support

* cmake : workarounds for cufile when CMake version < 3.25

* gg rebase fixup

* Loop in llama.cpp, fixed progress callback

* Attempt clang-tidy fix

* llama : fix vram size computation

* Add forgotten fclose()

---------

Co-authored-by: András Salamon <[email protected]>
Co-authored-by: Ilya Kurdyukov <[email protected]>
Co-authored-by: Tom Jobbins <[email protected]>
Co-authored-by: rankaiyx <[email protected]>
Co-authored-by: Stephan Walter <[email protected]>
Co-authored-by: DannyDaemonic <[email protected]>
Co-authored-by: Erik Scholz <[email protected]>
Co-authored-by: Georgi Gerganov <[email protected]>
Co-authored-by: David Kennedy <[email protected]>
Co-authored-by: Jason McCartney <[email protected]>
Co-authored-by: Evan Jones <[email protected]>
Co-authored-by: Maxime <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Zenix <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@prusnak prusnak prusnak approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@ott2 @prusnak
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.