Skip to content

Commit

Permalink
add comments
Browse files Browse the repository at this point in the history
  • Loading branch information
strengejacke committed Jun 25, 2024
1 parent d92db27 commit 63960ca
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
5 changes: 4 additions & 1 deletion R/eta_squared-main.R
Original file line number Diff line number Diff line change
Expand Up @@ -832,6 +832,7 @@ cohens_f_squared <- function(model,
include_intercept = include_intercept
)

## TODO: add back `effects = "fixed"` once the deprecation warning in parameters is removed
attr(out, "anova_type") <- tryCatch(attr(parameters::model_parameters(model, verbose = FALSE, es_type = NULL), "anova_type"),

Check warning on line 836 in R/eta_squared-main.R

View workflow job for this annotation

GitHub Actions / lint-changed-files / lint-changed-files

file=R/eta_squared-main.R,line=836,col=121,[line_length_linter] Lines should not be more than 120 characters. This line is 127 characters.
error = function(...) 1
)
Expand Down Expand Up @@ -863,6 +864,7 @@ cohens_f_squared <- function(model,

# TODO this should be in .anova_es.anvoa
# TODO the aoc method should convert to an anova table, then pass to anova
## TODO: add back `effects = "fixed"` once the deprecation warning in parameters is removed
params <- parameters::model_parameters(model, verbose = verbose, es_type = NULL)
out <- .es_aov_simple(as.data.frame(params),
type = type,
Expand All @@ -889,7 +891,8 @@ cohens_f_squared <- function(model,
verbose = TRUE,
include_intercept = FALSE,
...) {
params <- parameters::model_parameters(model, verbose = verbose, es_type = NULL)
## TODO: add back `effects = "fixed"` once the deprecation warning in parameters is removed
params <- parameters::model_parameters(model, verbose = verbose, es_type = NULL)
anova_type <- attr(params, "anova_type")
params <- as.data.frame(params)

Expand Down
1 change: 1 addition & 0 deletions R/eta_squared-methods.R
Original file line number Diff line number Diff line change
Expand Up @@ -257,6 +257,7 @@
ci = 0.95, alternative = "greater",
verbose = TRUE,
...) {
## TODO: add back `effects = "fixed"` once the deprecation warning in parameters is removed
params <- parameters::model_parameters(model, verbose = verbose, es_type = NULL)
anova_type <- attr(params, "anova_type")

Expand Down

0 comments on commit 63960ca

Please sign in to comment.