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

Add tutorials #44

Merged
merged 98 commits into from
Aug 21, 2023
Merged

Add tutorials #44

merged 98 commits into from
Aug 21, 2023

Conversation

ElePT
Copy link
Collaborator

@ElePT ElePT commented Aug 17, 2023

Summary

Fixes #14. Left as draft until local env issues that raise errors are fixed.

Details and comments

I'm including the IQPE example which does not use qiskit_algorithms, I could remove it, but I am not sure if we currently know where else to put it.

a-matsuo and others added 30 commits October 21, 2020 09:30
* added grover tutorial

* adding amplitude amplification

* added explanation of optional arguments

* changed the folder name

* finalized the tutorial

* change the file name

* finalize

Co-authored-by: Paul Nation <nonhermitian@gmail.com>
* Add algorithm and ml tutorials for Aqua

* Number qgan and add advanced vqe

* Fix notebook link

* Change IBMQ to mock simulator

* Add folders to index.rst

* Fix warnings in grover notebook

Co-authored-by: Manoel Marques <manoel.marques@ibm.com>
Co-authored-by: Paul Nation <nonhermitian@gmail.com>
### Summary
Fix issue Qiskit/qiskit-tutorials#1095 

### Details and comments
Delete the wrong and meaningless part of the sentence ". which it has to do the same thing".
algorithms: fix typo on grover jupyter notebook
Fixed Broken Links in "01_algorithms_introduction" tutorial.
Updated Some Old Links in "03_vqe_simulation_with_noise"
Added a extra Link and Corrected a Link in some tutorials
…utorials#1152)

* fix math mode of tutorials/circuits_advanced/09_pulse_simulator_duffing_model.ipynb

* fix toc of tutorials of operator flow

* simplify

* remove toc of tutorials of circuits_advanced

* fix toc of 06_building_pulse_schedules

* fix math mode

* fix math mode and code style in a link

* fix sum, sin, and cos

* fix typos
…nd grover tutorials (Qiskit/qiskit-tutorials#1163)

* Fix spelling in simulator tutorials and math mode of opflow tutorial

* Fix math mode and list of Grover tutorial

Co-authored-by: SooluThomas <soolu.elto@gmail.com>
Aqua imports replaced by QIskit Terra classes.

Chemistry, Finance, Machine Learning and Optimization were
removed as all those were converted and migrated to the new application repositories

* Remove Aqua references from notebooks

* Change algorithm/operator tutorials to use Aer

* compute gradients with fixed operator

* QAOA change weights to 1 and remove get_graph_solution

* Remove parallel disable statement & print oracle
Co-authored-by: SooluThomas <soolu.elto@gmail.com>
qbuits -> qubits
inizialize -> initialize
quntum -> quantum
x-measurment -> x-measurement
Fix a few typos "previous", "need" (twice), "measurement" (twice)
@woodsp-ibm
Copy link
Member

For the final tutorial (which I cannot view a diff here to comment inside it as mentioned before), shall we change the title to drop Qiskit out of it - we say Qiskit Primitives in there, which is correct still, and so I think we should update the title its currently Qiskit Gradient Framework using Primitives I could go for The Gradient Framework - as all the logic here is using primitives now so it seems less than needed to spell it out in the title, but as long as we drop Qiskit, whatever you like....

ElePT and others added 5 commits August 21, 2023 09:25
Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
Copy link
Member

@woodsp-ibm woodsp-ibm left a comment

Choose a reason for hiding this comment

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

LGTM. Thx @ElePT

@coveralls
Copy link

Pull Request Test Coverage Report for Build 5927078174

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 5 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.04%) to 89.975%

Files with Coverage Reduction New Missed Lines %
qiskit_algorithms/optimizers/aqgd.py 5 84.55%
Totals Coverage Status
Change from base Build 5897828464: -0.04%
Covered Lines: 6453
Relevant Lines: 7172

💛 - Coveralls

@ElePT ElePT merged commit 37dafdd into qiskit-community:main Aug 21, 2023
ElePT added a commit that referenced this pull request Aug 21, 2023
This was referenced Aug 21, 2023
@mergify
Copy link
Contributor

mergify bot commented Aug 22, 2023

⚠️ The sha of the head commit of this PR conflicts with #46. Mergify cannot evaluate rules on this PR. ⚠️

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.

Migrate tutorials from qiskit-tutorials