Skip to content

Commit

Permalink
Fix bug with first user enrolling in the adaptive section (#47)
Browse files Browse the repository at this point in the history
  • Loading branch information
idegtiarov authored Jun 11, 2019
1 parent 7e2d72d commit 9534d5c
Showing 1 changed file with 13 additions and 4 deletions.
17 changes: 13 additions & 4 deletions edx_adapt/api/resources/model_resources.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,12 @@ def post(self):
args = param_parser.parse_args()
course = args['course_id']
user = args['user_id']
default_param = None

try:
default_param = self.repo.get_model_params(course)
except DataException:
logger.debug("Default model's parameters are not found in the course description.")
default_param = None

try:
skills_list = self.repo.get_skills(course)
Expand All @@ -89,11 +94,15 @@ def post(self):
)
try:
key = self.create_search_key(course, user)
default_param = self.repo.get(key) or self.repo.get_model_params(course)
default_param = self.repo.get(key)
logger.debug(
"User has been already enrolled in the course, adapt will use already chosen parameters: {}".
format(default_param)
)
except DataException:
logger.debug(
"Default model's parameters are not found, adapt enrolls user with parameters: {}".format(
args['params']
"User has not been enrolled in the course yet, adapt enrolls user with parameters: {}".format(
default_param if default_param else args['params']
)
)
pass
Expand Down

0 comments on commit 9534d5c

Please sign in to comment.