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

Event Feed judgement name/title missing first 5 chars #697

Open
lane55 opened this issue Feb 25, 2023 · 0 comments
Open

Event Feed judgement name/title missing first 5 chars #697

lane55 opened this issue Feb 25, 2023 · 0 comments
Assignees
Labels
bug Something isn't working CLICS CCS Update Changes to update to the CLICS specification medium-high Should be consider for next release Small LOE Less than an hour to code, test and push a PR
Milestone

Comments

@lane55
Copy link
Contributor

lane55 commented Feb 25, 2023

Describe the issue:

The judgement name is missing the first 5 character for "No" judgements
the Yes judgement is

{"type":"judgement-types", "id":"pc2-5", "op":"create", "data": {"id":"RTE","name":"ime Error","penalty":true,"solved":false}}
"name":"ime Error"
should be:
"name":"Run-time Error"

Root cause is that judgement names can have "No - " (5 chars) in the front, the code
is removing the 5 chars unconditionally.

Example of all all judgements JSON is below.

To Reproduce:

use reject.ini with

Compilation Error|CE
Run-time Error|RTE
Time limit Exceeded|TLE
Wrong Answer|W

start server
start ef
use View Event Feed to view event feed

Expected behavior:

should show full judgement names,thus:
{"type":"judgement-types", "id":"pc2-3", "op":"create", "data": {"id":"AC","name":"Accepted","penalty":false,"solved":true}}
{"type":"judgement-types", "id":"pc2-4", "op":"create", "data": {"id":"CE","name":"Compilation Error","penalty":false,"solved":false}}
{"type":"judgement-types", "id":"pc2-5", "op":"create", "data": {"id":"RTE","name":"Run-time Error","penalty":true,"solved":false}}
{"type":"judgement-types", "id":"pc2-6", "op":"create", "data": {"id":"TLE","name":"Time limit Exceeded ","penalty":true,"solved":false}}
...

Actual behavior:

Shows

{"type":"judgement-types", "id":"pc2-3", "op":"create", "data": {"id":"AC","name":"Accepted","penalty":false,"solved":true}}
{"type":"judgement-types", "id":"pc2-4", "op":"create", "data": {"id":"CE","name":"lation Error","penalty":false,"solved":false}}
{"type":"judgement-types", "id":"pc2-5", "op":"create", "data": {"id":"RTE","name":"ime Error","penalty":true,"solved":false}}
{"type":"judgement-types", "id":"pc2-6", "op":"create", "data": {"id":"TLE","name":"limit Exceeded","penalty":true,"solved":false}}
{"type":"judgement-types", "id":"pc2-7", "op":"create", "data": {"id":"WA","name":" Answer","penalty":true,"solved":false}}
{"type":"judgement-types", "id":"pc2-8", "op":"create", "data": {"id":"NO","name":"tput","penalty":true,"solved":false}}
{"type":"judgement-types", "id":"pc2-9", "op":"create", "data": {"id":"IO","name":"plete Output","penalty":true,"solved":false}}
{"type":"judgement-types", "id":"pc2-10", "op":"create", "data": {"id":"EO","name":"sive Output","penalty":true,"solved":false}}
{"type":"judgement-types", "id":"pc2-11", "op":"create", "data": {"id":"IOF","name":"rect output format","penalty":true,"solved":false}}
{"type":"judgement-types", "id":"pc2-12", "op":"create", "data": {"id":"OCS","name":" - Contact Staff","penalty":true,"solved":false}}

Environment:

Log Info:

Screenshots:

Additional context:

{"type":"judgement-types", "id":"pc2-3", "op":"create", "data": {"id":"AC","name":"Accepted","penalty":false,"solved":true}}
{"type":"judgement-types", "id":"pc2-4", "op":"create", "data": {"id":"CE","name":"lation Error","penalty":false,"solved":false}}
{"type":"judgement-types", "id":"pc2-5", "op":"create", "data": {"id":"RTE","name":"ime Error","penalty":true,"solved":false}}
{"type":"judgement-types", "id":"pc2-6", "op":"create", "data": {"id":"TLE","name":"limit Exceeded","penalty":true,"solved":false}}
{"type":"judgement-types", "id":"pc2-7", "op":"create", "data": {"id":"WA","name":" Answer","penalty":true,"solved":false}}

@lane55 lane55 added bug Something isn't working CLICS CCS Update Changes to update to the CLICS specification Small LOE Less than an hour to code, test and push a PR labels Feb 25, 2023
@lane55 lane55 changed the title Event Feed judgement name/title missing first 5 Event Feed judgement name/title missing first 5 chars Feb 25, 2023
@lane55 lane55 added the medium-high Should be consider for next release label Feb 28, 2023
@johnbrvc johnbrvc self-assigned this Nov 2, 2023
@troy2914 troy2914 added this to the 9.11.0 milestone Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CLICS CCS Update Changes to update to the CLICS specification medium-high Should be consider for next release Small LOE Less than an hour to code, test and push a PR
Projects
None yet
Development

No branches or pull requests

3 participants