Pamela Fox

Results 771 comments of Pamela Fox

Going to close this issue since we haven't heard back. Please comment or open a new issue if you find an error in the logs that you need help deciphering....

@tonybaloney Another similar PR just came in: https://github.com/Azure-Samples/azure-search-openai-demo/pull/561 (I'm guessing its basically the same, as the rules probably default to empty lists) Do you know if there are security implications...

@tonybaloney Ah I ran the sec check on the latest PR, and it does indeed think its a bad change security-wise: https://github.com/Azure-Samples/azure-search-openai-demo/pull/561/checks?check_run_id=16182143029

I wonder if this is an underlying issue with the OpenAI deployments API, should networkAcls really be required?

We've now merged a PR with these changes: https://github.com/Azure-Samples/azure-search-openai-demo/pull/561 So I'll close this one. Thank you @den130 and sorry it took so long for merging. We're more comfortable with the...

Yes, you should be able to change the prompt in the appropriate file, as @Tibap says. Closing this issue, please comment if you were unable to find the correct file.

@tonybaloney I wonder if that was an intermittent error? It's listed at that exact URL on: https://mcr.microsoft.com/en-us/product/devcontainers/python/about I also just started a Codespace from this branch and it succeeded: ```...

The best way to debug a 500 is to look at the App Service logs, specifically the default_docker logs. The README has info on doing that, both at the bottom...

If you do not want those sources, you should modify the prompt in https://github.com/Azure-Samples/azure-search-openai-demo/blob/36478266c221930257a7a4888c097ba1b58b4926/app/backend/approaches/chatreadretrieveread.py#L30 and re-deploy the backend. And yes, we changed the request format to be a closer match...