-
Notifications
You must be signed in to change notification settings - Fork 304
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
Programming exercises
: Add Artemis intelligence consistency check
#10209
Programming exercises
: Add Artemis intelligence consistency check
#10209
Conversation
…-pipeline' into feature/communication/rephrasing-pipeline
…-pipeline' into feature/programming-exercises/rephrase-action-for-problem-statement
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code lgtm, just one small nit-pick
src/main/java/de/tum/cit/aet/artemis/iris/service/IrisConsistencyCheckService.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reapprove latest changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on TS1, worked as described.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on Ts1. It works as described. 👍
Programming exercises
: Add Artemis Intelligence consistency checkProgramming exercises
: Add Artemis intelligence consistency check
Important
This PR builds on #10156, merge afterwards. The Pyris code is already merged and should work on TS1, TS3, and TS5 (TS3 had some issues, which might still persist. Use TS1 or TS5)
Checklist
General
Server
I added multiple integration tests (Spring) related to the features (with a high test coverage).-> Follow-up due to time reasonsClient
I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.-> Follow-up due tue time reasonsauthorities
to all new routes and checked the course groups for displaying navigation elements (links, buttons).Motivation and Context
Although we try our best when creating a programming exercise sometimes there are inconsistencies between problem statement, template repository, solution repository and so one. Those mostly happen when updating an exercise to a different theme or generally forgetting to change something. To catch those inconsistencies, we want to use LLMs to detect them and make the instructors aware about them.
Description
Adds an Artemis Intelligence markdown action on the problem statement of programming exercises called "Check consistency". The action triggers a consistency check pipeline on the Pyris side and displays the result below the problem statement in a dismissible alert.
Steps for Testing
Prerequisites:
Make sure to deploy to a test server with Iris enabled (i.e. TS1, TS5 or TS3 if it is fixed already)
QuickSort
instead ofMergeSort
etc.)Testserver States
Note
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.
Review Progress
Code Review
Manual Tests
Test Coverage
Will be done as follow-up.
Screenshots
Summary by CodeRabbit
Release Notes: Artemis Intelligence Consistency Check Feature
New Features
User Interface
Backend Improvements