Maciej Dębski added a comment - 2013-11-7 11:03 - edited Well, every sorting field you mention is a different case :P 1. score - I believe we shall allow db ordering for scores. I think it shall be solved by adding db_order method to ScoreValue, returning int, which could be used for sorting. Then, add a field to Submission which on save() will be updated from score.db_order. Obviously some scores may not implement db_order, ex. always returning None from db_order(). I know we currently use Very Flexible Design For Scores, but it shall not prevent us from implementing some useful features where it is possible. BTW, i cannot think of score which could not be mapped homomorphically to int... 2. execution time - O.o it could be done pretty easily - you may annotate qs in admin, then sort by annotated. However, we shall decide if we want it, as it costs a bit db work. 3. source code length - it seems ok to me to add source_length int field to ProgramSubmission and update it on save() Caution: 1 and 3 needs not only scheme, but also data migration ^ review pls

Szymon Acedański added a comment - 2013-11-7 12:19 Ad. 2, 3. I think we shouldn't add 2 and 3 to the admin submission view, which is already cluttered. I'd follow the SIO1 solution with separate pages for these stats. Ad. 1. Database representations of ScoreValues should be by design naturally orderable (lexicographically), without any extra logic. Then we could use normal ordering in Django. I think this design may not be followed in practice, which I think is worth fixing.

Szymon Acedański added a comment - 2013-11-7 12:22 Ad. 3. The source_length fields sounds good. I'm no sure if the data migration is worth the hassle. Maybe it's ok to make it work only for new submissions.

Maciej Dębski added a comment - 2013-11-7 12:26 2,3: it would be extremaly useful to be able to dynamically select visible columns in admin. Unfortunalety django-admin does not support this, maybe it could be added somehow? For sure there exist replacements which does, eg. django-xadmin. 1. Ok, I will create separate ticket for it. 3. Ok, I will create separate ticket for adding field.

Szymon Acedański added a comment - 2013-11-7 12:31 - edited I like this idea: "2,3: it would be extremaly useful to be able to dynamically select visible columns in admin. Unfortunalety django-admin does not support this, maybe it could be added somehow? For sure there exist replacements which does, eg. django-xadmin." But I don't know at the moment how to do this or if this is feasible. Unfortunately it doesn't sound like a high enough priority for now. But, maybe there's some easy solution I don't know?

Szymon Acedański added a comment - 2016-11-9 18:00 Sorting by score is already done.

Szymon Acedański added a comment - 2020-04-27 16:28 This issue has been automatically closed as Obsolete due to no activity for 365 days. Feel free to reopen it or create a new one if it's still relevant.