fix: readme updated quota, python version recommendation, fabric deployment guide (#520)

* fix: update setup_env.sh to use sudo for script permissions and removed azure function tools; improve user email retrieval in create_sql_user_and_role.sh; clean up requirements

* fix: update setup_env.sh to ensure execute permissions for quota_check_params.sh; adjust Deployment Guide token capacity recommendation and Python version range

* update FabricDeployment.md

* update FabricDeployment.md

* update FabricDeployment.md

* update parameter name for Azure deployment in CAdeploy.yml

* updated the allowed regions for Azure OpenAI location and removed gpt-4o model

* update quota_check.md to increase GPT-4o-mini token capacity and modify default regions in quota_check_params.sh

---------

Co-authored-by: Abdul-Microsoft <v-amujeebta@microsoft.com>

Author: Harsh-Microsoft

.devcontainer/setup_env.sh

@@ -5,6 +5,7 @@ git pull
 
 # provide execute permission to quotacheck script
 sudo chmod +x ./infra/scripts/checkquota.sh
+sudo chmod +x ./infra/scripts/quota_check_params.sh
 
 # Add the path to ~/.bashrc for persistence
 if ! grep -q '/opt/mssql-tools18/bin' ~/.bashrc; then

.github/workflows/CAdeploy.yml

@@ -115,7 +115,7 @@ jobs:
           az deployment group create \
             --resource-group ${{ env.RESOURCE_GROUP_NAME }} \
             --template-file infra/main.bicep \
-            --parameters AZURE_ENV_OPENAI_LOCATION=${{ env.AZURE_LOCATION }}
+            --parameters AzureOpenAILocation=${{ env.AZURE_LOCATION }}
 
       - name: List KeyVaults and Store in Array
         id: list_keyvaults

docs/DeploymentGuide.md

@@ -24,7 +24,7 @@ Here are some example regions where the services are available: East US, East US
 ### [Optional] Quota Recommendations  
 By default, the **Gpt-4o-mini model capacity** in deployment is set to **30k tokens**, so we recommend
 
-> **For Global Standard | GPT-4o-mini - the capacity to at least 100k tokens post-deployment for optimal performance.**
+> **For Global Standard | GPT-4o-mini - increase the capacity to at least 150k tokens post-deployment for optimal performance.**
 
 To adjust quota settings, follow these [steps](AzureGPTQuotaSettings.md)
 
@@ -80,7 +80,7 @@ If you're not using one of the above options for opening the project, then you'l
 1. Make sure the following tools are installed:
     - [PowerShell](https://learn.microsoft.com/en-us/powershell/scripting/install/installing-powershell?view=powershell-7.5) <small>(v7.0+)</small> - available for Windows, macOS, and Linux.
     - [Azure Developer CLI (azd)](https://aka.ms/install-azd)
-    - [Python 3.9+](https://www.python.org/downloads/)
+    - [Python 3.9 to 3.11](https://www.python.org/downloads/)
     - [Docker Desktop](https://www.docker.com/products/docker-desktop/)
     - [Git](https://git-scm.com/downloads)
     - [Microsoft ODBC Driver 18 for SQL Server](https://learn.microsoft.com/en-us/sql/connect/odbc/download-odbc-driver-for-sql-server?view=sql-server-ver16)

docs/FabricDeployment.md

@@ -47,17 +47,19 @@
 2.  Select Fabric Workspace
 3.  Click `Select`
 4.  After publish is complete, navigate to Fabric Workspace
-5.  Click `...` next to the Semantic model for Power BI report
+5.  You will find the report in the Fabric Workspace under the name **WealthAdvisor-Client360Report**.  
+6.  Use the report's filtering options to analyze client data.
+<!-- 5.  Click `...` next to the Semantic model for Power BI report
 6.  Click on `Settings` 
 7.  Click on `Edit credentials` (under Data source credentials)
 8.  Select `OAuth2` for the Authentication method
 9.  Select option for `Privacy level setting for this data source`
 10. Click `Sign in`
 11. Navigate back to Fabric workspace and click on Power BI report.
 12. Click on `File` -> `Embed Report` -> `Website or Portal` from the top menu.
-13. Copy the first line from the popup with title `Here's a link you can use to embed this content.` for use later in step 5.6.
+13. Copy the first line from the popup with title `Here's a link you can use to embed this content.` for use later in step 5.6. -->
 
-## Step 5: Update the Power BI URL in Azure App Service configuration
+<!-- ## Step 5: Update the Power BI URL in Azure App Service configuration
 1. Launch the Azure Portal [Azure Portal](https://portal.azure.com/).
 2. Enter `Resource Groups` in the top search bar.
 
@@ -80,4 +82,4 @@
 
 8. Click on `Overview` from the left menu. Then click on `Restart` button in the top menu. Then click on `Yes` in the pop-up message. 
 
-   ![Application Restart](images/fabric/AppServiceRestart.png)
+   ![Application Restart](images/fabric/AppServiceRestart.png) -->

docs/quota_check.md

@@ -1,7 +1,7 @@
 ## Check Quota Availability Before Deployment
 
 Before deploying the accelerator, **ensure sufficient quota availability** for the required model.
-> **For Global Standard | GPT-4o-mini - the capacity to at least 100K tokens for optimal performance.**
+> **For Global Standard | GPT-4o-mini - increase the capacity to at least 150K tokens for optimal performance.**
 
 ### Login if you have not done so already
 ```

infra/main.bicep

@@ -21,7 +21,6 @@ param deploymentType string = 'GlobalStandard'
 @description('Name of the GPT model to deploy:')
 @allowed([
   'gpt-4o-mini'
-  'gpt-4o'
 ])
 param gptModelName string = 'gpt-4o-mini'
 
@@ -50,7 +49,7 @@ param embeddingDeploymentCapacity int = 80
 param imageTag string = 'latest'
 
 //restricting to these regions because assistants api for gpt-4o-mini is available only in these regions
-@allowed(['australiaeast','eastus', 'eastus2','francecentral','japaneast','norwayeast','southindia', 'swedencentral','uksouth', 'westus', 'westus3'])
+@allowed(['australiaeast','eastus', 'eastus2','francecentral','japaneast','swedencentral','uksouth', 'westus', 'westus3'])
 @description('Azure OpenAI Location')
 param AzureOpenAILocation string = 'eastus2'
 

infra/scripts/quota_check_params.sh

@@ -93,7 +93,7 @@ az account set --subscription "$AZURE_SUBSCRIPTION_ID"
 echo "🎯 Active Subscription: $(az account show --query '[name, id]' --output tsv)"
 
 # Default Regions to check (Comma-separated, now configurable)
-DEFAULT_REGIONS="eastus,uksouth,eastus2,northcentralus,westus,westus2,southcentralus,canadacentral,australiaeast,swedencentral"
+DEFAULT_REGIONS="australiaeast,eastus,eastus2,francecentral,japaneast,swedencentral,uksouth,westus,westus3"
 IFS=',' read -r -a DEFAULT_REGION_ARRAY <<< "$DEFAULT_REGIONS"
 
 # Read parameters (if any)