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

Update minimum catalyst version #5320

Merged

Conversation

erick-xanadu
Copy link
Contributor

Context: pennylane==0.35.0 and pennylane-catalyst==0.4.1 have some failing test.

Description of the Change: Update the minimum version of catalyst.

Benefits: Versions are compatible.

Copy link
Contributor

github-actions bot commented Mar 6, 2024

Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md with:

  • A one-to-two sentence description of the change. You may include a small working example for new features.
  • A link back to this PR.
  • Your name (or GitHub username) in the contributors section.

Copy link

codecov bot commented Mar 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.63%. Comparing base (3bb04be) to head (1e7665f).
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5320      +/-   ##
==========================================
- Coverage   99.63%   99.63%   -0.01%     
==========================================
  Files         400      400              
  Lines       37132    36870     -262     
==========================================
- Hits        36998    36735     -263     
- Misses        134      135       +1     

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

Copy link
Contributor

@dime10 dime10 left a comment

Choose a reason for hiding this comment

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

This looks good from my side, although the PennyLane team should have a look as well, especially at the implications (bug fix release?).

@erick-xanadu erick-xanadu enabled auto-merge (squash) March 8, 2024 17:12
@erick-xanadu erick-xanadu merged commit a4f0fff into master Mar 8, 2024
40 checks passed
@erick-xanadu erick-xanadu deleted the eochoa/2024-04-06/update-catalyst-minimum-version branch March 8, 2024 17:45
Mandrenkov pushed a commit that referenced this pull request Mar 8, 2024
**Context:** `pennylane==0.35.0` and `pennylane-catalyst==0.4.1` have
some failing test.

**Description of the Change:** Update the minimum version of catalyst.

**Benefits:** Versions are compatible.
Mandrenkov added a commit that referenced this pull request Mar 8, 2024
**Context:**

There are two compatibility issues with the v0.35.0 release of PennyLane:
1. The minimum required Catalyst version is now v0.5.0 (#5320).
2. Lightning simulators need special handling of diagonalizing gates when performing sampling measurements (#5343).

This PR cherry-picks the squashed commit corresponding to (1) into the RC branch as the change necessary for (2) is already being made directly to the RC branch.

**Description of the Change:**

* Bumped the PennyLane Python package version to `v0.35.1`.
* Cherry-picked the following commits into the RC branch: a4f0fff and 79ce676.

**Benefits:**

* The latest PennyLane release works with the latest releases of Catalyst and Lightning.

**Possible Drawbacks:**

None.

**Related GitHub Issues:**

None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants