jonasrauber / eagerpy

PyTorch, TensorFlow, JAX and NumPy — all of them natively using the same code
https://eagerpy.jonasrauber.de
MIT License
693 stars 39 forks source link

Define __matmul__ operator. #37

Closed eserie closed 3 years ago

codecov[bot] commented 3 years ago

Codecov Report

Merging #37 (cf6696a) into master (4656839) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #37   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           16        16           
  Lines         1753      1743   -10     
=========================================
- Hits          1753      1743   -10     
Impacted Files Coverage Δ
eagerpy/tensor/jax.py 100.00% <ø> (ø)
eagerpy/tensor/numpy.py 100.00% <ø> (ø)
eagerpy/tensor/pytorch.py 100.00% <ø> (ø)
eagerpy/tensor/tensorflow.py 100.00% <ø> (ø)
eagerpy/tensor/tensor.py 100.00% <100.00%> (ø)

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 4656839...cf6696a. Read the comment docs.

jonasrauber commented 3 years ago

Nice, this allows us to get rid of the framework-specific code duplication for matmul 👍 I guess this breaks compatibility with some older versions of PyTorch, TensorFlow, NumPy that didn't support __matmul__, but I think we can live with that.

jonasrauber commented 3 years ago

resolves #35