Skip to content

DA-626: update Azure #25

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
Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

DA-626: update Azure #25

wants to merge 4 commits into from

Conversation

teetangh
Copy link
Collaborator

  • Added support for loading credentials from .env file
  • Improved Azure OpenAI configuration with optional environment variables
  • Enhanced error handling for OpenAI API calls
  • Updated notebook to use python-dotenv for credential management
  • Increased embedding dimensions from 1536 to 3072 in index configuration
  • Minor text corrections in markdown cells

@teetangh
Update Azure RAG notebook with environment variable support and impro…
e0217b0 
…ved error handling

- Added support for loading credentials from .env file
- Improved Azure OpenAI configuration with optional environment variables
- Enhanced error handling for OpenAI API calls
- Updated notebook to use python-dotenv for credential management
- Increased embedding dimensions from 1536 to 3072 in index configuration
- Minor text corrections in markdown cells
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 29, 2025 •
edited
Loading

Caution

Notebooks or Frontmatter Files Have Been Modified

  • Please ensure that a frontmatter.md file is accompanying the notebook file, and that the frontmatter is up to date.
  • These changes will be published to the developer portal tutorials only if frontmatter.md is included.
  • Proofread all changes before merging, as changes to notebook and frontmatter content will update the developer tutorial.

1 Notebook Files Modified:

Notebook File Frontmatter Included?
azure/RAG_with_Couchbase_and_AzureOpenAI.ipynb ✅

0 Frontmatter Files Modified:

Frontmatter File
Note: frontmatter will be checked and tested in the Test Frontmatter workflow.

@teetangh teetangh self-assigned this Jan 29, 2025
@teetangh teetangh changed the title DAD 626 - update Azure DA-626: update Azure Jan 29, 2025
@teetangh
cleaned azure tutorial
b591cfe
@teetangh
updated azure tutorial with auto batching
f73847b
@teetangh
Refactor Azure RAG notebook with improved collection setup and error …
de27ebd 
…handling

- Updated collection setup function to handle bucket, scope, and collection creation dynamically
- Reduced embedding dimensions from 3072 to 1536 in vector search index
- Simplified error handling in search index and RAG chain creation
- Removed unnecessary imports and batch processing mode configuration
- Enhanced logging and error messages for better debugging
- Improved overall code readability and error resilience
@nithishr
Copy link
Contributor

nithishr commented Feb 14, 2025 •
edited
Loading

@teetangh Same comments as AWS Bedrock(#24 (comment)). Addtionally,

  • The text mentions to use the batch size of 10 but the code is using a batch size of 100
  • The initial RAG response is not matching the query if you follow the notebook serially.

@teetangh teetangh marked this pull request as draft February 23, 2025 17:23
@teetangh teetangh added the help wanted Extra attention is needed label Mar 10, 2025
@teetangh
Copy link
Collaborator Author

teetangh commented Mar 10, 2025 •
edited
Loading

@deniswsrosa Need more quota to rerun the tutorial

@AayushTyagi1
Copy link
Contributor

Please merge main before merging this work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@teetangh teetangh

Labels
help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@teetangh @nithishr @AayushTyagi1