{"comment":{"url":"https://api.github.com/repos/terrytangyuan/forecast/comments/12409712","html_url":"https://github.com/terrytangyuan/forecast/commit/028b063a9d629dd08c7755d2ac4ae88a49f221b0#commitcomment-12409712","id":12409712,"user":{"login":"robjhyndman","id":127518,"avatar_url":"https://avatars.githubusercontent.com/u/127518?v=3","gravatar_id":"","url":"https://api.github.com/users/robjhyndman","html_url":"https://github.com/robjhyndman","followers_url":"https://api.github.com/users/robjhyndman/followers","following_url":"https://api.github.com/users/robjhyndman/following{/other_user}","gists_url":"https://api.github.com/users/robjhyndman/gists{/gist_id}","starred_url":"https://api.github.com/users/robjhyndman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/robjhyndman/subscriptions","organizations_url":"https://api.github.com/users/robjhyndman/orgs","repos_url":"https://api.github.com/users/robjhyndman/repos","events_url":"https://api.github.com/users/robjhyndman/events{/privacy}","received_events_url":"https://api.github.com/users/robjhyndman/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"028b063a9d629dd08c7755d2ac4ae88a49f221b0","created_at":"2015-07-29T00:11:40Z","updated_at":"2015-07-29T00:11:40Z","body":"Remove the CV function. The results are incorrect for non-linear models. \r\n\r\nAs gam returns an object of class c(\"gam\",\"glm\",\"lm\"), I'm wondering if we can get away with forecast.gam being essentially a call to forecast.lm. As it is, there is too much duplicated code. We either need forecast.lm and forecast.gam calling a utility function to do most of the work, or forecast.gam being a wrapper to forecast.lm."}}