pxt-microbit icon indicating copy to clipboard operation
pxt-microbit copied to clipboard

Teacher Tools: The results incorrect when set result as N/A or Needs work and then turn on the Auto-Run

Open v-katherine opened this issue 10 months ago • 2 comments

Describe the bug Teacher Tools: The results incorrect when set result as N/A or Needs work and then turn on the Auto-Run

Steps to reproduce the behavior 1.Navigate to MakeCode Teacher Tool (microbit.org) and make sure the turn off the Auto-Run. 2.Enter Project Link and submit. 3.Choose the Rubric, and switch to Results tab. 4. Change some rubrics results as N/A or Needs work. 5.Turn on Auto-Run and observe the results.

Actual behavior: The results which changed to N/A or Needs work are incorrect after turning on Auto-Run.

Expected behavior: All rubrics will be evaluated and outcome correct results.

Screen Recording: resultincorrect

Additional context: 1.OS: Windows11 2.makecode.microbit,org version: 7.0.6 3.Microsoft MakeCode version: 10.0.18

v-katherine avatar Apr 25 '24 07:04 v-katherine

The results are correct when set results as N/A or Needs work and then click Run Runincorrect

v-katherine avatar Apr 25 '24 07:04 v-katherine

This is by design, if the teacher specifically sets a result then it gets priority. But we can chat about this as a team, since it does feel a bit inconsistent that pressing "run" does overwrite it but auto-run does not.

thsparks avatar Apr 30 '24 18:04 thsparks

Lets be consistent for both auto-run and run.

abchatra avatar Jul 24 '24 20:07 abchatra

Proposal:

Make Run and Auto-run status-aware After the user has manually set the status of a criteria to anything other than Not Started, Run and Auto-run will no longer evaluate that criteria. Manually setting status back to Not Started clears the flag, allowing Run and Auto-run to evaluate and re-evaluate the criteria in question.

Exception for single-criteria evaluation If the user clicks the Run button on the criteria itself (single-criteria run), then we will evaluate the criteria regardless of status.

Thoughts?

eanders-ms avatar Aug 23 '24 20:08 eanders-ms

Move the autorun to the right and name auto run on load.

abchatra avatar Aug 23 '24 21:08 abchatra

I verified this bug in the latest version:7.0.29 and it has been fixed.

fionama100 avatar Aug 29 '24 05:08 fionama100