clvcooke / MatrixMath

A from scratch implementation of basic matrix manipulation and operations in C++
0 stars 2 forks source link

Should there be a separate hpp/cpp file for utility functions #9

Open clvcooke opened 9 years ago

clvcooke commented 9 years ago

What does everyone think of moving quite a few of the functions out of the matrix class and into a new hpp/cpp utility functions file. This would allow us to make use of the functions without being restricted to having to create a Matrix object to do so.

ericye16 commented 9 years ago

What functions do you have in mind?

On Fri, 5 Dec 2014 17:57 Colin Langdon Vernon Cooke < notifications@github.com> wrote:

What does everyone think of moving quite a few of the functions out of the matrix class and into a new hpp/cpp utility functions file. This would allow us to make use of the functions without being restricted to having to create a Matrix object to do so.

— Reply to this email directly or view it on GitHub https://github.com/clvc/MatrixMath/issues/9.

RowanFerrabee commented 9 years ago

Ex: Matrix transpose(Matrix); rather than Matrix Matrix::transpose();

clvcooke commented 9 years ago

Possibly the inverse, transpose, determinant and cof. But now that I think about it the implementation wouldn't change much. The code would look more like normal math, but almost nothing else would change. Now that I think about it, its probably not worth it just to make the code look more like my math notes

RowanFerrabee commented 9 years ago

Well the whole point of this class is to make Matrices easy and intuitive to use for coders so it may be worth it to add those.