Pamela Fox

Results 809 comments of Pamela Fox

I just merged a different approach to private deployment which sets up a P2S VPN gateway that you can use for deployment, which works with the current NetworkACLs. But perhaps...

Nope, I'll fix it! We usually go with some sort of JSON-lines mime-type. Thanks for noticing!

@fabod You'll need to run `python -m pytest --snapshot-update` per the steps in CONTRIBUTING.md I tried to do that myself and update the PR, but since you forked off the...

I incorporated your suggestion in this PR: https://github.com/Azure-Samples/azure-search-openai-demo/pull/2709 And also made a few other changes for consistency. Thanks for raising this! I'll close the PR now.

cc @vhvb1989 for ideas on how we can improve the situation in our repo. Have you tried the flag that Victor added in that PR?

I think that'd be quite doable! If I was going to add that, I would start with a table with lat/lng coordinates. Then when the user asked the question, I'd...

Woot! I know this is a very popular request, thanks for the PR, I'll review it this week. In the meantime, if anyone subscribed tries it out as well, please...

I'm reviewing now, have made a few cosmetic changes and added a test. I'll add a few more tests tomorrow and test my deploy. Have also asked a Cosmos DB...

Update: I found an issue with the authentication decorator being used for the /items/id get and delete routes and have fixed it. It's working well for me locally now. I'll...

@fujita-h I added pytest for 97% coverage of CosmosDB. While adding the tests, I made a few changes: * 400 instead of 405 for CosmosDB feature disabled (I think 405...