Skip to content

E2609: Add calibration participant and review comparison APIs#323

Open
isaacmartin7777-maker wants to merge 14 commits intoexpertiza:mainfrom
ruju4a:main
Open

E2609: Add calibration participant and review comparison APIs#323
isaacmartin7777-maker wants to merge 14 commits intoexpertiza:mainfrom
ruju4a:main

Conversation

@isaacmartin7777-maker
Copy link
Copy Markdown

Summary

  • add instructor-only calibration endpoints to create/list calibration participants and start/save instructor calibration reviews
  • return calibration submission artifacts in JSON, including submitted hyperlinks and files for each calibration participant
  • add calibration report payloads for comparing instructor and student reviews, including rubric metadata and per-item agreement/distribution summaries
  • persist and use for_calibration response map state to support calibrated assignment flows and student task visibility

Why

This PR implements the backend support for calibration assignments. Instructors can add a calibration participant by username, create the gold-standard review for that submission, and provide the frontend with the data needed to display submission content and compare student reviews against the instructor review.

Test Plan

  • run request specs for calibration response maps
  • run request specs for calibration reports
  • verify calibration participants are created idempotently and tagged with for_calibration
  • verify calibration responses can be started, saved, and submitted
  • verify calibration report JSON includes instructor response, student responses, submitted content, and per-item summaries

Isaac Martin and others added 14 commits March 19, 2026 19:55
Implement GET /assignments/:id/calibration_submission
- Add calibration_reports_controller with show action for calibration data
- Add ensures_instructor_assignment_participant concern
- Add app/services: CalibrationPerItemSummary and CalibrationSubmittedContent
- Add calibration_reports_spec and spec/services tests
- Add docker-compose.prod.yml for production deployment
- Update routes.rb: add calibration_reports route and instructor_response endpoint
- Update assignments, calibration_response_maps, and all other controllers
- Update Gemfile: add redis gem, move faker to all groups, align with Rails 8
- Update models, helpers, serializers, config, and specs throughout
- Update Dockerfile, README, GitHub workflows, rubocop, rspec config

Made-with: Cursor
@github-actions
Copy link
Copy Markdown

5 Warnings
⚠️ Pull request is too big (more than 500 LoC).
⚠️ Pull request has duplicated commit messages.
⚠️ Schema changes detected without a corresponding DB migration.
⚠️ Pull request modifies config or setup files: config/database.yml, setup.sh.
⚠️ RSpec tests seem shallow (single it blocks or no context). Consider improving test structure.

Generated by 🚫 Danger

@isaacmartin7777-maker isaacmartin7777-maker changed the title Add calibration participant and review comparison APIs E2609: Add calibration participant and review comparison APIs Mar 31, 2026
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