Instructor task for resetting student attemps

[Ignacio Lozano]

Hi,

I saw that some features in the instructor panel creates InstructorTask, which helps me to log strong changes of a users' score. For example, rescoring problems generate the instructor task that you can see in the Tasks table or in Django admin.



On the opposite, "reset_student_attempts" doesn't generate any InstructorTask, so it is impossible to know if user score was changed by any instructor.

Could be interesting to generate an Instructor task for every Instructor operation in the Instructor panel? I know it is no too difficult, but it makes sense?

Thanks!

[David Ormsbee]

Hi Ignacio,

InstructorTasks are fairly heavy weight, and meant for long running jobs. The idea was to make sure the course teams had regular progress updates on how far along the task was, and that they could not monopolize overall system resources by accidentally launching a dozen concurrent tasks to calculate student grades. It's a bit cumbersome to use for small things like resetting the score, which is essentially a single row update.

I think that the functionality you're talking about actually makes a lot of sense as its own standalone piece. I guess my suggestion is that rather than moving reset_student_attempts to become an InstructorTask, that you create a new functionality for an audit log, and have both InstructorTasks and reset_student_attempts (and any number of other admin actions) write to it as necessary. It doesn't have to be an actual log file -- it could be backed by a model we can query, so long as we're not too liberal in what we write there.

Does that general approach sound reasonable? I'm curious if folks in the community have already done this.

Take care.

Dave

--
You received this message because you are subscribed to the Google Groups "General Open edX discussion" group.
To view this discussion on the web visit https://groups.google.com/d/msgid/edx-code/30c05059-b29f-4222-869c-6f04fef16718%40googlegroups.com.

[Ignacio Lozano]

Yeah, it sounds good to me. You're right, InstructorTasks are for high weight tasks.

I didn't found anything done about this improvement. It is not so critical but it would be interesting, I will add to our backlog

Regards.