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

include benchmark suite and automation for multiple os #1431

Open
wants to merge 3 commits into
base: rc
Choose a base branch
from

Conversation

nilslice
Copy link

@nilslice nilslice commented Mar 11, 2022

Description

Adds automated benchmarks using the corpus of Quil code found in quilc's repository. The benchmarks focus on the parsing time exclusively, and only use valid Quil programs. These benchmarks will be used to compare performance against quil-rust.

Mention: rigetti/quil-rs#36

Checklist

  • The PR targets the rc branch (not master).
  • The above description motivates these changes.
  • All new and existing tests pass locally and on the PR's checks.
  • All code follows Black style and obeys flake8 conventions.

@nilslice nilslice requested a review from a team as a code owner March 11, 2022 23:21
@nilslice nilslice force-pushed the sm/benchmarks branch 5 times, most recently from 2203a73 to dd6dc23 Compare March 11, 2022 23:37
Copy link
Contributor

@genos genos left a comment

Choose a reason for hiding this comment

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

A few questions, more about my ignorance of GitHub CI than anything 😅

.github/workflows/bench.yml Show resolved Hide resolved
benchmarks/test_parser.py Outdated Show resolved Hide resolved
benchmarks/test_parser.py Outdated Show resolved Hide resolved
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.

2 participants