v0.35.1_rc

[vincentmr]

Before submitting

Please complete the following checklist when submitting a PR:

• 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!

• 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.

• Ensure that the test suite passes, by running make test.

• Add a new entry to the .github/CHANGELOG.md file, summarizing the
  change, and including a link back to the PR.

• 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:
Release v0.35.1 introduces more comprehensive testing for finite-shot execution. Some bugs that were uncovered are fixed in each backend.

Description of the Change:
Add finite-shot execution tests and correctly apply diagonalizing gates for observables that require it.

Benefits:

Possible Drawbacks:

Related GitHub Issues:

[codecov]

Codecov Report

Attention: Patch coverage is 32.95455% with 59 lines in your changes are missing coverage. Please review.

Project coverage is 97.43%. Comparing base (ef0638a) to head (287027f).

Files	Patch %	Lines
...ane_lightning/lightning_kokkos/lightning_kokkos.py	3.33%	29 Missing ⚠️
...ylane_lightning/lightning_qubit/lightning_qubit.py	14.70%	29 Missing ⚠️
pennylane_lightning/core/lightning_base.py	80.00%	1 Missing ⚠️

Additional details and impacted files

@@                 Coverage Diff                 @@
##           v0.35.0_release     #633      +/-   ##
===================================================
- Coverage            98.70%   97.43%   -1.27%     
===================================================
  Files                  169      203      +34     
  Lines                24020    29445    +5425     
===================================================
+ Hits                 23708    28689    +4981     
- Misses                 312      756     +444     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[vincentmr]

@mlxd Yep, about time to add that label ;)

[AmintorDusko]

Thank you, @vincentmr! Nice work here!

[maliasadi]

why is this lazy set to False required?

[vincentmr]

@albi3ro suggested this change. The doc says

If lazy=False, operation-specific adjoint decompositions are first attempted.

which makes it more likely to give another "native" operation, instead of a nested operation class. So I think in essence this forces, for example, qml.Adjoint(qml.RX(0.1)) to qml.RX(-0.1) instead of something fancier.

[maliasadi]

🎉