[scalar_dynamic] Update editorial suggestions

longye-tian commented 5 months ago

Dear John @jstac,

This pull request updates the editorial suggestion in #426. The details are shown as follows.

(Delete %matplotlib inline)

If $n$ is an integer -> if $n$ is a positive integer
Change the duplicated heading for 22.2.3. Dynamic systems.
Capitalize the first letter in the section headings.
where $a, b$ are fixed constants -> where $a, b$ are constants. Sometimes they are referred to as parameters
$t \geq 0$ -> t = 0, 1, 2
Arbitrary $t$ to non-negative integer $t$
For example, recall how we previously studied -> For example, in the simple Solow-Swan growth model.
$k$ is capital stock -> $k$ is capital stock per capita
a fixed dynamic system consisting -> a dynamic system consisting
Use commas instead of full stops below the sentence: "For example, for the linear model $x_{t+1} = a x_t + b$, you can use the definition to check that"
45 degree -> 45-degree
Add labels to axis in plot45 function xt and x{t+1}
Put the global variables into function $g$
Missing full stop in 'study the trajectories'
Change the Solow model to Solow-Swan model

Best ❤️ Longye

netlify[bot] commented 5 months ago

Name	Link
Latest commit	516c2af5a08aca72091731f7f7501b7be0235dd3
Latest deploy log	https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/662e0209d937740008116653
Deploy Preview	https://deploy-preview-430--taupe-gaufre-c4e660.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

jstac commented 5 months ago

Thanks @longye-tian, much appreciated! This all looks good to me.

@mmcky , could you please guide this through the merge process and close any relevant issues?

mmcky commented 5 months ago

thanks @longye-tian this code cell is not executing in scalar-dynam

------------------
xmin, xmax = 0, 4  # Suitable plotting region.

plot45(g, xmin, xmax, 0, num_arrows=0)
------------------

due to a key error

KeyError't+1'

github-actions[bot] commented 5 months ago

mmcky commented 5 months ago

@longye-tian do you have any time to work on these comments?

longye-tian commented 5 months ago

@longye-tian do you have any time to work on these comments?

#426 (comment)

Hi @mmcky , I can work on these comments as well :))

I think the other comments in #426 require a separate pull request. I will create a new pull request for other comments.

Best, Longye

mmcky commented 5 months ago

thanks @longye-tian -- @jstac this PR is ready for review.

jstac commented 5 months ago

Many thanks @longye-tian , much appreciated. Merging.

QuantEcon / lecture-python-intro