Фоменко Степан Владимирович zzuummaa
  • Joined on 2020-04-23
zzuummaa commented on pull request SHS/pip#38 2020-09-24 14:38:07 +03:00
pimathvector_tests

Ничего не понятно в документации.

zzuummaa commented on pull request SHS/pip#38 2020-09-24 14:38:07 +03:00
pimathvector_tests

Документация не соответствует коду: вычисляется скалярное произведение а не его модуль.

zzuummaa approved SHS/pip#38 2020-09-24 14:38:07 +03:00
pimathvector_tests
zzuummaa commented on pull request SHS/pip#38 2020-09-24 12:27:05 +03:00
pimathvector_tests

При сравнении объектов класса PIMathVector с эелментами вещественных типов оператор == работает не корректно, сравнивая элементы на прямую через ==.

zzuummaa commented on pull request SHS/pip#38 2020-09-24 12:05:50 +03:00
pimathvector_tests

В тестах заменить магические константы чем то осмысленным.

zzuummaa commented on pull request SHS/pip#38 2020-09-24 11:52:14 +03:00
pimathvector_tests

Нужно постараться избегать рекурсивных определений. Пример PIMathVector::length() и PIMathVector::size(). По описанию невозможно понять что делают методы.

zzuummaa commented on pull request SHS/pip#38 2020-09-24 11:23:38 +03:00
pimathvector_tests

Просьба описывать в документации логику возвращения this более точно. Например метод PIMathVectorT::fill():

/**
* @brief Method that fills a vector with a value
*
* @param v value of which the vector is filled
* @return vector of type PIMathVectorT filled with "v"
*/
_CVector & fill(const Type & v) {PIMV_FOR(i, 0) c[i] = v; return *this;}

В блоке @return описание слишком расплывчато и не отражает того факта, что возвращаемый вектор- это тот же самый вектор, у которого был вызван метод fill().

Необходимо уточнить в документации логику возвращаемого значения для классов PIMathMatrixT, PIMathMatrix, PIMathVectorT, PIMathVector

zzuummaa pushed to master at zzuummaa/multithread_experiments 2020-09-17 17:34:57 +03:00
477b057424 Fix for SHSTK
zzuummaa pushed to master at zzuummaa/concurrent_lib 2020-09-15 12:33:41 +03:00
0ad6357f9c Improve docs & CMakeLists.txt
zzuummaa pushed to tests at SHS/pip 2020-09-03 16:57:51 +03:00
f1a0a3ec4a Fix code formatting & grammar mistakes