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

Updating docs for qiskit-transpiler-service #1435

Merged
merged 13 commits into from
May 29, 2024

Conversation

cbjuan
Copy link
Member

@cbjuan cbjuan commented May 28, 2024

Updating the documentation to reflect some of the changes we've implemented for the beta version of the service and to include citation reference

@qiskit-bot
Copy link
Contributor

One or more of the following people are relevant to this code:

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.

Thanks!

docs/transpile/qiskit-transpiler-service.mdx Outdated Show resolved Hide resolved
from qiskit.circuit.library import EfficientSU2
from qiskit_transpiler_service.transpiler_service import TranspilerService

circuit = EfficientSU2(101, entanglement="circular", reps=1).decompose()
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there a simpler circuit you can use? With examples, it's useful to keep simple the less relevant details. Here, we only want to express that we have a QuantumCircuit.

Copy link
Member Author

Choose a reason for hiding this comment

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

I do agree we could do it with any QuantumCircuit, yet we want to showcase from the beginning this works for big circuits, that's the reason for that particular one

docs/transpile/ai-transpiler-passes.mdx Show resolved Hide resolved
cbjuan and others added 3 commits May 28, 2024 17:46
docs/transpile/ai-transpiler-passes.mdx Outdated Show resolved Hide resolved
docs/transpile/ai-transpiler-passes.mdx Outdated Show resolved Hide resolved
docs/transpile/qiskit-transpiler-service.mdx Outdated Show resolved Hide resolved
docs/transpile/qiskit-transpiler-service.mdx Outdated Show resolved Hide resolved
docs/transpile/qiskit-transpiler-service.mdx Outdated Show resolved Hide resolved
docs/transpile/qiskit-transpiler-service.mdx Outdated Show resolved Hide resolved
docs/transpile/qiskit-transpiler-service.mdx Outdated Show resolved Hide resolved
cbjuan and others added 2 commits May 29, 2024 12:09
Co-authored-by: Rebecca Dimock <66339736+beckykd@users.noreply.github.com>
Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
@Eric-Arellano
Copy link
Collaborator

@cbjuan I've invited you with write access. You can merge this when you're ready, and then I'll deploy the docs.

@cbjuan
Copy link
Member Author

cbjuan commented May 29, 2024

Thank you @Eric-Arellano! Merging

@cbjuan cbjuan added this pull request to the merge queue May 29, 2024
@abbycross abbycross removed this pull request from the merge queue due to a manual request May 29, 2024
@abbycross
Copy link
Collaborator

Sorry - I'm just back from being out sick and want to give this a pass. Will merge asap I promise! I wonder if we should be saying "AI-powered synthesis passes" rather than "AI synthesis passes", per @JulesMurphy 's caution not to leave AI unmodified -- thoughts???

@cbjuan
Copy link
Member Author

cbjuan commented May 29, 2024

Yes, that's right @abbycross. In general we should say always AI-powered. Thanks for the suggestion and for jumping in right on time :)

Co-authored-by: abbycross <across@us.ibm.com>
@abbycross abbycross added this pull request to the merge queue May 29, 2024
Merged via the queue into Qiskit:main with commit e435161 May 29, 2024
4 checks passed
@cbjuan cbjuan deleted the updating-qiskit-transpiler-service-docs branch May 29, 2024 14:59
github-merge-queue bot pushed a commit that referenced this pull request May 29, 2024
…1458)

In #1435 I did not update the status of the Qiskit transpiler service
from "alpha" to "beta".


This PR fixes it

---------

Co-authored-by: abbycross <across@us.ibm.com>
frankharkins pushed a commit to frankharkins/documentation that referenced this pull request Jul 22, 2024
Updating the documentation to reflect some of the changes we've
implemented for the beta version of the service and to include citation
reference

---------

Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
Co-authored-by: Rebecca Dimock <66339736+beckykd@users.noreply.github.com>
Co-authored-by: abbycross <across@us.ibm.com>
frankharkins pushed a commit to frankharkins/documentation that referenced this pull request Jul 22, 2024
…iskit#1458)

In Qiskit#1435 I did not update the status of the Qiskit transpiler service
from "alpha" to "beta".


This PR fixes it

---------

Co-authored-by: abbycross <across@us.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants