Merge pull request #12 from Azure-Samples/codespace-fixes

Fixes from Codespace deployment

Author: pamelafox

README.md

@@ -142,7 +142,7 @@ Since the local app uses OpenAI models, you should first deploy it for the optim
 2. Run the FastAPI backend:
 
     ```shell
-    fastapi dev src/api/main.py
+    python3 -m uvicorn fastapi_app:create_app --factory --reload
     ```
 
     Or you can run "Backend" in the VS Code Run & Debug menu.

infra/main.bicep

@@ -17,25 +17,12 @@ param principalId string = ''
 
 @minLength(1)
 @description('Location for the OpenAI resource')
+// Look for gpt-35-turbo 0125 on the availability table:
 // https://learn.microsoft.com/en-us/azure/ai-services/openai/concepts/models#standard-deployment-model-availability
 @allowed([
-  'australiaeast'
-  'brazilsouth'
   'canadaeast'
-  'eastus'
-  'eastus2'
-  'francecentral'
-  'japaneast'
   'northcentralus'
-  'norwayeast'
-  'southafricanorth'
   'southcentralus'
-  'southindia'
-  'swedencentral'
-  'switzerlandnorth'
-  'uksouth'
-  'westeurope'
-  'westus'
 ])
 @metadata({
   azd: {

src/entrypoint.sh

@@ -1,3 +1,3 @@
 #!/bin/bash
 set -e
-python3 -m gunicorn fastapi_app:create_app()
+python3 -m gunicorn "fastapi_app:create_app()"

src/fastapi_app/postgres_engine.py

@@ -40,7 +40,7 @@ async def create_postgres_engine_from_env(azure_credential=None) -> AsyncEngine:
         host=os.environ["POSTGRES_HOST"],
         username=os.environ["POSTGRES_USERNAME"],
         database=os.environ["POSTGRES_DATABASE"],
-        password=os.environ["POSTGRES_PASSWORD"],
+        password=os.environ.get("POSTGRES_PASSWORD"),
         sslmode=os.environ.get("POSTGRES_SSL"),
         azure_credential=azure_credential,
     )

src/fastapi_app/query_rewriter.py

@@ -68,7 +68,7 @@ def extract_search_arguments(chat_completion: ChatCompletion):
             if function.name == "search_database":
                 arg = json.loads(function.arguments)
                 search_query = arg.get("search_query")
-                if "price_filter" in arg:
+                if "price_filter" in arg and arg["price_filter"]:
                     price_filter = arg["price_filter"]
                     filters.append(
                         {
@@ -77,7 +77,7 @@ def extract_search_arguments(chat_completion: ChatCompletion):
                             "value": price_filter["value"],
                         }
                     )
-                if "brand_filter" in arg:
+                if "brand_filter" in arg and arg["brand_filter"]:
                     brand_filter = arg["brand_filter"]
                     filters.append(
                         {