Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option for ondemand mode to do a full judging in 1 buttonpress #2380

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

vmcj
Copy link
Member

@vmcj vmcj commented Mar 17, 2024

Normally we would need 2 button presses, 1 to start judging and the other after the lazy eval finished to go further. In case we shadow onDemand we are probably more interested in full submissions as you want to see the behaviour on all testcases.

In case of an analyst they would probably also want to know how far off a participant is.

@vmcj vmcj force-pushed the analyst_full_judgement branch from 3b96dbf to b4885d2 Compare March 17, 2024 19:29
vmcj added 2 commits March 19, 2024 19:41
Normally we would need 2 button presses, 1 to start judging and the
other after the `lazy` eval finished to go further. In case we shadow
onDemand we are probably more interested in full submissions as you want
to see the behaviour on all testcases.

In case of an analyst they would probably also want to know how far off
a participant is.
@vmcj vmcj force-pushed the analyst_full_judgement branch from b4885d2 to adccd5a Compare March 19, 2024 18:46
Copy link
Member

@meisterT meisterT left a comment

Choose a reason for hiding this comment

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

Can you please add a screenshot to the PR?

@vmcj
Copy link
Member Author

vmcj commented Mar 22, 2024

Can you please add a screenshot to the PR?

image

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