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

Reorganise providers #615

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Reorganise providers #615

wants to merge 6 commits into from

Conversation

frankharkins
Copy link
Member

Summary

This PR:

  • Removes the "Provider" tag as it's now its own section
  • Defines "Provider" as a package that implements Qiskit's provider interface
  • Adds a "Hardware" tag to distinguish from circuit simulators

Questions

Should we add a local / cloud simulator tag to distinguish between the two?

@Eric-Arellano
Copy link
Collaborator

local / cloud simulator tag

The local/cloud part doesn't seem as relevant. Maybe a "Simulator" tag in general?

Copy link
Collaborator

@Eric-Arellano Eric-Arellano left a comment

Choose a reason for hiding this comment

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

This needs to regenerate the site, right?

@frankharkins
Copy link
Member Author

We have a Circuit simulator tag for that purpose. However I've since realised we already have a Hardware tag that means something else. Maybe these get a Hardware provider tag?

This needs to regenerate the site, right?

That happens on push to main. Thinking about it now, we should actually take the generated website out of version control.

@Eric-Arellano
Copy link
Collaborator

However I've since realised we already have a Hardware tag that means something else.

Hm, what does it mean?

Thinking about it now, we should actually take the generated website out of version control.

Ah, yes please in a dedicated PR explaining the change.

@frankharkins
Copy link
Member Author

Hm, what does it mean?

"For quantum hardware design or quantum control". We could expand the definition though

@Eric-Arellano
Copy link
Collaborator

Is this still relevant?

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.

None yet

2 participants