-
Notifications
You must be signed in to change notification settings - Fork 3
Closed
Labels
code-qualityCode quality improvementsCode quality improvementsgood first issueGood for newcomersGood for newcomersloggingLogging improvementsLogging improvements
Description
Summary
The worker/transcription.py file uses 27+ print() statements instead of Python's logging module.
Current Behavior
print(f"Loading Whisper model: {WHISPER_MODEL}...")
print("Model loaded successfully")
print(f" Transcribing: {audio_path.name}")Expected Behavior
logger.info(f"Loading Whisper model: {WHISPER_MODEL}...")
logger.info("Model loaded successfully")
logger.debug(f"Transcribing: {audio_path.name}")Files to Modify
worker/transcription.py(~27 print statements)
Implementation Steps
- Add
import loggingand create a module logger:logger = logging.getLogger(__name__) - Replace all
print()calls with appropriate log levels - Run tests:
pytest tests/test_transcription*.py
Acceptance Criteria
- No
print()statements remain in the file - All logging uses appropriate log levels
- Existing tests pass
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
code-qualityCode quality improvementsCode quality improvementsgood first issueGood for newcomersGood for newcomersloggingLogging improvementsLogging improvements