Skip to content

UPSTREAM PR #18866: convert_hf_to_gguf.py: refactor modify_tensors to call super#938

Open
loci-dev wants to merge 5 commits intomainfrom
upstream-PR18866-branch_am17an-refactor-modify-tensor
Open

UPSTREAM PR #18866: convert_hf_to_gguf.py: refactor modify_tensors to call super#938
loci-dev wants to merge 5 commits intomainfrom
upstream-PR18866-branch_am17an-refactor-modify-tensor

Conversation

@loci-dev
Copy link
Copy Markdown

Mirrored from ggml-org/llama.cpp#18866

Prepare for #18740 and possibly other weight merges. This PR is just a refactor, and shouldn't change any functionality

@loci-review
Copy link
Copy Markdown

loci-review bot commented Jan 16, 2026

Explore the complete analysis inside the Version Insights

Based on the analysis, no functions were identified with measurable performance changes between the base and target versions. This indicates no meaningful performance impact from the code changes.

@loci-dev loci-dev force-pushed the main branch 15 times, most recently from 2b0c660 to ab110c2 Compare January 19, 2026 12:16
@loci-dev loci-dev force-pushed the upstream-PR18866-branch_am17an-refactor-modify-tensor branch from c6c23ce to 13e3fac Compare January 19, 2026 13:46
@loci-review
Copy link
Copy Markdown

loci-review bot commented Jan 19, 2026

Explore the complete analysis inside the Version Insights

@loci-dev loci-dev force-pushed the main branch 6 times, most recently from 80b1525 to ddecb43 Compare January 20, 2026 11:11
@loci-dev loci-dev force-pushed the upstream-PR18866-branch_am17an-refactor-modify-tensor branch from d58771b to 492fe83 Compare January 20, 2026 12:47
@loci-review
Copy link
Copy Markdown

loci-review bot commented Jan 22, 2026

Analysis didn’t complete successfully. Explore Version Insights for details.

@loci-dev loci-dev force-pushed the main branch 29 times, most recently from 881552d to d592984 Compare January 26, 2026 15:13
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.

2 participants