New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Spatialreg #847
Spatialreg #847
Conversation
Retain SEs of any fixest input, including fixest summary
- Adjustments to tidying fixest summaries - Make `glance.fixest` results depend on the type of model estimated. - Clarify that augment only works on `feols`, `feglm` or `femlm` models
Also simplify tidy.mfx() using fix_data_frame()
- Break out detection of fixest summary - Add some documentation about optional arguments and which types of models `augment.fixest` can handle.
#' @export | ||
#' @seealso [augment()] | ||
#' @family spatialreg tidiers | ||
augment.sarlm <- function(x) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add a ...
argument here for compatibility with the generic and @template param_unused_dots
roxygen tag?
I I like you udoated my function. If it is ok I extended your script for another model suppoerted
We don't actually need a three-level if() statement. We can simply append if the value exists, and skip if it doesn't. This allows for much simpler code
Merged! Thanks.🙂 |
Cool! Thank you for your work. |
This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue. |
This adds documented methods for spatial autoregressive models of class
sarlm
estimated with thespatialreg
package. Begins from code supplied to #841, with some adjustments for style and adding confidence interval on estimates.