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

Persist Structured Grading Instructions (Modeling) #373

Open
wants to merge 20 commits into
base: develop
Choose a base branch
from

Conversation

LeonWehrhahn
Copy link
Contributor

Motivation and Context

A new structured_grading_criterion table is introduced to persist generated structured grading instructions and eliminating redundant computations for every feedback requests.

Description

A new structured_grading_criterion table is introduced to persist generated structured grading instructions, preventing redundant computation. This table stores:

  • exercise_id: Foreign key referencing the exercises table.
  • instructions_hash: A hash of relevant exercise data (problem statement, grading instructions, sample solution, max points, bonus points) to uniquely identify the instructions. This hash allows for invalidation and regeneration of cached instructions when underlying exercise data changes.
  • structured_grading_criterion: The JSON representation of the StructuredGradingCriterion object.
  • lms_url

Steps for Testing

Test if structured grading instructions for Modeling exercises are now persisted after the first feedback request for a particular exercise

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.


Screenshots

LeonWehrhahn and others added 19 commits November 16, 2024 19:30
…tionships; fix foreign key references and ensure proper inheritance structure.
…remove debug prints, update caching logic, and change serialization method for structured grading instructions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

1 participant