jeromematthewcelestine / hetagentsaggshocks

Demonstration of the Reiter method for solving models with heterogeneous agents and aggregate shocks in general equilibrium. Solves a simple model of firm investment with persistent aggregate and idiosyncratic productivity shocks. UPDATE: Consider looking at https://github.com/jeromematthewcelestine/hadsge for Reiter-method code instead.
11 stars 15 forks source link

Merge `run_investment.m` and `MAIN.m`? #3

Open jagman88 opened 7 years ago

jagman88 commented 7 years ago

Mostly personal preference, but in a folder with a bunch of files, I really like to have the top-level file named something really obvious with capitalization like MAIN.m or START.m or RUN.m or something along those lines (i.e. something that's an obvious focal point for people coming to the file who want to get started right away). Could also go for MAIN_investment.m or RUN_investment.m.

If we did this, we could merge (or delete one of) run_investment.m and MAIN.m.

Thoughts?

jeromematthewcelestine commented 7 years ago

Sounds good. Please implement any "better practice" stuff you think is a good idea. I'd like to move away from a set of scripts towards one script plus a set of functions/classes, if possible.

By the way, I had capitalized "investment" in those filenames because I was thinking of Investment as the model name. It might be helpful to break things up into the multipurpose stuff and the model-specific stuff, though I'm not sure how feasible that would be.

jagman88 commented 7 years ago

Let's talk about it a little bit more tomorrow and then go ahead.

Wow, I didn't even know Matlab could handle classes until you mentioned it. Given that they're available, it would definitely be possible to shift everything in that direction. Don't know how much work that would be, though.

Ah, I see about the capitalization. What you suggest sounds good, so let's think about what would work as generic vs. specific functions, too.