Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign upRequest to change cplot.lm to cplot.default #84
Closed
Labels
Milestone
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
lukesonnet commentedJan 26, 2018
Changing
cplot.lm
tocplot.default
would let objects of an arbitrary S3 class that look like an lm object to take advantage of this great function without requiring users to usemargins:::cplot.lm(obj)
.This would be super helpful unless there's a justification for keeping it as cplot.lm!
Here's a MWE for estimatr: