search

harrelfe / Hmisc

Harrell Miscellaneous
Other
205 stars 81 forks source link

minor README correction spelling - trunPOSIXt to truncPOSIXt ( with 'c' ) #86

Closed AndreMikulec closed 6 years ago

AndreMikulec commented 6 years ago

I hit an eror on this.

Written - trunPOSIXt

# https://cran.r-project.org/web/packages/Hmisc/NEWS
# changes in version 4.1-0 (2017-12-19)
#  * round.POSIXt,trunc.POSIXt: changed to roundPOSIXt, trunPOSIXt to avoid conflicts with R base

But

https://github.com/harrelfe/Hmisc/blob/master/R/dates.s
truncPOSIXt <- function(. . .

And

https://github.com/harrelfe/Hmisc/blob/master/NAMESPACE
export(. . . ,truncPOSIXt, . . .

So, the README needs the name corrected from(no 'c') trunPOSIXt to truncPOSIXt(with 'c')

NOTE: Thanks for trunc.POSIXt and truncPOSIXt

 Hmisc::trunc.POSIXt(., units='months')

is useful ( and package base does not do 'months'/'years' )

Thank You, Andre Mikulec Andre_Mikulec@Hotmail.com

harrelfe commented 6 years ago

Thanks for catching that