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

Add override UI settings configuration #36

Merged
merged 1 commit into from
Aug 15, 2024
Merged

Conversation

SmartManoj
Copy link
Owner

@SmartManoj SmartManoj commented Aug 15, 2024

Add override_UI_setttings configuration to the application.

  • opendevin/core/config.py

    • Add override_UI_setttings attribute to the AppConfig class.
    • Set the default value of override_UI_setttings to False.
  • opendevin/server/session/session.py

    • Check the value of override_UI_setttings before overriding the default LLM configuration.
    • If override_UI_setttings is False, override the LLM model, API key, and base URL with values from the request arguments.

For more details, open the Copilot Workspace session.

Add `override_UI_setttings` configuration to the application.

* **opendevin/core/config.py**
  - Add `override_UI_setttings` attribute to the `AppConfig` class.
  - Set the default value of `override_UI_setttings` to `False`.

* **opendevin/server/session/session.py**
  - Check the value of `override_UI_setttings` before overriding the default LLM configuration.
  - If `override_UI_setttings` is `False`, override the LLM model, API key, and base URL with values from the request arguments.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/SmartManoj/Kevin?shareId=XXXX-XXXX-XXXX-XXXX).
@SmartManoj SmartManoj merged commit 289a577 into kevin Aug 15, 2024
10 checks passed
@SmartManoj SmartManoj deleted the add-override-ui-settings branch August 15, 2024 13:15
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.

1 participant