search

trnnick / nnfor

31 stars 15 forks source link

Moving "forecast" from "Depends" to "Imports" #11

Open ngupta23 opened 4 years ago

ngupta23 commented 4 years ago

I was wondering if the forecast package could be moved from "Depends" to "Imports" in the DESCRIPTION file. I am building a package around nnfor and it is giving me an error, probably because some forecast function is being overwritten by another package in my "imports" list. I found an article that states that "imports" is safer than "depends", hence this request.

https://stackoverflow.com/questions/8637993/better-explanation-of-when-to-use-imports-depends

Thanks, Nikhil

ngupta23 commented 4 years ago

Hi Dr. Kourentzes,

I found one more reference to not including packages in the 'Depends' section here:

"Unless there is a good reason otherwise, you should always list packages in Imports not Depends. That’s because a good package is self-contained, and minimises changes to the global environment (including the search path)."

It would be great if this could be changed in the nnfor package.

Thanks!