Azure-Samples / azure-search-openai-javascript

A TypeScript sample app for the Retrieval Augmented Generation pattern running on Azure, using Azure AI Search for retrieval and Azure OpenAI and LangChain large language models (LLMs) to power ChatGPT-style and Q&A experiences.
MIT License
252 stars 130 forks source link

Approach radio buttons state not being passed to search API #185

Closed JasonHaley closed 8 months ago

JasonHaley commented 8 months ago

This issue is for a: (mark with an x)

- [x ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

azd auth login azd up

go to static web app select any of the default search questions and click the search buttoon open the though process verify there is text like "[DynamicTool]" in there Open the Developer Settings in upper right corner Notice the "Retrieve-Then-Read" is selected

The ask-read-retrieve-read is the approach that uses the Dynamic Tool.

You can try changing the approach in the Developer Settings, but it does not seem to work.

Also the OneShot.tsx file does have the rrr hard-coded.

Expected/desired behavior

Change the approach being used when the Developer Setting is changed

JasonHaley commented 8 months ago

This can be closed, it was fixed this week.