we will move to Knapsack later