Skip to content

Improve polygon loader and update testlib.h - #1725

Open
pxsit wants to merge 4 commits into
cms-dev:mainfrom
pxsit:update-testlib
Open

pxsit wants to merge 4 commits into
cms-dev:mainfrom
pxsit:update-testlib

Conversation

@pxsit

@pxsit pxsit commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor
  • Added support for importing Polygon batch graders.
  • Improved test file handling by using the test path patterns specified in the Polygon package.
  • Added a check to reject Polygon interactors, with a clear error message.
  • Imported HEADER_EXTS and LANGUAGES from cms.grading.languagemanager to support the above enhancements.
  • Updated the patch to improve checker output for partial scores, ensure proper message formatting, and clarify that interactors are not supported. Argument order for CMS is enforced, and checker exit codes and messages are improved for CMS integration.

Copilot AI lite review requested due to automatic review settings September 27, 2026 17:45

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@pxsit
pxsit requested a balanced review from Copilot September 27, 2026 17:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@codecov

codecov Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 3.50877% with 55 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.62%. Comparing base (cc9dfaf) to head (36cf2ad).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
cmscontrib/loaders/polygon.py 3.50% 55 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1725      +/-   ##
==========================================
- Coverage   56.72%   56.62%   -0.11%     
==========================================
  Files         344      344              
  Lines       28307    28360      +53     
==========================================
+ Hits        16057    16058       +1     
- Misses      12250    12302      +52     
Flag Coverage Δ
functionaltests 0.00% <0.00%> (ø)
unittests 56.62% <3.50%> (-0.11%) ⬇️

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

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants