-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix/turtorials typo and bugs #108
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also add
@@ -12,7 +12,7 @@ Welcome to the Qamomile tutorial! This guide will help you get started with Qamo | |||
|
|||
- **Qiskit** | |||
- **Quri-parts** | |||
|
|||
- **Qutip** | |||
|
|||
|
|||
## Tutorials |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also update the title to "Write Hamiltonian Algebraically"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
"source": [] | ||
"source": [ | ||
"# References\n", | ||
"[1] Farhi, Edward, Jeffrey Goldstone, and Sam Gutmann. \"A quantum approximate optimization algorithm.\" arXiv preprint arXiv:1411.4028 (2014).\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also add the link to the paper?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added
Change
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess you might just have forgottten to commit the change.
docs/quickstart.ipynb
Outdated
"- For Qiskit integration: `pip install qamomile[qiskit]`\n", | ||
"- For Quri Parts integration: `pip install qamomile[quri-parts]`" | ||
"- For Quri Parts integration: `pip install qamomile[quri-parts]`\n", | ||
"- For Qutip integration: `pip install qamomile[qutip]`" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you update this part to pip install "qamomile[qiskit]"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I missed the convergence in the quickstart; however, @ahao27 fixed it.
Description:
This pull request introduces a series of corrections of typo and clarifications to the documentation. These changes aim to enhance readability and accuracy across the documentation.
Changes:
qutip
as an additional quantum circuit SDK for broader coverage.Capitalized "algebraically" for consistency.
Updated # Get exp(-i beta H_C) circuit without mixer circuit to # Get exp(-i gamma H_P) circuit without mixer circuit.