Find a better name for KTensor

OpenMined / KotlinSyft

The official Syft worker for secure on-device machine learning

https://www.openmined.org

Apache License 2.0

85 stars 28 forks source link

Find a better name for KTensor #296

Closed galalen closed 3 years ago

galalen commented 3 years ago

Description

KTensor is implemented in dataloader API, it's just a data holder that contains shape and data will help in construct IValue which is used by torch mobile. We're discussing rename it to SyftTensor if the name wouldn't be used for something else.

Additional Context

Add any other context or screenshots.

vkkhare commented 3 years ago

this issue aligns with a larger issue #214

galalen commented 3 years ago

this issue aligns with a larger issue #214

For this one, we can just use torch mobile Tensor class

galalen commented 3 years ago

KTensor is no longer used in the data loader API.