Refactoring Codebase

[czgdp1807]

Description of the problem

Currently the code base has mixed up CUDA kernels with C++ class and the API is too confusing. Some changes are to be made summarised in the following points,

• Remove fill method from both Matrix and Vector classes - The first phase of refactoring should include removing fill method from both of these classes and their GPU counter parts and shifting it to operations module. This will avoid kernel calls inside class methods and the APIs for Matrix and Vector will be unambiguous.

• Shift product and multiply functions to operations module - The reason for doing this are similar to the above.

• Discuss API for using streams i.e., working on Using non-default streams in CUDA #2. Currently, methods like, Vector.fill decipher from the block_size whether to use GPU or not. Instead separate functions should be used for GPU and CPU with clean APIs.

Example of the problem

References/Other comments

[purva98]

Hello, can me and my teammate riddhi99 also try contributing to this current open issue?

[czgdp1807]

@purva98 There is an open PR already working on this issue. See, #13

[sreshu]

Is this issue still open?

[czgdp1807]

Yes. @sreshu