Conversation
DimaTrushin
left a comment
DimaTrushin
left a comment
There was a problem hiding this comment.
Обрати внимание, что я не повторял замечания. Если одна и та же ошибка всречалась несколько раз, я комментировал лишь единожды. Потому я ожидаю, что ты поправишь все аналогичные замечания всюду.
Я отсмотрел только property_model.cpp.
Во-первых, у тебя должен быть загружен полностью проект. А именно:
файлы для настройки системы сборки. Если это cmake, то соответственно CMakeList.txt файл.
У тебя должны быть включены все файлы для управления git, в частности .gitignore и .gitsubmodules
У тебя должны быть включены все файлы для работы с кодом такие как .clang-format и clang-tidy
Проект в репозитории после клонирования должен собираться по нажатию одной кнопки.
DimaTrushin
left a comment
DimaTrushin
left a comment
There was a problem hiding this comment.
Обрати внимание, что я не повторял замечания. Если одна и та же ошибка всречалась несколько раз, я комментировал лишь единожды. Потому я ожидаю, что ты поправишь все аналогичные замечания всюду.
Добавил несколько замечаний
|
Я не пофиксил вертикальные пробелы и не загрузил eigen 3.3.4 как подмудуль. Это я успею и после 1 апреля, а сейчас нужно хоть дописать сам код) |
|
Я быстро накидал main для мниста (сори, за using namespace ...). У меня 93% за 1.5 часа. Это не очень, да? |
2 participants
No description provided.