OpenMined / PySyft-TensorFlow

SOON TO BE DEPRECATED - The TensorFlow bindings for PySyft
Apache License 2.0
57 stars 11 forks source link

Abstract duplicate code in TensorFlowTensor & TensorFlowVariable #22

Open jvmncs opened 4 years ago

jvmncs commented 4 years ago

To do this properly, we'll want to move some of the code in TorchTensor from syft/frameworks/torch/tensors/interpreters/native.py into the AbstractTensor at syft/generic/tensor.py, and then call most of our code for these classes from the AbstractTensor base for TFTensor & TFVar.

jvmncs commented 4 years ago

KerasObject seems to fulfill similar functions as well -- the class hierarchy here should probably be reviewed