Remove magritte dependency

[nmercadeb]

All date operations work with clock functions except for this: https://github.com/OHDSI/CohortConstructor/blob/c82dbcc23e039a89450af71591db7fdec2d058e2/R/trimDemographics.R#L200C9-L206C11

I am not sure if it's just that I don't know how to do it, or if it is a bug? @edward-burn

library(duckdb)
#> Loading required package: DBI
library(here)
#> here() starts at C:/Users/nuriamb/AppData/Local/Temp/RtmpgpNqUU/reprex-886449ad49d3-loyal-rat
library(CDMConnector)
library(dplyr)
#> 
#> Attaching package: 'dplyr'
#> The following objects are masked from 'package:stats':
#> 
#>     filter, lag
#> The following objects are masked from 'package:base':
#> 
#>     intersect, setdiff, setequal, union
library(DBI)
library(clock)
#> 
#> Attaching package: 'clock'
#> The following object is masked from 'package:CDMConnector':
#> 
#>     as_date

Sys.setenv(EUNOMIA_DATA_FOLDER = here())

con <- dbConnect(duckdb(), dbdir = eunomia_dir())
#> Downloading GiBleed
#> 
#> Download completed!
#> Creating CDM database
#> C:/Users/nuriamb/AppData/Local/Temp/RtmpgpNqUU/reprex-886449ad49d3-loyal-rat/GiBleed_5.3.zip

tableLocal <- tibble(date_col = as.Date(c("2000-01-01", "2000-01-02")), num_col = 1:2L)
tableLocal |>
  mutate(
    new_date_1 = add_days(.data$date_col, .data$num_col),
    new_date_2 = add_days(.data$date_col, 1:2)
  )
#> # A tibble: 2 × 4
#>   date_col   num_col new_date_1 new_date_2
#>   <date>       <int> <date>     <date>    
#> 1 2000-01-01       1 2000-01-02 2000-01-02
#> 2 2000-01-02       2 2000-01-04 2000-01-04

dbWriteTable(con, "table", tableLocal)

tableSQL <- tbl(con, sql("SELECT *FROM main.table")) 

tableSQL |>
  mutate(
    new_date_1 = add_days(.data$date_col, .data$num_col),
    new_date_2 = add_days(.data$date_col, 1:2)
  )
#> Error in `collect()`:
#> ! Failed to collect lazy table.
#> Caused by error:
#> ! rapi_execute: Failed to run query
#> Error: Conversion Error: Could not convert string 'num_col day' to INTERVAL

^{Created on 2024-10-23 with reprex v2.1.1}

[edward-burn]

@nmercadeb this needs to stay until the next release of dbplyr, but we could leave that and merge all the clean up you have done (and then when time comes it will be easy to remove the dependency on magrittr)