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

Update Getting Started page #1236

Merged
merged 3 commits into from
Sep 4, 2023

Conversation

woodsp-ibm
Copy link
Member

Summary

The Getting Started page currently shows installation via the metapackage. This changes the text a bit and changes to directly installing using pip

Comparable change to that done in ML qiskit-community/qiskit-machine-learning#681

Details and comments

docs/getting_started.rst Outdated Show resolved Hide resolved

As Qiskit Nature depends on Qiskit, you can though simply install it into your
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
As Qiskit Nature depends on Qiskit, you can though simply install it into your
As Qiskit Nature depends on Qiskit, you can also simply install it into your

I am not sure how to understand the though in this sentence. I assume you meant smth along the lines of also. If not, feel free to rephrase differently.

Copy link
Member Author

Choose a reason for hiding this comment

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

Again I wanted to update this text but not do too much. For "you can though" you can read it as "you can instead" ie you do not not have to go through the step of separately installing Qiskit.

Co-authored-by: Max Rossmannek <[email protected]>
@mergify mergify bot merged commit b4c829d into qiskit-community:main Sep 4, 2023
16 checks passed
@woodsp-ibm woodsp-ibm deleted the getting_started branch September 4, 2023 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants