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

Benchmark interpret vs compiled code evaluation #14

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

ReubenJ
Copy link
Member

@ReubenJ ReubenJ commented Feb 20, 2024

As I was working on HerbBenchmarks.jl I realized we might be spending a lot of our time interpreting built expressions (candidate programs). This PR is a start at benchmarking our default interpreter so that we have some concrete numbers to talk about when making changes to it.

So far, the PR makes the package compliant with PackageBenchmark.jl's structure it expects, making the following possible:

using Pkg
Pkg.activate(; temp=true)
Pkg.add("PkgBenchmark")
Pkg.add("BenchmarkTools")
Pkg.add(PackageSpec(name="HerbInterpret", rev="benchmark-interpret-vs-compiled"))
using PkgBenchmark

import HerbInterpret

benchmarkpkg(HerbInterpret)

TODO:

@ReubenJ ReubenJ self-assigned this Feb 23, 2024
@ReubenJ ReubenJ force-pushed the benchmark-interpret-vs-compiled branch from 9017138 to e86c8c9 Compare October 7, 2024 12:34
Copy link

codecov bot commented Oct 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 28.57%. Comparing base (584ea42) to head (c3d295a).

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #14   +/-   ##
=======================================
  Coverage   28.57%   28.57%           
=======================================
  Files           2        2           
  Lines         105      105           
=======================================
  Hits           30       30           
  Misses         75       75           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ReubenJ ReubenJ force-pushed the benchmark-interpret-vs-compiled branch from 9d88ceb to c3d295a Compare October 7, 2024 15:06
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.

1 participant