Skip to content

Fill gaps in account management functionality and add test coverage#64

Closed
daka1510 wants to merge 1 commit intoQiskit:mainfrom
daka1510:account-mgmt-2
Closed

Fill gaps in account management functionality and add test coverage#64
daka1510 wants to merge 1 commit intoQiskit:mainfrom
daka1510:account-mgmt-2

Conversation

@daka1510
Copy link
Copy Markdown
Contributor

@daka1510 daka1510 commented Dec 16, 2021

Summary

Follow-up PR for #49 with the goal to fill below gaps we left out in the initial version

  • add unit test coverage for account management related code
  • tweak behavior so that it matches the latest acceptance criteria notebook
  • refactor code to use accounts module in favor of credentials module

Details and comments

Relates to #2

@daka1510 daka1510 changed the title Tweak account management and add test coverage Fill gaps in account management functionality and add test coverage Dec 16, 2021
@jyu00
Copy link
Copy Markdown
Collaborator

jyu00 commented Dec 17, 2021

@daka1510 FYI I'm replacing Credentials with ClientParameters which has all the data for a client to connect to the server. It looks a lot like Account, but I thought it'd be nice to separate what's actually being saved on disk and passed to the client modules.

@daka1510
Copy link
Copy Markdown
Contributor Author

This PR largely conflicts with #62 from Jessie and is superseded by some changes therein. Will therefore close this draft PR.

@daka1510 daka1510 closed this Dec 21, 2021
@daka1510 daka1510 deleted the account-mgmt-2 branch December 21, 2021 12:36
blakejohnson pushed a commit to blakejohnson/qiskit-ibm-runtime that referenced this pull request May 26, 2023
* Put the schemas and backend requirements under a spec field

* Remove bad comma from vqe_metadata.json
ihincks pushed a commit to ihincks/qiskit-ibm-runtime that referenced this pull request Oct 29, 2025
* pyproject

* removed NLV3 models

* remove executor transport

* 0.1.0 --> 0.1

* decoders

* revert

* schemas

* default image
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.

2 participants