search

rosshamish / classtime

university schedule generation and course data API
MIT License
16 stars 2 forks source link

Only one electives group gets a course in each schedule #108

Closed rosshamish closed 8 years ago

rosshamish commented 9 years ago

Repro:

Generate a schedule with:

Expected:

Math 101 and Biol 108 are both scheduled

Actual:

Only one will be scheduled. Could be either.

  1. Write a test in tests/classtime/brain/scheduling/test_schedule_generator.py to reproduce this
  2. Fix the bug in classtime/brain/scheduling/schedule_generator.py. The bug is in _schedule_electives() at https://github.com/rosshamish/classtime/blob/master/classtime/brain/scheduling/schedule_generator.py#L125