diff --git a/docs/overview/python-package.rst b/docs/overview/python-package.rst index fae6a91be..ecd810825 100644 --- a/docs/overview/python-package.rst +++ b/docs/overview/python-package.rst @@ -52,6 +52,39 @@ More specifically, it can be through the following functions: * mgr.eval_async__def - Runs operation asynchronously under a new anonymous sequence * seq.record_ - Records operation in sequence (requires sequence to be in recording mode) +Tensor Component +------------------ + +The `kp.Tensor` component provides utilities to load and manage data into GPU memory. + +The primary interface to the GPU image leverage `np.array` containers which wrap the GPU memory. + +One of the key things to take into consideration is the GPU memory and resource management that is provided by Kompute - namely the `kp.Tensor` allows for the memory to be managed until the python object refcount goes down to zero or is explicitly destroyed with the `destroy()` function. + +Another thing to bare in mind is that when the `.data()` function is called, the numpy array would add an extra refcount, and the underlying resources won't be destroyed until that object is destroyed. This is shown more intuitively in the example below: + +.. code-block:: python + :linenos: + + m = kp.Manager() + + t = m.tensor([1,2,3]) + + td = t.data() + + del t + + td + # this is OK + + assert td.base.is_init() == True # OK + + m.destroy() # Frees all memory inside tensors + + assert td.base.is_init() == False # Consistent to expected setup + + del td # Now this calls tensor destructor as refcount reaches 0 + Log Level Configuration ^^^^^^ diff --git a/python/src/main.cpp b/python/src/main.cpp index 84d874594..e05e19320 100644 --- a/python/src/main.cpp +++ b/python/src/main.cpp @@ -95,18 +95,17 @@ PYBIND11_MODULE(kp, m) { py::class_>(m, "Tensor", DOC(kp, Tensor)) .def("data", [](kp::Tensor& self) { // Non-owning container exposing the underlying pointer - py::str dummyDataOwner; // Explicitly request data to not be owned by np switch (self.dataType()) { case kp::Tensor::TensorDataTypes::eFloat: - return py::array(self.size(), self.data(), dummyDataOwner); + return py::array(self.size(), self.data(), py::cast(&self)); case kp::Tensor::TensorDataTypes::eUnsignedInt: - return py::array(self.size(), self.data(), dummyDataOwner); + return py::array(self.size(), self.data(), py::cast(&self)); case kp::Tensor::TensorDataTypes::eInt: - return py::array(self.size(), self.data(), dummyDataOwner); + return py::array(self.size(), self.data(), py::cast(&self)); case kp::Tensor::TensorDataTypes::eDouble: - return py::array(self.size(), self.data(), dummyDataOwner); + return py::array(self.size(), self.data(), py::cast(&self)); case kp::Tensor::TensorDataTypes::eBool: - return py::array(self.size(), self.data(), dummyDataOwner); + return py::array(self.size(), self.data(), py::cast(&self)); default: throw std::runtime_error("Kompute Python data type not supported"); } diff --git a/python/test/test_tensor_types.py b/python/test/test_tensor_types.py index fb11083fd..ee645d850 100644 --- a/python/test/test_tensor_types.py +++ b/python/test/test_tensor_types.py @@ -207,3 +207,26 @@ def test_type_unsigned_int(): assert np.all(tensor_out.data() == arr_in_a * arr_in_b) +def test_tensor_numpy_ownership(): + + arr_in = np.array([1, 2, 3]) + + m = kp.Manager() + + t = m.tensor(arr_in) + + # This should increment refcount for tensor sharedptr + td = t.data() + + assert td.base.is_init() == True + assert np.all(td == arr_in) + + del t + + assert td.base.is_init() == True + assert np.all(td == arr_in) + + m.destroy() + + assert td.base.is_init() == False +