stats19 icon indicating copy to clipboard operation
stats19 copied to clipboard

Midnight

Open wengraf opened this issue 1 year ago • 3 comments

In forming a datetime (by combining a date and a time), the package is making only a date where time is "00:00". For my current purposes, this doesn't matter (I'm just going to add 1 second to crashes at 00:00), but there appears to be a different approach in how R handles midnight datetimes (2024-5-10 00:00:00) and how STATS19 does.

wengraf avatar May 10 '24 13:05 wengraf

Oh.. Weird. Do you have an example bit of code that generates an incorrect result?

That will help demo when it's fixed :crossed_fingers:

Robinlovelace avatar May 10 '24 15:05 Robinlovelace

Hi @Robinlovelace

Here is what I mean...

`# Load packages if (!require("pacman")) install.packages("pacman") pacman::p_load(lubridate, tidyr, dplyr, sf, stats19, tidyverse)

crash.link <- "https://data.dft.gov.uk/road-accidents-safety-data/dft-road-casualty-statistics-collision-1979-latest-published-year.csv" min.year.of.study <- 2021 options(timeout=10000) downloader::download(crash.link, "crash.csv") crash.df <- read.csv("crash.csv") crash.df <- subset(crash.df, crash.df$accident_year >= min.year.of.study) crash.df <- stats19::format_collisions(crash.df) crash.df$datetime[crash.df$time == "00:00"]`

wengraf avatar May 10 '24 15:05 wengraf

Thanks, will take a look (and the other issue).

Keep them coming!

Robinlovelace avatar May 10 '24 15:05 Robinlovelace