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

MBS-9255: Add ai spellcheck #7

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

Conversation

PM84
Copy link
Contributor

@PM84 PM84 commented Jul 20, 2024

Hi @marcusgreen

i added an option to the question to get a spellcheck of the answer.

This is now in a draft state, and I'll be happy to get your feedback!

This is what it should look like:

image

image

@PM84 PM84 force-pushed the MBS-9255_add_ai_spellcheck branch from 7cf93ea to c02059f Compare July 20, 2024 23:15
*/
private function get_spellchecking(array $response):string {
$fullaiprompt = $this->build_full_ai_spellchecking_prompt($response['answer']);
$ai = new local_ai_manager\manager('feedback');
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will fall over when using my back end because local_ai_manager does not exist. I have played with putting in a check to see if the class exists. However with the Moodle 4.5 AI subsystem due soon, I would like a way of checking for either my system, the mebis system, or the core system. Best wishes
Marcus

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