Skip to content

CP3108 SR AI-powered marking #3126

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

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

EugeneOYZ1203n
Copy link

@EugeneOYZ1203n EugeneOYZ1203n commented Apr 9, 2025

Description

Creates a generate AI comments field in the Grading editor and workspace, also includes fields to enable LLM Grading and add a course specific API key in the course configuration panels.

Type of change

  • Added a AI Comment generation box in the Grading Editor and Workspace for Programming questions
  • Added enable LLM Grading and LLM API Key fields to course configuration with buttons in the Dropdown Create Course UI and in the Course Configuration Panel
  • Added 3 functions in Request Saga to link to backend

How to test

Nil. I don't know how to test frontend, do advise on what I should include.

Checklist

Do advise as well on how to update the documentation. Thanks

  • I have tested this code
  • I have updated the documentation

@coveralls
Copy link

coveralls commented Apr 17, 2025

Pull Request Test Coverage Report for Build 15554325324

Details

  • 3 of 43 (6.98%) changed or added relevant lines in 6 files are covered.
  • 2 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.07%) to 30.778%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/pages/academy/grading/subcomponents/GradingWorkspace.tsx 0 1 0.0%
src/commons/dropdown/DropdownCreateCourse.tsx 0 2 0.0%
src/pages/academy/adminPanel/subcomponents/CourseConfigPanel.tsx 0 2 0.0%
src/pages/academy/grading/subcomponents/GradingCommentSelector.tsx 0 4 0.0%
src/commons/sagas/RequestsSaga.ts 3 11 27.27%
src/pages/academy/grading/subcomponents/GradingEditor.tsx 0 23 0.0%
Files with Coverage Reduction New Missed Lines %
src/pages/academy/adminPanel/subcomponents/CourseConfigPanel.tsx 1 3.03%
src/pages/academy/grading/subcomponents/GradingEditor.tsx 1 0.0%
Totals Coverage Status
Change from base Build 15551817270: -0.07%
Covered Lines: 4847
Relevant Lines: 14859

💛 - Coveralls

@RichDom2185 RichDom2185 self-requested a review April 17, 2025 19:51
Copy link
Member

@RichDom2185 RichDom2185 left a comment

Choose a reason for hiding this comment

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

Is there a backend PR to accompany this frontend PR?

@RichDom2185 RichDom2185 added the blocked Something else needs pass review first label Apr 17, 2025
@EugeneOYZ1203n
Copy link
Author

Is there a backend PR to accompany this frontend PR?

This is the accompanying backend PR:
source-academy/backend#1248

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Something else needs pass review first
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants