Skip to content

[User] shadow variable #1363

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Closed
maddin200 opened this issue May 8, 2023 · 1 comment · Fixed by #1367
Closed

[User] shadow variable #1363

maddin200 opened this issue May 8, 2023 · 1 comment · Fixed by #1367

Comments

@maddin200
Copy link

/llama.cpp line 973

Code:
{
const auto & hparams = model.hparams; // <-- also used line 892
prusnak added a commit that referenced this issue May 8, 2023
@prusnak
llama: fix shadowing
10632d8 
fixes #1363
@prusnak prusnak mentioned this issue May 8, 2023
Merged
@prusnak
Copy link
Collaborator

prusnak commented May 8, 2023

fix in #1367

ggerganov pushed a commit that referenced this issue May 8, 2023
@prusnak
llama : fix hparams shadow (#1367)
003ba2f 
fixes #1363
@Bearsaerker Bearsaerker mentioned this issue Mar 12, 2025
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

llama: fix shadowing ggml-org/llama.cpp
2 participants
@prusnak @maddin200