Skip to content

[Feature:InstructorUI] Autograding component averages#12969

Merged
bmcutler merged 10 commits into
mainfrom
autograding-component-averages
Jul 3, 2026
Merged

[Feature:InstructorUI] Autograding component averages#12969
bmcutler merged 10 commits into
mainfrom
autograding-component-averages

Conversation

@dagemcn

@dagemcn dagemcn commented Jun 29, 2026

Copy link
Copy Markdown
Member

Why is this Change Important & Necessary?

Closes Issue #12360.

What is the New Behavior?

Prior to the change:
image
After the change:
image

What steps should a reviewer take to reproduce or test the bug or new feature?

  1. Run submitty_install_site
  2. Log in as instructor
  3. Go to any gradeable with autograding (best to test with are Autograder Hidden and Extra Credit and Autograde and TA Homework (C System Calls))
  4. Go to the statistics and charts page
  5. Click on the new tab
  6. Ensure all new features work correctly, the new tab, the checkboxes, the new filters

Automated Testing & Documentation

Cypress tests should be added for this, I am working on this

Other information

This takes advantage of the previously unused autograding_testcase and autograding_testcase_data tables implemented in PR #12660. If a gradeable was built before that PR was merged, they will not have results in the new tab.

@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 57 lines in your changes missing coverage. Please review.
✅ Project coverage is 21.32%. Comparing base (01f8919) to head (d2a3c2c).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main   #12969      +/-   ##
============================================
- Coverage     21.35%   21.32%   -0.04%     
- Complexity     9990    10002      +12     
============================================
  Files           275      275              
  Lines         37317    37374      +57     
  Branches        505      505              
============================================
  Hits           7969     7969              
- Misses        28847    28904      +57     
  Partials        501      501              
Flag Coverage Δ
autograder 21.16% <ø> (ø)
js 1.98% <ø> (ø)
migrator 100.00% <ø> (ø)
php 20.37% <0.00%> (-0.04%) ⬇️
python_submitty_utils 80.08% <ø> (ø)
submitty_daemon_jobs 91.13% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dandrecollins07-ctrl dandrecollins07-ctrl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested various features all working properly, code looks good. Approving.

@github-project-automation github-project-automation Bot moved this from Seeking Reviewer to Awaiting Maintainer Review in Submitty Development Jun 30, 2026

@bmcutler bmcutler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@dagemcn
The order of the tabs in the screenshot is inconsistent.
I would expect:
manual grading components
manual grading histogram
autograding components
autograding histogram
combined histogram

@github-project-automation github-project-automation Bot moved this from Awaiting Maintainer Review to Work in Progress in Submitty Development Jul 2, 2026
@bmcutler
bmcutler merged commit c33d434 into main Jul 3, 2026
44 of 52 checks passed
@bmcutler
bmcutler deleted the autograding-component-averages branch July 3, 2026 04:48
@github-project-automation github-project-automation Bot moved this from Work in Progress to Done in Submitty Development Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants