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

Migrate comments when changing grading scale. #515

Open
nthiery opened this issue Mar 19, 2024 · 2 comments · Fixed by #550
Open

Migrate comments when changing grading scale. #515

nthiery opened this issue Mar 19, 2024 · 2 comments · Fixed by #550
Assignees

Comments

@nthiery
Copy link

nthiery commented Mar 19, 2024

Use case: @hivert started to grade a question using a "direct" grading scale. At some point he
realized that an "hybrid" grading scale would be more appropriate. When changing
the grading scale however, all comments dissapeared, both from the comment list and
the student copies; which scared a bit @hivert. Gladly, they were still in the database, and
reappeared when switching back to a direct grading scale :-)

Would it be possible instead to migrate the comments when changing from one scale to the other?
If there is some loss of information, like how many points are attached to a comment, that's
presumably ok, especially if there is a warning about it.

@barais
Copy link
Collaborator

barais commented Mar 19, 2024

Use case: @hivert started to grade a question using a "direct" grading scale. At some point he
realized that an "hybrid" grading scale would be more appropriate.

Yes changing the type of grading during a scoring session is not supported. Migrating the comments is possible.  I think the best option in the future is to match everything on the hybrid model. It can possibly unify everything.

As a patch, I will add the ability to go from direct or postive/negative to hybrid but not in the other way. For positive/negative to hybrid, I can keep the scoring. For direct, I will add a new hybrid comment called score with step and grade to keep the grade. I will keep all textcomment to hybrid comment with score to 0%

@nthiery
Copy link
Author

nthiery commented Mar 19, 2024

Thanks for the quick feedback! Yes direct->hybrid migration will do very well for now.

@barais barais linked a pull request Jun 18, 2024 that will close this issue
@barais barais reopened this Jun 18, 2024
@barais barais self-assigned this Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants