-
Notifications
You must be signed in to change notification settings - Fork 180
feat-logger-update #1173
feat-logger-update #1173
Conversation
|
@nguyenhoangthuan99 Please link this PR to issue via keyword - I've helped to do this in the Description |
@nguyenhoangthuan99 A few questions:
I imagine cortex.cpp may have some embedded use-cases in the future, where devs may have diskspace constraints for logs (difficult for me to imagine now with >1gb model sizes, but this may change in the future) |
Hi @dan-homebrew , Average each line will contains < 300 characters , each character is 1 byte. So 100,000 lines will be 300 * 100,000/(1024*1024) ~ 30Mb |
This looks good to me - let's include it as a default param in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
Great job! |
Fixes #1153