-
-
Notifications
You must be signed in to change notification settings - Fork 22
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[lp_intro] update suggestions (#474)
* [lp_intro] update suggestions This pull request update some of the the lp_intro.md lecture according to #469 (mainly involving typos, styles) In particular, the changes include: - [x] Unbold non-definition words and use italics for emphases - [x] Remove capitalization in the titles - [x] Add period for the first sentence in example 1 - [x] use "`ortools.linear_solver`" - [x] "... using it's status" -> "... using its status" - Round the print in the format string for printing. - "three year" -> "three-year" - "Let's us" -> "Let's" - "LP" -> "Linear programming" - In the dot point in the "useful transformations" section, remove the capitalization of the second word. - 'bounds' -> `bounds` - We did not discuss complementary slackness, so we can delete it. One thing I find is that the column header is automatically in bold, so I cannot fix it by changing the md file. * [lp_intro] update notations Dear John @jstac, This pull request finishes the issue #469 by - add the notation for $z$, $A_{eq},b_{eq}, A_{ub},b_{ub}$. - change the orange to black to correspond to the changes in the visualization plot. Best, Longye
- Loading branch information
1 parent
a291267
commit 39594d7
Showing
1 changed file
with
34 additions
and
36 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
39594d7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 Deployed on https://66851daf6a58b6d22cb79d35--taupe-gaufre-c4e660.netlify.app