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

Raise if unsupported #111

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Raise if unsupported #111

wants to merge 6 commits into from

Conversation

amichuda
Copy link
Collaborator

  • Raises NotImplementedError is a model not in the allowed_models tuple isn't input into wildboottest

Fixes #58.

@amichuda amichuda requested a review from s3alfisc June 26, 2024 02:15
@amichuda
Copy link
Collaborator Author

amichuda commented Jun 26, 2024

oh sorry, I created this branch from the wrong one. I can rebase, but if you can approve #106 , then this one should just involve the last commit!

I can also close this PR if that is easier, let me know, sorry!

@s3alfisc
Copy link
Member

Looks good as well =)

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.

Throw error when model passed to wildboottest() is not of type statsmodels.regression.linear_model.OLS
2 participants