Ale0430 / MiniBuoy-App

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

unify.datetime() changed #54

Open Ale0430 opened 7 months ago

Ale0430 commented 7 months ago

unify.datetime has been changed in Aleschanges branch to a mutate(...) variation also using lubridate parse, including the two most common datetime formas of MSR and hobo files (mdy_HMS, ymd_HMS), though more formats could be added if needed.

unify.datetime() is a function to standardise 12 hr clock to 24 hr clock. as originally written, on R version 4.3.2 the function removes entirely the datetime column for MSR files, causing the app to fail.

ACTIONS: I'm now running only R v4.2.3, can someone call my branch and run the app using R v4.3.2 to assess the app performance with the changes on datetime?

mcwimm commented 4 months ago

Tested with R 4.2.2 and 4.2.3 (latest version). For me the test data sets worked (but I assume they are in the correct time format).