TensorNetwork icon indicating copy to clipboard operation
TensorNetwork copied to clipboard

Added pinv to tensorflow backend

Open raunakbhupal opened this issue 4 years ago • 4 comments

This is done with respect to issue: #844 pinv function is added to tensornetwork/backends/tensorflow/tensorflow_backend.py The pinv function has been referred from https://numpy.org/doc/stable/reference/generated/numpy.linalg.pinv.html Please do have a look and let me know if should make any other change. Thank you.

raunakbhupal avatar Apr 01 '21 17:04 raunakbhupal

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Apr 01 '21 17:04 google-cla[bot]

@googlebot I signed it!

raunakbhupal avatar Apr 01 '21 17:04 raunakbhupal

@alewis Can you please review this and let me know if there are any changes to be made?

raunakbhupal avatar Apr 01 '21 18:04 raunakbhupal

Codecov Report

Merging #917 (14016a1) into master (6ac507a) will decrease coverage by 0.00%. The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #917      +/-   ##
==========================================
- Coverage   98.12%   98.11%   -0.01%     
==========================================
  Files         129      129              
  Lines       22790    22792       +2     
==========================================
+ Hits        22362    22363       +1     
- Misses        428      429       +1     
Impacted Files Coverage Δ
...rnetwork/backends/tensorflow/tensorflow_backend.py 97.68% <50.00%> (-0.56%) :arrow_down:

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 6ac507a...14016a1. Read the comment docs.

codecov-io avatar Apr 02 '21 07:04 codecov-io