Skip to content

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.

3 participants