Skip to content

BackendV2 additional properties#8318

Closed
gadial wants to merge 8 commits into
Qiskit:mainfrom
gadial:backend_v2_properties_setup
Closed

BackendV2 additional properties#8318
gadial wants to merge 8 commits into
Qiskit:mainfrom
gadial:backend_v2_properties_setup

Conversation

@gadial
Copy link
Copy Markdown
Contributor

@gadial gadial commented Jul 11, 2022

Summary

The goal of this PR is to extend BackendV2 with additional properties present in BackendV1 and currently missing, to ensure qiskit-experiments can transition to BackendV2 usage. Properties that only make sense in the context of FakeBackendV2 will be added there.

Details and comments

  • Support for the control_channels property.

@gadial gadial requested review from a team and jyu00 as code owners July 11, 2022 14:04
@qiskit-bot
Copy link
Copy Markdown
Collaborator

Thank you for opening a new pull request.

Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient.

While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone.

One or more of the the following people are requested to review this:

  • @Qiskit/terra-core

@gadial gadial changed the title BackendV2 additional properties [WIP] BackendV2 additional properties Jul 11, 2022
@coveralls
Copy link
Copy Markdown

coveralls commented Jul 18, 2022

Pull Request Test Coverage Report for Build 2789742793

  • 8 of 10 (80.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.006%) to 83.992%

Changes Missing Coverage Covered Lines Changed/Added Lines %
qiskit/providers/fake_provider/fake_backend.py 7 9 77.78%
Totals Coverage Status
Change from base Build 2785570863: 0.006%
Covered Lines: 55910
Relevant Lines: 66566

💛 - Coveralls

@gadial gadial changed the title [WIP] BackendV2 additional properties BackendV2 additional properties Jul 25, 2022
@mtreinish mtreinish self-assigned this Jul 25, 2022
@jakelishman
Copy link
Copy Markdown
Member

This appears to have been obsoleted by #8444, but feel free to re-open if I've missed something.

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.

6 participants