search

park-junha / GradPlanner

An automated four-year undergraduate degree scheduler.
http://gradplanner.us
GNU General Public License v3.0
2 stars 0 forks source link

satisfies variable in buildPlan only stores first element of scuClass satisfies member #41

Closed park-junha closed 4 years ago

park-junha commented 4 years ago

Currently, the scuClass object's satisfies member is a list and can only contain one element. The following code in the buildPlan() method of the FourYearPlan class declares a variable within the method called satisfies such that it only takes the first element of the satisfies member of an scuClass object:

satisfies = self.metadata['required'][cID].getSatisfies()[0] if self.feasible(cID, terms[quarter][0], year) and self.preferenceMet(quarterMap, satisfiesMap[satisfies]):

This issue may become a problem when in the future, scuClass objects have a satisfies member with multiple elements.

park-junha commented 4 years ago

doubleDip method added, somewhat fixes this issue, but scuClass objects can still only be initialized with one satisfies member. Closing for now