-
-
Notifications
You must be signed in to change notification settings - Fork 199
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
constructLognormalIncomeProcessUnemployment(parameters) is not a method? #538
Comments
If it matters, it is quite specific to the particular case of the IndShockConsumerType and does not necessarily apply to superclasses that might inherit from it. |
*Subclasses ^^
Yes, this is something that's existed since literally the first week of
HARK, but shouldn't be there. It's a vestige of a previous design
structure, and we never fixed it. This is the same reason that discrete
distributions are nested lists with a particular structure, rather than a
DiscreteDistribution class that has various capabilities. We started out
very early on with a classless structure, and these are two bad vestiges of
that.
…On Fri, Feb 21, 2020 at 3:43 PM Christopher Llorracc Carroll < ***@***.***> wrote:
If it matters, it is quite specific to the particular case of the
IndShockConsumerType and does not necessarily apply to superclasses that
might inherit from it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#538?email_source=notifications&email_token=ADKRAFMJMI4IZBBYOFSBUXLREA4F7A5CNFSM4KZJJDXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMUA2BA#issuecomment-589827332>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADKRAFKPBU6BM7YULZ46HVDREA4F7ANCNFSM4KZJJDXA>
.
|
Cool. Thanks for confirming. Seems like now is a good time to pay down the technical debt--kind of funny, for a library about consumption, borrowing, etc. Honestly, I'm very happy doing this kind of work. |
move constructLogNormalIncomeProcess into model as method, fixes #538
constructLognormalIncomeProcessUnemployment(parameters)
is a fucntion defined in the ConsIndShockModel module.It is called from within the AgentType with
constructLognormalIncomeProcessUnemployment(self)
--as if it's a method.But it's not structured as a method. It's a standalone function.
This is one example of several other cases where something that is used much like a method is defined as a standalone function instead.
This is odd design.
The text was updated successfully, but these errors were encountered: