Update 3T2022 DEDP certificates and grades in MicroMasters

mitodl / micromasters

Portal for learners and course teams to access MITx Micromasters® programs

https://mm.mit.edu

BSD 3-Clause "New" or "Revised" License

28 stars 16 forks source link

Update 3T2022 DEDP certificates and grades in MicroMasters #5275

Closed pdpinch closed 1 year ago

pdpinch commented 1 year ago

Legacy DEDP courses have separate grades for the online course, exam, and combined final grade. As of 3T2022, the online course and exam are combined into one grade, which is the same as the combined final grade.

However, in 3T2022, the combined final grade and its passing status does not reflect the requirement that the learner pass the final exam. Some one-off adjustments will be necessary.

Acceptance Criteria:

[ ] Come up with a plan to import and migrate the necessary data to provide passing status, certificates, and a correct learner record for learners who took the 3T2022 exams.
[ ] Update this PR description with the plan.

related issues:

https://github.com/mitodl/mitxonline/issues/1315

pdpinch commented 1 year ago

I believe if we freeze the grades as is, we will have a correct course grade, but the passing status will be incorrect
Can we import final exam grades using the existing script?
Instead of combined final grades which are a combination of the course grade and exam grade, can we copy the course grade into the combined final grade? It will be the correct score, but not the correct passing status

pdpinch commented 1 year ago

We can write a management command to import the combined final grades and passing status.

annagav commented 1 year ago

There are two different issues here:

Disabling the CombinedFinalGrade functionality for DEDP courses, #5276
Be able to show the correct grade and the passing status. As far as I understand, the grade on edx is correct, and the passing status is incorrect. So in this case what we could do is after we freeze the final grades we can update the passing status of the grades.

Can we import final exam grades using the existing script?

@pdpinch I think we should just show the edx grade and completely remove the "exam grade" and "combined grade" from the dashboard.

Let me know if that makes sense.

pdpinch commented 1 year ago

@pdpinch I think we should just show the edx grade and completely remove the "exam grade" and "combined grade" from the dashboard.

Let me know if that makes sense.

I think this is the right direction going forward, which I guess means that we don't need to import the final grades for 3T2022 (and beyond) at all.

But I'm not sure what the plan is for the legacy grades (before 3T2022). We can discuss on #5276.

pdpinch commented 1 year ago

Be able to show the correct grade and the passing status. As far as I understand, the grade on edx is correct, and the passing status is incorrect. So in this case what we could do is after we freeze the final grades we can update the passing status of the grades.

Does the final grade model already include a Boolean for passing status?

annagav commented 1 year ago

Yes it does

annagav commented 1 year ago

@pdpinch

Instead of combined final grades which are a combination of the course grade and exam grade, can we copy the course grade into the combined final grade? It will be the correct score, but not the correct passing status

We cannot populate edx grade into the combined final grade, because the combined final grade represents the best grade for course over all course runs and exams, and not a grade for a single course run. Options I am considering: 1) Make combined final grade optional, if it exists display it if not then also okay. Decide whether the user passed the course on not based on the existence of a certificate. 2) use the new grades to compare to existing combined final grade, if the score is higher than before, then create/update combined final grade.

annagav commented 1 year ago

@pdpinch do we want to keep MicromastersCourseCertificates generation? Based on what do we decide to generate it? Or do we get certificates from edx now for DEDP, the same as we do for non-fa programs?

pdpinch commented 1 year ago

do we want to keep MicromastersCourseCertificates generation?

No, I don’t want to keep it. I would prefer to direct users to MITx Online to get their certificates (same as we do with edX).

However, I don’t know if we have a reliable API in MITx Online’s open edX that we can use for this purpose.

annagav commented 1 year ago

So the dashboard should display something like this now:

pdpinch commented 1 year ago

I think the dashboard display is OK, given the constraints.

We do need to freeze the grades. We should test a couple of users to see if it works as expected.

pdpinch commented 1 year ago

@annagav I put off freezing the grades until you were back from vacation. Can we freeze the grades now?

pdpinch commented 1 year ago

@annagav can this be closed now?