Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial PR: azure-search-openai-demo with dotnet backend #2

Merged
merged 55 commits into from
Apr 6, 2023

Conversation

dluc
Copy link
Member

@dluc dluc commented Apr 6, 2023

Initial PR: azure-search-openai-demo with dotnet backend

liamca and others added 30 commits February 8, 2023 13:00
Update README.md to highlight important note
Adding metadata to measure provisions/deploys of this app template
Adding metadata to measure telemetry
need to switch for pwsh windows forward slash './data/*'  otherwise no data is processed

PS /Users/alexzeltov/git/azure-search-openai-demo> python ./scripts/prepdocs.py '.\data\*' --searchkey "XZnvXI20iBoBfucFfsObjMTvVYXny0txh6n81NvwnGAzSeBUiHAL" --storageaccount $env:AZURE_STORAGE_ACCOUNT --container $env:AZURE_STORAGE_CONTAINER --searchservice $env:AZURE_SEARCH_SERVICE --index $env:AZURE_SEARCH_INDEX -v
Ensuring search index gptkbindex exists
Search index gptkbindex already exists
Processing files...
PS /Users/alexzeltov/git/azure-search-openai-demo> python ./scripts/prepdocs.py './data/*' --searchkey "XZnvXI20iBoBfucFfsObjMTvVYXny0txh6n81NvwnGAzSeBUiHAL" --storageaccount $env:AZURE_STORAGE_ACCOUNT --container $env:AZURE_STORAGE_CONTAINER --searchservice $env:AZURE_SEARCH_SERVICE --index $env:AZURE_SEARCH_INDEX -v 
Ensuring search index gptkbindex exists
Search index gptkbindex already exists
Processing files...
Processing './data/Northwind_Standard_Benefits_Details.pdf'
        Uploading blob for page 0 -> Northwind_Standard_Benefits_Details-0.pdf
        Uploading blob for page 1 -> Northwind_Standard_Benefits_Details-1.pdf
Notice of how to create a free Azure subscription
* Update prepdocs hook to use AzureDeveloperCliCredential
* Using tenant id as argument for prepdocs
* Allow existing resources
* Update readme and add roles.ps1
* Add links to roles
* remove comment
* Initial codespaces support
* Add linux scripts, update azure.yaml
* Create venv for backend
* Readme updates
* Add az cli to devcotnainer for roles script
* Add codespaces option to readme
* Minor readme and script updates
* Fix up prepdocs.sh
* Remove python detection and update .sh env load
* Add troubleshooting
* Update scripts to use pip/python from venv
* Update ps to use -m
* Add azd login
* Readme update with screenshots
* Add gitattributes
* Make .sh impls consistent
* Removing second tenant id
* support lin/win from prepdocs hook
* Update template version
* unwanted
* update start.ps1
* windows
* windows
* py3 fix
* patch
* win
* Tweak pre-reqs

---------

Co-authored-by: Victor Vazquez <[email protected]>
anatolip and others added 25 commits March 22, 2023 10:59
…tables, and document layout (#37)

* Add Form Recognizer integration. Keep local PDF parser option.

* Add conversion of Form Recognizer tables in to HTML tables understandable by ChatGPT
Add table splitting logic to make table split across sessions less.

* Add FormRecognizer service into bicep deployments
Add Cognitive Service Users role to make Form Recognizer work with DefaultAzureCredential.

* Add Form Recognizer service as parameteres for prepdocs script.

* Add image of the table with health plan cost into Benefit_Options.pdf. Now user can ask following questions:
- What is cost difference between plans?
- I don't have any dependents. What would be savings if I switch to Standard?

* Add additional prompt to return tabular data as html table.
Add table format for answer in CSS.

* Update ReadMe with information about Form Recognizer cost.

* Fix spellings

* Add html escaping inside html table generation
Parameter name is not valid and causing authentication error.
@dluc dluc merged commit 21843b2 into main Apr 6, 2023
@dluc dluc deleted the u/xiaoyun/dotnet branch April 6, 2023 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.