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

Integrate LlamaIndex in finesse-frontend #79

Closed
4 tasks
Tracked by #91
k-allagbe opened this issue Jan 30, 2024 · 4 comments · Fixed by #96
Closed
4 tasks
Tracked by #91

Integrate LlamaIndex in finesse-frontend #79

k-allagbe opened this issue Jan 30, 2024 · 4 comments · Fixed by #96
Assignees
Labels
user-visible user-visible (requires QA)

Comments

@k-allagbe
Copy link
Member

k-allagbe commented Jan 30, 2024

In finesse-frontend, we are adding LlamaIndex as a new search source. This will involve integrating it into the debug panel for source selection and implementing backend calls for retrieving search results.

We should also take this opportunity to remove the simulated source.

Tasks

  • Remove the simulated option from the debug panel.
  • Include LlamaIndex in the debug panel for source selection.
  • Conduct thorough testing to ensure accurate retrieval and integration of search results from LlamaIndex.
  • Update project documentation to reflect the addition and usage of LlamaIndex.

Acceptance Criteria

  • The simulated source is no longer available.
  • The system correctly fetches and displays search results from LlamaIndex without significant performance issues.
  • The debug panel UI clearly shows LlamaIndex as an available search source.
  • Documentation is updated to include details on LlamaIndex integration and usage.
@k-allagbe k-allagbe added the user-visible user-visible (requires QA) label Jan 30, 2024
@k-allagbe k-allagbe added this to the 2024Feb milestone Jan 30, 2024
@nmeraihi
Copy link
Member

I suggest that we keep only two options for now;

  • Use Azure AI searche (activated by default)
  • LlamaIndex (deactivated until we fix the results)

@k-allagbe
Copy link
Member Author

I suggest that we keep only two options for now;

* [x]  Use Azure AI searche (activated by default)

* [ ]  LlamaIndex (deactivated until we fix the results)

I changed the description to reflect that.

@k-allagbe
Copy link
Member Author

I suggest that we keep only two options for now;

* [x]  Use Azure AI searche (activated by default)

* [ ]  LlamaIndex (deactivated until we fix the results)

@rngadam fixed ai-cfia/ailab-db#38. Should we still remove it from the debug panel?

@k-allagbe k-allagbe self-assigned this Mar 27, 2024
@k-allagbe k-allagbe changed the title Integrate LlamaIndex as a Search Source in finesse-frontend Integrate LlamaIndex in finesse-frontend Mar 27, 2024
@k-allagbe k-allagbe removed this from the 2024Feb milestone Mar 27, 2024
k-allagbe added a commit that referenced this issue Mar 27, 2024
k-allagbe added a commit that referenced this issue Mar 27, 2024
k-allagbe added a commit that referenced this issue Mar 27, 2024
@k-allagbe k-allagbe linked a pull request Mar 27, 2024 that will close this issue
4 tasks
k-allagbe added a commit that referenced this issue Mar 27, 2024
@nmeraihi
Copy link
Member

oui parfait!

k-allagbe added a commit that referenced this issue Apr 2, 2024
k-allagbe added a commit that referenced this issue Apr 2, 2024
…e-frontend

issue #79: integrate llamaindex in finesse-frontend
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
user-visible user-visible (requires QA)
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants