-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Diagnostic test session #16489
Diagnostic test session #16489
Conversation
…nto diagnostic_player
…tic_test_algorithm
…nto diagnostic_test_algorithm
…tic_test_algorithm
Unassigning @U8NWXD since the review is done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, @U8NWXD thank you for the review, I have addressed the comments.
Please take a look at the nearest time when you are available.
Thank you.
Thanks @Nik-09 ! This LGTM |
Hi @Nik-09, this PR is ready to be merged. Please address any remaining comments prior to merging, and feel free to merge this PR once the CI checks pass and you're happy with it. Thanks! |
Overview
The PR adds the UI to give the diagnostic test from the diagnostic test player page. And, based on the performance in the test, 0, or 1, or 2 topics were recommended.
Essential Checklist
Proof that changes are correct
Updated screenshots
two.mp4
three.mp4
Existing exploration editor and player
exploration.mp4
Existing practice session
practice_session.mp4
PR Pointers