demographicsCohort ignores ageGroup names

[catalamarti]

cdm <- CohortConstructor::mockCohortConstructor()
#> Note: method with signature 'DBIConnection#Id' chosen for function 'dbExistsTable',
#>  target signature 'duckdb_connection#Id'.
#>  "duckdb_connection#ANY" would also be valid

demographics <- CohortConstructor::demographicsCohort(
  cdm = cdm, 
  name = "demographics", 
  ageRange = list("name1" = c(0, 19), "name2" = c(20, Inf))
)
#> ! cohort columns will be reordered to match the expected order:
#>   cohort_definition_id, subject_id, cohort_start_date, and cohort_end_date.
#> ℹ Building new trimmed cohort
#> Adding demographics information
#> Creating initial cohort
#> Trim age
#> ✔ Cohort trimmed

CohortConstructor::settings(demographics)
#> # A tibble: 2 × 3
#>   cohort_definition_id cohort_name    age_range
#>                  <int> <chr>          <chr>    
#> 1                    1 demographics_1 0_19     
#> 2                    2 demographics_2 20_Inf

^{Created on 2024-10-17 with reprex v2.0.2}

[edward-burn]

So would you want two settings?

• age_name (name of list, or age_range as is now if no name provided)
• age_range (as is)

[catalamarti]

yep happy with that