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

Add pulse simulator test case to verify_wheels.py #584

Closed
mtreinish opened this issue Feb 5, 2020 · 0 comments · Fixed by #588
Closed

Add pulse simulator test case to verify_wheels.py #584

mtreinish opened this issue Feb 5, 2020 · 0 comments · Fixed by #588
Assignees
Labels
enhancement New feature or request

Comments

@mtreinish
Copy link
Member

mtreinish commented Feb 5, 2020

What is the expected behavior?

The tools/verify_wheels.py script: https://github.com/Qiskit/qiskit-aer/blob/master/tools/verify_wheels.py is what we use to validate are built binaries prior to upload to pypi. It just quickly sanity checks each simulator and validates that everything compiled works and is accessible from python. However, it was originally written prior to the introduction of the pulse simulator. To make sure that we are building binaries that include a functional pulse simulator we should add a test case that exercises it to the script. This doesn't need to be exhaustive just something the puts together a small schedule and checks the results to validate it works.

The existing cases in there for the qasm, statevector, and unitary simulators just took the contents from a single unit test and converted them to a standalone test case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants