From 8505ea50fda3b82f231526bde9b986a065ff1b8d Mon Sep 17 00:00:00 2001
From: unknown <kirar.kartikeya1@gmail.com>
Date: Tue, 20 Feb 2024 16:53:56 +0530
Subject: [PATCH 1/5] updating document for tm_variable_browser

---
 R/tm_variable_browser.R    | 49 +++++++++++++++++++-------------------
 man/tm_variable_browser.Rd | 34 +++++++++++++-------------
 2 files changed, 41 insertions(+), 42 deletions(-)

diff --git a/R/tm_variable_browser.R b/R/tm_variable_browser.R
index 71730bff8..bac172d19 100644
--- a/R/tm_variable_browser.R
+++ b/R/tm_variable_browser.R
@@ -1,37 +1,36 @@
-#' Variable Browser Teal Module
+#' Variable browser `teal` module
 #'
-#' The variable browser provides a table with variable names and labels and a
-#' plot that visualizes the content of a particular variable.
-#' specifically designed for use with `data.frames`.
+#' Module provides provides a detailed summary and visualization of variable distributions
+#' for `data.frame` objects, with interactive features to customize analysis.
 #'
-#' @details Numeric columns with fewer than 30 distinct values can be treated as either factors
-#' or numbers with a checkbox allowing users to switch how they are treated (if < 6 unique values
+#' @details Numeric columns with fewer than 30 distinct values can be treated as either categorical
+#' or numeric with a checkbox allowing users to switch how they are treated (if < 6 unique values
 #' then the default is categorical, otherwise it is numeric).
 #'
 #' @inheritParams teal::module
 #' @inheritParams shared_params
-#' @param parent_dataname (`character(1)`) If this `dataname` exists in `datasets_selected`
-#'   then an extra checkbox will be shown to allow users to not show variables in other datasets
-#'   which exist in this `dataname`.
-#'   This is typically used to remove `ADSL` columns in `CDISC` data. In non `CDISC` data this
-#'   can be ignored. Defaults to `"ADSL"`.
-#' @param datasets_selected (`character`) A vector of datasets which should be
-#'   shown and in what order. Names in the vector have to correspond with datasets names.
-#'   If vector of length zero (default) then all datasets are shown.
-#'   Note: Only datasets of the `data.frame` class are compatible; using other types will cause an error.
+#' @param parent_dataname (`character(1)`) string specifying a parent dataset.
+#' If it exists in `datasets_selected`then an extra checkbox will be shown to
+#' allow users to not show variables in other datasets which exist in this `dataname`.
+#' This is typically used to remove `ADSL` columns in `CDISC` data.
+#' In non `CDISC` data this can be ignored. Defaults to `"ADSL"`.
+#' @param datasets_selected (`character`) vector of datasets which should be
+#' shown, in order. Names must correspond with datasets names.
+#' If vector of length zero (default) then all datasets are shown.
+#' Note: Only `data.frame` objects are compatible; using other types will cause an error.
 #'
 #' @aliases
-#'   tm_variable_browser_ui,
-#'   tm_variable_browser_srv,
-#'   tm_variable_browser,
-#'   variable_browser_ui,
-#'   variable_browser_srv,
-#'   variable_browser
-#'
+#' tm_variable_browser_ui,
+#' tm_variable_browser_srv,
+#' tm_variable_browser,
+#' variable_browser_ui,
+#' variable_browser_srv,
+#' variable_browser
 #'
 #' @export
 #'
 #' @examples
+#' library(teal.widgets)
 #'
 #' data <- teal_data()
 #' data <- within(data, {
@@ -42,12 +41,12 @@
 #' datanames(data) <- datanames
 #' join_keys(data) <- default_cdisc_join_keys[datanames]
 #'
-#' app <- teal::init(
+#' app <- init(
 #'   data = data,
 #'   modules(
-#'     teal.modules.general::tm_variable_browser(
+#'     tm_variable_browser(
 #'       label = "Variable browser",
-#'       ggplot2_args = teal.widgets::ggplot2_args(
+#'       ggplot2_args = ggplot2_args(
 #'         labs = list(subtitle = "Plot generated by Variable Browser Module")
 #'       ),
 #'     )
diff --git a/man/tm_variable_browser.Rd b/man/tm_variable_browser.Rd
index 5b12bd007..ce16fa8a7 100644
--- a/man/tm_variable_browser.Rd
+++ b/man/tm_variable_browser.Rd
@@ -8,7 +8,7 @@
 \alias{variable_browser_ui,}
 \alias{variable_browser_srv,}
 \alias{variable_browser}
-\title{Variable Browser Teal Module}
+\title{Variable browser \code{teal} module}
 \usage{
 tm_variable_browser(
   label = "Variable Browser",
@@ -23,16 +23,16 @@ tm_variable_browser(
 \item{label}{(\code{character(1)}) Label shown in the navigation item for the module or module group.
 For \code{modules()} defaults to \code{"root"}. See \code{Details}.}
 
-\item{datasets_selected}{(\code{character}) A vector of datasets which should be
-shown and in what order. Names in the vector have to correspond with datasets names.
+\item{datasets_selected}{(\code{character}) vector of datasets which should be
+shown, in order. Names must correspond with datasets names.
 If vector of length zero (default) then all datasets are shown.
-Note: Only datasets of the \code{data.frame} class are compatible; using other types will cause an error.}
+Note: Only \code{data.frame} objects are compatible; using other types will cause an error.}
 
-\item{parent_dataname}{(\code{character(1)}) If this \code{dataname} exists in \code{datasets_selected}
-then an extra checkbox will be shown to allow users to not show variables in other datasets
-which exist in this \code{dataname}.
-This is typically used to remove \code{ADSL} columns in \code{CDISC} data. In non \code{CDISC} data this
-can be ignored. Defaults to \code{"ADSL"}.}
+\item{parent_dataname}{(\code{character(1)}) string specifying a parent dataset.
+If it exists in \code{datasets_selected}then an extra checkbox will be shown to
+allow users to not show variables in other datasets which exist in this \code{dataname}.
+This is typically used to remove \code{ADSL} columns in \code{CDISC} data.
+In non \code{CDISC} data this can be ignored. Defaults to \code{"ADSL"}.}
 
 \item{pre_output}{(\code{shiny.tag}, optional)\cr
 with text placed before the output to put the output into context. For example a title.}
@@ -47,16 +47,16 @@ The argument is merged with options variable \code{teal.ggplot2_args} and defaul
 For more details see the vignette: \code{vignette("custom-ggplot2-arguments", package = "teal.widgets")}}
 }
 \description{
-The variable browser provides a table with variable names and labels and a
-plot that visualizes the content of a particular variable.
-specifically designed for use with \code{data.frames}.
+Module provides provides a detailed summary and visualization of variable distributions
+for \code{data.frame} objects, with interactive features to customize analysis.
 }
 \details{
-Numeric columns with fewer than 30 distinct values can be treated as either factors
-or numbers with a checkbox allowing users to switch how they are treated (if < 6 unique values
+Numeric columns with fewer than 30 distinct values can be treated as either categorical
+or numeric with a checkbox allowing users to switch how they are treated (if < 6 unique values
 then the default is categorical, otherwise it is numeric).
 }
 \examples{
+library(teal.widgets)
 
 data <- teal_data()
 data <- within(data, {
@@ -67,12 +67,12 @@ datanames <- c("ADSL", "ADTTE")
 datanames(data) <- datanames
 join_keys(data) <- default_cdisc_join_keys[datanames]
 
-app <- teal::init(
+app <- init(
   data = data,
   modules(
-    teal.modules.general::tm_variable_browser(
+    tm_variable_browser(
       label = "Variable browser",
-      ggplot2_args = teal.widgets::ggplot2_args(
+      ggplot2_args = ggplot2_args(
         labs = list(subtitle = "Plot generated by Variable Browser Module")
       ),
     )

From e25650e3c2d1485eb52056933c550e67aca7f929 Mon Sep 17 00:00:00 2001
From: unknown <kirar.kartikeya1@gmail.com>
Date: Wed, 21 Feb 2024 14:19:13 +0530
Subject: [PATCH 2/5] fixing the doc

---
 R/tm_variable_browser.R    | 106 ++++++++++---------------------------
 man/create_sparklines.Rd   |  22 +-------
 man/tm_variable_browser.Rd |   4 ++
 man/validate_input.Rd      |   2 +-
 man/var_missings_info.Rd   |   4 +-
 5 files changed, 37 insertions(+), 101 deletions(-)

diff --git a/R/tm_variable_browser.R b/R/tm_variable_browser.R
index bac172d19..ec76be68e 100644
--- a/R/tm_variable_browser.R
+++ b/R/tm_variable_browser.R
@@ -27,7 +27,7 @@
 #' variable_browser_srv,
 #' variable_browser
 #'
-#' @export
+#' @return Object of class `teal_module` to be used in `teal` applications.
 #'
 #' @examples
 #' library(teal.widgets)
@@ -55,6 +55,9 @@
 #' if (interactive()) {
 #'   shinyApp(app$ui, app$server)
 #' }
+#'
+#' @export
+#'
 tm_variable_browser <- function(label = "Variable Browser",
                                 datasets_selected = character(0),
                                 parent_dataname = "ADSL",
@@ -94,7 +97,8 @@ tm_variable_browser <- function(label = "Variable Browser",
   )
 }
 
-# ui function
+# UI function for the variable browser module.
+# It includes dynamic visualization and analysis options for dataset variables.
 ui_variable_browser <- function(id,
                                 pre_output = NULL,
                                 post_output = NULL) {
@@ -164,6 +168,8 @@ ui_variable_browser <- function(id,
   )
 }
 
+# Server function for the variable browser module.
+# Generates interactive plots and variable summaries based on selected datasets and processing user inputs and data.
 srv_variable_browser <- function(id,
                                  data,
                                  reporter,
@@ -511,12 +517,12 @@ srv_variable_browser <- function(id,
 
 #' Summarizes missings occurrence
 #'
-#' Summarizes missings occurrence in vector
+#' Summarizes missings occurrence in vector.
 #' @param x vector of any type and length
-#' @return text describing \code{NA} occurrence.
+#' @return Text describing \code{NA} occurrence.
 #' @keywords internal
 var_missings_info <- function(x) {
-  return(sprintf("%s [%s%%]", sum(is.na(x)), round(mean(is.na(x) * 100), 2)))
+  sprintf("%s [%s%%]", sum(is.na(x)), round(mean(is.na(x) * 100), 2))
 }
 
 #' S3 generic for \code{sparkline} widget HTML
@@ -529,6 +535,8 @@ var_missings_info <- function(x) {
 #' @param width \code{numeric} the width of the \code{sparkline} widget (pixels)
 #' @param ... \code{list} additional options passed to bar plots of \code{jquery.sparkline};  see
 #' \href{https://omnipotent.net/jquery.sparkline/#common}{\code{jquery.sparkline docs}}
+#' @param bar_spacing \code{numeric} the spacing between the bars (in pixels)
+#' @param bar_width \code{numeric} the width of the bars (in pixels)
 #'
 #' @return character variable containing the HTML code of the \code{sparkline} HTML widget
 #' @keywords internal
@@ -540,25 +548,13 @@ create_sparklines <- function(arr, width = 150, ...) {
   UseMethod("create_sparklines")
 }
 
-#' Default method for \code{\link{create_sparklines}}
-#'
-#'
 #' @export
-#' @keywords internal
 #' @rdname create_sparklines
 create_sparklines.default <- function(arr, width = 150, ...) {
-  return(as.character(tags$code("unsupported variable type", class = "text-blue")))
+  as.character(tags$code("unsupported variable type", class = "text-blue"))
 }
 
-#' Generates the HTML code for the \code{sparkline} widget
-#'
-#' @param bar_spacing \code{numeric} the spacing between the bars (in pixels)
-#' @param bar_width \code{numeric} the width of the bars (in pixels)
-#'
-#' @return \code{character} with HTML code for the \code{sparkline} widget
-#'
 #' @export
-#' @keywords internal
 #' @rdname create_sparklines
 create_sparklines.Date <- function(arr, width = 150, bar_spacing = 5, bar_width = 20, ...) {
   arr_num <- as.numeric(arr)
@@ -589,16 +585,7 @@ create_sparklines.Date <- function(arr, width = 150, bar_spacing = 5, bar_width
   )
 }
 
-#' Generates the HTML code for the \code{sparkline} widget
-#'
-#'
-#' @param bar_spacing \code{numeric} the spacing between the bars (in pixels)
-#' @param bar_width \code{numeric} the width of the bars (in pixels)
-#'
-#' @return \code{character} with HTML code for the \code{sparkline} widget
-#'
 #' @export
-#' @keywords internal
 #' @rdname create_sparklines
 create_sparklines.POSIXct <- function(arr, width = 150, bar_spacing = 5, bar_width = 20, ...) {
   arr_num <- as.numeric(arr)
@@ -629,16 +616,7 @@ create_sparklines.POSIXct <- function(arr, width = 150, bar_spacing = 5, bar_wid
   )
 }
 
-#' Generates the HTML code for the \code{sparkline} widget
-#'
-#'
-#' @param bar_spacing \code{numeric} the spacing between the bars (in pixels)
-#' @param bar_width \code{numeric} the width of the bars (in pixels)
-#'
-#' @return \code{character} with HTML code for the \code{sparkline} widget
-#'
 #' @export
-#' @keywords internal
 #' @rdname create_sparklines
 create_sparklines.POSIXlt <- function(arr, width = 150, bar_spacing = 5, bar_width = 20, ...) {
   arr_num <- as.numeric(arr)
@@ -669,46 +647,19 @@ create_sparklines.POSIXlt <- function(arr, width = 150, bar_spacing = 5, bar_wid
   )
 }
 
-
-#' Generates the HTML code for the \code{sparkline} widget
-#'
-#' Coerces a character vector to factor and delegates to the \code{create_sparklines.factor}
-#'
-#'
-#' @return \code{character} with HTML code for the \code{sparkline} widget
-#'
 #' @export
-#' @keywords internal
 #' @rdname create_sparklines
 create_sparklines.character <- function(arr, ...) {
-  return(create_sparklines(as.factor(arr)))
+  create_sparklines(as.factor(arr))
 }
 
-
-#' Generates the HTML code for the \code{sparkline} widget
-#'
-#' Coerces logical vector to factor and delegates to the \code{create_sparklines.factor}
-#'
-#'
-#' @return \code{character} with HTML code for the \code{sparkline} widget
-#'
 #' @export
-#' @keywords internal
 #' @rdname create_sparklines
 create_sparklines.logical <- function(arr, ...) {
-  return(create_sparklines(as.factor(arr)))
+  create_sparklines(as.factor(arr))
 }
 
-
-#' Generates the \code{sparkline} HTML code
-#'
-#' @param bar_spacing \code{numeric} spacing between the bars (in pixels)
-#' @param bar_width \code{numeric} width of the bars (in pixels)
-#'
-#' @return \code{character} with HTML code for the \code{sparkline} widget
-#'
 #' @export
-#' @keywords internal
 #' @rdname create_sparklines
 create_sparklines.factor <- function(arr, width = 150, bar_spacing = 5, bar_width = 20, ...) {
   decreasing_order <- TRUE
@@ -742,13 +693,7 @@ create_sparklines.factor <- function(arr, width = 150, bar_spacing = 5, bar_widt
   )
 }
 
-#' Generates the \code{sparkline} HTML code
-#'
-#'
-#' @return \code{character} with HTML code for the \code{sparkline} widget
-#'
 #' @export
-#' @keywords internal
 #' @rdname create_sparklines
 create_sparklines.numeric <- function(arr, width = 150, ...) {
   if (any(is.infinite(arr))) {
@@ -759,8 +704,7 @@ create_sparklines.numeric <- function(arr, width = 150, ...) {
   }
 
   arr <- arr[!is.na(arr)]
-  res <- sparkline::spk_chr(unname(arr), type = "box", width = width, ...)
-  return(res)
+  sparkline::spk_chr(unname(arr), type = "box", width = width, ...)
 }
 
 #' Summarizes variable
@@ -768,6 +712,7 @@ create_sparklines.numeric <- function(arr, width = 150, ...) {
 #' Creates html summary with statistics relevant to data type. For numeric values it returns central
 #' tendency measures, for factor returns level counts, for Date  date range, for other just
 #' number of levels.
+#'
 #' @param x vector of any type
 #' @param numeric_as_factor \code{logical} should the numeric variable be treated as a factor
 #' @param dt_rows \code{numeric} current/latest `DT` page length
@@ -854,10 +799,10 @@ var_summary_table <- function(x, numeric_as_factor, dt_rows, outlier_definition)
   }
 }
 
-
 #' Plot variable
 #'
 #' Creates summary plot with statistics relevant to data type.
+#'
 #' @inheritParams shared_params
 #' @param var vector of any type to be plotted. For numeric variables it produces histogram with
 #' density line, for factors it creates frequency plot
@@ -1024,6 +969,8 @@ plot_var_summary <- function(var,
   plot_main
 }
 
+#' @noRd
+#' @keywords internal
 is_num_var_short <- function(.unique_records_for_factor, input, data_for_analysis) {
   length(unique(data_for_analysis()$data)) < .unique_records_for_factor && !is.null(input$numeric_as_factor)
 }
@@ -1034,7 +981,7 @@ is_num_var_short <- function(.unique_records_for_factor, input, data_for_analysi
 #' @param plot_var (`list`) list of a data frame and an array of variable names
 #' @param data (`tdata`) the datasets passed to the module
 #'
-#' @returns `logical` TRUE if validations pass; a Shiny validation error otherwise
+#' @returns `logical` TRUE if validations pass; a `shiny` validation error otherwise
 #' @keywords internal
 validate_input <- function(input, plot_var, data) {
   reactive({
@@ -1051,6 +998,8 @@ validate_input <- function(input, plot_var, data) {
   })
 }
 
+#' @noRd
+#' @keywords internal
 get_plotted_data <- function(input, plot_var, data) {
   dataset_name <- input$tabset_panel
   varname <- plot_var$variable[[dataset_name]]
@@ -1083,7 +1032,6 @@ render_tabset_panel_content <- function(datanames, parent_dataname, output, data
 
 #' Renders a single tab in the left-hand side tabset panel
 #'
-#' @description
 #' Renders a single tab in the left-hand side tabset panel. The rendered tab contains
 #' information about one dataset out of many presented in the module.
 #'
@@ -1130,7 +1078,6 @@ render_tab_header <- function(dataset_name, output, data) {
 
 #' Renders the table for a single dataset in the left-hand side tabset panel
 #'
-#' @description
 #' The table contains column names, column labels,
 #' small summary about NA values and `sparkline` (if appropriate).
 #'
@@ -1249,7 +1196,6 @@ render_tab_table <- function(dataset_name, parent_dataname, output, data, input,
 
 #' Creates observers updating the currently selected column
 #'
-#' @description
 #' The created observers update the column currently selected in the left-hand side
 #' tabset panel.
 #'
@@ -1269,6 +1215,8 @@ establish_updating_selection <- function(datanames, input, plot_var, columns_nam
   })
 }
 
+#' @noRd
+#' @keywords internal
 get_bin_width <- function(x_vec, scaling_factor = 2) {
   x_vec <- x_vec[!is.na(x_vec)]
   qntls <- stats::quantile(x_vec, probs = c(0.1, 0.25, 0.75, 0.9), type = 2)
@@ -1280,6 +1228,8 @@ get_bin_width <- function(x_vec, scaling_factor = 2) {
   if (isTRUE(x_span / binwidth >= 2)) binwidth else x_span / 2
 }
 
+#' @noRd
+#' @keywords internal
 custom_sparkline_formatter <- function(labels, counts) {
   htmlwidgets::JS(
     sprintf(
diff --git a/man/create_sparklines.Rd b/man/create_sparklines.Rd
index 2d7be08ce..18cc1080b 100644
--- a/man/create_sparklines.Rd
+++ b/man/create_sparklines.Rd
@@ -38,34 +38,16 @@ create_sparklines(arr, width = 150, ...)
 \item{...}{\code{list} additional options passed to bar plots of \code{jquery.sparkline};  see
 \href{https://omnipotent.net/jquery.sparkline/#common}{\code{jquery.sparkline docs}}}
 
-\item{bar_spacing}{\code{numeric} spacing between the bars (in pixels)}
+\item{bar_spacing}{\code{numeric} the spacing between the bars (in pixels)}
 
-\item{bar_width}{\code{numeric} width of the bars (in pixels)}
+\item{bar_width}{\code{numeric} the width of the bars (in pixels)}
 }
 \value{
 character variable containing the HTML code of the \code{sparkline} HTML widget
-
-\code{character} with HTML code for the \code{sparkline} widget
-
-\code{character} with HTML code for the \code{sparkline} widget
-
-\code{character} with HTML code for the \code{sparkline} widget
-
-\code{character} with HTML code for the \code{sparkline} widget
-
-\code{character} with HTML code for the \code{sparkline} widget
-
-\code{character} with HTML code for the \code{sparkline} widget
-
-\code{character} with HTML code for the \code{sparkline} widget
 }
 \description{
 Generates the \code{sparkline} HTML code corresponding to the input array.
 For numeric variables creates a box plot, for character and factors - bar plot.
 Produces an empty string for variables of other types.
-
-Coerces a character vector to factor and delegates to the \code{create_sparklines.factor}
-
-Coerces logical vector to factor and delegates to the \code{create_sparklines.factor}
 }
 \keyword{internal}
diff --git a/man/tm_variable_browser.Rd b/man/tm_variable_browser.Rd
index ce16fa8a7..555055cc5 100644
--- a/man/tm_variable_browser.Rd
+++ b/man/tm_variable_browser.Rd
@@ -46,6 +46,9 @@ The argument is merged with options variable \code{teal.ggplot2_args} and defaul
 
 For more details see the vignette: \code{vignette("custom-ggplot2-arguments", package = "teal.widgets")}}
 }
+\value{
+Object of class \code{teal_module} to be used in \code{teal} applications.
+}
 \description{
 Module provides provides a detailed summary and visualization of variable distributions
 for \code{data.frame} objects, with interactive features to customize analysis.
@@ -81,4 +84,5 @@ app <- init(
 if (interactive()) {
   shinyApp(app$ui, app$server)
 }
+
 }
diff --git a/man/validate_input.Rd b/man/validate_input.Rd
index d1872efcf..37ab1310d 100644
--- a/man/validate_input.Rd
+++ b/man/validate_input.Rd
@@ -14,7 +14,7 @@ validate_input(input, plot_var, data)
 \item{data}{(\code{tdata}) the datasets passed to the module}
 }
 \value{
-\code{logical} TRUE if validations pass; a Shiny validation error otherwise
+\code{logical} TRUE if validations pass; a \code{shiny} validation error otherwise
 }
 \description{
 Validates the variable browser inputs
diff --git a/man/var_missings_info.Rd b/man/var_missings_info.Rd
index c814a510c..70426bf8d 100644
--- a/man/var_missings_info.Rd
+++ b/man/var_missings_info.Rd
@@ -10,9 +10,9 @@ var_missings_info(x)
 \item{x}{vector of any type and length}
 }
 \value{
-text describing \code{NA} occurrence.
+Text describing \code{NA} occurrence.
 }
 \description{
-Summarizes missings occurrence in vector
+Summarizes missings occurrence in vector.
 }
 \keyword{internal}

From 8d91cec510b085b1f2dda693aa229df2def824f8 Mon Sep 17 00:00:00 2001
From: unknown <kirar.kartikeya1@gmail.com>
Date: Fri, 23 Feb 2024 15:01:17 +0530
Subject: [PATCH 3/5] reverting catpital case and adding tags

---
 R/tm_variable_browser.R    | 8 +++++---
 man/tm_variable_browser.Rd | 2 +-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/R/tm_variable_browser.R b/R/tm_variable_browser.R
index 4d962f94d..4e349ea96 100644
--- a/R/tm_variable_browser.R
+++ b/R/tm_variable_browser.R
@@ -38,7 +38,7 @@
 #'   )
 #' )
 #'
-#' # General data example
+#' # general data example
 #' data <- teal_data()
 #' data <- within(data, {
 #'   iris <- iris
@@ -116,7 +116,8 @@ tm_variable_browser <- function(label = "Variable Browser",
 }
 
 # UI function for the variable browser module.
-# It includes dynamic visualization and analysis options for dataset variables.
+#' @noRd
+#' @keywords internal
 ui_variable_browser <- function(id,
                                 pre_output = NULL,
                                 post_output = NULL) {
@@ -187,7 +188,8 @@ ui_variable_browser <- function(id,
 }
 
 # Server function for the variable browser module.
-# Generates interactive plots and variable summaries based on selected datasets and processing user inputs and data.
+#' @noRd
+#' @keywords internal
 srv_variable_browser <- function(id,
                                  data,
                                  reporter,
diff --git a/man/tm_variable_browser.Rd b/man/tm_variable_browser.Rd
index 3475bf7f3..df04c9e79 100644
--- a/man/tm_variable_browser.Rd
+++ b/man/tm_variable_browser.Rd
@@ -66,7 +66,7 @@ tm_variable_browser_module <- tm_variable_browser(
   )
 )
 
-# General data example
+# general data example
 data <- teal_data()
 data <- within(data, {
   iris <- iris

From 5b8b2f0f97b7d7bc25d1f0e361714446e066cb8b Mon Sep 17 00:00:00 2001
From: unknown <kirar.kartikeya1@gmail.com>
Date: Mon, 26 Feb 2024 13:05:33 +0530
Subject: [PATCH 4/5] use statistical term

---
 R/tm_variable_browser.R    | 6 +++---
 man/tm_variable_browser.Rd | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/R/tm_variable_browser.R b/R/tm_variable_browser.R
index 4e349ea96..85ade3695 100644
--- a/R/tm_variable_browser.R
+++ b/R/tm_variable_browser.R
@@ -3,9 +3,9 @@
 #' Module provides provides a detailed summary and visualization of variable distributions
 #' for `data.frame` objects, with interactive features to customize analysis.
 #'
-#' Numeric columns with fewer than 30 distinct values can be treated as either categorical
-#' or numeric with a checkbox allowing users to switch how they are treated (if < 6 unique values
-#' then the default is categorical, otherwise it is numeric).
+#' Numeric columns with fewer than 30 distinct values can be treated as either discrete
+#' or continuous with a checkbox allowing users to switch how they are treated(if < 6 unique values
+#' then the default is discrete, otherwise it is continuous).
 #'
 #' @inheritParams teal::module
 #' @inheritParams shared_params
diff --git a/man/tm_variable_browser.Rd b/man/tm_variable_browser.Rd
index df04c9e79..088b4b85e 100644
--- a/man/tm_variable_browser.Rd
+++ b/man/tm_variable_browser.Rd
@@ -51,9 +51,9 @@ Module provides provides a detailed summary and visualization of variable distri
 for \code{data.frame} objects, with interactive features to customize analysis.
 }
 \details{
-Numeric columns with fewer than 30 distinct values can be treated as either categorical
-or numeric with a checkbox allowing users to switch how they are treated (if < 6 unique values
-then the default is categorical, otherwise it is numeric).
+Numeric columns with fewer than 30 distinct values can be treated as either discrete
+or continuous with a checkbox allowing users to switch how they are treated(if < 6 unique values
+then the default is discrete, otherwise it is continuous).
 }
 \examples{
 library(teal.widgets)

From a6099c6c68636be545ad5992fe6fc67fcdf72951 Mon Sep 17 00:00:00 2001
From: unknown <kirar.kartikeya1@gmail.com>
Date: Mon, 26 Feb 2024 15:15:36 +0530
Subject: [PATCH 5/5] removing aliases

---
 R/tm_variable_browser.R    | 8 --------
 man/tm_variable_browser.Rd | 6 ------
 2 files changed, 14 deletions(-)

diff --git a/R/tm_variable_browser.R b/R/tm_variable_browser.R
index 85ade3695..2cbc949a8 100644
--- a/R/tm_variable_browser.R
+++ b/R/tm_variable_browser.R
@@ -19,14 +19,6 @@
 #' If vector of length zero (default) then all datasets are shown.
 #' Note: Only `data.frame` objects are compatible; using other types will cause an error.
 #'
-#' @aliases
-#' tm_variable_browser_ui,
-#' tm_variable_browser_srv,
-#' tm_variable_browser,
-#' variable_browser_ui,
-#' variable_browser_srv,
-#' variable_browser
-#'
 #' @examples
 #' library(teal.widgets)
 #'
diff --git a/man/tm_variable_browser.Rd b/man/tm_variable_browser.Rd
index 088b4b85e..9ea518a85 100644
--- a/man/tm_variable_browser.Rd
+++ b/man/tm_variable_browser.Rd
@@ -2,12 +2,6 @@
 % Please edit documentation in R/tm_variable_browser.R
 \name{tm_variable_browser}
 \alias{tm_variable_browser}
-\alias{tm_variable_browser_ui,}
-\alias{tm_variable_browser_srv,}
-\alias{tm_variable_browser,}
-\alias{variable_browser_ui,}
-\alias{variable_browser_srv,}
-\alias{variable_browser}
 \title{Variable browser \code{teal} module}
 \usage{
 tm_variable_browser(