-
Notifications
You must be signed in to change notification settings - Fork 23
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
I978: Update generated results file according to new WF honor degrees proposed by Bill #987
I978: Update generated results file according to new WF honor degrees proposed by Bill #987
Conversation
1. If WF style rankings then Highest Honors, High Honors and Honors are assigned, based on new "Bill" rules. 2. Standings Record Comparator now uses case insensitive comparator for account names. 3. Account Name Comparator is updated to make accent insensitive.
…r compare which was not perfect
1. Removed wf2017 from test data as it was not compliant with Bill's new rules 2. Added wf47 data and new results.tsv generated by DOMjudge compliant with Bill's new rules 3. Updated algorithm to consider 0 solved problems as "Honorable" no matter what 4. For non-WFGroups consider only isHonor (equivalent to isRanked) 5. Updated expected results of test cases for 0 solved problems cases.
…ors with multiple solvd problems
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.
Reviewed code, and tested with 2 different contests using different # of medals, etc. Tried both TSV and CSV files. Seems to work as currently desired.
@SamanwaySadhu You can add your name as the author on the new module you created for the CSV report.
Credit where credit is due! |
This comment was marked as resolved.
This comment was marked as resolved.
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.
Went through the code part looks good to me. Most of the changes appear to be in data.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
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.
Looks good. Appears to work.
Description of what the PR does
Highest Honors
,High Honors
andHonors
instead of simplyRanked
based on the rules proposed by Bill.results.csv
file as well.Issue which the PR addresses
Fixes #978
Environment in which the PR was developed (OS, IDE, Java version, etc.)
Windows 10, Eclipse 2021-12 R, JDK 8u381 (1.8), C++ 13.2.0
Precise steps for testing the PR (i.e., how to demonstrate that it works correctly)
profiles
folder andprofiles.properties
files.Run Contest
andFinalize
tab to toggle between WF group rankings checkbox.Reports
in eitherAdmin
orServer
.results.csv
orresults.tsv
report.Comment
Checking success for JUnits is enough to test functionality of this PR. The JUnits have been recently curated based on WF 47 and the corresponding
results.tsv
in testdata has been generated by DOMjudge based on the new Bill's rules. Check the synthetic data of JUnits as well to review how code will behave under various edge cases.