Skip to content

Feature/pdf export whisper fallback#965

Open
KolaSailaja wants to merge 3 commits into
imDarshanGK:mainfrom
KolaSailaja:feature/pdf-export-whisper-fallback
Open

Feature/pdf export whisper fallback#965
KolaSailaja wants to merge 3 commits into
imDarshanGK:mainfrom
KolaSailaja:feature/pdf-export-whisper-fallback

Conversation

@KolaSailaja

Copy link
Copy Markdown

📌 Related Issue

Fixes #870


🔍 Description

This PR adds PDF export support for chat/session exports and implements a graceful fallback when Whisper is unavailable.

Changes made

  • Added pdf to the supported ExportFormat enum.

  • Implemented PDF generation for export endpoints.

  • Added fallback handling when the Whisper dependency is unavailable:

    • Catches missing Whisper import errors.
    • Logs a warning instead of failing the export.
    • Continues generating a text-only PDF export.
  • Updated backend dependencies to include the required PDF generation library.

  • Added tests covering PDF export functionality and the Whisper fallback behavior.


📂 Files Changed

  • backend/models/schemas.py
  • backend/routes/export.py
  • backend/requirements.txt
  • backend/requirements_fixed.txt
  • backend/tests/test_pdf_export.py

🧪 Testing

  • Verified PDF export generation.
  • Verified export succeeds when Whisper is unavailable.
  • Verified existing export formats remain unaffected.
  • Added automated tests for the new functionality.

📸 Screenshots

N/A (Backend feature)


✅ Checklist

  • Implemented the requested functionality.
  • Added tests for the modified path.
  • Preserved existing functionality.
  • Followed the project's coding style.
  • Kept changes focused on the requested feature.

@vercel

vercel Bot commented Jul 9, 2026

Copy link
Copy Markdown

@KolaSailaja is attempting to deploy a commit to the Darshan's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

Add fallback when Whisper is missing to PDF export

1 participant