-
-
Notifications
You must be signed in to change notification settings - Fork 62
Histogram trick #38
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
Histogram trick #38
Conversation
Can we come up with a better name than |
Don't know, I just like the "trick" |
"approximation" is not that fancy |
9144db8
to
45eaeca
Compare
Looks like we need a pre-commit hook. There's a missing empty line at the end of one of the files |
I agree with @ricardoV94, |
Ok, I'll will take that in account! |
I seem to have finished (on the converging path) the implementation for the histogram approximation. I converged with the following API
|
blocked by #44 |
The PR is ready for review |
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.
LGTM!
Add histogram trick to experimental utils. The idea is to reduce computations for huge data with smart preprocessing
Features: