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

updating LLM configuration to get details from dotenv file #77

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

BethanyJep
Copy link

@BethanyJep BethanyJep commented Feb 3, 2025

Purpose

Bug fix - configuring LLM

Description

I was getting ValueError: No LLM service properly configured

solution: added the following to the config file:
from dotenv import load_dotenv

Load environment variables from .env file

load_dotenv()

Related Issue

none

Does this introduce a breaking change?

When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.

  • Yes
  • No

Type of change

  • Bugfix
  • Feature
  • Code style update (e.g., formatting, local variables)
  • Refactoring (no functional or API changes)
  • Documentation content changes
  • Other... Please describe:

Checklist

Before submitting your pull request, please confirm the following:

  • I have thoroughly tested my changes: I confirm that I have run the code and manually tested all affected areas.
  • All existing tests pass: I have run all tests and confirmed that nothing is broken.
  • I have added new tests (if applicable): I have written tests that cover the new functionality introduced by my code changes.
  • I have followed the Co-op Translators coding conventions: My code adheres to the style guide and coding conventions outlined in the repository.
  • I have documented my changes (if applicable): I have updated the documentation to reflect the changes where necessary.

Additional context

error details
Screenshot 2025-01-30 151615

@skytin1004
Copy link
Collaborator

skytin1004 commented Feb 4, 2025

Hi @BethanyJep,

Thank you for sharing the issue! May I ask which version of Co-op Translator you are using?
I believe this issue has been fixed since version 0.6.2 or later, but I'd appreciate it if you could confirm.

From what I know, this issue occurred between versions 0.6.0 and 0.6.1, and those versions have since been blocked. If you’re using a different version, I may need to investigate further.

Thank you again for your contribution!

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.

2 participants