Skip to content
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

Merge #224 did not bring in the final version #225

Merged
merged 2 commits into from
Feb 16, 2019
Merged

Conversation

econ-ark
Copy link
Collaborator

@econ-ark econ-ark commented Feb 16, 2019

Some final tweaks were omitted from the merged version; this brings them in

Release Notes: No need for notes on this one; just does what #224 was supposed to do

mnwhite and others added 2 commits February 14, 2019 13:26
All of the consumption-saving model files used a style of import that didn't work when the file was run directly (rather than called as a module), even though they have a __main__ block (and main() function).  This has now been fixed.

Also mostly removed extraneous file RepAgentModel.py, which seems to be an old name of ConsRepAgentModel.py.  This file now simply imports all of ConsRepAgentModel and warns the user to use that instead.
@llorracc llorracc merged commit 7afa2c1 into master Feb 16, 2019
@llorracc llorracc deleted the QuickImportFix branch February 16, 2019 16:40
@shaunagm
Copy link
Contributor

I've been assigning release note responsibilities to the author of PRs, but this author is a ghost. (Maybe it's the econ-ark-owner account, and someone was logged into it accidentally?) Anyway, can either @llorracc or @mnwhite take responsibility for writing a quick summary?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants