-
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 rewriting for problem statement
#10156
Programming exercises
: Add Artemis intelligence rewriting for problem statement
#10156
Conversation
…-pipeline' into feature/communication/rephrasing-pipeline
…phrase-action-for-problem-statement' 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.
lgtm
The base branch was changed.
Programming exercises
: Add Artemis Intelligence rewriting for problem statement Programming exercises
: Add Artemis intelligence rewriting for problem statement
…tion-for-problem-statement
Caution Review failedThe pull request is closed. WalkthroughThis pull request introduces enhancements to the Changes
Sequence DiagramsequenceDiagram
participant Component as ProgrammingExerciseEditableInstructionComponent
participant Route as ActivatedRoute
participant ProfileService
participant ArtemisIntelligenceService
Component->>Route: Get courseId from route
Route-->>Component: Return courseId
Component->>ProfileService: Request profile info
ProfileService-->>Component: Return profile details
Component->>Component: Compute Iris enabled status
Component->>ArtemisIntelligenceService: Fetch Iris actions
ArtemisIntelligenceService-->>Component: Return available actions
Possibly related PRs
Suggested Labels
Suggested Reviewers
📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (5)
✨ Finishing Touches
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Important
This PR builds on #10157, please also review that one since it contains most of the code for the rewriting pipeline
Note
This PR depends on the counterpart in Iris. For testing, deploy Pyris#192 via
deploy:pyris-test
if it is not already deployed (lock:pyris-test
). You can use TS1, TS3, TS5 for testing since it has Iris enabled.Checklist
General
Client
Motivation and Context
We want to make it easier to provide high quality problem statements to students while reducing the effort needed by a instructors. For this we want integrated AI assistance to help us rewrite and improve the problem statement.
Description
Adds the Artemis Intelligence rewrite action to the problem statement when creating or editing a programming exercise.
Steps for Testing
Prerequisites:
Make sure that Pyris#192 is deployed
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
Unchanged
Screenshots
Summary by CodeRabbit
New Features
Tests
Improvements
OnInit
lifecycle hook in programming exercise instruction component