Check validity of exercise points after admitting deadline deviation or extra attempts

[PasiSa]

When student is admitted deadline extension, the exercise points for already submitted tasks should be checked and updated automatically. Currently this requires manual clicking.

Also, if student is given extra attempts, exercise points should be checked for submissions that exceeded the previous limit.

[PasiSa]

Did some editing and moved modification problem into separate issue #902

[PasiSa]

It was noted in a meeting today, that re-adjusting points, e.g. removal of late penalties, can be done without needing to rerun the grading process.

[PasiSa]

Issue 694 in EDIT Jira

[markkuriekkinen]

It was noted in a meeting today, that re-adjusting points, e.g. removal of late penalties, can be done without needing to rerun the grading process.

Do you mean that this already can be done in the system or that this is another idea for the future implementation? I don't think we currently have a well-automated feature for this use case. It is possible to manually assess submissions or manually send them to auto-regrading, but that is too time-consuming for the teacher.

The idea was that if the student has already made submissions, but they did not get points either due to the late penalty (requires that late submissions are allowed) or becoming an unofficial submission (requires that unofficial submissions are enabled in the category), then either

1. teacher adds a deadline deviation and A+ should then recognize automatically that the affected submissions must be regraded, OR
2. teachers presses a new button "accept late points for this student" and A+ automatically removes late penalties from the affected submissions and converts them to normal, official submissions.

I like 2) because:

• the submission has already been graded and sending it to auto-regrading is unnecessary
• A+ already knows the potentially existing late penalty or the unofficial submission status
• teacher does not need to add any new deadline deviations: deviations are intended for the use case when the student needs to make new submissions, but now the submissions already exist and the student is not supposed to make new submissions
• the new button could do this for all submissions in the whole module (exercise round) at once

[PasiSa]

The idea was that if the student has already made submissions, but they did not get points either due to the late penalty (requires that late submissions are allowed) or becoming an unofficial submission (requires that unofficial submissions are enabled in the category), then either

1. teacher adds a deadline deviation and A+ should then recognize automatically that the affected submissions must be regraded, OR
2. teachers presses a new button "accept late points for this student" and A+ automatically removes late penalties from the affected submissions and converts them to normal, official submissions.

I like 2) because:

With poor phrasing I think I intended to mean the same as (2)

[markkuriekkinen]

Relevant comment from the duplicate issue: #482 (comment)

The old discussion copied to the issue description mentions personal deadline extensions, but on the other hand, it seems unnecessary to create any deadline extensions when the submission already exists in the database. Deadline extensions should only be used when we want the student to submit again after the deadline.

Another idea:

• There is a button in the inspect submission page for accepting late or unofficial submissions as "normal submissions".

• The button is available to teachers and if the exercise allows assistant grading, then also to assistants.

• The submission record in the database contains the original points set by the grading service as well as the service max points. Verify this first so that I remember this correctly. If this is the case, then we should be able to set the points again in A+ without sending the submission again to the service for regrading.

• When the button is pressed, A+ could set the points based on the original service points again without any late penalties. Unofficial submissions would be set to the "ready" status. The service points may use a different maximum, hence they must be scaled again to the A+ max points.

• Ensure that the related points caches are updated or invalidated when the submissions points are changed.