Update #1
Merged
Update #1
Conversation
This file contains hidden or 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
Update coding suggestion in issue #392, more specifically, this pull request is related to - Set the default values in `create_cagan_adaptive_model` instead of global parameters. (Some parameters are still global and move to the Experiment 1 section as they are not part of the `create_cagan_adaptive_model`) - change `inv` to `solve` (I also modified the name of `solve` function to `solve_cagan_adaptive` to avoid confusion with the `solve` function in `np`) - remove extra print when checking the stability of dynamics. - in the section of 14.6. Technical condition for stability and add a new section that focuses on experiments. - give names to matrices and equations so that they can be referred in the code comment (I add comment in code to suggest matrix D is the coefficient matrix of equation (14.8)) - Add comments in this function and give titles to subplots. - Change subtitle "Forcast error" -> "Forcast error and model computation" - Change the time notations for equations in the section "Experiment 1"
This pull request updates the title reference.
Co-authored-by: Matt McKay <[email protected]>
* Promote lectures to the intro series and update toc, bib, data files. * enable build reports on failure for latex workflow * remove interpolation section that is not related. * Tom's March 19 morning edits of some lectures * Humphrey's 20 Mar updates on math aligned syntax * Tom's March 20 edits of intro lectures * Tom's edits on March 21 of lectures in intro repo branch * update captions for graphs in lectures * Tom's afternoon March 21 edits of intro lectures * Tom's edit of toc file temporarily to remove two less complete lectures * update doc reference for XXX in text --------- Co-authored-by: mmcky <[email protected]> Co-authored-by: thomassargent30 <[email protected]>
* FIX: aligned issues * fix aligned * fixes for spelling, generaly tidy up
* [prob_dist] update according to editorial suggestions This commit update according to issue #402 . In particular, it mainly updates the typos, and missing colons. * [prob_dist] Update on parenthesis * Update prob_dist.md
[cagan_adaptive] Update the code suggestions
Address comments for `inflation_history` lecture
* [inquality] Review of lecture and incorporate updates * update plots * more plot updates * add quantecon back in for now re: gini_coefficient * @mmcky edits * review and minor edits * fix execution issue * minor updates to code output * updates * Compare inquality across countries * add plotly install for preview * supress mimetype warnings * TMP: temporarily disable warnings for preview * TMP: enable require.js * @mmcky review and proof-read * review of executable version in jupyter lab * address @jstac comment 1 * drop dynamics from title of section * use matplotlib default cycler * update gini to Gini * set x and y labels * review in jupyter-lab * tweaks from review of preview * remove incorrect statement * minor edit * improve narrative * incorporate some of @jstac comments * fix equation in prf:definition * address remaining @jstac direct comments * save branch conflict * @mmcky final review * update figures with mystnb figure settings * review from html * fix line continuation * Incorporate @jstac feedback and comments * reinstate hardline error on warning for html build on CI * Fix: duplicate label * add ignore on myst.domain warnings * address @jstac feedback * misc --------- Co-authored-by: John Stachurski <[email protected]>
* update cagan ree * remove extra annotations in the code * remove double spaces * remove figsize * Revert "remove figsize" This reverts commit c2afe22. * improve code comments based on feedback * update feedback for the second section * update labels and descriptions * update request on `the` * update graph function and set global dpi * distinguish the names of the two plotting functions * switch to tuple whenever possible * revert changes on dpi settings
* Use map instead of applymap * Update lectures/inflation_history.md --------- Co-authored-by: mmcky <[email protected]> Co-authored-by: Matt McKay <[email protected]>
Resolve most of the suggestions
[lln_clt] Update editorial suggestions
Revised under Editorial suggestion #414
[monte_carlo] updates and editorial suggestions
[heavy_tail] Update editorial suggestions
… assets (#424) * MAINT: refactor graphviz assets to lecture locations * update markov_chains_I * update markov_chain_II * update networks * clean up * manage merge conflict and ensure black edges
…545) * [laffer_adaptive] update style and spelling - change 'it reverse' to 'it reverses' - change the subtitle to lower case to match with the style - change 'pseudo code' to 'pseudo-code' to match with the title for consistency - change 'limiting values exists' to 'limiting values exist' - add hyphen to steady state when using is as adjective for consistency. * [lake_model] update spelling - change 'the below graph' to 'the graph below' for better word ordering - change 'long run growth' to 'long-run growth' for consistency
* [intro_supply_demand] Update typos * [intro_supply_demand] example admonition add two example admonitions
…#546) * [markov_chains_II] spelling and example admontion - change 'Benhabib el al' to 'Benhabib et al' * [markov_chains_II] example admonitions add example admonitions that includes a figure. * [mle] spelling and example admonition - change 'log-normal' to 'lognormal' for consistency - add one example admonition. * [markov_chains_II] add another example admonition add example admonition for irreducibility * [money_inflation] Spelling update - change 'take the form of what are' to 'take the form of what is' - change 'qualitive outcome' to 'qualitative outcome' - change 'it reverse the pervese' to 'it reverses the perverse' - change 'theses tools' to 'these tools' - change 'steady state rate' to 'steady-state rate' for consistency * [money_inflation_nonlinear] Update spelling - change 'stationary inflation rate that assert' to 'stationary inflation rate that asserts' - change 'it reverse the pervese' to 'it reverses the perverse' - change 'qualitive' to 'qualitative'.
…r July (#525) * upload french_rev from tom_edits * remove redundent files * update TOC * Tom's July 6 edits of french revolution lecture * Tom's edits of french revolution lecture July 7 * Tom's July 22 edits of french revolution lecture * revert one change in the text * rebuild with cite * update fig labels * Tom's July 23 edits of some lectures and bib file * update solution and quote directives * fix a small typo * make the last graph clearer * update style in line with style guide * update based on feedback --------- Co-authored-by: Humphrey Yang <[email protected]> Co-authored-by: thomassargent30 <[email protected]> Co-authored-by: Humphrey Yang <[email protected]>
* RMV: removes ak2 lecture and redirects to the intermediate series * add dependency for collab run
* address feedbacks from reading group * update based on feedback * remove "for"
…-theme (#493) * [geom_series] remark_admonition Dear John, This pull request is for #442. I use the geom_series.md lecture as an example to test the current admonition style. Best, Longye * TST: install main branch version of sphinx-proof * test with new css definition * test the new css styling for remark * test updated quantecon-book-theme * upgrade quantecon-book-theme==0.7.2 * update branch until released for sphinx-proof * enable sphinx-proof==0.2.0 * update software for collab testing * remark_money_inflation Add remark admonition for money_inflation.md * remark_unpleasent Add remark admonition to unpleasent.md * remark_money_inflation_nonlinear Add remark admonition for money_inflation_nonlinear.md --------- Co-authored-by: mmcky <[email protected]> Co-authored-by: Matt McKay <[email protected]>
* Update RA list * add reading group
* MAINT: upgrade python=3.12 and anaconda==2024.06-1 * fix error in anaconda distribution tag
* fix `prf:theorem` in markov_chains_II * fix a small bug in indexing for prob_dist
* MAINT: upgrade to anaconda=2024.10 * tmp: disable build cache * Revert "tmp: disable build cache" This reverts commit 3543206.
Bumps [dawidd6/action-download-artifact](https://github.com/dawidd6/action-download-artifact) from 6 to 7. - [Release notes](https://github.com/dawidd6/action-download-artifact/releases) - [Commits](dawidd6/action-download-artifact@v6...v7) --- updated-dependencies: - dependency-name: dawidd6/action-download-artifact dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* ENH: update software stack * pin sphinx-reredirects * tmp: remove build cache * Revert "tmp: remove build cache" This reverts commit 78d7391.
* FIX: yf change to labels and using close price * migrate Adj Close to Close
* fix string syntax warnings * update one missing case
Bumps [dawidd6/action-download-artifact](https://github.com/dawidd6/action-download-artifact) from 7 to 8. - [Release notes](https://github.com/dawidd6/action-download-artifact/releases) - [Commits](dawidd6/action-download-artifact@v7...v8) --- updated-dependencies: - dependency-name: dawidd6/action-download-artifact dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* update cons_smooth * change names * update text * add tax_smooth * update tax smooth lecture * update lecture * update tax lecture * J to W * update text * Tom's Jan30 edits of tax-smooth lecture * update graphs in tax_smooth * update graphs for cons_smooth * remove the commented text * remove capitalization * Tom's Jan 31 edits of two smoothing lectures --------- Co-authored-by: Humphrey Yang <[email protected]> Co-authored-by: thomassargent30 <[email protected]> Co-authored-by: Humphrey Yang <[email protected]>
Bumps [dawidd6/action-download-artifact](https://github.com/dawidd6/action-download-artifact) from 8 to 9. - [Release notes](https://github.com/dawidd6/action-download-artifact/releases) - [Commits](dawidd6/action-download-artifact@v8...v9) --- updated-dependencies: - dependency-name: dawidd6/action-download-artifact dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.