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

Use qiskitrc credentials if default config doesn't exist #370

Merged
merged 19 commits into from
Jun 22, 2022

Conversation

kt474
Copy link
Member

@kt474 kt474 commented Jun 13, 2022

Summary

If the qiskit-ibm.json config file doesn't exist or doesn't have any valid accounts, credentials will be copied over from a qiskitrc file, given that it exists.

Details and comments

Fixes #301

@coveralls
Copy link

coveralls commented Jun 13, 2022

Pull Request Test Coverage Report for Build 2539113838

  • 17 of 17 (100.0%) changed or added relevant lines in 2 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.1%) to 65.925%

Files with Coverage Reduction New Missed Lines %
qiskit_ibm_runtime/accounts/management.py 1 98.11%
Totals Coverage Status
Change from base Build 2537385393: 0.1%
Covered Lines: 2666
Relevant Lines: 4044

💛 - Coveralls

@kt474 kt474 changed the title [WIP] Use qiskitrc credentials if default config doesn't exist Use qiskitrc credentials if default config doesn't exist Jun 16, 2022
@kt474 kt474 requested a review from rathishcholarajan June 16, 2022 16:20
@kt474
Copy link
Member Author

kt474 commented Jun 16, 2022

@rathishcholarajan should be ready for initial review, let me know if this is the implementation we want to go with.

@rathishcholarajan rathishcholarajan added the Changelog: New Feature Include in the Added section of the changelog label Jun 17, 2022
@kt474 kt474 requested a review from rathishcholarajan June 17, 2022 19:23
Copy link
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.

@kt474 Looks good so far, just remembered that we need to copy these kwargs as well, proxies and verify https://github.com/Qiskit/qiskit-ibmq-provider/blob/master/qiskit/providers/ibmq/ibmqfactory.py#L240

@kt474 kt474 requested a review from rathishcholarajan June 20, 2022 01:31
test/unit/test_account.py Outdated Show resolved Hide resolved
@kt474 kt474 requested a review from rathishcholarajan June 20, 2022 20:25
@rathishcholarajan rathishcholarajan added this to the 0.6.0 milestone Jun 20, 2022
@kt474 kt474 requested a review from rathishcholarajan June 22, 2022 00:51
Copy link
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! Good job @kt474!

@rathishcholarajan rathishcholarajan merged commit bac910b into Qiskit:main Jun 22, 2022
blakejohnson pushed a commit to blakejohnson/qiskit-ibm-runtime that referenced this pull request May 26, 2023
* backendv2 name

* black

* fix variable
@kt474 kt474 deleted the load-qiskitrc-creds branch July 13, 2023 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: New Feature Include in the Added section of the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Runtime should load IQX credentials from Qiskit if available
3 participants