Skip to content
This repository has been archived by the owner on Aug 19, 2023. It is now read-only.

Add depth tracking to random hex circuit benchmarks #201

Merged
merged 1 commit into from
Apr 22, 2019

Conversation

mtreinish
Copy link
Member

Summary

This commit adds 2 new benchmarks that use the same random hex circuit
setup but instead of measuring the time it takes to transpile() it
tracks the depth of the output circuit. This will serve as an example
for how we can track other transpiler performance metrics over time in
terra (not just depth).

Details and comments

This commit adds 2 new benchmarks that use the same random hex circuit
setup but instead of measuring the time it takes to transpile() it
tracks the depth of the output circuit. This will serve as an example
for how we can track other transpiler performance metrics over time in
terra (not just depth).
Copy link
Member

@1ucian0 1ucian0 left a comment

Choose a reason for hiding this comment

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

LGTM

@ajavadia ajavadia merged commit 7b4a8bd into Qiskit:master Apr 22, 2019
@mtreinish mtreinish deleted the add-depth-benchmarks branch April 22, 2019 14:10
jakelishman pushed a commit to jakelishman/qiskit-terra that referenced this pull request Aug 1, 2023
…tapackage#201)

This commit adds 2 new benchmarks that use the same random hex circuit
setup but instead of measuring the time it takes to transpile() it
tracks the depth of the output circuit. This will serve as an example
for how we can track other transpiler performance metrics over time in
terra (not just depth).
jakelishman pushed a commit to jakelishman/qiskit-terra that referenced this pull request Aug 11, 2023
…tapackage#201)

This commit adds 2 new benchmarks that use the same random hex circuit
setup but instead of measuring the time it takes to transpile() it
tracks the depth of the output circuit. This will serve as an example
for how we can track other transpiler performance metrics over time in
terra (not just depth).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants