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

remove cstwMPC from HARK codebase, bye bye #666

Merged
merged 4 commits into from
Apr 30, 2020

Conversation

MridulS
Copy link
Member

@MridulS MridulS commented Apr 28, 2020

closes #449 #415

cstwMPC stays as a REMARK and datasets moved to HARK.datasets.

@llorracc
Copy link
Collaborator

I think we should let @mnwhite actually execute the merge, just to give him the satisfaction of finally accomplishing a long-sought goal. But @sbenthall can review first.

@sbenthall
Copy link
Contributor

this passes my review.
@mnwhite please do the honors at your convenience

@mnwhite
Copy link
Contributor

mnwhite commented Apr 28, 2020 via email

@llorracc
Copy link
Collaborator

@mnwhite you might lose your privileges to merge this if you don't find a meme fast enough!

@mnwhite
Copy link
Contributor

mnwhite commented Apr 30, 2020 via email

@sbenthall sbenthall merged commit 9d19960 into econ-ark:master Apr 30, 2020
@sbenthall sbenthall mentioned this pull request May 1, 2020
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.

split cstwMPC into general tools and example code
4 participants