---
# System prepended metadata

title: python_test_tasks review for andrey_danilishin

---

# 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`