-
Notifications
You must be signed in to change notification settings - Fork 303
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
Adaptive learning
: Extend learning path feature to recommend sequence of competencies
#7091
Conversation
…-path/participation/exercises
…-path/participation/exercises
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.
Retested with Postgres. Works :)
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.
Tested in Testing Session 👍
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.
lgtm now
The base branch was changed.
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.
Approve code after changes
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.
re-approve code
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.
looks good to me
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.
LGTM
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.
Tested the changes since c522378 on ts2, the code of these changes looks fine and the alerts are displayed properly.
Checklist
General
Server
Client
authorities
to all new routes and checked the course groups for displaying navigation elements (links, buttons).Motivation and Context
We want to provide a sequential order of lecture units and exercises for students participating via the learning paths feature.
Description
This PR adds the first step towards generating this sequential order by recommending an order of the competencies.
The LearningPathService is now able to analyze the student's progress and assign utilities to the corresponding competency. The utility accounts for upcoming due dates, prior competencies, relation types between competencies, and the mastery progress. The utilities can then be used to construct the ordering.
Steps for Testing
Prerequisites:
View
-buttonReview Progress
Performance Review
Code Review
Manual Tests
Test Coverage
Client
Server
�
Screenshots
As this PR doesn't change the UI there are no screenshots of shiny new components, just a comparison between the old graph received from the server (in participation view) and the new one:
Previously:
Now: