search

zedtaha / nardl

nardl:An R package to estimate the nonlinear cointegrating autoregressive distributed lag model
14 stars 9 forks source link

CUSUM plot code should move in a separate function #11

Closed JoFAM closed 6 years ago

JoFAM commented 6 years ago

Right now you have to rerun the entire model if you want to see the CUSUM plot. It would make sense to make this plot available without having to rerun the model and all tests, eg as a plot() function for a model object, or a plot() function for the test itself. This would still leave the option of using graph=TRUE in nardl(), but would give more access to the plot for the user of the package.

JoFAM commented 6 years ago

Made cusum a proper S3 method in commit 3aa6265a98178b3070cf46a16158be30e2822b53

JoFAM commented 6 years ago

Made sumsq a proper S3 method in commit bb4977b