search

Ale0430 / MiniBuoy-App

Its another app
GNU General Public License v3.0
0 stars 0 forks source link

Removing packages #16

Closed cailadd90 closed 1 year ago

cailadd90 commented 1 year ago

Removed 4 redundant packages (timetk, kknn, kernlab, and caret).

summarise_by_time() function from the timetk package was being used to simultaneously aggregate dates and summarise the data. Now switched to using lubridate's floor_date() and summarise() that yeild the same results.

Note, runsd() of the caTools could be replaced with rollapply() in the zoo package (also being used for na.approx()). However, the function was running very slowly, so decided to stick with runsd().

mcwimm commented 1 year ago

Perfect! Will be merged after PR #13 is merged (already rebased).