-
Notifications
You must be signed in to change notification settings - Fork 140
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
Support new primitive based algorithms. #425
Comments
Tutorials and migration guide have just been merged #448. So, I think we can close this issue. |
Yes. I think so. Maybe though, if you note above I mentioned for the GroverOptimzer the direct use of Grover algorithm from Terra. When the GroverOptimzer was originally done the Grover at that time was not as flexible. But now its likely that the Grover algorithm can be used here rather than the custom logic/circuits here. There was no time to look into this though for the current release but it may be worth raising a separate issue to cover this so it can be looked at later. |
Thank you for reminding me of |
That certainly was the most recent discussion, yes. The idea has been floating around for a while though. |
I made #456 for the discussion |
What is the expected enhancement?
Support primitive based algorithms such as the new QAOA and SamplingVQE etc accordingly.
Update GroverOptimizer to move away from QuantumInstance use, potentially by more directly using Grover from Terra.
Update tutorials correspondingly
Update readme sample
Migration guide as applicable
GroverOptimizer
withSampler
#437The text was updated successfully, but these errors were encountered: