forked from qiskit-community/qiskit-aqua
-
Notifications
You must be signed in to change notification settings - Fork 5
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
stefan-woerner
merged 12 commits into
stefan-woerner:master
from
qiskit-community:master
Jul 8, 2020
Merged
fetch from origin #124
stefan-woerner
merged 12 commits into
stefan-woerner:master
from
qiskit-community:master
Jul 8, 2020
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
) * 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]>
* return includes is_meas * add reno
* 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]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Details and comments