Completely untested, will adapt construct_income_process on next commit as test example.

Holy crap, construct() worked on the first try! Only IncShkDstn has been tested, and none of the "error catching" features have been looked at. But hey, it works!

Should pass these tests now, but formatting is probably still wrong.

Can now collapse a DiscreteDistribution to a single dimension; PermShkDstn and TranShkDstn are made this way now. Moved bespoke constructor dictionary lines out of init. Future plan: each AgentType subclass should have default constructors dictionary in its parameter dictionary.

One test was failing due to lack of seed.

All tests pass except for calc_jacobian, which needs a separate fix.

Missed this one. Jacobian test will still be failing.

All test should pass now. Added new optional input for AgentType.solve() that allows the user to decline to run the presolve routine. For most of our classes, presolve might update the terminal solution and/or run update/constructor methods, but this is fatal to what calc_jacobian wants to do.

I think something about pytest just changed while I was working on this branch, because three tests needed to have a [0] added to them (to reference the only element in the array) in order to be correct. Nothing changed, but the result of these tests did. I'm confused.

One assertEqual test was off by 1e-16 on Python 3.9, but not 3.10. No idea why.

Also added retro support for using [None] rather than None for aXtraExtra.

pLvlGrid and pLvlNextFunc are now built with the constructors framework. This means that the *only* difference between the AgentType subclasses in ConsGenIncProcessModel is their default dictionary. GenInc tests pass locally, committing to see what else I broke.

Because dictionaries now include constructor methods, two extra lines were needed. Also made the example problems consistently infinite horizon so that the comparison is relevant.

This makes object used for the solution not depend on simulation parameter. Test results were slightly adjusted to account for grid change. Tests passed on the commit immediately prior to this one, hence the *very* specific commit.

The portfolio / risky-asset model constructed inputs have been ported into the constructors framework. I didn't put the simulation-only object AdjustDstn into this form, nor did I put the one-liner ShareGrid constructor. The latter should probably be moved later-- it's not hard, I just got lazy when it's a one line method to make a uniform grid on [0,1].

Fixed a spelling error in one place, so had to fix it where it was imported.

Due to presence of constructors, two examples needed a different dictionary as their base.

Tests pass, committing to see which examples break.

Tests pass, hopefully examples will too.

AgentType now has a helper method called describe_constructors that prints to screen the status of constructed objects, including their names, the required primitive inputs, and whether each value exists.

Also deleted long-unused commented out code.

Prior commit was actually trivial, as it didn't delete the prior update_solution_terminal!

Very silly

Never make last second changes before committing, kids. You will never get it right and only embarrass yourself through the commit history.

…tors

Minor errors in code and dictionaries due to changes in prior commit.

Also had to change dictionary import in tests. Notebooks might break.

Too much copy-pasta, something ended up zero that should have been one.

One is an extremely basic binary state specification, the other is a "ratchet" specification in which transitions only go one direction, and only one step at a time.

Same issue as in earlier commit, but in an example notebook. Just imported and used correct dictionary.

Construct method is now more robust, with an option to force through errors and record them in a separate dictionary. Also made other small changes/fixes.

This commit adds one missing (trivial) constructor and adds a few "counterexample" constructors that *could* be used for other parameter specifications. There are many, many more that could be written.

Also added one missing line of documentation.