-
-
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
makeEulerErrorFunc never used #892
Comments
I can be rewritten with calcExpectations, but I don't think it should be
deleted. It has a trivial `notImplementedError` definition in many models
because those classes inherit from IndShockConsumerType, the only class
it's properly defined for. Maybe it's in KinkedRType as well? The base
definition in ConsIndShockModel would be incorrect for any other model.
This is one of those features that could eventually have a "magic" version
once everything is equation-ified.
…On Thu, Dec 24, 2020 at 12:46 PM Sebastian Benthall < ***@***.***> wrote:
makeEulerErrorFunc is defined for many models, but never called or tested.
https://github.com/econ-ark/HARK/search?q=makeEulerErrorFunc&type=code
Should this code be removed?
Alternatively, can it be rewritten more simply using calcExpectations?
See #625 <#625>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#892>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADKRAFKAOMMS7AM34S7Y6ILSWN473ANCNFSM4VINHSFQ>
.
|
I see now that it contains redundant information with the solution code. I'm trying to figure out how to consolidate the solution code use calcExpectations. See #893 I would rather rewrite the solution code to use a well-contained euler function than remove the well-contained euler function. But I think the redundant information should be removed. I think we are closer to a 'magic' version than you might think, but the first step is to tighten what's already there. |
Add a note in the documentation explaining why this method is here. |
documented the role of makeEulerErrorFunc. Fixes #892
makeEulerErrorFunc
is defined for many models, but never called or tested.https://github.com/econ-ark/HARK/search?q=makeEulerErrorFunc&type=code
Should this code be removed?
Alternatively, can it be rewritten more simply using
calcExpectations
? See #625The text was updated successfully, but these errors were encountered: