Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Grader does not see exercise status enrollment from the index.yaml during grading #118

Open
markkuriekkinen opened this issue Sep 7, 2021 · 0 comments
Labels
area: config Related to course configuration area: grading quizzes Quiz feature effort: days Solving this issue could take days. Starting with the creation of a new branch to the merging experience: moderate required knowledge estimate priority: low requester: Aalto teacher The issue is raised by a teacher from Aalto University type: bug This is a bug.

Comments

@markkuriekkinen
Copy link
Contributor

markkuriekkinen commented Sep 7, 2021

If the exercise status is set to "enrollment" by using the append_content variable in conf.py, then the grader code in the form feedback does not know that the exercise is for enrollment. The feedback contains the "correct/incorrect" lines that have been removed in enrollment exercises. In other words, the is_enrollment_exercise only sees the status defined in the exercise config.yaml and it is unaware of the exercise status in index.yaml. (Using append_content adds the status only in index.yaml.)

def is_enrollment_exercise(self):
return self.exercise.get('status', '') in ('enrollment', 'enrollment_ext')

If the status "enrollment" is added in the questionnaire RST directive or manually in the exercise config.yaml, then it works.

@markkuriekkinen markkuriekkinen added area: config Related to course configuration area: grading quizzes Quiz feature effort: days Solving this issue could take days. Starting with the creation of a new branch to the merging experience: moderate required knowledge estimate priority: low requester: Aalto teacher The issue is raised by a teacher from Aalto University type: bug This is a bug. labels Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: config Related to course configuration area: grading quizzes Quiz feature effort: days Solving this issue could take days. Starting with the creation of a new branch to the merging experience: moderate required knowledge estimate priority: low requester: Aalto teacher The issue is raised by a teacher from Aalto University type: bug This is a bug.
Projects
None yet
Development

No branches or pull requests

1 participant