Skip to content
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

do we want to check on a.args, s.args, etc separately; or just leave it up to auto.arima() etc to cope? #13

Closed
ellisp opened this issue Mar 19, 2016 · 2 comments

Comments

@ellisp
Copy link
Owner

ellisp commented Mar 19, 2016

Just putting this as an issue while cleaning the comment out of the code

@dashaub
Copy link
Collaborator

dashaub commented Mar 19, 2016

I was thinking we'd pass the arguments on to the auto.arima/ets/nnetar/stlm/tbats functions as-is. Those functions already do very robust error-handling, so any invalid arguments in those lists will give the user a very descriptive error anyway. Duplicating that validation wouldn't seem advisable unless there is a compelling reason.

@ellisp
Copy link
Owner Author

ellisp commented Mar 20, 2016

I agree.

@ellisp ellisp closed this as completed Mar 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants