Skip to content

Fix "_init_model" resseting params of Command Prompt #99

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 5 commits into from
Feb 19, 2025

Conversation

SirDaXll
Copy link
Contributor

@SirDaXll SirDaXll commented Feb 17, 2025

#97 Not the main problem, but one caused by it. Fixed a problem, in these new commit (version 1.3.1.dev13), that occurred when calling pwcpp from the command line, where there was a line in the code (_init_model) which resets what was entered to certain default values ​​(for example, I want to transcribe in Spanish, but it returns it to English with "[Spanish]" and "(speaking in foreign language)"). For this, different lines were added to the code to capture the logs provided by the program in the console (it captured the parameters, but restarted them). Finally, the problem was solved by deleting a line (actually commented), and I deleted the log captures that I placed to identify the problem. I tested the program in the command prompt and in a .py file with different parameters, and there were no errors.

absadiki and others added 4 commits January 30, 2025 01:47
chore(repair-wheel): make the repair-wheel logic optional absadiki#95
fix(print_progress): print progress callback absadiki#97
Fix issue with parameters resetting to default values when executing in the command prompt.
Copy link
Owner

@absadiki absadiki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @SirDaXll for the PR.
Please delete that line and also if you can squash the commit history into one commit or fork the latest main branch and push your fix.

@SirDaXll SirDaXll closed this Feb 19, 2025
@SirDaXll SirDaXll reopened this Feb 19, 2025
@SirDaXll
Copy link
Contributor Author

@absadiki The "whisper.cpp" folder won't let me squash commits.

@absadiki
Copy link
Owner

Okey. no worries.
Thanks @SirDaXll.

@absadiki absadiki merged commit 2cdf91a into absadiki:fix/print-progress Feb 19, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants