feat: use tutor configs as django settings#1
Closed
Henrrypg wants to merge 6 commits intohpg/add-tutor-pluginfrom
Closed
feat: use tutor configs as django settings#1Henrrypg wants to merge 6 commits intohpg/add-tutor-pluginfrom
Henrrypg wants to merge 6 commits intohpg/add-tutor-pluginfrom
Conversation
Member
felipemontoya
left a comment
There was a problem hiding this comment.
the 2 things I dont like (as per our call)
- too many variables, we can try something like what django does for DATABASES
- even though we are writing it tutor first, we should make it safe for someone to not use tutor, which it currently is not as the variables jump from tutor to backend code
f2c9310 to
484f600
Compare
62b5e4b to
c0ca9d2
Compare
5ccd34c to
d117c81
Compare
feat: add tutor plugin and package installation
e4dafbb to
98566c7
Compare
--- adding simple configuration with key only for openai and anthropic and override capacity
98566c7 to
a00ac4d
Compare
863263b to
f9eed65
Compare
f9eed65 to
d82be77
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates the way configuration settings for AI model integration are managed in the
openedx_ai_extensionsplugin. The main change is moving from hardcoded values in the backend to dynamic, configurable settings that can be managed through Tutor and environment variables. This makes the plugin more flexible and easier to configure for different environments.Key changes include:
Configuration Management Improvements:
OPENEDX_AI_EXTENSIONS_VERSION,OPENEDX_AI_EXTENSIONS_API_KEY,OPENEDX_AI_EXTENSIONS_MODEL,OPENEDX_AI_EXTENSIONS_TEMPERATURE,OPENEDX_AI_EXTENSIONS_LLM_FUNCTION) to the Tutor patch file, enabling external configuration of AI model settings.plugin.py), so they are available and can be overridden as needed.Backend Refactoring:
OPENEDX_AI_EXTENSIONS_*) instead of hardcoded values for the AI model, API key, temperature, and LLM function in the workflow configuration.plugin_settingsfunction, making the backend rely solely on the externally provided settings.