Join GitHub today
GitHub is home to over 28 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
Comments
leeper
added
the
enhancement
label
Jan 26, 2018
leeper
added this to the v0.4.0 milestone
Jan 26, 2018
leeper
closed this
in
c1ed68d
Aug 1, 2018
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: