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

expand BackendProperties in PassManagerConfig #3626

Closed
ajavadia opened this issue Dec 18, 2019 · 5 comments
Closed

expand BackendProperties in PassManagerConfig #3626

ajavadia opened this issue Dec 18, 2019 · 5 comments
Labels
type: feature request New feature or request

Comments

@ajavadia
Copy link
Member

As @taalexander astutely pointed out, it makes more sense for the transpiler to accept individual properties rather than a monolithic BackendProperties. For example readout_errors, cx_errors, gate_durations, etc, etc. This would be in the same vain as how it currently accepts basis_gates and coupling_map, rather than BackendConfiguration.

Of course transpile(circuit, backend) would still work, but now we would be allowed to override specific properties.

This should be done after #3035. It applies to the NoiseAdaptiveLayout pass and the CrosstalkAdaptiveSchedule pass.

@ajavadia ajavadia added the good first issue Good for newcomers label Dec 18, 2019
@1ucian0 1ucian0 added the type: feature request New feature or request label Feb 14, 2020
@BlakeGerard
Copy link

So essentially, the goal is to allow the direct passing of the parameters in BackendProperties init method to the transpiler?

@taalexander
Copy link
Contributor

I'm not positive I still agree with this earlier sentiment as the Backend has evolved to be a representation of a target machine for Qiskit (not the transport representation it once was). Perhaps this should be closed now @ajavadia?

@BlakeGerard
Copy link

Ah ok, I'm just looking around for good first issues.

@1ucian0 1ucian0 removed the good first issue Good for newcomers label Sep 20, 2021
@mrvee-qC
Copy link

Update for Santa:elf:- QGoGP

✅ As mentioned above, there have been significant changes in the code structure as to what Backend signifies now! If OP does not have an update on the same, we can safely close this issue! 😄

Will update comment/add more comments if cause is found/fixed!

Python version 3.9.7
qiskit-terra version: 0.19.1

@jakelishman
Copy link
Member

At the very least, this issue is somewhat superseded by work on BackendV2, and we have other trackers for how we want to handle transpilation with respect to those backends (the main one being #7113). I'll close this in favour of #7113.

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

No branches or pull requests

6 participants