Skip to content

Commit

Permalink
Fixing cmd errors
Browse files Browse the repository at this point in the history
  • Loading branch information
jacobmoore5067 committed Jan 22, 2024
1 parent 3eeac5b commit 6ec66df
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 14 deletions.
20 changes: 10 additions & 10 deletions R/hts_prep_data.R
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
#' @param threshold Threshold to define outliers. Default is 0.975.
#' @param remove_missing Whether to remove missing values from the summary.
#' Default is TRUE.
#' @param missing_value Missing value to remove. Default is 995.
#' @param missing_values Missing values to remove. Default is 995.
#' @param not_imputable Value representing 'Not imputable' to remove. Default
#' is -1.
#' @param strataname Name of strata name to bring in. Default is NULL.
Expand Down Expand Up @@ -106,15 +106,15 @@ hts_prep_data = function(summarize_var = NULL,

summarize_var = variables_dt[shared_name == summarize_var, variable]

}

for(i in 1:length(summarize_var)){

if(var_dt[,class(get(summarize_var[i]))] != 'integer'){
for(i in 1:length(summarize_var)){

message("Checkbox variables must have integer values")

stop()
if(var_dt[,class(get(summarize_var[i]))] != 'integer'){

message("Checkbox variables must have integer values")

stop()

}

}

Expand Down Expand Up @@ -215,7 +215,7 @@ hts_prep_data = function(summarize_var = NULL,

}

if(byvar_table[,class(get(var))] != 'integer'){
if(byvar_is_shared & byvar_table[,class(get(var))] != 'integer'){

message("Checkbox variables must have integer values")

Expand Down
4 changes: 2 additions & 2 deletions man/hts_prep_data.Rd

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

5 changes: 3 additions & 2 deletions man/hts_remove_missing_data.Rd

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

0 comments on commit 6ec66df

Please sign in to comment.