-
-
Notifications
You must be signed in to change notification settings - Fork 54
STYLE: Fix emphasis vs definitions style guide compliance (Issue #721) #723
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
Conversation
- Changed all first-use technical terms from italic to bold - Complies with QuantEcon style guide: bold for definitions, italic for emphasis - Related to issue #721
- Changed technical terms from italic to bold on first definition - Fixed: probability distributions, companion matrix, vector autoregression, deterministic/indeterministic seasonal, linear time trend, moving average, martingale with drift, unconditional mean/variance-covariance matrix, ensemble, cross-sectional average, autocovariance function, stationary distribution, covariance stationary, ergodicity, Markov property, conditional covariance matrix, discrete Lyapunov - Related to issue #721
- Changed Kolmogorov's strong law from italic to bold (named theorem) - Changed variance-covariance matrix from italic to bold (definition) - Related to issue #721
- Changed Gordon formula from italic to bold (named formula) - Changed Lucas tree model terms (tree, fruit, shares, dividend) to bold - Changed infinite horizon, call option, strike price to bold (definitions) - Note: kept 'exercises' and 'not to exercise' as italic (emphasis of choice) - Related to issue #721
- Changed 'Markov perfect equilibrium' from italic to bold in formal definition - Related to issue #721
- Changed 'values' from italic to bold when introducing value functions concept - Note: 'Bellman equation', 'policy', 'reservation wage' already use bold - Related to issue #721
- Changed 'parametric class' from italic to bold (technical concept) - Changed 'Poisson regression' from italic to bold (named model) - Changed 'cumulative normal distribution' from italic to bold (technical term) - Note: 'maximum likelihood estimates' already uses bold - Related to issue #721
- Changed 'exogenous' from italic to bold (key econometric term) - Changed 'marginal effect' from italic to bold (technical definition) - Changed 'the sum of squared residuals' from italic to bold (OLS definition) - Note: 'omitted variable bias', 'multivariate regression model', 'endogeneity', 'two-stage least squares', 'instrument' already use bold - Related to issue #721
Changes per #721: - rational expectations equilibrium (first introduction) - perceived law of motion - actual law of motion - belief function - Euler equation - transversality condition - recursive competitive equilibrium - planning problem All terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - backward shift (operator definition) - lag (operator definition) - forward shift (operator definition) - explosive solution All terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - second-order linear difference equation - national output identity - consumption function - accelerator - accelerator coefficient - aggregate demand - aggregate supply - business cycles - stochastic linear difference equation - marginal propensity to consume - steady state - random - stochastic - shocks - disturbances - second-order scalar linear stochastic difference equation - characteristic polynomial - zeros - roots All terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - transmission rate - infection rate - recovery rate - effective reproduction number All terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - propagation mechanism Term changed from italic to bold as it is a definition per style guide.
Changes per #721: - activities - goods - input matrix - output matrix - intensity - goods used in production - total outputs - productive - cost - revenue - costs - revenues - irreducibility All terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - Lifecycle patterns - Within-cohort heterogeneity - Cross-cohort interactions All terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - numeraire Term changed from italic to bold as it is a definition per style guide.
Changes per #721: - exogenous (cake_eating_egm.md) - adapted (cake_eating_stochastic.md) - state (cake_eating_stochastic.md) - control (cake_eating_stochastic.md) - topologically conjugate (cake_eating_time_iter.md) All terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - career (career.md) - job (career.md) - aggregation theory (cass_koopmans_1.md) All terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - recursion - multiplicative decomposition Terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - prior distributions - posterior distributions - speculative behavior - ex dividend - Short sales are prohibited - Harsanyi Common Priors Doctrine All terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - reservation wage (odu.md) - reservation wage functional equation (odu.md) - matrix (opt_transport.md) - vector (opt_transport.md) All terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - prior (kalman.md) - filtering distribution (kalman.md) - predictive (kalman.md) - Kalman gain (kalman.md) - predictive distribution (kalman.md) - savings (ifp_advanced.md) All terms changed from italic to bold as they are definitions per style guide.
Changes per #721: - Household (changed from italic to bold - also fixed typo 'Frim' to 'Firm') - Firm Terms changed from italic to bold as they are economic agents being defined per style guide.
Changes per #721: - conditionally (as part of 'conditionally independently and identically distributed') Term changed from italic to bold as it is a definition per style guide.
@jstac If you had time to do a few quick I built an agent to review each lecture in a very systematic way using lots of |
This comment was marked as outdated.
This comment was marked as outdated.
|
📖 Netlify Preview Ready! Preview URL: https://pr-723--sunny-cactus-210e3e.netlify.app (c6a0887) 📚 Changed Lecture Pages: ak2, ak_aiyagari, cake_eating_egm, cake_eating_stochastic, cake_eating_time_iter, career, cass_fiscal, cass_koopmans_1, exchangeable, ifp_advanced, kalman, likelihood_bayes, linear_algebra, linear_models, lln_clt, markov_asset, markov_perf, mccall_model, mle, morris_learn, odu, ols, opt_transport, rational_expectations, re_with_feedback, samuelson, sir_model, uncertainty_traps, von_neumann_model |
|
@jstac with a fresh brain this morning I have gone through this PR in more detail. It's actually quite tricky! but I think there should be less If you have time would you be able to cycle through my checkboxes and tick the ones you agree with? |
|
Does checking the box mean i agree with the change or that I agree with the designation of emphasis? |
|
roger that @mmcky . Done. Some of these are borderline. |
Based on PR review feedback with checked [x] emphasis comments, reverted the following terms from bold back to italic (emphasis, not definitions): - linear_models.md: ergodicity (concept emphasis) - markov_asset.md: tree, fruit, shares, dividend (metaphorical emphasis) - mccall_model.md: values (concept emphasis) - mle.md: parametric class (emphasis not definition) - morris_learn.md: prior/posterior distributions, speculative behavior, ex dividend, Short sales, Harsanyi Common Priors Doctrine (emphasis) - ols.md: exogenous, marginal effect (emphasis not definitions) - rational_expectations.md: rational expectations equilibrium (intro emphasis), perceived/actual law of motion (intro emphasis, formal definitions come later) - samuelson.md: second-order linear difference equation, national output identity, consumption function, accelerator, accelerator coefficient, aggregate demand/supply, random, stochastic, shocks, disturbances (emphasis not definitions) These are emphasis on concepts or references, not formal definitions.
Additional Revisions Based on Review FeedbackI've processed all the checked Files Reverted (8 files, commit 8d221fd)linear_models.md:
markov_asset.md:
mccall_model.md:
mle.md:
morris_learn.md:
ols.md:
rational_expectations.md:
samuelson.md:
Pattern ClarificationThe distinction is now clearer:
Total revisions so far:
The remaining 15 files contain correct formal definitions. |
|
📖 Netlify Preview Ready! Preview URL: https://pr-723--sunny-cactus-210e3e.netlify.app (8d221fd) 📚 Changed Lecture Pages: ak2, cake_eating_stochastic, cake_eating_time_iter, career, cass_fiscal, kalman, likelihood_bayes, linear_algebra, linear_models, lln_clt, markov_asset, markov_perf, mle, odu, ols, rational_expectations, re_with_feedback, samuelson, sir_model, uncertainty_traps, von_neumann_model |
Fixed the following typos: - linear_algebra.md: removed extra '.*.' after 'square' and 'symmetric' - linear_algebra.md: removed extra '.l.' after 'diagonal' - sir_model.md: removed extra 'd)' after 'infected)' - von_neumann_model.md: removed extra '.).' after 'consumed)' - von_neumann_model.md: removed extra '****' after 'outputs' - von_neumann_model.md: removed extra 'es' from 'activitieses' → 'activities'
Typo FixesFixed 7 typos that were accidentally introduced during the formatting changes (commit f0264ea): linear_algebra.md:
sir_model.md:
von_neumann_model.md:
All changes have been reviewed and corrected. The PR should now be clean and ready for final review. |
mmcky
left a comment
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.
@jstac OK -- this now looks like a net positive improvement. I will merge this later today as it touches a lot of files.
|
📖 Netlify Preview Ready! Preview URL: https://pr-723--sunny-cactus-210e3e.netlify.app (f0264ea) 📚 Changed Lecture Pages: ak2, cake_eating_stochastic, cake_eating_time_iter, career, cass_fiscal, kalman, likelihood_bayes, linear_algebra, linear_models, lln_clt, markov_asset, markov_perf, mle, odu, ols, rational_expectations, re_with_feedback, samuelson, sir_model, uncertainty_traps, von_neumann_model |
|
Thanks @mmcky , please go ahead when ready. |
This PR addresses Issue #721 by systematically reviewing all lecture files and ensuring compliance with the QuantEcon style guide for emphasis and definitions.
Summary
Changes Made
Changed technical terms from italic to bold when they are being defined (not for emphasis), following the style guide rule:
**bold**for definitions*italic*for emphasisFiles Modified (29 total)
Key files with multiple changes:
linear_algebra.md- 32 changes (20 definition terms)linear_models.md- 20 changes (16 definition terms)samuelson.md- 19 changesvon_neumann_model.md- 13 changesrational_expectations.md- 8 changesImpact
Testing
All changes are purely cosmetic (Markdown formatting) and do not affect:
Closes #721