lhenneman / hyspdisp

4 stars 4 forks source link

negate_util.R #29

Closed schoolAccountMajaG closed 4 years ago

schoolAccountMajaG commented 5 years ago

What are your thoughts on file negate_util.R.? Should it be kept there? The function is used in Hyspdisp_fac_model_parallel for example.

schoolAccountMajaG commented 5 years ago

https://github.com/lhenneman/hyspdisp/blob/master/R/negate_util.R#L1

schoolAccountMajaG commented 5 years ago

While you’re free to arrange functions into files as you wish, the two extremes are bad: don’t put all functions into one file and don’t put each function into its own separate file. (It’s OK if some files only contain one function, particularly if the function is large or has a lot of documentation.). File names should be meaningful and end in .R.

http://r-pkgs.had.co.nz/r.html