ubc / iPeer

Peer Evaluation System
http://ipeer.ctlt.ubc.ca
Other
17 stars 18 forks source link

Result Release Randomization #623

Open timkato opened 5 years ago

timkato commented 5 years ago

SN ENHC0011932 Pete Ostafischuk provided some astute feedback: Students are noticing that they get scores/comments in the same order for every peer evaluation within their group throughout the term. Pete suggests it should be randomized each time to avoid students figuring out extra information about which group member gave them which score/comment.

kitsook commented 4 years ago

This should be fixed without code change since the PHP 7 upgrade. The existing code uses shuffle to randomize the result. But the random number generator wasn't seeded properly and thus randomization wasn't effective.

Since PHP 7.1, the shuffle function changed to use a diff random number generator mt_rand which is seeded automatically. So the randomization should be working now.

I tested it on my dev machine with simple / rubric / mixed eval. The sorting order is diff every time the student reload the result page.