Skip to content

Conversation

@TudorCRL
Copy link
Owner

No description provided.

Copy link

@jamied157 jamied157 left a comment

Choose a reason for hiding this comment

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

Looks good, few small things in the examples and I don't know how we deal with sample rate

raise RequestError("missing speechmatics python module: install using `pip install speechmatics-python`")

wav_data = audio_data.get_wav_data(
convert_rate=None if audio_data.sample_rate >= 16000 else 16000 # audio samples must be at least 16 kHz

Choose a reason for hiding this comment

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

You might know better than me - is this correct? Do we deal with sample rate >= 16000 ourselves?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Not sure, let me check with some of the heads

Copy link
Owner Author

Choose a reason for hiding this comment

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

I asked Dan, he reckons we don't get anything less than 8000, but may be able to support lower than that. It's not documented anywhere that I can see. Maybe we just let the runtime deal with it and pass any errors to users?

Choose a reason for hiding this comment

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

Yeah sounds good to me, we can keep an eye on the repo and see if anyone is having issues if it gets merged

Copy link
Owner Author

Choose a reason for hiding this comment

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

Removed that now, will resolve and merge and then open a PR into the upstream

@TudorCRL TudorCRL merged commit 614b921 into master Feb 21, 2023
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.

3 participants