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

Proposed Change to setup.py for Environment Variables #117

Open
VAN613 opened this issue Dec 19, 2024 · 0 comments
Open

Proposed Change to setup.py for Environment Variables #117

VAN613 opened this issue Dec 19, 2024 · 0 comments

Comments

@VAN613
Copy link

VAN613 commented Dec 19, 2024

Hello,

I would like to propose a change to the setup.py file. Currently, the file contains the following lines:

deployment_id = "text-embedding-ada-002"  # Example deployment ID
model_name = "text-embedding-ada-002"

I would like to modify it to use environment variables as follows:

import os
deployment_id = os.getenv("AZURE_OPENAI_EMBEDDING_DEPLOYMENT")
model_name = os.getenv("AZURE_OPENAI_EMBEDDING_MODEL")

Is there any issue with making this change? Thank you for your consideration.

Best regards,

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

No branches or pull requests

1 participant