Skip to content

Conversation

@CatalinaAlbornoz
Copy link
Contributor

Fix list of generators in the Ising-type Lie algebras section

Closes #1563 .

Fix list of generators in the Ising-type Lie algebras section
@github-actions
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!

@CatalinaAlbornoz CatalinaAlbornoz requested review from KetpuntoG, Qottmann and daniela-angulo and removed request for daniela-angulo November 21, 2025 20:54
@CatalinaAlbornoz CatalinaAlbornoz enabled auto-merge (squash) November 21, 2025 20:56
@github-actions
Copy link

github-actions bot commented Nov 21, 2025

Your preview is ready 🎉!

You can view your changes here

Deployed at: 2025-11-25 18:27:32 UTC

@CatalinaAlbornoz CatalinaAlbornoz requested review from dwierichs and removed request for KetpuntoG November 21, 2025 20:58
@Qottmann
Copy link
Collaborator

I noticed that one_qubit_decomposition raises an odd warning:

U_target = qml.matrix(qml.RZ(-0.5, 0))
decomp = qml.ops.one_qubit_decomposition(U_target, 0, rotations="XYX")
/Users/korbinian.kottmann/Xanadu/pennylane/pennylane/math/decomposition.py:92: RuntimeWarning: invalid value encountered in scalar divide
  2 * math.arccos(math.clip(-math.imag(U[..., 0, 1]) / math.sin(half_lam_plus_phi), -1, 1)),

unrelated to this PR though

Copy link
Contributor

@dwierichs dwierichs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@CatalinaAlbornoz CatalinaAlbornoz merged commit a08e97e into master Nov 25, 2025
12 checks passed
@CatalinaAlbornoz CatalinaAlbornoz deleted the catalina-dla-typo-fix branch November 25, 2025 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Typo in DLA demo

5 participants