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

feat: llama-index code migration! #65

Merged
merged 11 commits into from
Mar 13, 2024
Merged

Conversation

amindadgar
Copy link
Member

  • We needed to update codes based on the latest update of llama-index library.
  • We also, added skip to previously gdrive etl test cases as because the task was paused and yet to be completed.

- We needed to update codes based on the latest update of llama-index library.
- We also, added skip to previously gdrive etl codes as because it's task was paused and yet to be completed.
@amindadgar amindadgar linked an issue Mar 11, 2024 that may be closed by this pull request
@amindadgar amindadgar linked an issue Mar 11, 2024 that may be closed by this pull request
@amindadgar amindadgar requested review from TjitsevdM and cyri113 March 11, 2024 08:06
The backend lib now contains the llama-index-legacy which is for using its PGVectorStore and airflow was not supporting the new vectorstore version because of sqlalchemy!
Following best practices to make each DAG file more efficient.
That was because airflow was raising an error and couldn't bring the previous dags up.
- One of the github commit data relations was changed.
- When no data was available deletion was not producing empty string, which is fixed now.
- The summary type was added in other commits.
- The COMMITED relationships typo was fixed on staging database.
@amindadgar amindadgar linked an issue Mar 13, 2024 that may be closed by this pull request
@cyri113 cyri113 merged commit 76eef06 into main Mar 13, 2024
16 checks passed
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.

Update llama-index library dependency test case failing: test_discord_summary_transform_daily_summaries.py
3 participants