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

Use Moodle 4.5 Core AI subsystem #6

Open
wants to merge 4 commits into
base: forked
Choose a base branch
from
Open

Use Moodle 4.5 Core AI subsystem #6

wants to merge 4 commits into from

Conversation

PhMemmel
Copy link
Member

@PhMemmel PhMemmel commented Nov 5, 2024

Replaces #5

marcusgreen and others added 3 commits November 3, 2024 23:08
Conditionally use tool_aiconnect or moodle 4.5 coreai subsystem
A new function called perform_request checks the usemebisai setting
and if it is set then the mebis-lp ai subsystem is used, otherwise the
Core AI subsystem is used. The tool_aiconnect approach has been removed
@PhMemmel PhMemmel self-assigned this Nov 5, 2024
@marcusgreen
Copy link

This is looking good, I will read through in detail.

@marcusgreen
Copy link

I am at the edge of my git understanding. What is the next step in getting this merged so we are synchronised. Do you create a pull on my repo?

@PhMemmel
Copy link
Member Author

PhMemmel commented Nov 6, 2024

Hi @marcusgreen, we will internally test and review my latest changes. Once we're done, we're gonna do some cleanup on the commit history of our forked branch and send you a final PR to your repo from the cleaned up branch.

@marcusgreen
Copy link

I now have all your commits in my repo
https://github.com/marcusgreen/moodle-qtype_aitext/commits/405_core_ai/
So I will do a bit more testing myself.

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