Skip to content
This repository has been archived by the owner on Aug 19, 2023. It is now read-only.

Component status #116

Merged
merged 9 commits into from
Feb 19, 2019
Merged

Component status #116

merged 9 commits into from
Feb 19, 2019

Conversation

jaygambetta
Copy link
Member

@jaygambetta jaygambetta commented Feb 18, 2019

@ajavadia and @nonhermitian and @diego-plan9 can you look at this and see if the table is correct.

@chriseclectic can you add a table for the aer provider (this can be a follow-on pr).

@derivation
Copy link
Contributor

@jaygambetta when you call for additional work from others on pull requests you open, would you like the PR to be merged as a minimum viable product and for me to spin up issues, or keep the PR open until the others have added their contributions to it?

@jaygambetta
Copy link
Member Author

I am happy to have the minimum viable product and for you to spin up issues. Smaller pr's are always better (fewer conflicts) easier to review.

@jaygambetta
Copy link
Member Author

jaygambetta commented Feb 18, 2019

I am also happy for you to edit directly (close or and change). The goal of this is to have something somewhere which says the status of things as people always ask what is stable in qiskit and what is not.

@diego-plan9
Copy link
Member

The table looks reasonable to me 👍

@ajavadia
Copy link
Member

Table looks good.

I think we can have the compiler interface stable in 0.9 (along with transpiler API).

@nonhermitian
Copy link
Collaborator

I think this is good. I thought the visualizations were stable as of last release, but 0.8 is fine as well.

Copy link
Contributor

@derivation derivation left a comment

Choose a reason for hiding this comment

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

I'll spin up an issue to make sure info on Aer is accurate.

@jaygambetta
Copy link
Member Author

ok cool. I'm good to merge this and make aer a follow up. As we need to also make a follow up for aqua (@pistoia )

@derivation when you are good with the structure and such just approve and merge

@jaygambetta
Copy link
Member Author

@nonhermitian it is more where it should be located. I think it should be in visualizations not tools/visualizations

@derivation derivation merged commit cfe182f into Qiskit:master Feb 19, 2019
jakelishman pushed a commit to jakelishman/qiskit-terra that referenced this pull request Aug 11, 2023
* Components

* Fixing table

* Small update

* Update components.rst

* Update index.rst
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.

None yet

5 participants