-
Notifications
You must be signed in to change notification settings - Fork 11
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
predictions(., hypothesis) arguments inconsistency bw python and R #142
Comments
lol this is embarrassing. Crazy week, but I'll look into it soon. Should be an easy fix. Or maybe @artiom-matvei has time to take a look. |
Might be a b0 vs b1 issue. I think we decided it was necessary to make a breaking change so that Python used zero indexing. |
@vincentarelbundock is there a way for me to update the replication code? |
From @drsundar.bsky.social#corrected code chunk: avg_predictions(m, predictions(m, predictions(m, |
Thanks everyone for confirming that the index of hypotheses in Python must start at I will collect a list of errors and typos found in the JSS here: vincentarelbundock/marginaleffects#1293 After a while, I'll try to update the package at JSS, but I'm not sure if they will let me. |
The accompanying python replication code to the JSS article does not work at the following two lines
with
NameError: name 'b2' is not defined
.A notebook demo-ing the correct python code either in the repo or the marginal effect zoo would be great.
The text was updated successfully, but these errors were encountered: