Skip to content

Commit

Permalink
fix formula warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
pachadotdev committed Feb 16, 2024
1 parent 637e9b9 commit 69818eb
Show file tree
Hide file tree
Showing 21 changed files with 158 additions and 137 deletions.
2 changes: 1 addition & 1 deletion R/feglm.R
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@
#' Linear Models with High-Dimensional k-Way Fixed Effects". ArXiv e-prints.
#' @examples
#' mod <- feglm(
#' trade ~ dist + lang + cntg + clny | exp_year + imp_year,
#' trade ~ log_dist + lang + cntg + clny | exp_year + imp_year,
#' trade_panel,
#' family = poisson(link = "log")
#' )
Expand Down
2 changes: 1 addition & 1 deletion R/felm.R
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
#'
#' @examples
#' mod <- felm(
#' log(trade) ~ dist + lang + cntg + clny | exp_year + imp_year,
#' log(trade) ~ log_dist + lang + cntg + clny | exp_year + imp_year,
#' trade_panel
#' )
#'
Expand Down
2 changes: 1 addition & 1 deletion R/fenegbin.R
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
#' @examples
#' # same as the example in fepoisson but with overdispersion/underdispersion
#' mod <- fenegbin(
#' trade ~ dist + lang + cntg + clny | exp_year + imp_year,
#' trade ~ log_dist + lang + cntg + clny | exp_year + imp_year,
#' trade_panel
#' )
#'
Expand Down
2 changes: 1 addition & 1 deletion R/fepoisson.R
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
#' @examples
#' # same as the example in feglm but with less typing
#' mod <- fepoisson(
#' trade ~ dist + lang + cntg + clny | exp_year + imp_year,
#' trade ~ log_dist + lang + cntg + clny | exp_year + imp_year,
#' trade_panel
#' )
#'
Expand Down
7 changes: 5 additions & 2 deletions R/generics_vcov.R
Original file line number Diff line number Diff line change
Expand Up @@ -35,15 +35,18 @@ vcov.feglm <- function(
type <- match.arg(type)

# Extract cluster from formula
cl.vars <- attr(terms(object[["formula"]], rhs = 3L), "term.labels")
# it is totally fine not to have a cluster variable
suppressWarnings({
cl.vars <- attr(terms(object[["formula"]], rhs = 3L), "term.labels")
})
k <- length(cl.vars)
if (isTRUE(k >= 1L) && type != "clustered") {
type <- "clustered"
# add overwrite warning msg
message(
paste(
"There are clustering variables in the model formula.",
"The 'type' argument will be overwritten to 'clustered'."
"The 'type' argument will be overwritten to 'cluster'."
)
)
}
Expand Down
Binary file modified data/trade_panel.rda
Binary file not shown.
25 changes: 25 additions & 0 deletions dev/create_dataset.r
Original file line number Diff line number Diff line change
Expand Up @@ -17,4 +17,29 @@ trade_panel = trade_panel %>%
select(exp_year, imp_year, trade, dist, cntg, lang, clny, rta) %>%
mutate(rta = as.integer(rta))

trade_panel = trade_panel %>%
mutate(
exporter = substr(exp_year, 1, 3),
importer = substr(imp_year, 1, 3)
) %>%
filter(importer != exporter) %>%
select(-exporter, -importer)

use_data(trade_panel, overwrite = T)

load_all()

trade_panel = trade_panel %>%
mutate(
exporter = substr(exp_year, 1, 3),
importer = substr(imp_year, 1, 3),
pair = paste(exporter, importer, sep = "-"),
log_dist = log(dist)
) %>%
select(exp_year, imp_year, pair, trade, log_dist, everything()) %>%
select(-exporter, -importer, -dist)

trade_panel$year = as.integer(substr(trade_panel$exp_year, 4, 7))

trade_panel = trade_panel %>%
select(year, everything())
45 changes: 12 additions & 33 deletions docs/articles/intro.html

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

2 changes: 1 addition & 1 deletion docs/pkgdown.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,5 @@ pkgdown: 2.0.7
pkgdown_sha: ~
articles:
intro: intro.html
last_built: 2024-02-15T23:37Z
last_built: 2024-02-16T03:35Z

Loading

0 comments on commit 69818eb

Please sign in to comment.