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

Last updates to Intro to Qiskit and what are patterns pages #1507

Merged
merged 12 commits into from
Jun 20, 2024

Conversation

javabster
Copy link
Collaborator

Just added some images and did some reformatting

@abbycross abbycross self-requested a review June 10, 2024 14:42
### Circuit building tools
([`qiskit.circuit`](/api/qiskit/circuit))
For initializing and manipulating registers, circuits, instructions, gates, parameters, and control flow objects.
- Circuit building tools ([`qiskit.circuit`](/api/qiskit/circuit)) - For initializing and manipulating registers, circuits, instructions, gates, parameters, and control flow objects.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- Circuit building tools ([`qiskit.circuit`](/api/qiskit/circuit)) - For initializing and manipulating registers, circuits, instructions, gates, parameters, and control flow objects.
- Circuit-building tools ([`qiskit.circuit`](/api/qiskit/circuit)) - For initializing and manipulating registers, circuits, instructions, gates, parameters, and control flow objects.

### Quantum Information
([`qiskit.quantum_info`](/api/qiskit/quantum_info))
A toolkit for working with quantum states, operators and channels, using exact calculations (no sampling noise). Use this module to specify input observables and analyse fidelity of outputs from primitives queries.
- Quantum Information ([`qiskit.quantum_info`](/api/qiskit/quantum_info)) - A toolkit for working with quantum states, operators and channels, using exact calculations (no sampling noise). Use this module to specify input observables and analyse fidelity of outputs from primitives queries.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- Quantum Information ([`qiskit.quantum_info`](/api/qiskit/quantum_info)) - A toolkit for working with quantum states, operators and channels, using exact calculations (no sampling noise). Use this module to specify input observables and analyse fidelity of outputs from primitives queries.
- Quantum information ([`qiskit.quantum_info`](/api/qiskit/quantum_info)) - A toolkit for working with quantum states, operators, and channels, using exact calculations (no sampling noise). Use this module to specify input observables and analyze fidelity of outputs from primitives queries.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure if the "I" in Information should be capitalized as part of a package name or not - suggesting the change here since we try to keep things lower-case as much as possible, but feel free to change it back

### Transpiler
([`qiskit.transpiler`](/api/qiskit/transpiler))
For transforming and adapting quantum circuits to suit specific device topology and/or optimizing for execution on real quantum systems.
- Transpiler ([`qiskit.transpiler`](/api/qiskit/transpiler)) - For transforming and adapting quantum circuits to suit specific device topology and/or optimizing for execution on real quantum systems.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- Transpiler ([`qiskit.transpiler`](/api/qiskit/transpiler)) - For transforming and adapting quantum circuits to suit specific device topology and/or optimizing for execution on real quantum systems.
- Transpiler ([`qiskit.transpiler`](/api/qiskit/transpiler)) - For transforming and adapting quantum circuits to suit specific device topology, and optimizing for execution on real quantum systems.

Copy link
Collaborator

Choose a reason for hiding this comment

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

(avoiding and/or bc it doesn't translate well)

Copy link
Collaborator

@abbycross abbycross left a comment

Choose a reason for hiding this comment

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

Made some tiny nit suggestions, otherwise GTG 🪩



---


As we move from bespoke circuit construction to utility-scale workflows, the flexibility and ease with which Qiskit Patterns allows one to compose the different steps of the pattern opens quantum computing to a wide variety of applications and techniques for easy use by quantum computational scientists.
As the field moves from bespoke circuit construction to utility-scale workflows, the flexibility and ease with which Qiskit patterns allow users to compose the different steps of the pattern opens quantum computing to a wide variety of applications and techniques for easy use by quantum computational scientists.
Copy link
Collaborator

Choose a reason for hiding this comment

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

flagging this fix I made (from "As we move" to "As the field moves") to avoid using "we" -- making sure it's palatable or maybe there's a better way to reword it @javabster

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.

I didn't review the content itself - but the infra-side looks good!

Could you please organize the images better to be like public/images/qiskit-patterns/map.svg, ../qiskit-patterns/overview.svg, etc? We want to use folders with public/images so that it's more organized.

scripts/patterns-reorg/page_content.py Outdated Show resolved Hide resolved
@Eric-Arellano Eric-Arellano added this pull request to the merge queue Jun 20, 2024
Merged via the queue into main with commit 1f2a2c2 Jun 20, 2024
3 checks passed
@Eric-Arellano Eric-Arellano deleted the intro-updates branch June 20, 2024 14:25
frankharkins pushed a commit to frankharkins/documentation that referenced this pull request Jul 22, 2024
)

Just added some images and did some reformatting

---------

Co-authored-by: abbycross <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[ NEW PAGE ] What are Qiskit Patterns? Replace "What is Qiskit" section
4 participants