Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add some logging utilities to capture whisper.cpp output #112

Merged
merged 3 commits into from
Mar 15, 2024
Merged

Add some logging utilities to capture whisper.cpp output #112

merged 3 commits into from
Mar 15, 2024

Conversation

tazz4843
Copy link
Owner

No description provided.

@thewh1teagle
Copy link
Contributor

thewh1teagle commented Jan 11, 2024 •
edited
Loading

The module whisper_sys_log is private not sure how should I use it. Tried to install your branch with the features enabled and it didn't worked.

@thewh1teagle
Copy link
Contributor

I cloned your repo, make the modules public but still it doesn't hide it

image

@tazz4843
Copy link
Owner Author

To be clear, you did call at least one of install_whisper_log_trampoline or install_whisper_tracing_trampoline?

@thewh1teagle
Copy link
Contributor

Yes

@tazz4843
Copy link
Owner Author

Gonna go ahead and merge, hopefully it's resolved with some fixes from other PRs. If not, let me know and I'll go ahead and dig into it.

@tazz4843 tazz4843 merged commit 586c517 into master Mar 15, 2024
@tazz4843 tazz4843 deleted the log-trampolines branch November 6, 2024 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tazz4843 @thewh1teagle