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.

Sign up for GitHub

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

Jump to bottom

Marktab 20230919 #13

Merged
merged 5 commits into from
Sep 20, 2023
Merged

Marktab 20230919 #13

merged 5 commits into from
Sep 20, 2023

Conversation

marktab
Copy link
Collaborator

@marktab marktab commented Sep 19, 2023

Update code for vector search (drawn from MSUS repository):

  • Updated utils.py (preserving Andy Cox semantic search function)
  • Updated notebooks 1-3 to synch with MSUS (new vector search code)
  • Added notebook zero to check air-gapped environments (designed to be robust for all air-gapped environments)

@marktab
Resource Validation
99c6fe1 
This code provides basic Azure environment checking
@marktab
Update utils.py
e2ce051 
Synch utils.py with MSUS repository (required for new notebook) -- added conditional logic for processing search_results only if 'value' is populated.

Preserved Andy Cox function for validating semantic search capabilities
@marktab
Notebook 1
abb276f 
Add refreshed version of notebook one which has updated vector search code.
@marktab
Notebook 2 & 3 Refresh
8e958b3 
Aligning with MSUS repository:  new code for vector search
@marktab
Update credentials.env
caac5be 
Added form recognizer (document intelligence) prompts
@marktab
Copy link
Collaborator Author

marktab commented Sep 19, 2023

@cheruvubh please review the credentials.env, and then merge

@marktab marktab merged commit d2a6bd0 into main Sep 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cheruvu1 cheruvu1 cheruvu1 approved these changes

@cheruvubh cheruvubh cheruvubh approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@marktab @cheruvu1 @cheruvubh