Skip to content

Add judge input to custom checker#17

Merged
rayokamoto merged 4 commits intomainfrom
add-judge-input-to-custom-checker
Apr 29, 2026
Merged

Add judge input to custom checker#17
rayokamoto merged 4 commits intomainfrom
add-judge-input-to-custom-checker

Conversation

@xxori
Copy link
Copy Markdown
Collaborator

@xxori xxori commented Apr 28, 2026

As defined here, custom checkers can take a kwargs object containing additional info. One particularly useful one I would like the cli to provide is the input file, as this is useful in a lot of checking scripts. Additionally, we cannot pass it through argv as we passed the other information, as it will be too large, so I use stdin piping here. We may need to switch to some other solution, as judge or process output may also be too large in some cases, but I forsee this being far more unlikely.

@xxori xxori requested review from Copilot and rayokamoto and removed request for rayokamoto April 28, 2026 14:51
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds support for providing the test case input to Python-based custom checkers in aucpl problem test, aligning with the concept of “additional checker info” via kwargs.

Changes:

  • Pipe the test input file into the checker runner via stdin and forward it to check(...) as judge_input.
  • Update the CLI’s embedded Python wrapper to read stdin and pass judge_input into the checker call.
  • Update README custom checker signature documentation.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
crates/cli/src/problem/test.rs Pipes input file to checker via stdin and passes it to check as judge_input.
README.md Updates custom-checker documentation to mention the new input parameter.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread crates/cli/src/problem/test.rs
Comment thread crates/cli/src/problem/test.rs
Comment thread README.md Outdated
xxori and others added 2 commits April 29, 2026 11:09
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@rayokamoto rayokamoto merged commit 630b0ed into main Apr 29, 2026
1 check passed
@rayokamoto rayokamoto deleted the add-judge-input-to-custom-checker branch April 29, 2026 02:55
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.

3 participants