Skip to content
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

added dummy X reg # 15 #18

Merged
merged 5 commits into from Jun 5, 2018
Merged

added dummy X reg # 15 #18

merged 5 commits into from Jun 5, 2018

Conversation

vviers
Copy link
Contributor

@vviers vviers commented Jun 1, 2018

solves #15,

I kept it simple

Copy link
Member

@floswald floswald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very good!

rect(xleft = x, ybottom = y,
xright = x + abs(errors), ytop = y + errors, density = -1,
col = rgb(red = 1, green = 0, blue = 0, alpha = 0.05), border = NA)
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it shows a warning

Warning in plot.xy(xy.coords(x, y), type = type, ...) :
  "add" is not a graphical parameter

sidebarPanel(sliderInput("i_dummy", "Intercept", min = 0,
max = 5, step = .25, value = 5),
sliderInput("s_dummy", "Slope", min = -1,
max = 5, step = .25, value = .25),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

start at a better slope or it's hard to get started. 3?

@floswald
Copy link
Member

floswald commented Jun 4, 2018

a comment to the older plots in that PR: for Fit with no slope it would be great if you could fix the x-axis for both versions - with and without demeaned data. it would be great to just see the cloud of points shift. the changing xaxix masks this a bit.

@vviers
Copy link
Contributor Author

vviers commented Jun 4, 2018

Ok, will do

@vviers
Copy link
Contributor Author

vviers commented Jun 4, 2018

Done, will push this pm.

@floswald floswald merged commit dc389a7 into ScPoEcon:master Jun 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants