Skip to content

feat(CRater): Ping endpoint#2177

Merged
hirokiterashima merged 8 commits into
developfrom
issue-2159-ping-endpoint
May 29, 2025
Merged

feat(CRater): Ping endpoint#2177
hirokiterashima merged 8 commits into
developfrom
issue-2159-ping-endpoint

Conversation

@Aaron-Detre
Copy link
Copy Markdown
Contributor

@Aaron-Detre Aaron-Detre commented May 12, 2025

Changes

  • When you enter a step with a Dialog Guidance or CRater Open Response component, PingEndpointService collects the item IDs.
  • When the student focuses on the textbox, the client sends ping requests for all collected item IDs to the API.
  • Pings will be sent every 4 minutes and 55 seconds regardless of whether or not they are still focusing on the textbox or not.
    • For simplicity + edge case where student types out response > unfocuses > model sleeps > refocuses > sends immediately
  • When the student leaves the step, the item IDs are cleared out and pings are no longer sent.

Test

  • Make sure that ~30 seconds after focusing on a "berkeley_" item ID that hasn't been used recently, scoring requests come back quickly without the cold start waiting time.
  • Requires this PR for the API.

Closes #2159

@Aaron-Detre Aaron-Detre added the enhancement New feature of any size or improvement (UI, performance, security) label May 12, 2025
@Aaron-Detre Aaron-Detre marked this pull request as ready for review May 13, 2025 18:34
Copy link
Copy Markdown
Member

@hirokiterashima hirokiterashima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hirokiterashima hirokiterashima merged commit e3a850c into develop May 29, 2025
6 checks passed
@hirokiterashima hirokiterashima deleted the issue-2159-ping-endpoint branch May 29, 2025 18:54
@hirokiterashima
Copy link
Copy Markdown
Member

🎉 This PR is included in version 5.188.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature of any size or improvement (UI, performance, security) released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(CRater): Ping endpoint

2 participants