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

Remove CPhase + Tidy up src #717

Merged
merged 20 commits into from
May 10, 2024
Merged

Remove CPhase + Tidy up src #717

merged 20 commits into from
May 10, 2024

Conversation

maliasadi
Copy link
Member

@maliasadi maliasadi commented May 7, 2024

  • Remove CPhase in favour of CPhaseShift in Lightning devices. qml.CPhase is an alias for qml.ControlledPhaseShift and can be removed from the the list of supported ops and toml files.
  • Update the name of CI C++ and Python tests to include the matrix strategy info in the list of required actions.
  • Tidy up a few files and scripts.

[sc-62667]
[sc-58634]

Copy link
Contributor

github-actions bot commented May 7, 2024

Hello. You may have forgotten to update the changelog!
Please edit .github/CHANGELOG.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
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.

Thanks Ali!

Copy link

codecov bot commented May 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.35%. Comparing base (279f7e8) to head (5380a99).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #717      +/-   ##
==========================================
+ Coverage   93.91%   95.35%   +1.44%     
==========================================
  Files         107       91      -16     
  Lines       16244    14073    -2171     
==========================================
- Hits        15255    13419    -1836     
+ Misses        989      654     -335     

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

@maliasadi maliasadi added the ci:build_wheels Activate wheel building. label May 7, 2024
@maliasadi maliasadi requested a review from vincentmr May 7, 2024 21:16
@maliasadi maliasadi changed the title Remove CPhase + Add the release notes to docs + Tidy up src Remove CPhase + Tidy up src May 7, 2024
Copy link
Member

@multiphaseCFD multiphaseCFD left a comment

Choose a reason for hiding this comment

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

Thanks @maliasadi ! LGTM!

Copy link
Contributor

@vincentmr vincentmr left a comment

Choose a reason for hiding this comment

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

A couple questions, but ready to approve ... so I'll approve. Thanks @maliasadi , nice one.

Makefile Outdated Show resolved Hide resolved
.github/CHANGELOG.md Show resolved Hide resolved
@maliasadi maliasadi merged commit c164f2e into master May 10, 2024
96 of 104 checks passed
@maliasadi maliasadi deleted the maa/fix-cphase-minor branch May 10, 2024 20:25
multiphaseCFD pushed a commit that referenced this pull request May 13, 2024
* Remove CPhase; Tidy up NDPermuter, gh scripts, and doc

* Add release_notes to docs

* Auto update version from '0.37.0-dev0' to '0.37.0-dev2'

* Update changelog

* Auto update version from '0.37.0-dev2' to '0.37.0-dev3'

* trigger ci

* Revert change in config.h

* Update

* Update

* Update

* Auto update version from '0.37.0-dev3' to '0.37.0-dev4'

* Fix _version

* Auto update version from '0.37.0-dev6' to '0.37.0-dev7'

* Update format

* trigger ci

---------

Co-authored-by: ringo-but-quantum <github-ringo-but-quantum@xanadu.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:build_wheels Activate wheel building.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants