-
Notifications
You must be signed in to change notification settings - Fork 46
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
initial implementation of UnivariateProbabilitySimulator #28
initial implementation of UnivariateProbabilitySimulator #28
Conversation
… their signatures and behaviours will differ across subclasses
- unit tests still to be written - docstrings to be reviewed/updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks okay. I gave it a test run on my simulated classification data, and the result is below. Seems to work as expected. Shape for something linear on the log-odds scale looks about right, and the line appears to centre around the positive class probability of 0.564.
Before merging a couple of things:
# Conflicts: # src/facet/simulation/_simulation.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for implementing the changes!
Closes #13.