> On Jan 17, 2024, at 3:46 AM, Josiah Parry <josiah.pa...@gmail.com> wrote:
>
> Hey folks! I've received note that a package of mine is failing tests on
> oldrel.
>
> Check results:
> https://www.r-project.org/nosvn/R.check/r-oldrel-windows-x86_64/arcgisutils-00check.html
>
> I think I've narrowed it down to the way that I've written the test which
> uses `as.POSIXct(Sys.Date(), tz = "UTC")`.
>
That's not where it fails - it fails in
today <- Sys.Date()
today_ms <- date_to_ms(today)
as.POSIXct(today_ms / 1000)
which is equivalent to
as.POSIXct(as.numeric(Sys.Date()) * 86400)
and that is only supported since R 4.3.0 - from NEWS:
as.POSIXct(<numeric>) and as.POSIXlt(.) (without specifying origin) now work.
so you have to add R >= 4.3.0 or use .POSIXct() instead.
I didn't check your other tests so you may have more of the same ...
Cheers,
Simon
> If I understand the R release changelog correctly, this behavior did not
> exist prior to R 4.3.0.
>
> as.POSIXlt(<Date>) now does apply a tz (time zone) argument, as does
>> as.POSIXct(); partly suggested by Roland Fuß on the R-devel mailing list.
>
>
> https://cran.r-project.org/doc/manuals/r-release/NEWS.html
>
> Does this check out? If so, would be more effective to modify the test to
> use a the character method of `as.POSIXct()`?
>
> [[alternative HTML version deleted]]
>
> ______________________________________________
> R-package-devel@r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-package-devel
>
______________________________________________
R-package-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel
