Skip to content

Conversation

@TeetoToast
Copy link
Contributor

@TeetoToast TeetoToast commented Apr 10, 2025

Linear Ticket

Linear Ticket

Description

Need new database queries so that we can grab information relevant to the instructor dashboard.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would change existing functionality)

Changes

Added queries to api/lib/redisHelper.mjs, and added a new .mjs file to test those queries. The new queries aim to:

  1. Get average score on an assignment 

  2. Get maximum score on an assignment

  3. Get minimum (non n/a) score on an assignment

  4. Get top k students in terms of score for an assignment

  5. Get top k students in terms of total course score

Testing

Tested manually in docker. The output is below:
Screenshot 2025-04-10 at 6 05 06 AM

If one wants to manually check these results as I did, use this HAID sheet. Instruction on how to do so can be found in the api/testAssignmentQueries.mjs file.

Checklist

  • My branch name matches the format: <ticket-id>/<brief-description-of-change>
  • My PR name matches the format: [<ticket-id>] <brief-description-of-change>
  • I have added doc-comments to all new functions (JSDoc for JS and Docstrings for Python)
  • I have reviewed all of my code
  • My code only contains major changes related to my ticket

Screenshots/Video

Additional Notes

@linear
Copy link

linear bot commented Apr 10, 2025

@TeetoToast TeetoToast marked this pull request as ready for review April 10, 2025 13:21
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