Skip to content

Conversation

@llorracc
Copy link
Collaborator

@wdu9, @akshayshanker , could you both review?

@wdu9, there are some html comments embedded for you. Search CDC.
Please ensure your pull request adheres to the following guidelines:

  • Tests for new functionality/models or Tests to reproduce the bug-fix in code.
  • Updated documentation of features that add new functionality.
  • Update CHANGELOG.md with major/minor changes.

@llorracc llorracc requested a review from wdu9 October 26, 2024 22:07
@akshayshanker
Copy link
Collaborator

@wdu9 do you want to make your particular edits in response to the @CDC2WXD comments in SSJ_explanation.ipynb, and then I can review?

@llorracc, I did a pretty extensive tidy up of KS-HARK-Presentation.ipynb after you had done these edits. I suggest keeping the KS-HARK-Presentation.ipynb I edited. Let's quickly discuss in person next Zoom.

@akshayshanker
Copy link
Collaborator

Scratch that --- I will address the comments including the ones to @wdu9.

@mnwhite
Copy link
Contributor

mnwhite commented May 21, 2025

I think this PR is going to be harder to resolve. It looks like it's from even earlier than the prior one, and the files have been changed substantially since then. The "new" file in this PR might actually be an old version of a different file, under an old name. I guess Akshay will tackle this in a fresh branch?

Funny enough, CDC's primary comment from June 2024 was exactly what I wrote in my review on the other PR (unaware Chris wrote it here): the model needs to be explained in math and/or words.

@llorracc
Copy link
Collaborator Author

Thanks @mnwhite

Do you have a complete inventory of the HANK related notebooks and material in HARK? If so, is that inventory described somewhere that would be easy to find for someone looking for precisely this material? If not, I think it would be very useful to have such an inventory.

@mnwhite
Copy link
Contributor

mnwhite commented May 21, 2025

The SSJ and HANK-adjacent notebooks were moved from /examples/SSJ-example/ to /examples/ConsNewKeynesianModel/ in the master branch. There are currently five notebooks there:

  • HANKFiscal_example.ipynb
  • Jacobian_example.ipynb
  • KS-HARK-presentation.ipynb
  • SSJ_explanation.ipynb
  • Transition_Matrix_Example.ipynb

I think KrusellSmith_solved_by_HANK.ipynb in this branch is a very old/preliminary version of what is now HANKFiscal_example.ipynb, but don't hold me to that.

mnwhite added a commit that referenced this pull request May 28, 2025
This is a recreation of the substantive changes from #1505 . Requested / suggested changes that CDC left as comments were not made.
@mnwhite
Copy link
Contributor

mnwhite commented May 28, 2025

Substantive changes have been incorporated via #1568

@mnwhite mnwhite closed this May 28, 2025
@github-project-automation github-project-automation bot moved this from In progress to Done in Issues & PRs May 28, 2025
@mnwhite mnwhite reopened this May 28, 2025
@github-project-automation github-project-automation bot moved this from Done to In progress in Issues & PRs May 28, 2025
@mnwhite mnwhite removed Priority: High Version: 0.16 for now-released v0.16.0 labels May 28, 2025
@mnwhite
Copy link
Contributor

mnwhite commented May 28, 2025

Re-opened because Akshay wants it left as to be completed.

@mnwhite
Copy link
Contributor

mnwhite commented Jun 4, 2025

Can this PR be archived now in some way? My understanding is that the remaining work on this is to address some comments from CDC that require new writing. Could those comments be moved to an issue so that we can close out a PR that will never be merged because it's been superceded?

@llorracc
Copy link
Collaborator Author

llorracc commented Jun 5, 2025

@mnwhite, I'm OK if you want to turn them into an issue. You should be able to compare he commit before this one to this commit to see what I changed/edited.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

4 participants