cms-dev / cms

Contest Management System
http://cms-dev.github.io/
GNU Affero General Public License v3.0
896 stars 362 forks source link

Identify old submission results #150

Open stefano-maggiolo opened 11 years ago

stefano-maggiolo commented 11 years ago

If an admin changes something in a task (read: dataset after they are merged), there is no way to discern the submission results that were computed with the new task/dataset data and those computed after. Proposal: add an "up_to_date" field in the submission results table to be flipped when something changes in the task/dataset, and allow admins to re-evaluate only those submission.

lw commented 11 years ago

I don't like this idea that much because:

Anyway, if at the end we decide we want something like that, I'd prefer to do it in a way that was proposed somewhere some time ago: store a "hash" for each result that is derived from the digests of the files that were used (+ other useful information) and use that to determine if a result is outdated.

giomasce commented 11 years ago

Il 12/05/2013 20:45, Luca Wehrstedt ha scritto:

  • We've always given great powers (and great responsibilities) to contest administrators, because they know best. This seems to go in the opposite direction.

I mostly agree with your remarks; only thing: adding such a tool is not necessarily something "removing" admin's powers; it may just be something to help them use the powers properly.

In this particular case, I acknowledge that your hashing proposal is better, although I would need to think better about it to decide whether I really like it or not (I don't remember what and whether I already commented about it).

Gio.

Giovanni Mascellani mascellani@poisson.phc.unipi.it Pisa, Italy

Web: http://poisson.phc.unipi.it/~mascellani Jabber: g.mascellani@jabber.org / giovanni@elabor.homelinux.org

stefano-maggiolo commented 11 years ago

adding such a tool is not necessarily something "removing" admin's powers; it may just be something to help them use the powers properly

That was the idea, to inform them and making sure they don't forget to think about this. Remember that they might also think that CMS handle this automatically. In particular, I'm not too worried about false positive here.