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

Added power function to Jax backend and test. #884

Merged
merged 13 commits into from
Dec 7, 2020

Conversation

LuiGiovanni
Copy link
Contributor

@LuiGiovanni LuiGiovanni commented Dec 5, 2020

This is in relation to #840
Specifically adding to the Jax backend files, where I implement a power function and added the test function for said power function.

@google-cla google-cla bot added the cla: yes label Dec 5, 2020
@LuiGiovanni LuiGiovanni marked this pull request as draft December 5, 2020 08:31
@LuiGiovanni LuiGiovanni marked this pull request as ready for review December 5, 2020 09:30
@codecov-io
Copy link

codecov-io commented Dec 6, 2020

Codecov Report

Merging #884 (3624ccf) into master (2d78c61) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #884   +/-   ##
=======================================
  Coverage   98.00%   98.00%           
=======================================
  Files         129      129           
  Lines       22604    22619   +15     
=======================================
+ Hits        22152    22167   +15     
  Misses        452      452           
Impacted Files Coverage Δ
tensornetwork/backends/jax/jax_backend.py 98.85% <100.00%> (+<0.01%) ⬆️
tensornetwork/backends/jax/jax_backend_test.py 99.27% <100.00%> (+0.01%) ⬆️

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 2d78c61...3624ccf. Read the comment docs.

Copy link
Contributor

@alewis alewis left a comment

Choose a reason for hiding this comment

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

LGTM

@alewis alewis merged commit 436db76 into google:master Dec 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants