chat-with-your-data-solution-accelerator icon indicating copy to clipboard operation
chat-with-your-data-solution-accelerator copied to clipboard
verified publisher icon Azure-Samples

feat: Remove LangChain from question answer tool

Open cecheta opened this issue 1 year ago • 1 comments

Required for #750

Purpose

  • This PR removes LangChain from the Question Anwer Tool
  • Changes the example user/assistant messages to system messages, as from the docs
  • Updates unit and functional tests

Does this introduce a breaking change?

  • [ ] Yes
  • [x] No

How to Test

  • Ask questions to LLM that require Question Answer Tool

cecheta avatar May 17 '24 12:05 cecheta

Coverage

Coverage Report •
FileStmtsMissCoverMissing
code/backend/batch/utilities/helpers
   llm_helper.py421173%40–41, 50, 61–62, 73, 86–87, 94, 113, 123
code/backend/batch/utilities/tools
   question_answer_tool.py550100% 
TOTAL242767272% 

Tests Skipped Failures Errors Time
205 0 :zzz: 0 :x: 0 :fire: 12.584s :stopwatch:

github-actions[bot] avatar May 17 '24 12:05 github-actions[bot]