Skip to content
This repository has been archived by the owner on Dec 31, 2024. It is now read-only.

Add speech router and transcribe endpoint #23

Merged
merged 2 commits into from
Dec 5, 2023
Merged

Conversation

ReinderVosDeWael
Copy link
Contributor

This pull request adds a new speech router and a transcribe endpoint to the project. The speech router includes the necessary views for handling speech-related requests, and the transcribe endpoint uses OpenAI's Whisper API to transcribe audio files. This feature allows users to convert audio files into text format. The transcribe endpoint accepts audio files and returns the transcription as a string. This PR also includes the necessary dependencies and configurations for the new functionality.

@codecov-commenter
Copy link

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (4017f17) 94.72% compared to head (4800fd9) 94.73%.

Files Patch % Lines
src/linguaweb_api/microservices/openai.py 66.66% 2 Missing ⚠️
src/linguaweb_api/routers/speech/controller.py 96.77% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
+ Coverage   94.72%   94.73%   +0.01%     
==========================================
  Files          17       19       +2     
  Lines         417      475      +58     
==========================================
+ Hits          395      450      +55     
- Misses         22       25       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ReinderVosDeWael ReinderVosDeWael merged commit 75ca707 into main Dec 5, 2023
5 checks passed
@ReinderVosDeWael ReinderVosDeWael deleted the feature/speech branch December 5, 2023 17:58
@ReinderVosDeWael ReinderVosDeWael linked an issue Dec 5, 2023 that may be closed by this pull request
8 tasks
Comment on lines +11 to +12
apt-get install -y ffmpeg && \
pip install poetry && \
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
apt-get install -y ffmpeg && \
pip install poetry && \
apt-get install -y ffmpeg
RUN pip install poetry && \

I'd do separate run layers for apt and pip/poetry so it can cache when poetry breaks (way more likely than apt breaking)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Task: Speaking Endpoints
3 participants