Skip to content

gguf-split : add basic checks #9499

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Sep 15, 2024
Merged

gguf-split : add basic checks #9499

merged 2 commits into from
Sep 15, 2024

Conversation

slaren
Copy link
Member

@slaren slaren commented Sep 15, 2024

  • Do not overwrite existing files
  • Do not ignore extraneous arguments

Fixes #9498 #8264

@slaren slaren merged commit e6deac3 into master Sep 15, 2024
57 checks passed
@slaren slaren deleted the sl/gguf-merge-no-overwrite branch September 15, 2024 17:02
dsx1986 pushed a commit to dsx1986/llama.cpp that referenced this pull request Oct 29, 2024
* gguf-split : do not overwrite existing files when merging

* gguf-split : error when too many arguments are passed
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 15, 2024
* gguf-split : do not overwrite existing files when merging

* gguf-split : error when too many arguments are passed
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 18, 2024
* gguf-split : do not overwrite existing files when merging

* gguf-split : error when too many arguments are passed
Nexesenex pushed a commit to Nexesenex/croco.cpp that referenced this pull request Feb 25, 2025
* gguf-split : do not overwrite existing files when merging

* gguf-split : error when too many arguments are passed
Nexesenex added a commit to Nexesenex/croco.cpp that referenced this pull request May 24, 2025
gguf-split : improve --split and --merge logic (ggml-org#9619)

* make sure params --split and --merge are not specified at same time

* update gguf-split params parse logic

* Update examples/gguf-split/gguf-split.cpp

Co-authored-by: Xuan Son Nguyen <[email protected]>
Co-authored-by: slaren <[email protected]>

---------

gguf-split : add basic checks (ggml-org#9499)

* gguf-split : do not overwrite existing files when merging

* gguf-split : error when too many arguments are passed

Authored-by: slaren <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: can not merge gguf, gguf_init_from_file: invalid magic characters ''
2 participants