Closed maliasadi closed 2 months ago
Hello. You may have forgotten to update the changelog! Please edit .github/CHANGELOG.md with:
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 81.50%. Comparing base (
4b79587
) to head (8e117ff
). Report is 2 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@multiphaseCFD there are more things to fix in latest/latest :grimacing: I'll update the changelog right after getting latest/latest :green_circle:
@multiphaseCFD latest/latest :green_circle: :tada:
@vincentmr I've run them manually! But let's trigger them here too :+1:
Before submitting
Please complete the following checklist when submitting a PR:
[x] All new features must include a unit test. If you've fixed a bug or added code that should be tested, add a test to the
tests
directory![x] All new functions and code must be clearly commented and documented. If you do make documentation changes, make sure that the docs build and render correctly by running
make docs
.[x] Ensure that the test suite passes, by running
make test
.[x] Add a new entry to the
.github/CHANGELOG.md
file, summarizing the change, and including a link back to the PR.[x] Ensure that code is properly formatted by running
make format
.When all the above are checked, delete everything above the dashed line and fill in the pull request template.
Context:
PR https://github.com/PennyLaneAI/pennylane/pull/6021 removed code duplication for
BasisEmbedding
andBasisState
. As the resultBasisState
no longer decomposes toBasisStatePreparation
. This PR updates Python unit tests to support this generalization of basis state preparation.PR https://github.com/PennyLaneAI/pennylane/pull/6046 added a facade wrapper class for "legacy" devices. This PR is a follow up to PR #839 updating Multi-GPU LGPU device and tests.
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues: [sc-71917]