mirror of
https://github.com/Monadical-SAS/reflector.git
synced 2025-12-20 20:29:06 +00:00
* fix: refactor modal API key configuration for better separation of concerns - Split generic MODAL_API_KEY into service-specific keys: - TRANSCRIPT_API_KEY for transcription service - DIARIZATION_API_KEY for diarization service - TRANSLATE_API_KEY for translation service - Remove deprecated *_MODAL_API_KEY settings - Add proper validation to ensure URLs are set when using modal processors - Update README with new configuration format BREAKING CHANGE: Configuration keys have changed. Update your .env file: - TRANSCRIPT_MODAL_API_KEY → TRANSCRIPT_API_KEY - LLM_MODAL_API_KEY → (removed, use TRANSCRIPT_API_KEY) - Add DIARIZATION_API_KEY and TRANSLATE_API_KEY if using those services * fix: update Modal backend configuration to use service-specific API keys - Changed from generic MODAL_API_KEY to service-specific keys: - TRANSCRIPT_MODAL_API_KEY for transcription - DIARIZATION_MODAL_API_KEY for diarization - TRANSLATION_MODAL_API_KEY for translation - Updated audio_transcript_modal.py and audio_diarization_modal.py to use modal_api_key parameter - Updated documentation in README.md, CLAUDE.md, and env.example * feat: implement auto/modal pattern for translation processor - Created TranscriptTranslatorAutoProcessor following the same pattern as transcript/diarization - Created TranscriptTranslatorModalProcessor with TRANSLATION_MODAL_API_KEY support - Added TRANSLATION_BACKEND setting (defaults to "modal") - Updated all imports to use TranscriptTranslatorAutoProcessor instead of TranscriptTranslatorProcessor - Updated env.example with TRANSLATION_BACKEND and TRANSLATION_MODAL_API_KEY - Updated test to expect TranscriptTranslatorModalProcessor name - All tests passing * refactor: simplify transcript_translator base class to match other processors - Moved all implementation from base class to modal processor - Base class now only defines abstract _translate method - Follows the same minimal pattern as audio_diarization and audio_transcript base classes - Updated test mock to use _translate instead of get_translation - All tests passing * chore: clean up settings and improve type annotations - Remove deprecated generic API key variables from settings - Add comments to group Modal-specific settings - Improve type annotations for modal_api_key parameters * fix: typing * fix: passing key to openai * test: fix rtc test failing due to change on transcript It also correctly setup database from sqlite, in case our configuration is setup to postgres. * ci: deactivate translation backend by default * test: fix modal->mock * refactor: implementing igor review, mock to passthrough
44 lines
1.6 KiB
Python
44 lines
1.6 KiB
Python
import httpx
|
|
|
|
from reflector.processors.audio_diarization import AudioDiarizationProcessor
|
|
from reflector.processors.audio_diarization_auto import AudioDiarizationAutoProcessor
|
|
from reflector.processors.types import AudioDiarizationInput, TitleSummary
|
|
from reflector.settings import settings
|
|
|
|
|
|
class AudioDiarizationModalProcessor(AudioDiarizationProcessor):
|
|
INPUT_TYPE = AudioDiarizationInput
|
|
OUTPUT_TYPE = TitleSummary
|
|
|
|
def __init__(self, modal_api_key: str | None = None, **kwargs):
|
|
super().__init__(**kwargs)
|
|
if not settings.DIARIZATION_URL:
|
|
raise Exception(
|
|
"DIARIZATION_URL required to use AudioDiarizationModalProcessor"
|
|
)
|
|
self.diarization_url = settings.DIARIZATION_URL + "/diarize"
|
|
self.modal_api_key = modal_api_key
|
|
self.headers = {}
|
|
if self.modal_api_key:
|
|
self.headers["Authorization"] = f"Bearer {self.modal_api_key}"
|
|
|
|
async def _diarize(self, data: AudioDiarizationInput):
|
|
# Gather diarization data
|
|
params = {
|
|
"audio_file_url": data.audio_url,
|
|
"timestamp": 0,
|
|
}
|
|
async with httpx.AsyncClient() as client:
|
|
response = await client.post(
|
|
self.diarization_url,
|
|
headers=self.headers,
|
|
params=params,
|
|
timeout=None,
|
|
follow_redirects=True,
|
|
)
|
|
response.raise_for_status()
|
|
return response.json()["diarization"]
|
|
|
|
|
|
AudioDiarizationAutoProcessor.register("modal", AudioDiarizationModalProcessor)
|