# python_test_tasks review for andrey_danilishin
комментарии/рекомендации к решению
- code_review
- не нужно было извлекать данные из архива
- `*.py`
- исправления корректны
- `Dockerfile`
- не проведено никаких исправлений рекомендую посмотреть лучшие практики [здесь](https://testdriven.io/blog/docker-best-practices/)
- `requirements.txt`
- pandas - надо было зафиксировать версию `pandas==***`
- tessellation
- решение не проходит тесты для `desk_size=[8, 6]` смотри описание для запуска проверки решения
- для решения можно почитать статью [How good is the Warnsdorff’s knight’s tour heuristic?](https://arxiv.org/pdf/0803.4321.pdf)
- весь код написан одной функцией, это может быть одной из причин почему не получилось решить задачу, иногда проще код разбивать на более небольшие задачи и решать каждую отдельно
- так же рекомендую при написании кода опираться на гайдлайны больших компаний, таких как [google](https://google.github.io/styleguide/pyguide.html)
- можно было бы попробовать использовать numpy для упрощения и ускорения некоторых операций с матрицами см. примеры в `test_solution.py`
- общее
- сразу добавить все файлы/директории связанные с настройкой IDE или специфичными только для одного разработчика данным в .gitignore, такие как `.idea`