Skip to content

Commit

Permalink
Merge pull request #281 from OHDSI/issue_268
Browse files Browse the repository at this point in the history
name changed to summariseConceptIdCounts
  • Loading branch information
catalamarti authored Dec 22, 2024
2 parents db3a67e + 2c32313 commit f82be40
Show file tree
Hide file tree
Showing 11 changed files with 150 additions and 176 deletions.
4 changes: 2 additions & 2 deletions NAMESPACE
Original file line number Diff line number Diff line change
Expand Up @@ -9,18 +9,18 @@ export(plotInObservation)
export(plotObservationPeriod)
export(plotRecordCount)
export(settings)
export(summariseAllConceptCounts)
export(summariseClinicalRecords)
export(summariseConceptCounts)
export(summariseConceptIdCounts)
export(summariseConceptSetCounts)
export(summariseInObservation)
export(summariseMissingData)
export(summariseObservationPeriod)
export(summariseOmopSnapshot)
export(summariseRecordCount)
export(suppress)
export(tableAllConceptCounts)
export(tableClinicalRecords)
export(tableConceptIdCounts)
export(tableMissingData)
export(tableObservationPeriod)
export(tableOmopSnapshot)
Expand Down
1 change: 0 additions & 1 deletion R/plotRecordCount.R
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,6 @@ plotRecordCount <- function(result,
plot.margin = ggplot2::margin(t = 5, r = 5, b = 30, l = 5)
)


}else{
p <- result |>
visOmopResults::barPlot(x = "variable_name",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,10 +30,10 @@
#' con <- dbConnect(duckdb(), eunomiaDir())
#' cdm <- cdmFromCon(con = con, cdmSchema = "main", writeSchema = "main")
#'
#' summariseAllConceptCounts(cdm, "condition_occurrence")
#' summariseConceptIdCounts(cdm, "condition_occurrence")
#' }
#'
summariseAllConceptCounts <- function(cdm,
summariseConceptIdCounts <- function(cdm,
omopTableName,
countBy = "record",
year = FALSE,
Expand Down Expand Up @@ -108,7 +108,7 @@ summariseAllConceptCounts <- function(cdm,

omopgenerics::dropSourceTable(cdm = cdm, name = dplyr::starts_with(prefix))

return(result)
return(result)
}) |>
purrr::compact()

Expand Down
4 changes: 2 additions & 2 deletions R/tableAllConceptCounts.R → R/tableConceptIdCounts.R
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
#' Create a visual table from a summariseAllConceptCounts() result.
#' Create a visual table from a summariseConceptIdCounts() result.
#' @param result A summarised_result object.
#' @param type Type of formatting output table, either "gt" or "flextable".
#' @return A gt or flextable object with the summarised data.
#' @export
#'
#'
tableAllConceptCounts <- function(result,
tableConceptIdCounts <- function(result,
type = "gt") {
# initial checks
rlang::check_installed("visOmopResults")
Expand Down
4 changes: 2 additions & 2 deletions _pkgdown.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,8 @@ reference:
- contents:
- summariseConceptSetCounts
- plotConceptSetCounts
- summariseAllConceptCounts
- tableAllConceptCounts
- summariseConceptIdCounts
- tableConceptIdCounts
- subtitle: Mock Database
desc: Create a mock database to test the OmopSketch package
- contents:
Expand Down
Binary file modified man/figures/README-unnamed-chunk-3-1.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

19 changes: 0 additions & 19 deletions man/tableAllConceptCounts.Rd

This file was deleted.

19 changes: 19 additions & 0 deletions man/tableConceptIdCounts.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

142 changes: 0 additions & 142 deletions tests/testthat/test-summariseAllConceptCounts.R

This file was deleted.

Loading

0 comments on commit f82be40

Please sign in to comment.