Skip to content

modifications to the librispeech-parakeet#1

Open
bhargav191098 wants to merge 2 commits into
danbev:parakeet-supportfrom
bhargav191098:bench_changes
Open

modifications to the librispeech-parakeet#1
bhargav191098 wants to merge 2 commits into
danbev:parakeet-supportfrom
bhargav191098:bench_changes

Conversation

@bhargav191098

Copy link
Copy Markdown
  • Writing a run_eval.py file to do all the things that the eval.mk file (eval.mk does download librispeech, extract and run eval.py file) does, so that this runs on Windows PowerShell with no installation steps.
  • Moving away model load logic out of the audio file load logic so that we can load once and do inference on multiple audio files : this mirrors how whisper-cli works.
    +The eval.mk was expecting cuda-89 build, directory but since the build-dir name is user specified, it made sense to make the cli exe build directory as a required parameter to the benchmark py script.

fprintf(stderr, "Loading Parakeet model from: %s\n", params.model.c_str());
}

struct parakeet_context * pctx = parakeet_init_from_file_with_params(params.model.c_str(), ctx_params);

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I've added this to the main branch as discovered a bug in the process releated to the lstm state not being cleared. I'll add you as co-author to the PR 👍

@@ -0,0 +1,270 @@
#!/usr/bin/env python3

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I'd prefer to leave this out of this commit and we can do this in a follow up commit. The original PR is very large and it will take reviewers some time to go through. Keeping this the same as whispers test code will make this a little easier I think. I was able to run the make version on windows so it will work there with some dependency installations.

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