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

Fixes for 0.31 release #835

Merged
merged 8 commits into from
Jun 23, 2023
Merged

Fixes for 0.31 release #835

merged 8 commits into from
Jun 23, 2023

Conversation

mudit2812
Copy link
Contributor

@mudit2812 mudit2812 commented Jun 22, 2023

Summary:

  • ensemble_multi_qpu.py prints a warning due to permuting a zipped array of features and classes with ragged dimensions. This PR updates the code so that the permutation for the feature array and class array happens separately so that the warning is not raised.
  • Updated tutorial_quantum_circuit_cutting.py to remove use of the do_queue keyword.

@mudit2812 mudit2812 marked this pull request as ready for review June 22, 2023 14:26
@mudit2812 mudit2812 requested a review from a team June 22, 2023 14:35
Copy link
Contributor

@timmysilv timmysilv left a comment

Choose a reason for hiding this comment

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

snazzy fix 🕴️

@github-actions
Copy link

github-actions bot commented Jun 22, 2023

Thank you for opening this pull request.

You can find the built site at this link.

Deployment Info:

  • Pull Request ID: 835
  • Deployment SHA: bfeafc4fccf5ec84e68d3983435b5cb8eab90a14
    (The Deployment SHA refers to the latest commit hash the docs were built from)

Note: It may take several minutes for updates to this pull request to be reflected on the deployed site.

@mudit2812 mudit2812 marked this pull request as draft June 22, 2023 18:28
@mudit2812 mudit2812 changed the title Remove ragged array warning from Ensemble Classification demo Fixes for 0.31 release Jun 22, 2023
@mudit2812 mudit2812 marked this pull request as ready for review June 22, 2023 19:12
Copy link
Contributor

@timmysilv timmysilv 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 getting rid of queuing there! I suggested the removal of an extra print, but it didn't remove the line below it so black might fail. lgtm otherwise

demonstrations/tutorial_quantum_circuit_cutting.py Outdated Show resolved Hide resolved
@mudit2812 mudit2812 requested review from albi3ro and a team and removed request for albi3ro June 23, 2023 19:13
@mudit2812 mudit2812 enabled auto-merge (squash) June 23, 2023 20:59
Copy link
Contributor

@BorjaRequena BorjaRequena left a comment

Choose a reason for hiding this comment

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

Nice fix! It even looks cleaner than before 😄

@mudit2812 mudit2812 merged commit 08206a6 into dev Jun 23, 2023
@mudit2812 mudit2812 deleted the 0.31_changes_mp branch June 23, 2023 21:13
josh146 added a commit that referenced this pull request Jun 27, 2023
* Development branch

* Bumped requirement package versions (#667)

* Bumped jax versions

* Updated neural-tangents

* Merge dev in master following v0.28.0 release of PennyLane (#671) (#672)

* Development branch

* Bumped requirement package versions (#667)

* Bumped jax versions

* Updated neural-tangents

* Apply suggestions from code review

Co-authored-by: Josh Izaac <[email protected]>
Co-authored-by: GitHub Nightly Merge Action <[email protected]>
Co-authored-by: Rashid N H M <[email protected]>

Co-authored-by: Jay Soni <[email protected]>
Co-authored-by: Josh Izaac <[email protected]>
Co-authored-by: GitHub Nightly Merge Action <[email protected]>

* Update chemical reactions demo to use pyscf for openshell H3 (#704)

* use pyscf for openshell H3

* change date and fix interface warning

* Change ibm device in Quantum volume (#703)

* Ibm oslo

* Update to new hardware

* Change to Lima

* Typo

* Replace

* One below

* Update demos to remove warnings (#705)

* Added interface, removed deprecated functionality

* Addressing PR comments

---------

Co-authored-by: Romain Moyard <[email protected]>

* Update classical shadows (#708)

* More

* typo

* Trigger CI

* lightning req no depth

* Trigger CI

* Update SPSA demo to count circuit executions manually. (#706)

* new version of SPSA demo

* Update demonstrations/spsa.py

Co-authored-by: Josh Izaac <[email protected]>

* simplify code, update outputs and figures

* Apply suggestions from code review

Co-authored-by: Romain Moyard <[email protected]>

* review and tweaking

* make spsa an executing tutorial

* move file

* fix links

* fix some more links

* author bio

* line breaks, undo link change, fixing them actually

* links...

* links again... bio

* update to count executions manually

* reword device executions -> circuit executions. avoid recreating device and qnode

* print executions

---------

Co-authored-by: Josh Izaac <[email protected]>
Co-authored-by: Romain Moyard <[email protected]>

* Updated adaptive circuits demo for new return type (#735)

* Update automatically run demos to be able to execute with v0.30 (#741)

* fix toric code and classical kernels demos (#740)

* FIx photonics and noisy demos (#742)

* FIx two demos

* small change

* wrap relevant results in np.array (#743)

* Updated demos to work with return types (#744)

* Update demos for new return types (#754)

* remove pennylane-sf from dependencies

* try to use developement lightning

* force upgrade of lightnign

* make tutorial_gbs non-executable

---------

Co-authored-by: Romain Moyard <[email protected]>
Co-authored-by: Matthew Silverman <[email protected]>
Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: lillian542 <[email protected]>

* remove nlopt from list of requirements (#755)

* use dev version of lightning for demo checker (#759)

* Improve formatting of output in toric code demo (#762)

* round outputs on toric code

* Update demonstrations/tutorial_toric_code.py

Co-authored-by: Mudit Pandey <[email protected]>

---------

Co-authored-by: Matthew Silverman <[email protected]>
Co-authored-by: Mudit Pandey <[email protected]>

* use lightning in quantumvolume; fix qnspsa for new return; use H.sparse_matrix (#757)

* use lightning in quantumvolume; fix qnspsa for new return

* stop using sparse_hamiltonian

* Update manually run demos (#766)

* Added changes to manual demos

* Updated output as found in rerun

* Trigger CI

* Add warnings to pennylane-sf plugins and expected output (#761)

* add warnings to pennylane-sf plugins, add output

* Apply suggestions from code review

Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Josh Izaac <[email protected]>

* add warning to quantum neural net

---------

Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Josh Izaac <[email protected]>

* dev (#824)

* Update requirements.txt (#825)

* Update resource estimation demo for non-cubic systems (#823)

* Change private methods in adjoint diff tutorial (#832)

* adjoint diff tutorial improvement

* Apply suggestions from code review

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Matthew Silverman <[email protected]>

* update date

---------

Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: Matthew Silverman <[email protected]>

* Add fermionic operators demo (#813)

* Fixes for 0.31 release (#835)

* Updated ensemble demo

* Trigger ci

* Update circuit cutting demo

* Updated stop_recording

* Update tutorial to not rely on tape context

* Update demonstrations/tutorial_quantum_circuit_cutting.py

Co-authored-by: Matthew Silverman <[email protected]>

---------

Co-authored-by: Matthew Silverman <[email protected]>

* Modify image in fermionic demo (#838)

* use new image

* Added hero_illustrationd folder to _static, added fermionic_ops_hero.png, modified tutorial_fermionic_operators.py and metadata to add hero image and delete previous image.

---------

Co-authored-by: CatalinaAlbornoz <[email protected]>

* use pinned versions of pennylane and plugins (#840)

---------

Co-authored-by: GitHub Nightly Merge Action <[email protected]>
Co-authored-by: Rashid N H M <[email protected]>
Co-authored-by: Jay Soni <[email protected]>
Co-authored-by: soranjh <[email protected]>
Co-authored-by: Romain Moyard <[email protected]>
Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: David Wierichs <[email protected]>
Co-authored-by: trbromley <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Matthew Silverman <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Tom Bromley <[email protected]>
Co-authored-by: Guillermo Alonso-Linaje <[email protected]>
Co-authored-by: CatalinaAlbornoz <[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.

4 participants