-
Notifications
You must be signed in to change notification settings - Fork 40
Open
Labels
P2Medium Priority, not urgent but should be on the near-term agendMedium Priority, not urgent but should be on the near-term agendoptimizationAdding or improving optimization methodsAdding or improving optimization methodsrobustnessMake the code more robustMake the code more robusttestingAdding a new test or fixing an existing oneAdding a new test or fixing an existing one
Description
We should add unit tests which check that the deterministic subproblems are solved correctly.
Right now, I only see optimization test for strictly convex functions in test_optimizer. A basic gradient descent would work for that; things could go wrong in more specialized methods that would not be detected by minimizing a convex function.
Originally posted by @unalmis in #1181 (review)
Metadata
Metadata
Assignees
Labels
P2Medium Priority, not urgent but should be on the near-term agendMedium Priority, not urgent but should be on the near-term agendoptimizationAdding or improving optimization methodsAdding or improving optimization methodsrobustnessMake the code more robustMake the code more robusttestingAdding a new test or fixing an existing oneAdding a new test or fixing an existing one