fix attemp

[zhuoyuT]

Fixes #0000

• [ ] I have read and agree to the Code of Conduct, and have followed the Contributing guidelines.

Pull Request Type

• [ ] Bug fix (please link related issues if applicable)
• [ ] New feature
• [ ] Question update
• [ ] Other

Summary

[danieldanielecki]

@eduardconstantin, what do you think?

The user has reported a bug via Reddit, @zhuoyuT might you confirm that's what is the PR about?

[eduardconstantin]

someone else implemented the scoring, I believe this happens because some questions have multiple answers and the score is divided by the number of answers

[zhuoyuT]

@danieldanielecki Hey it is, but this is not final. I commented out the shuffle function and reduced the question count for debugging purpose, I will try to update it during the weekend, but feel free to adjust on top of my commit.

@eduardconstantin the issue I encountered was the point of the last question was never added

[eduardconstantin]

even better

[danieldanielecki]

@danieldanielecki Hey it is, but this is not final. I commented out the shuffle function and reduced the question count for debugging purpose, I will try to update it during the weekend, but feel free to adjust on top of my commit.

@eduardconstantin the issue I encountered was the point of the last question was never added

thanks for the clarification, will be waiting for the next commits :)

[zhuoyuT]

@eduardconstantin @danieldanielecki PR updated, please check

[danieldanielecki]

@eduardconstantin @danieldanielecki PR updated, please check

Is it now 100% fixed & ready?

[danieldanielecki]

What I understood before this fix when there was a question with multiple answers correct, the total result wasn't calculated correctly. Now it should be fixed, and easily noticed by seeing 100% for such tests in Practice Mode.

Might you please confirm?

cc: @eduardconstantin

[eduardconstantin]

What I understood before this fix when there was a question with multiple answers correct, the total result wasn't calculated correctly. Now it should be fixed, and easily noticed by seeing 100% for such tests in Practice Mode.

Might you please confirm?

cc: @eduardconstantin

I assumed that was the issue, I haven't checked

[danieldanielecki]

I tested, and it was indeed the case.

However, regardless of the number of multiple-choice questions, the maximum score was always 96.67/100. Merged.

[danieldanielecki]

and it's no longer the case, thank you so much for the contribution @zhuoyuT

@eduardconstantin, I also made the counter of answered questions in the top left a bit more explicit by adding ANSWERED:. Thank you as well, for the input!

[eduardconstantin]

and it's no longer the case, thank you so much for the contribution @zhuoyuT

@eduardconstantin, I also made the counter of answered questions in the top left a bit more explicit by adding ANSWERED:. Thank you as well, for the input!

not a bad idea, but now the top part is not centered anymore :(

[danieldanielecki]

and it's no longer the case, thank you so much for the contribution @zhuoyuT

@eduardconstantin, I also made the counter of answered questions in the top left a bit more explicit by adding ANSWERED:. Thank you as well, for the input!

not a bad idea, but now the top part is not centered anymore :(

uhh good feedback, missed it. Any idea how to handle both?

[danieldanielecki]

and it's no longer the case, thank you so much for the contribution @zhuoyuT

@eduardconstantin, I also made the counter of answered questions in the top left a bit more explicit by adding ANSWERED:. Thank you as well, for the input!

not a bad idea, but now the top part is not centered anymore :(

used emoji instead https://github.com/Ditectrev/Practice-Exams-Platform/commit/5ac85c3817f1466e6997898e315455e2e1caade9 and now having best of both ;)

[danieldanielecki]

What I understood before this fix when there was a question with multiple answers correct, the total result wasn't calculated correctly. Now it should be fixed, and easily noticed by seeing 100% for such tests in Practice Mode.

Might you please confirm?

cc: @eduardconstantin

I meant Exam Mode of course. Practice Mode has no results shown.