TimFinucane / Ainur

Solution for parallel task scheduling problem, inspired by the tale of the Lord of the Rings
Other
5 stars 0 forks source link

Remove unused LowerBound method #69

Closed TimFinucane closed 6 years ago

TimFinucane commented 6 years ago

I propose we remove the estimate method from lower bounds without a nodesToVisit, as it is unused and currently just another method we have to support elsewhere, and on a similar note change the argument of nodesToVisit to a HashSet because it seems that all our algorithms use HashSet instead of list for storing the nodes

emipeanz commented 6 years ago

Related to #60

TimFinucane commented 6 years ago

Emi did this.