[SIO-1345] Serialized scores lexical ordering shall match __cmp__ ordering - SIO2

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: Current Version
Fix Version/s: None
Component/s: OIOIOI
Labels:
None

Description

ex. now IntegerScore is serialized ex. as int:99 or int:100,
so string ordering does not correspond to natural one.

Then, ordering in admin could be added.

Issue Links

blocks

SIO-1329 sorting submissions execution time / source code length for admins

Activity

Descending order - Click to sort in ascending order

Hide

Permalink

Gerrit Gerrit added a comment - 2013-11-9 1:18

Change I5bc7d5a69480f4b69043e0003c97d7363322e5b4, patchset 9
https://gerrit.sio2project.mimuw.edu.pl/1857

~~SIO-1345~~ Serialized scores lexical ordering shall match __cmp__ ordering

Change-Id: I5bc7d5a69480f4b69043e0003c97d7363322e5b4

Show

Gerrit Gerrit added a comment - 2013-11-9 1:18 Change I5bc7d5a69480f4b69043e0003c97d7363322e5b4, patchset 9 https://gerrit.sio2project.mimuw.edu.pl/1857 SIO-1345 Serialized scores lexical ordering shall match __cmp__ ordering Change-Id: I5bc7d5a69480f4b69043e0003c97d7363322e5b4

Hide

Permalink

Gerrit Gerrit added a comment - 2013-11-9 0:47

Change I5bc7d5a69480f4b69043e0003c97d7363322e5b4, patchset 8
https://gerrit.sio2project.mimuw.edu.pl/1857

~~SIO-1345~~ Serialized scores lexical ordering shall match __cmp__ ordering

Change-Id: I5bc7d5a69480f4b69043e0003c97d7363322e5b4

Show

Gerrit Gerrit added a comment - 2013-11-9 0:47 Change I5bc7d5a69480f4b69043e0003c97d7363322e5b4, patchset 8 https://gerrit.sio2project.mimuw.edu.pl/1857 SIO-1345 Serialized scores lexical ordering shall match __cmp__ ordering Change-Id: I5bc7d5a69480f4b69043e0003c97d7363322e5b4

Hide

Permalink

Gerrit Gerrit added a comment - 2013-11-8 21:11

Change I5bc7d5a69480f4b69043e0003c97d7363322e5b4, patchset 7
https://gerrit.sio2project.mimuw.edu.pl/1857

~~SIO-1345~~ Serialized scores lexical ordering shall match __cmp__ ordering

Change-Id: I5bc7d5a69480f4b69043e0003c97d7363322e5b4

Show

Gerrit Gerrit added a comment - 2013-11-8 21:11 Change I5bc7d5a69480f4b69043e0003c97d7363322e5b4, patchset 7 https://gerrit.sio2project.mimuw.edu.pl/1857 SIO-1345 Serialized scores lexical ordering shall match __cmp__ ordering Change-Id: I5bc7d5a69480f4b69043e0003c97d7363322e5b4

Hide

Permalink

Gerrit Gerrit added a comment - 2013-11-8 20:26

Change I5bc7d5a69480f4b69043e0003c97d7363322e5b4, patchset 6
https://gerrit.sio2project.mimuw.edu.pl/1857

~~SIO-1345~~ Serialized scores lexical ordering shall match __cmp__ ordering

Change-Id: I5bc7d5a69480f4b69043e0003c97d7363322e5b4

Show

Gerrit Gerrit added a comment - 2013-11-8 20:26 Change I5bc7d5a69480f4b69043e0003c97d7363322e5b4, patchset 6 https://gerrit.sio2project.mimuw.edu.pl/1857 SIO-1345 Serialized scores lexical ordering shall match __cmp__ ordering Change-Id: I5bc7d5a69480f4b69043e0003c97d7363322e5b4

Hide

Permalink

Gerrit Gerrit added a comment - 2013-11-8 0:08

Change I5bc7d5a69480f4b69043e0003c97d7363322e5b4, patchset 5
https://gerrit.sio2project.mimuw.edu.pl/1857

~~SIO-1345~~ Serialized scores lexical ordering shall match __cmp__ ordering

Change-Id: I5bc7d5a69480f4b69043e0003c97d7363322e5b4

Show

Gerrit Gerrit added a comment - 2013-11-8 0:08 Change I5bc7d5a69480f4b69043e0003c97d7363322e5b4, patchset 5 https://gerrit.sio2project.mimuw.edu.pl/1857 SIO-1345 Serialized scores lexical ordering shall match __cmp__ ordering Change-Id: I5bc7d5a69480f4b69043e0003c97d7363322e5b4

8 older comments

Hide

Permalink

Maciej Matraszek added a comment - 2013-11-7 17:57

It might not always be straightforward. With normal descending sorting you only need to zero-fill the value, but in ACM-style contests we use ordering like: ORDER BY accepted_solutions DESC, total_time ASC.
Take a look at AcmScore.to_repr here: https://gerrit.sio2project.mimuw.edu.pl/#/c/1744/12/oioioi/acm/score.py
I have tried to implement such ordering there (but forgot the ASC sorting).

One solution I can think of is to digit-negate the time (00123 -> 99876), but that's ugly ugly.

Show

Maciej Matraszek added a comment - 2013-11-7 17:57 It might not always be straightforward. With normal descending sorting you only need to zero-fill the value, but in ACM-style contests we use ordering like: ORDER BY accepted_solutions DESC, total_time ASC. Take a look at AcmScore.to_repr here: https://gerrit.sio2project.mimuw.edu.pl/#/c/1744/12/oioioi/acm/score.py I have tried to implement such ordering there (but forgot the ASC sorting). One solution I can think of is to digit-negate the time (00123 -> 99876), but that's ugly ugly.

People

Assignee:

Szymon Acedański

Reporter:

Maciej Dębski

Votes:

0 Vote for this issue

Watchers:

4 Start watching this issue

Dates

Created:

2013-11-7 13:40

Updated:

2014-09-29 21:41

Resolved:

2013-11-9 16:45

Agile

View on Board