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
Stop using QNodeCollection
in ExpvalCost
#3687
Conversation
Hello. You may have forgotten to update the changelog!
|
[sc-32650] |
Codecov Report
@@ Coverage Diff @@
## master #3687 +/- ##
==========================================
- Coverage 99.74% 99.73% -0.01%
==========================================
Files 324 324
Lines 28396 28399 +3
==========================================
+ Hits 28324 28325 +1
- Misses 72 74 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me! Please update changelog-dev.md
as well. Assuming all the tests pass, I think it's ready for approval.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 🎉
* 🔧 refactor (ExpvalCost): Stop using QNodeCollection. * ⏪ revert (ExpvalCost): Change back to qnodes. * 🐛 fix (hamiltonian_expand): Small fix. * 🧪 tests (ExpvalCost): Fix test. * Update tests/optimize/test_qng.py * Update tests/optimize/test_optimize_shot_adaptive.py * Update tests/test_vqe.py * Apply suggestions from code review * 🧪 tests (ExpvalCost): Fix test. * 🔧 refactor (SquaredErrorLoss): Stop using qml.map. * ⏪ revert (ExpvalCost): Revert. * 🐛 fix (ExpvalCost): Small fix. * 🔧 refactor (ExpvalCost): Refactor.
Stop using deprecated
qml.map
.