Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

Rename qiskit_ibm to qiskit_ibm_provider#244

Merged
rathishcholarajan merged 1 commit intoQiskit:mainfrom
daka1510:rename
Mar 2, 2022
Merged

Rename qiskit_ibm to qiskit_ibm_provider#244
rathishcholarajan merged 1 commit intoQiskit:mainfrom
daka1510:rename

Conversation

@daka1510
Copy link
Copy Markdown
Collaborator

Summary

Rename qiskit_ibm to qiskit_ibm_provider.

Details and comments

Solves #235

"metadata": {},
"source": [
"The `qiskit-ibm`'s implementation of `BaseBackend` is [`IBMBackend`](https://qiskit.org/documentation/stubs/qiskit_ibm.IBMBackend.html#qiskit_ibm.IBMBackend). It accepts additional parameters to the `run()` method:\n",
"The `qiskit-ibm`'s implementation of `BaseBackend` is [`IBMBackend`](https://qiskit.org/documentation/stubs/qiskit_ibm_provider.IBMBackend.html#qiskit_ibm_provider.IBMBackend). It accepts additional parameters to the `run()` method:\n",
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yes, these inks will not work. But pointing to qiskit_ibm is also confusing. So decided to leave it like this for now...

Copy link
Copy Markdown
Member

@rathishcholarajan rathishcholarajan left a comment

Choose a reason for hiding this comment

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

@daka1510 Could you please resolve merge conflicts (I merged other open PRs)?

@daka1510
Copy link
Copy Markdown
Collaborator Author

daka1510 commented Mar 1, 2022

Will resolve conflicts as soon as #243 and #241 have been integrated.

Copy link
Copy Markdown
Member

@rathishcholarajan rathishcholarajan left a comment

Choose a reason for hiding this comment

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

LGTM!

@rathishcholarajan rathishcholarajan merged commit adece22 into Qiskit:main Mar 2, 2022
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.

2 participants