Skip to content
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

Merge dev into master #1302

Merged
merged 40 commits into from
Jan 15, 2025
Merged

Merge dev into master #1302

merged 40 commits into from
Jan 15, 2025

Conversation

PietropaoloFrisoni
Copy link
Contributor

No description provided.

actions-user and others added 30 commits November 7, 2024 09:31
**Summary:**

Updating deprecated code that was introduced from a recent deprecation
PennyLaneAI/pennylane#6549.

[sc-77590]
…1266)

Summary:

Updating deprecated code that was introduced from a recent deprecation
PennyLaneAI/pennylane#6583.

[sc-76836]
As the name suggests.

Demo outputs were confirmed to look the same.
This PR updates `pennylane-qrack` and `bluequbit` to the most recent
versions, including changes and updates implemented after the recent
removals of deprecated functionality in PennyLane.
 
This should resolve all the failures on the `dev` branch.

[sc-79563]
**Title:**
Remove QubitDevice warnings, which will be removed in the 0.40PL
**Summary:**
as title
**Relevant references:**
#PennyLaneAI/pennylane#6537
**Possible Drawbacks:**
None
**Related GitHub Issues:**
**Summary:**
The PR updates QSVT on hardware how-to to use PennyLane `poly_to_angles`
function following [sc-79728].

---------

Co-authored-by: Guillermo Alonso-Linaje <[email protected]>
This PR updates "intro to qsvt" and "apply qsvt" to be compatible with
the updated qsvt functionality.
[sc-72638]

---------

Co-authored-by: Soran Jahangiri <[email protected]>
Title:
Update qchem demos to be JAX or JAX/JIT compatible

Summary:
As we phase out autograd, we need to ensure that our current qchem demos
can work with either vanilla numpy or jax. Demos that are JIT compatible
were made to use JIT as well. For demos that don't have any
differentiation, we opt to use vanilla numpy.

Notes: the qubit tapering demo is not JIT compatible due to 2 issues: 1)
Wire ordering is not ascending (Not a bug, but might need attention) 2)
qml.taper_operation() uses the Exp operator, which has a conditional
statement (PennyLaneAI/pennylane#5993)

Relevant references:

Possible Drawbacks:

Related GitHub Issues:
[[sc-69776](https://app.shortcut.com/xanaduai/story/69776)]

---------

Co-authored-by: GitHub Nightly Merge Action <[email protected]>
Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Mikhail Andrenkov <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: Ivana Kurečić <[email protected]>
Co-authored-by: Paul Finlay <[email protected]>
Co-authored-by: Jack Brown <[email protected]>
Co-authored-by: soranjh <[email protected]>
Co-authored-by: bellekaplan <[email protected]>
Co-authored-by: soranjh <[email protected]>
Doesn't build due to the fact that [this
PR](PennyLaneAI/pennylane#6096) is not yet
merged. JSON files need to be updated as well.

**Summary:**
Fixes remaining demos that checked for requires_grad. Won't build for
now due to missing a fix in [this
PR](PennyLaneAI/pennylane#6324)

[[sc-69776](https://app.shortcut.com/xanaduai/story/69776)]
[[sc-69778](https://app.shortcut.com/xanaduai/story/69778)]

---------

Co-authored-by: GitHub Nightly Merge Action <[email protected]>
Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Mikhail Andrenkov <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: Korbinian Kottmann <[email protected]>
Co-authored-by: Ivana Kurečić <[email protected]>
Co-authored-by: Paul Finlay <[email protected]>
Co-authored-by: Jack Brown <[email protected]>
Co-authored-by: bellekaplan <[email protected]>
Co-authored-by: obliviateandsurrender <[email protected]>
Co-authored-by: soranjh <[email protected]>
Co-authored-by: Soran Jahangiri <[email protected]>
This PR updates the Makefile to install PennyLane from the `rc` branch.

*This will be reverted after the feature freeze.*
actions-user and others added 9 commits January 9, 2025 20:59
As the title says.

---------

Co-authored-by: Isaac De Vlugt <[email protected]>
Co-authored-by: JerryChen97 <[email protected]>
Co-authored-by: Isaac De Vlugt <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Alex Preciado <[email protected]>
- Some of the `ops` print differently after switching from `Tensor` to
`Prod`. Cleaned that up.
- The link to the PennyLane-Qiskit device documentation in the Ensemble
PR is a dead link to a device we no longer have (`qiskit.ibmq`). In the
new plugin, we have `qiskit.remote`.

The second point could potentially be resolve differently. The device
name _is_ correct for the version of the plugin that the demo works
with, but the remote device should also be fine with that version. We
could also leave it mentioning `qiskit.ibmq` because it's an old demo,
and just delete the link 🤷‍♀️ But we don't want dead links.
Gets the Quantum Volume and how to use qiskit demos working

---------

Co-authored-by: Astral Cai <[email protected]>
**Context:**

Demo improvements encountered during the v0.40 quality assurance
process.

**Description of the Change:**

A few things were adjusted in this PR,
- Deprecated code update.
- Seed `nx.draw` drawings since they are drawn with random vertex
positions. This leads to false positives in our `qml-diff-checker`
automated script.
- Epoch count to reproduce features from an older version.
- Add note to `qrack.py` about performance being dependent on the
hardware used to execute the demo

---------

Co-authored-by: Alex Preciado <[email protected]>
The order of the operators in the lie closure changed with
PennyLaneAI/pennylane#6724 such that the
hard-coded indices are not correct anymore.

I changed the latter part where we pick the gate in the moment space
according to the $p = O_1 O_2$ that is not in the moment space to
automatically pick it from the list (instead of hard-coding the index).
However, the first choice of $p$ is hard to automate, the only thing
that matters is that $p = O_1 O_2$ is not in the DLA

Note for reviewers:
The important thing to look at is that the outputs are now again `True`
This PR is not completed yet. All plugins and packages must be released,
then I need to run the `poetry lock --no-update` command (but it's
useful to have a PR ready)
@PietropaoloFrisoni PietropaoloFrisoni added the ci:build-all-qml-demos Build all the Demos for this Pull Request label Jan 14, 2025
Copy link

👋 Hey, looks like you've updated some demos!

🐘 Don't forget to update the dateOfLastModification in the associated metadata files so your changes are reflected in Glass Onion (search and recommendations).

Please hide this comment once the field(s) are updated. Thanks!

Copy link

Thank you for opening this pull request.

You can find the built site at this link.

Deployment Info:

  • Pull Request ID: 1302
  • Deployment SHA: f9c2fc718b13d425e713c1c47bd5f23a77c76f5c
    (The Deployment SHA refers to the latest commit hash the docs were built from)

Note: It may take several minutes for updates to this pull request to be reflected on the deployed site.

@PietropaoloFrisoni PietropaoloFrisoni merged commit 5f3c0ef into master Jan 15, 2025
19 checks passed
@PietropaoloFrisoni PietropaoloFrisoni deleted the merge_dev_into_master branch January 15, 2025 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:build-all-qml-demos Build all the Demos for this Pull Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.