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

Migrate chat_with_video Project from Aana SDK #1

Merged
merged 9 commits into from
Aug 5, 2024
Merged

Conversation

movchan74
Copy link
Contributor

Summary:
Migrated chat_with_video project from Aana SDK.

Key Changes:

  1. Code Migration: Migrated all the functionality of the project to a separate repo.
  2. Database: Moved Extended Video Entity and all related entities to the project as well as their repositories. Added alembic setup for the project and migrations.
  3. Tests: Moved project tests to the repo but reused fixtures from the SDK.

Notes:

  • The database setup will serve as an example for other Aana Apps that need a database. My suggestion would be to add it to the app template.
  • Right now the app uses the remove_projects branch of the Aana SDK. We might want to change it later once remove_projects is merged and the new SDK is released.

Copy link
Contributor

@HRashidi HRashidi left a comment

Choose a reason for hiding this comment

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

We need to add a github action with gpu for tests.

@movchan74 movchan74 requested a review from evanderiel August 2, 2024 08:37
@movchan74 movchan74 merged commit bbe52d2 into main Aug 5, 2024
6 checks passed
@movchan74 movchan74 deleted the app_implementation branch August 5, 2024 08:23
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.

3 participants