-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactoring Objective to Cost #2649
Conversation
Whoops -- sorry, I take that back @naveenoid. Build problems, for example:
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending. Comments from Reviewable |
ready for review. |
@drake-jenkins-bot retest this please |
Reviewed 6 of 6 files at r1, 2 of 2 files at r2. drake/solvers/NloptSolver.cpp, line 46 [r2] (raw file):
The URL for the NLopt documentation should not change. Comments from Reviewable |
Review status: 7 of 8 files reviewed at latest revision, all discussions resolved. drake/solvers/NloptSolver.cpp, line 46 [r2] (raw file):
|
Reviewed 1 of 1 files at r3. Comments from Reviewable |
@sherm1 FYI, you can use in situations like this! Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending. Comments from Reviewable |
Thanks, @david-german-tri -- I'll until the build finishes. Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending. Comments from Reviewable |
Reviewed 2 of 2 files at r2, 1 of 1 files at r3. Comments from Reviewable |
Review status: all files reviewed at latest revision, all discussions resolved. Comments from Reviewable |
All usages of "Objective" refactored to "Cost" to fit with #2639
+@sammy-tri Feature review
+@sherm1 Platform review
This change is