Skip to content

Implement benchmark framework for gpython along with a couple of benchmarks #51

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

Merged
merged 1 commit into from
Jan 22, 2019

Conversation

ncw
Copy link
Collaborator

@ncw ncw commented Jan 22, 2019

This reworks the existing test framework so it can be used for
benchmarks too. Note that it now uses the subtest framework which
makes the test output much neater.

To run the benchmarks use

cd vm
go test -v -run XXX -bench .

Fixes #50

…hmarks

This reworks the existing test framework so it can be used for
benchmarks too.  Note that it now uses the subtest framework which
makes the test output much neater.

To run the benchmarks use

    cd vm
    go test -v -run XXX -bench .

Fixes #50
@ncw ncw requested a review from corona10 January 22, 2019 12:41
@codecov-io
Copy link

codecov-io commented Jan 22, 2019

Codecov Report

Merging #51 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #51   +/-   ##
=======================================
  Coverage   66.02%   66.02%           
=======================================
  Files          58       58           
  Lines       10246    10246           
=======================================
  Hits         6765     6765           
  Misses       3005     3005           
  Partials      476      476

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9985b49...69315d5. Read the comment docs.

Copy link
Collaborator

@corona10 corona10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!
LGTM

@corona10 corona10 merged commit 95617b7 into master Jan 22, 2019
@corona10 corona10 deleted the benchmarks branch January 22, 2019 12:46
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.

3 participants