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

V0.14.1 bump #1083

Merged
merged 8 commits into from
Feb 12, 2021
Merged

V0.14.1 bump #1083

merged 8 commits into from
Feb 12, 2021

Conversation

josh146
Copy link
Member

@josh146 josh146 commented Feb 12, 2021

  • Merges the bugfix PRs in the v0.14.1 milestone into a holding branch for the v0.14.1 release.

  • Bumps the version number

  • Updates the changelog

.github/CHANGELOG.md Outdated Show resolved Hide resolved
@@ -865,6 +884,7 @@ def qfunc_decorator(func):
interface=interface,
diff_method=diff_method,
mutable=mutable,
max_expansion=max_expansion,
Copy link
Contributor

Choose a reason for hiding this comment

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

I know this has already been code reviewed, but for my understanding - we have added the max_expansion attribute but have we implemented the effects of changing max_expansion?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep! The change is on line 781, max_expansion=10.

Copy link
Contributor

@trbromley trbromley left a comment

Choose a reason for hiding this comment

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

Just to check, the v0.14.1-bugfix branch is currently level with the v0.14.0 release?

Copy link
Contributor

@antalszava antalszava 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! 💯

Comment on lines +489 to +491
if all(c == 0 for c in coeffs) or not coeffs:
self.cost_fn = lambda *args, **kwargs: np.array(0)
return
Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you for relocating this! 🚀

tests/devices/test_default_qubit_jax.py Show resolved Hide resolved
Co-authored-by: Tom Bromley <49409390+trbromley@users.noreply.github.com>
Co-authored-by: antalszava <antalszava@gmail.com>
@codecov
Copy link

codecov bot commented Feb 12, 2021

Codecov Report

Merging #1083 (a95ad63) into v0.14.1-bugfix (4fffb1b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@               Coverage Diff               @@
##           v0.14.1-bugfix    #1083   +/-   ##
===============================================
  Coverage           97.75%   97.75%           
===============================================
  Files                 153      153           
  Lines               11517    11530   +13     
===============================================
+ Hits                11258    11271   +13     
  Misses                259      259           
Impacted Files Coverage Δ
pennylane/_version.py 100.00% <100.00%> (ø)
pennylane/devices/default_qubit_autograd.py 100.00% <100.00%> (ø)
pennylane/devices/default_qubit_jax.py 94.02% <100.00%> (+0.27%) ⬆️
pennylane/devices/default_qubit_tf.py 90.66% <100.00%> (+0.52%) ⬆️
pennylane/tape/qnode.py 99.23% <100.00%> (ø)
pennylane/vqe/vqe.py 93.88% <100.00%> (+0.10%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4fffb1b...63adb9b. Read the comment docs.

@josh146 josh146 merged commit 149a785 into v0.14.1-bugfix Feb 12, 2021
@josh146 josh146 deleted the v0.14.1-bump branch February 12, 2021 13:58
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