-
Notifications
You must be signed in to change notification settings - Fork 249
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
C++-ify rungard
.
#2344
C++-ify rungard
.
#2344
Conversation
This is the next steps of merging runguard and runpipe (https://docs.google.com/document/d/1WZRwdvJUamsczYC7CpP3ZIBU8xG6wNqYqrNJf7osxYs/edit). No functional changes intended. Small commits so it should be easy-ish to follow. |
Co-authored-by: Jaap Eldering <eldering@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I assume you tested this with doing a replay of an old contest?
I did some limited testing on a few interactive and non-interactive problems so far, but I would like to do more testing before merging. |
I did now a replay of a contest with ~1000 submissions. There was a single submission where the verdict changed and it was random (with |
No description provided.