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

Remove doc mention of judgehost restriction #2217

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

vmcj
Copy link
Member

@vmcj vmcj commented Nov 13, 2023

Added a small part about how to pick up judgings together with parallel judging.

restricted to certain contests, languages and problems, so that it can be
the case that a judgehost is available, but not judging an available
submission.
*judging* of this submission. Note that it is possible to
Copy link
Member

Choose a reason for hiding this comment

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

nit: I would phrase it like this:
"If parallel judging is enabled, multiple judgehosts may pick up and work on the same submission (if there is no queue of pending submissions)."

*judging* of this submission. Note that it is possible to
parallelize this when only one submission is waiting.
#. In case that multiple submissions are not judged yet the
load will be spread over all judgehosts. There is a slight delay
Copy link
Member

Choose a reason for hiding this comment

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

Suggested phrasing (how do I do this in this github UI directly?):
To avoid starving other teams from judgehost resources, submissions from teams that submit while they have other submissions in the judge queue will be penalized by downgrading their judging priority.

Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested phrasing (how do I do this in this github UI directly?): To avoid starving other teams from judgehost resources, submissions from teams that submit while they have other submissions in the judge queue will be penalized by downgrading their judging priority.

From the files tab, close to the line number you can drag. than you can comment there and there is a symbol [page with plus-minus] where you can change your suggestion.

doc/manual/judging.rst Outdated Show resolved Hide resolved
Added a small part about how to pick up judgings together with parallel judging.

This was left over from:
DOMjudge#1137

Co-authored-by: Thijs Kinkhorst <[email protected]>
@vmcj vmcj force-pushed the judgehost_restrictions_removed branch from 91e080e to b522548 Compare November 24, 2023 10:56
@vmcj vmcj added this pull request to the merge queue Nov 24, 2023
Merged via the queue into DOMjudge:main with commit c9fbcaa Nov 24, 2023
21 checks passed
@vmcj vmcj deleted the judgehost_restrictions_removed branch November 24, 2023 12:39
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