Skip to content

Change meaning of submissions/<verdict> directories #158

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 2 commits into from
Apr 10, 2024

Conversation

Tagl
Copy link
Collaborator

@Tagl Tagl commented Apr 10, 2024

Closes #87

| wrong_answer | Wrong answer for some test file. May be too slow or crash for other test files. |
| time_limit_exceeded | Too slow relative to the safety margin for some test file. May output wrong answers or crash for other test files. |
| run_time_error | Crashes for some test file. May output wrong answers or be too slow for other test files. | Very rarely useful.
| rejected | Is not accepted for any reason in the final verdict. |
Copy link
Member

Choose a reason for hiding this comment

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

Why did you not write this similar to the other? I.e.

Suggested change
| rejected | Is not accepted for any reason in the final verdict. |
| rejected | Is rejected (i.e. not accepted, for any reason) for some test file. |

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This does not have the desired effect for scoring problems, right? We want rejected to mean that it wasn't accepted in the end.

@niemela niemela merged commit 008e617 into Kattis:master Apr 10, 2024
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.

Meaning of submissions/<verdict> directories
2 participants