search

Biryukov239 / matrix-multiplication-kotlin

0 stars 0 forks source link

Нечитаемые названия переменных #3

Closed lounres closed 4 months ago

lounres commented 4 months ago

Я не смог сходу понять, что хранят переменные m, k, n, fst, snd, Заодно сокращения в Kotlin тоже не поощряются: в добавок к предыдущему firstMat, secondMat, resGPU, resCPU. Также непонятны функции roundUp и fit. К ним можно приписать коротенькую документацию хотя бы.

Biryukov239 commented 4 months ago

Поменял названия переменных и дописал короткую документацию к функциям.

lounres commented 4 months ago

Стало сильно лучше! Спасибо!

Ещё бы в firstMat, secondMat и resultMat были бы полностью дописаны слова, а также все эти правки применены к CPUMultiplier и GPUMultiplier, то было бы совсем замечательно.

Biryukov239 commented 4 months ago

Вроде поправил.

lounres commented 4 months ago

Стало сильно лучше! Спасибо! Мне всё равно не нравятся оставшиеся сокращения Mem и Opt. Но ладно, на это я могу забить.