Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhanced Chat with Video and Database Integration #34

Merged
merged 7 commits into from
Jan 24, 2024

Conversation

movchan74
Copy link
Contributor

Summary:
This Pull Request focuses on enhancing the chat with video functionality by integrating it with the database.

Key Changes:

  1. Database for Chat with Video Usecase: Transitioned the chat with video feature to use a database instead of file-based storage.

  2. Database Function Bug Fixes: Addressed and resolved several bugs in the database functions.

  3. Video Table Enhancements: Added 'title' and 'description' fields to the video table.

  4. New Endpoint for Video Deletion: Implemented a new endpoint to delete indexed videos using their 'media_id', facilitating easier content management.

  5. Pipeline Configuration Bug Fixes: Fixed various bugs in the pipeline configuration.

  6. Model Loading Speed Enhancement: Optimized the loading time for the BLIP2 deployment, significantly speeding up the model initialization process.

@movchan74 movchan74 marked this pull request as ready for review January 19, 2024 18:57
@evanderiel
Copy link
Collaborator

Does this fix the issue we saw with jobs hanging incomplete?

@movchan74
Copy link
Contributor Author

Does this fix the issue we saw with jobs hanging incomplete?

I think so. I've tested it and it seems to work. Ideally, we need to add integration tests but I would do it as a separate PR.

@movchan74 movchan74 requested a review from evanderiel January 23, 2024 10:38
Copy link
Collaborator

@evanderiel evanderiel left a comment

Choose a reason for hiding this comment

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

ok

@movchan74 movchan74 merged commit c0a10b9 into main Jan 24, 2024
2 checks passed
@movchan74 movchan74 deleted the chat_with_video_db branch January 24, 2024 09:09
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.

2 participants