Skip to content

add params to getDBCovariateData - #312

Merged
ginberg merged 4 commits into
developfrom
issue292
Aug 29, 2025
Merged

add params to getDBCovariateData#312
ginberg merged 4 commits into
developfrom
issue292

Conversation

@ginberg

@ginberg ginberg commented Aug 20, 2025

Copy link
Copy Markdown
Collaborator

to be able to specify a custom schema and table for createCohortBasedCovariateSettings
see #292

@ginberg
ginberg marked this pull request as ready for review August 21, 2025 10:02
@ginberg
ginberg requested review from anthonysena and jreps August 21, 2025 10:03
@ginberg
ginberg merged commit 79d3b51 into develop Aug 29, 2025
8 checks passed
ginberg added a commit that referenced this pull request Sep 2, 2025
* fix setting of continous covariates in case of multiple settings and add a test

* update test

* Optimizing tidyCovariates for large data. Fixes #308 (#309)

Co-authored-by: ginberg <ginberg@gmail.com>
Co-authored-by: Admin_mschuemi <Admin_mschuemi@its.jnj.com>

* cran submission file

* add params to getDBCovariateData (#312)

* #292 add function to replace cohort schema and table

* #292 add params

* add test

* doc

* v3.11.0

* update news

---------

Co-authored-by: Martijn Schuemie <schuemie@ohdsi.org>
Co-authored-by: Admin_mschuemi <Admin_mschuemi@its.jnj.com>
ginberg added a commit that referenced this pull request Oct 28, 2025
* fix setting of continous covariates in case of multiple settings and add a test

* update test

* Optimizing tidyCovariates for large data. Fixes #308 (#309)

Co-authored-by: ginberg <ginberg@gmail.com>
Co-authored-by: Admin_mschuemi <Admin_mschuemi@its.jnj.com>

* cran submission file

* add params to getDBCovariateData (#312)

* #292 add function to replace cohort schema and table

* #292 add params

* add test

* doc

* v3.11.0

* update news

* Making removal of redundant covariates reproducible (with unit test). Fixes #315 (#316)

Co-authored-by: Admin_mschuemi <Admin_mschuemi@its.jnj.com>

* version update

* prepare release

* CRAN file

---------

Co-authored-by: Martijn Schuemie <schuemie@ohdsi.org>
Co-authored-by: Admin_mschuemi <Admin_mschuemi@its.jnj.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants