Skip to content

Name colors #11573

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 1 commit into from
Feb 2, 2025
Merged

Name colors #11573

merged 1 commit into from
Feb 2, 2025

Conversation

ericcurtin
Copy link
Collaborator

It's more descriptive, use #define's so we can use compile-time concatenations.

@ggerganov
Copy link
Member

Wouldn't it be better if llama-run started using the common/log functionality instead?

I think llama-run should either fully use common and share functionality with the rest of the examples, or do not link common and implement everything from scratch to be self-contained.

It's more descriptive, use #define's so we can use compile-time
concatenations.

Signed-off-by: Eric Curtin <[email protected]>
@ericcurtin
Copy link
Collaborator Author

Wouldn't it be better if llama-run started using the common/log functionality instead?

I think llama-run should either fully use common and share functionality with the rest of the examples, or do not link common and implement everything from scratch to be self-contained.

Sure SGTM, I move the colors to log.h now.

@ericcurtin ericcurtin merged commit 84ec8a5 into master Feb 2, 2025
44 of 45 checks passed
@ericcurtin ericcurtin deleted the colors-description branch February 2, 2025 15:14
@ericcurtin
Copy link
Collaborator Author

Flakey build

tinglou pushed a commit to tinglou/llama.cpp that referenced this pull request Feb 13, 2025
It's more descriptive, use #define's so we can use compile-time
concatenations.

Signed-off-by: Eric Curtin <[email protected]>
orca-zhang pushed a commit to orca-zhang/llama.cpp that referenced this pull request Feb 26, 2025
It's more descriptive, use #define's so we can use compile-time
concatenations.

Signed-off-by: Eric Curtin <[email protected]>
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Feb 26, 2025
It's more descriptive, use #define's so we can use compile-time
concatenations.

Signed-off-by: Eric Curtin <[email protected]>
mglambda pushed a commit to mglambda/llama.cpp that referenced this pull request Mar 8, 2025
It's more descriptive, use #define's so we can use compile-time
concatenations.

Signed-off-by: Eric Curtin <[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.

2 participants