Skip to content
This repository has been archived by the owner on Feb 5, 2024. It is now read-only.

Rename qubitstatevector to stateprep #134

Merged
merged 31 commits into from
Aug 23, 2023
Merged

Rename qubitstatevector to stateprep #134

merged 31 commits into from
Aug 23, 2023

Conversation

multiphaseCFD
Copy link
Member

@multiphaseCFD multiphaseCFD commented Aug 21, 2023

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:

Description of the Change:

  • Introduce StatePrep in lieu of QubitStateVector in LightningGPU following recent changes in PennyLane PR#4450.
  • Deprecate Python 3.8.
  • Update Lightning imports.

Benefits:

Possible Drawbacks:

Related GitHub Issues:

@multiphaseCFD multiphaseCFD added the ci:use-multi-gpu-runner Enable usage of Multi-GPU runner for this Pull Request label Aug 21, 2023
@github-actions
Copy link

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.

@multiphaseCFD multiphaseCFD changed the title Rename stateprep Rename qubitstatevector to stateprep Aug 21, 2023
.github/workflows/tests_linux_x86.yml Show resolved Hide resolved
requirements.txt Show resolved Hide resolved
requirements.txt Outdated Show resolved Hide resolved
Copy link
Member

@mlxd mlxd left a comment

Choose a reason for hiding this comment

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

Great work here folks

@mlxd mlxd requested a review from AmintorDusko August 22, 2023 20:34
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.

Approved ;)

Copy link
Contributor

@AmintorDusko AmintorDusko left a comment

Choose a reason for hiding this comment

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

Looks good to me!

mpitests/test_adjoint_jacobian.py Show resolved Hide resolved
@vincentmr vincentmr merged commit 2cd8247 into main Aug 23, 2023
13 checks passed
@vincentmr vincentmr deleted the rename_stateprep branch August 23, 2023 12:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ci:use-multi-gpu-runner Enable usage of Multi-GPU runner for this Pull Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants