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

Put back Python_EXECUTABLE in setup.py. #461

Merged
merged 7 commits into from
Jun 21, 2023
Merged

Put back Python_EXECUTABLE in setup.py. #461

merged 7 commits into from
Jun 21, 2023

Conversation

vincentmr
Copy link
Contributor

@vincentmr vincentmr commented Jun 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:
The last dev release of Lightning has broken wheels. The incorrect Python version is found by CMake and hence all modules are built against Python 3.6, rather than the required version.

Description of the Change:
Specify version number from sys in setup.py.

Benefits:
Fixes wheels.

Possible Drawbacks:

Related GitHub Issues:

@github-actions
Copy link
Contributor

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.

@vincentmr vincentmr marked this pull request as ready for review June 21, 2023 17:08
@vincentmr
Copy link
Contributor Author

Should we update the changelog for this?

@mlxd
Copy link
Member

mlxd commented Jun 21, 2023

Should we update the changelog for this?

Yep :)

@codecov
Copy link

codecov bot commented Jun 21, 2023

Codecov Report

Merging #461 (aeb0d07) into master (1856f47) will not change coverage.
The diff coverage is 100.00%.

❗ Current head aeb0d07 differs from pull request most recent head 68c6300. Consider uploading reports for the commit 68c6300 to get more accurate results

@@           Coverage Diff           @@
##           master     #461   +/-   ##
=======================================
  Coverage   99.82%   99.82%           
=======================================
  Files          50       50           
  Lines        4667     4667           
=======================================
  Hits         4659     4659           
  Misses          8        8           
Impacted Files Coverage Δ
pennylane_lightning/_version.py 100.00% <100.00%> (ø)

@vincentmr vincentmr requested review from AmintorDusko and mlxd June 21, 2023 17:40
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 now! Thank you for that!

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.

Thanks for the quick turnaround @vincentmr
Feel free to merge when CI is done.

@vincentmr vincentmr merged commit 2a27cf6 into master Jun 21, 2023
@vincentmr vincentmr deleted the bugfix/pythonver branch June 21, 2023 17:57
vincentmr added a commit that referenced this pull request Jun 21, 2023
* Put back Python_EXECUTABLE in setup.py.

* Auto update version

* Trigger CI

* Fix PYTHON_EXECUTABLE case.

* Fix PYTHON_EXECUTABLE case MacOS.

* Update changelog [skip ci].

* Trigger CI

---------

Co-authored-by: Dev version update bot <github-actions[bot]@users.noreply.github.com>
mlxd added a commit that referenced this pull request Jun 26, 2023
* Initial commit for Lightning v0.31.0_rc

* Update changelog.

* Bump pennylane_requires to 0.30.

* Add git safe dir override for Kokkos docker builds (#452)

* Add git safe dir override for Kokkos docker builds

* Auto update version

* Update changelog

---------

Co-authored-by: Dev version update bot <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Vincent Michaud-Rioux <[email protected]>

* Specify PYTHON_EXECUTABLE in CMAKE_ARGS.

* Fix case of Python_EXECUTABLE CMAKE var.

* Put back Python_EXECUTABLE in setup.py. (#461)

* Put back Python_EXECUTABLE in setup.py.

* Auto update version

* Trigger CI

* Fix PYTHON_EXECUTABLE case.

* Fix PYTHON_EXECUTABLE case MacOS.

* Update changelog [skip ci].

* Trigger CI

---------

Co-authored-by: Dev version update bot <github-actions[bot]@users.noreply.github.com>

* Build against PL-rc branch.

* --force-reinstall autograd numpy==1.23.5

* Revert changes.

* Update changelog.

* Revert changes made on wrong branch.

* Trigger CI

* Auto update version

* Revert RC test branches to master

---------

Co-authored-by: Vincent Michaud-Rioux <[email protected]>
Co-authored-by: Dev version update bot <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Vincent Michaud-Rioux <[email protected]>
Co-authored-by: Amintor Dusko <[email protected]>
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.

3 participants