-
-
Notifications
You must be signed in to change notification settings - Fork 195
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
Input checking for MarkovConsumerType #83
Comments
That's because LivPrb and PermGroFac are time-varying, so they need to be On Tue, Sep 27, 2016 at 8:46 PM, dclow notifications@github.com wrote:
|
Just issued a PR to do this. If the PR gets merged (maybe after dealing with issues you find), we can close this issue. |
I will tackle these PRs this weekend. I also plan to prepare a rather On Wed, Oct 12, 2016 at 3:11 PM, dclow notifications@github.com wrote:
|
The PR that addressed this was merged back in October; closing. |
MarkovConsumerType could use more checking of its inputs. It can give very weird error messages if it receives inputs that are just slightly different than what they should be.
For example, I wasted a lot of time with an infinite horizon version before I figured out that PermGroFac and LivPrb should be lists (of length 1, whose only element is a np.array), but Rfree should just be a numpy array. Also, those arrays should be of shape (N,), where N is the number of Markov states. Using an array of shape (N,1) also produced a very obscure error message
I imagine putting in checks at the beginning to verify that inputs are the right types and shapes could save a lot of people some of the hassle I went through
The text was updated successfully, but these errors were encountered: