Skip to content

fetch from origin #124

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

Merged
merged 12 commits into from
Jul 8, 2020
Merged

Conversation

stefan-woerner
Copy link
Owner

Summary

Details and comments

manoelmarques and others added 12 commits June 24, 2020 13:18
)

* Fix Exchange/wiki providers, new Yahoo! provider, fix divide by 0

* add words to pylintdic to support latest pyenchant on mac

* Remove unused stockmarket parameter

* handle division by 0

* make pandas, quandl, yfinance optional

* fix spelling

* install pandas,quandl,yfinance on  aqua install

* expand not installed msgs
* factor in the coeff of ListOp

* fix coeff in composedop reduce

* state coeff must be squared (applied left and right)

* fix norm

* add test

* add reno
* Z2 symmetry fix for when no symmetries found

* Use classical solver for faster test
The legacy tutorial in the Qiskit/qiskit-tutorials repo for running
QSVM classification is failing with newer versions of aqua that use
cvxpy. The error message returned suggests setting the qcp flag to true
on solve(). This treats the problem as disciplined quasiconvex program
instead of a disciplined convex program and seems to avoid the hard
failure in the tutorial.

Fixes: Qiskit/qiskit-tutorials#928
* fix param binding of primitive

* add test

Co-authored-by: Manoel Marques <[email protected]>
* simplify and reduce, add equals to SummedOp, update tests

* directly use new opflow, no need to go via WPO

* update comments and docstrings

* directly use opflow

* don't do equality check in add

* directly use opflow

* change order in reduce

* fix qaoa

* add short test on summed op equality

* rm prints

* use set comparison, rename simplify to collapse_summands

* fix expected value, should be sqrt(2), not 2

* cast coeffs to complex

* add reno on equals

* fix mypy

* fix spell

* fix lint

* dont cast coefficient to complex

leads to problems if the coeff is exponentitated and not supposed to be complex

* use sum instead of reduce

* rm unused import

* move __hash__ to primitive op and base on repr

* use != over not ==

* add summed op test for different primitives

* check for opbase, not summedop

* adress changes from review

* explicitly raise an error upon ListOp input
* return identity op instead of the int 0

* fix spell

* add note that equals is not mathematically sound

Co-authored-by: Manoel Marques <[email protected]>
* Fix lint in controlled circuit

* Fixed change to Instruction.definition
* Remove Swaps From IQFT

* Remove Unused Variables from GroverOptimizer

Co-authored-by: Manoel Marques <[email protected]>
)

* slsqp implementation

* slsqp implementation

* slsqp implementation, linting, style, docstring

* tests, style, linting, spelling

* spelling

* review changes

* review changes

* doctests, reverting back some changes.

* doctests

* added a reno yaml file for slsqp

* multistart cobyla and new compatibility check

* weird spell checker

* more linting

* edited reno note, more on docstring.

* spelling

* more on multi start optimizer

* more on multi start optimizer

* fixed clip type in multistart_optimizer.py

Co-authored-by: Manoel Marques <[email protected]>
@stefan-woerner stefan-woerner merged commit 0e532bf into stefan-woerner:master Jul 8, 2020
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.

7 participants