Skip to content

Judging timestamps #3027

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

Merged
merged 3 commits into from
Jul 8, 2025
Merged

Judging timestamps #3027

merged 3 commits into from
Jul 8, 2025

Conversation

Kevinjil
Copy link
Contributor

@Kevinjil Kevinjil commented Jul 6, 2025

While we're at it, align title formatting with the format used in the title of the submission time.

The submission page shows a warning
The problem's testcases have changed since this judging has been performed. We recommend rejudging the whole problem.
but this warning does not cover all possible changes in the testcases (e.g. changing input / output files does not trigger this warning). Update to use the hashes to cover more cases.

Fixes #2764

Kevinjil added 3 commits July 6, 2025 14:20
Align title formatting with the format used in the title of the
submission time.
Enable reuse of the hash definition. No functional changes intended.
The page shows a warning
```The problem's testcases have changed since this judging has been performed. We recommend rejudging the whole problem.```
but this warning does not cover all possible changes in the testcases (e.g. changing input / output files does not trigger this warning).

Make this test more rigor by comparing the hashes.
@Kevinjil
Copy link
Contributor Author

Kevinjil commented Jul 6, 2025

image

@Kevinjil Kevinjil requested a review from meisterT July 6, 2025 13:49
@Kevinjil Kevinjil added this pull request to the merge queue Jul 8, 2025
Merged via the queue into DOMjudge:main with commit 6be1166 Jul 8, 2025
36 checks passed
@Kevinjil Kevinjil deleted the judging-timestamps branch July 8, 2025 17:08
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.

Show judging timestamp in the interface
5 participants