-
Notifications
You must be signed in to change notification settings - Fork 109
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
Benchmark refactoring, improving efficiency, enhanced reporting #107
Conversation
* Disabled Redis disc persistence and refactored integration test, fixes #95 * Added Bayes backend benchmarks
I'll take a look later today, thanks! |
I am making some changes here to better organize the code. |
@Ch4s3, I think I am done refactoring for now. Please go ahead for code review and merging if seems good. |
|
||
class BenchmarkReporter < Minitest::Reporters::RubyMateReporter | ||
def before_suite(suite) | ||
puts ([suite] + BayesianCommonBenchmarks.bench_range).join("\t") |
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.
I would have much preferred to extract this custom reporter into a separate file and potentially share that with other benchmarks in future, but the kind of data I am using in the before_suite
method will not be available outside, unless there is a way that I am unaware of.
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.
This is probably fine
* Remove residue after refactoring #107 * Speed up Docker image rebilding
NOPROGRESS
ENV hack