tests #36

Merged
andrey merged 70 commits from tests into master 2020-09-15 15:40:57 +03:00
Owner
No description provided.
andrey reviewed 2020-09-15 15:27:35 +03:00
andrey left a comment
Author
Owner

test

test
@@ -361,2 +791,2 @@
Type determinant(bool * ok = 0) const {
PIMathMatrix(const uint cols = 0, const uint rows = 0, const Type &f = Type()) { _V2D::resize(rows, cols, f); }
Author
Owner

Документация и тесты для конструкторов

Документация и тесты для конструкторов
andrey marked this conversation as resolved
@@ -363,0 +824,4 @@
}
/**
* @brief Creates a matrix whose row equal to vector
Author
Owner

Подробное описание

Подробное описание
andrey marked this conversation as resolved
@@ -363,0 +841,4 @@
/**
* @brief Set the selected column in matrix
*
Author
Owner

уточнить, Размер вектора должен соотвествовать размеру матрицы

уточнить, Размер вектора должен соотвествовать размеру матрицы
@@ -363,0 +853,4 @@
/**
* @brief Set the selected row in matrix
*
Author
Owner
[см](https://git.shs.tools/SHS/pip/pulls/36/files#issuecomment-1026)
@@ -363,0 +864,4 @@
}
/**
* @brief Method which changes selected rows in a matrix
Author
Owner

исправить

исправить
@@ -363,0 +920,4 @@
*
* @return true if matrix is null, else false
*/
bool isNull() const {
Author
Owner

переименовать в isZeroFilled

переименовать в isZeroFilled
@@ -363,0 +949,4 @@
* @param sm matrix for the compare
* @return if matrices are equal true, else false
*/
bool operator==(const _CMatrix &sm) const {
Author
Owner

добавить сравнение размера матриц

добавить сравнение размера матриц
@@ -363,0 +1051,4 @@
/**
* @brief Determinant of the matrix is calculated
*
Author
Owner

параметр (bool *ok = 0) не документирован

параметр (bool *ok = 0) не документирован
@@ -428,0 +1134,4 @@
*
* @return inverted matrix
*/
_CMatrix &invert(bool *ok = 0, _CMCol *sv = 0) {
Author
Owner

разобраться с _CMCol *sv = 0

разобраться с _CMCol *sv = 0
andrey marked this conversation as resolved
@@ -498,3 +1245,4 @@
}
/// Multiply matrices {CR x Rows0} on {Cols1 x CR}, result is {Cols1 x Rows0}
Author
Owner

Отформатировать и дополнить документацию

Отформатировать и дополнить документацию
andrey marked this conversation as resolved
@@ -18,2 +4,2 @@
else
{
template<typename Type>
bool cmpSquareMatrixWithValue(PIMathMatrix<double> matrix, Type val, int num) {
Author
Owner

Если уж делаете шаблонную функцию то должно быть тогда PIMathMatrix, иначе нафига вам вообще шаблоны тут?

Если уж делаете шаблонную функцию то должно быть тогда PIMathMatrix<Type>, иначе нафига вам вообще шаблоны тут?
andrey merged commit aae59106f0 into master 2020-09-15 15:40:57 +03:00
Sign in to join this conversation.