hombit / sai-tasks2019

0 stars 18 forks source link

upload KGO weather task #18

Closed SergeyZHR closed 4 years ago

SergeyZHR commented 4 years ago

В итоге я не нашёл ни одного сервиса, который бы делился нормальными прогнозами по api, а не своим непонятным усредением GFS ,UKMET, ICON и мб ECWMF с неизвестными коэффицентами, поэтому решил распарсить ансамбль моделей c windy Оно работает, осталось только сервер с иксами найти и добавить строчку в crontab :) а то windy просто так request'ом сохраняться не хочет, даже с юзерагентом, но костыль был найден на просторах гитхаба. пока обучена сохранять данные в pickle которые потом надо будет переобработать

hombit commented 4 years ago

Перед тем как разбираться с кодом, я бы хотел решить некоторые более важные вопросы. Код тоже требует множества улучшений, но это обсудим отдельно.

  1. Мне категорически не нравится идея использовать браузер, неужели этот ресурс настолько хорош и настолько ? У нас курс про питон, а не баш, нельзя обойтись без баша? Можно ли обойтись без отправки сочетаний клавиш браузеру и вообще работы с ним как с графиеским приложением, использовать headless версию браузера, например в Docker-контейнере?
  2. Заведите новую ветку, отличную от master, для пулл-реквеста.
  3. Нужно убрать все данные из репозитория, которые генерит код, они тут не нужны.
  4. Зачем используется darksky? Обязательно нужно пойти в свой аккаунт этого сервиса и инвалидировать api-ключ. Не храните секретную информацию в репозитарии, особенно публичном!
hombit commented 4 years ago

Я не эксперт, но кажется в таких случаях принято использовать что-то вроде Selenium