Skip to content

Conversation

@TinzyWinzy
Copy link

Batch Feedback Refactor: Feedback Only After Paper Submission

Overview

This PR implements a batch-only feedback workflow for The Examiner app, in line with the new requirements and performance constraints.
AI-generated feedback (both local and cloud-based) is now provided only after the user submits the entire paper, rather than after each individual question.

Key Changes

Batch Feedback Workflow
Per-question feedback is disabled.
A "Submit Paper" button is added; feedback is only generated after submission.
On submission:
All answers for the paper are collected.
Local AI evaluation is run for each answer in a batch.
Feedback is stored in the DB (no schema changes).
Groq (cloud) analysis is queued for each question.
A modal displays all feedback at once, updating in real-time as cloud feedback arrives.

UI/UX

Loading/progress dialog during batch processing.
Feedback modal shows all questions, answers, and feedback at once.
"Cloud analysis pending" message until Groq feedback is received.
Modal updates in-place as finalized feedback arrives.

Database

No schema changes; all changes are operational.
Relies on the existing answer_history table.

Error Handling

User-friendly error messages for missing data, DB errors, or AI failures.
UI is blocked during processing to prevent interaction.

Rationale

Performance:

Defers heavy AI processing to the end, masking latency and improving responsiveness, especially on low-spec hardware.

User Experience:

Smoother, less disruptive feedback experience.

Compliance:

Follows the Batch-Only Feedback Rule and other project requirements.

Additional Notes

No changes to the database schema.
Only files directly involved in the feedback workflow were modified.

Please see BatchFeedback_Refactor_Report.md for rationale and implementation notes.

Ready for review and merge.

…ion, not per-question. Implements holistic batch feedback modal, disables per-question feedback, and queues cloud analysis for all questions post-submission. No DB schema changes. [Batch-Only Feedback Rule]
@EmmS21
Copy link
Owner

EmmS21 commented Aug 25, 2025

Hey. I never saw this PR. I thought you just weren't being responsive.

Please hit me up on WhatsApp again. If this works then I owe you $50 and a proper final interview

@TinzyWinzy
Copy link
Author

Hi Bro thats okay, its been a while i can take a look

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