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

Remove qobj run in AerBackend #1724

Closed
wants to merge 2 commits into from
Closed

Conversation

hhorii
Copy link
Collaborator

@hhorii hhorii commented Feb 15, 2023

Summary

Remove qobj support in backend.run(), which was deprecated 2 years ago.

Details and comments

#1720

@hhorii hhorii added this to the Aer 0.12.0 milestone Feb 15, 2023
@hhorii hhorii changed the title deprecate qobj run in AerBackend Remove qobj run in AerBackend Feb 15, 2023
@hhorii
Copy link
Collaborator Author

hhorii commented Feb 15, 2023

Even If we remove qobj supports in arguments, qobj will be still necessary to execute pulse simulation and DASK-based qasm simulation. That is, assemble() will be internally called in run(). To generate qobj for pulse, arguments for assemble() is necessary. For example, verify_wheel.py executes pulse simulation as follows

    qobj = assemble([schedule],
                    backend=backend_sim,
                    qubit_lo_freq=[5.0],
                    meas_level=1,
                    meas_return='avg',
                    shots=1)
    results = backend_sim.run(qobj, system_model=system_model).result()

Note that pulse simulation requires different arguments for assemble() and run(). If we remove qobj supports, the same arguments will be specified and unfortunately pulse simulation doesn't work correctly.

I think that qobj should be removed after unsupport of pulse simulation (#1722).

@hhorii hhorii modified the milestones: Aer 0.12.0, Aer 0.13.0 Feb 15, 2023
@hhorii hhorii closed this Feb 20, 2023
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