Skip to content
This repository has been archived by the owner on Aug 21, 2023. It is now read-only.

Add Aqua operators tutorial #1068

Merged
merged 6 commits into from
Jan 21, 2021
Merged

Add Aqua operators tutorial #1068

merged 6 commits into from
Jan 21, 2021

Conversation

jlapeyre
Copy link
Contributor

No description provided.

@ikkoham
Copy link
Contributor

ikkoham commented Oct 28, 2020 •
edited
Loading

Thanks, can I check this tutorial?

@jlapeyre
Copy link
Contributor Author

@ikkoham Yes, please do check it.

ikkoham
ikkoham reviewed Oct 29, 2020
View reviewed changes
Copy link
Contributor

@ikkoham ikkoham left a comment

Choose a reason for hiding this comment

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

Very nice note. Let me point out a little.

tutorials/operators/01_operator_flow.ipynb Outdated Show resolved Hide resolved
tutorials/operators/01_operator_flow.ipynb Outdated Show resolved Hide resolved
tutorials/operators/01_operator_flow.ipynb Outdated Show resolved Hide resolved
tutorials/operators/01_operator_flow.ipynb Outdated Show resolved Hide resolved
tutorials/operators/01_operator_flow.ipynb Outdated Show resolved Hide resolved
tutorials/operators/01_operator_flow.ipynb Outdated Show resolved Hide resolved
@jlapeyre
Make corrections and changes in PR review
27a5ce9 
	modified:   tutorials/operators/01_operator_flow.ipynb
ikkoham
ikkoham previously approved these changes Nov 6, 2020
View reviewed changes
Copy link
Contributor

@ikkoham ikkoham left a comment

Choose a reason for hiding this comment

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

LGTM

@t-imamichi t-imamichi mentioned this pull request Nov 9, 2020
Closed
@t-imamichi
Copy link
Member

Qiskit advocates submitted a PR of a tutorial of operators. #1077
It seems started from Qiskit advocates project. qiskit-advocate/qiskit-advocate-projects#8

I hope these two groups collaborate.

@e-eight
Copy link

e-eight commented Nov 9, 2020 •
edited
Loading

Looks like we are duplicating efforts. I think we, the Qiskit advocates, would be happy to collaborate on this.

@jlapeyre
Copy link
Contributor Author

It would really be beneficial to have a tutorial on opflow operators committed sooner rather than later. Can we commit this one and work on integrating #1077 later ?

@stefan-woerner
Copy link
Contributor

@jlapeyre I agree, let's merge this version first so that we have a tutorial on the opflow available and then work on further improving it.

Since it is rather long, can you add a table of content to navigate it? Otherwise I think it is good for now.

@e-eight
Copy link

e-eight commented Nov 25, 2020

@jlapeyre @stefan-woerner Maybe we can make the non-overlapping parts of #1077 in to separate tutorial. Also it will be nice to have a plan for what tutorials would be nice to have, so that the two groups do not repeat the same things.

@nonhermitian
Copy link
Contributor

This has two H1 level headers that will break the formatting.

@jlapeyre
Copy link
Contributor Author

jlapeyre commented Dec 3, 2020

This has two H1 level headers that will break the formatting.

whoops, thought I removed that.

@nonhermitian nonhermitian self-requested a review January 21, 2021 14:16
@nonhermitian nonhermitian merged commit e0d492b into Qiskit:master Jan 21, 2021
nonhermitian added a commit that referenced this pull request Mar 1, 2021
@nonhermitian @tgag17
@neont21 @azeemba @yaelbh @mtreinish @chriseclectic @manoelmarques @mmcelaney @ewinston @danchepkwony @woodsp-ibm @kdk @schulta @divshacker @ljmf00 @lcapelluto @SooluThomas @jlapeyre
Backport tutorials into stable 0.23 branch (#1131)
a0fcb28 
* Fixed issue 1034 by correcting the typo

* Fixed issue 1065 by correcting the typo

* Add new line for list to be rendered properly

The two lists are not being rendered as lists so we add an extra newline for it to behave properly.

* Fix Aer tutorials (#1074)

* fixed setting of initial unitary

* fixed build noise model tutorial

* fixed tutorial applying noise to custom gate

* fixed extended stabilizer tutorial

* fixed the MPS tutorial

* Bopes tutorial fix (#1069)

* fix in bopes sampler tutorial

* kernel renaming for CI

Co-authored-by: Paul Nation <nonhermitian@gmail.com>

* fix qgans (#1071)

* fix qgans

* fix qgan code

* Pin docplex version in constraints.txt for CI job (#1082)

The recent new release of docplex has broken the released version of
aqua. There is a fix in aqua qiskit-community/qiskit-aqua#1420 which was released
in 0.8.1, but this hasn't been included in a metapackage release yet as
we're waiting for bugfix releases from all the other elements in the
next couple of days. Until that is included in a metapackage release we
won't be able to use the latest version of docplex in the CI job. This
commit adds a pin on the version of docplex used in a constraints file
to always use a known working version to unblock CI.

* Fix incorrect fidelity calculations in tomography tutorial (#1072)

* Fix tomography tutorial

* Remove noisy GST example

The old example can't use process fidelity function because it is comparing two noisy channels. My attempt to update using diamond norm distance instead cant be run in CI due to CVXPY solver not working without BLAS or CVXOPT.

* Remove header

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>

* updated device noise model tutorial (#1080)

Co-authored-by: Paul Nation <nonhermitian@gmail.com>

* Add seed to quantum_instance (#1081)

Co-authored-by: Paul Nation <nonhermitian@gmail.com>

* Remove legacy tutorials (#1085)

* Combine advanced circuits and pulse (#1086)

* Combine advanced circuits and pulse

* remove pulse from toc

* fix numbering for proper globing

* All Qiskit projects point to main CoC (#1088)

* remove unused variables (#1089)

* Fixes minor typo in the electronic structure  chemsitry tutorial

* Bump cell timeout in sphinx config (#1091)

* Bump cell timeout in sphinx config

The 07_asian_barrier_spread_pricing tutorial is quite slow and often
times out in CI because running the simulation takes very close to 3
minutes in the fastest case. But, the performance of the CI nodes is
quite variable since they run on virtual machines in azure's public
cloud. So many jobs fail randomly because of timeouts on that notebook.
This commit attempts to fix this by bumping the timeout 60 seconds which
hopefully will give us enough slack time in the job to handle slow CI
nodes.

* Make cell timeout an env var

* Add missing int cast for QISKIT_CELL_TIMEOUT (#1092)

In #1091 we added a new env var to enable downstream CI to adjust the
per cell timeout to make things more reliable in variable performance
environments. But in that PR we neglected to cast the value of the env
var, which is always a string, to an int. This commit fixes that
oversight so the new env var is actually usable.

* Algorithm and machine learning tutorials (#1087)

* 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>

* Remove deprecated DAGCircuit method usage. (#1093)

* Fix typo (#1095)

### Summary
Fix issue #1095 

### Details and comments
Delete the wrong and meaningless part of the sentence ". which it has to do the same thing".

* Add external collaborator (#1094)

* Fixed some typos on 04_vqe_advanced.ipynb

* Delete 05_getting_started_with_pulse.ipynb

* Add files via upload

* Delete 05_getting_started_with_pulse.ipynb

* Add files via upload

* Delete 05_getting_started_with_pulse.ipynb

* Add files via upload

* Fixed a typo on 04_vqe_advanced.ipynb

* algorithms: fix typo on grover jupyter notebook

* Migrate pulse_lib to library (#1107)

* Fixed all the broken links in "05_getting_started_with_pulse.ipynb." (#1102)

* Delete 05_getting_started_with_pulse.ipynb

* Add files via upload

* Delete 05_getting_started_with_pulse.ipynb

* Add files via upload

* Delete 05_getting_started_with_pulse.ipynb

* Add files via upload

Co-authored-by: Paul Nation <nonhermitian@gmail.com>

* Delete 06_building_pulse_instructions.ipynb

* Add files via upload

* Delete 07_building_pulse_schedules.ipynb

* Add files via upload

* Delete 07_building_pulse_schedules.ipynb

* Add files via upload

* Delete 08_adding_measurements.ipynb

* Add files via upload

* Delete 09_gathering_system_information.ipynb

* Add files via upload

* Delete 01_algorithms_introduction.ipynb

* Fixed Some Broken Links in this tutorial

* Delete 03_vqe_simulation_with_noise.ipynb

* Fix some Old Links in this tutorials

* Delete 7_examples_vehicle_routing.ipynb

* Fixed some old links

* Fixed some old links

* Delete docs which are not as useful anymore

* Reindex and rename files

* Add new page about pulse gates

* Modify existing files

* Add note about openpulse being enabled for pulse gates

* Add note about scheduling

* Add Aqua operators tutorial (#1068)

* Add Aqua operators tutorial

* Make corrections and changes in PR review
	modified:   tutorials/operators/01_operator_flow.ipynb

* Add table of contents

* Remove duplicate H1 header

Co-authored-by: Paul Nation <nonhermitian@gmail.com>

* Deleting file uploading new one

* Uploaded the file with a added extra link.

* Corrected Link

* Run pulse gates on 0.16.2

* Execute on terra version 0.16.2

* Update 02_portfolio_diversification.ipynb

* Update 02_portfolio_diversification.ipynb

* Update 02_portfolio_diversification.ipynb

* Delete 08_fixed_income_pricing.ipynb

* Update file with a working link

* Delete 08_fixed_income_pricing.ipynb

* Add files via upload

* Delete 08_fixed_income_pricing.ipynb

* With Updated Qiskit version

* Delete 10_qgan_option_pricing.ipynb

* Updated Few Links

* Pin cvxpy in CI (#1127)

* Pin cvxpy in CI

The latest release of cvxpy is causing job failures because of how they
changed their numpy requirement. This commit unblocks CI by just capping
the version installed in CI to just exclude the latest version.

* Pin numpy too

* Add operators to docs structure (#1126)

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>

* Remove numpy cap from tutorials job (#1139)

Co-authored-by: Tanya Garg <tanyagarg1701@gmail.com>
Co-authored-by: Peter J <peter.j@kakao.com>
Co-authored-by: Azeem Bande-Ali <A.BandeAli@gmail.com>
Co-authored-by: Yael Ben-Haim <yaelbh@il.ibm.com>
Co-authored-by: Panagiotis Barkoutsos <bpa@zurich.ibm.com>
Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
Co-authored-by: Christopher J. Wood <cjwood@us.ibm.com>
Co-authored-by: Manoel Marques <Manoel.Marques@ibm.com>
Co-authored-by: Maureen McElaney <mmcelaney@users.noreply.github.com>
Co-authored-by: ewinston <ewinston@us.ibm.com>
Co-authored-by: danchepkwony <dan.k.chepkwony@gmail.com>
Co-authored-by: Dan Chepkwony <67590846+danchepkwony@users.noreply.github.com>
Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
Co-authored-by: Kevin Krsulich <kevin.krsulich@ibm.com>
Co-authored-by: Shota Nakasuji <schulta.8715@gmail.com>
Co-authored-by: Dongsin Kim <ds.justin.kim@gmail.com>
Co-authored-by: Divyanshu Singh <55018955+divshacker@users.noreply.github.com>
Co-authored-by: Luís Ferreira <lsferreira169@gmail.com>
Co-authored-by: Lauren Capelluto <laurencapelluto@gmail.com>
Co-authored-by: SooluThomas <soolu.elto@gmail.com>
Co-authored-by: John Lapeyre <jlapeyre@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@stefan-woerner stefan-woerner stefan-woerner approved these changes

@nonhermitian nonhermitian nonhermitian approved these changes

@ikkoham ikkoham ikkoham left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@jlapeyre @ikkoham @t-imamichi @e-eight @stefan-woerner @nonhermitian