отделить логику создания, обновления и чтения данных about poptimizer_old HOT 10 CLOSED

https://github.com/WLM1ke/PortfolioOptimizer/blob/121e507e43627031797be6985a35d615aa68d96c/src/portfolio/getter/local_cpi.py#L84-L91

from poptimizer_old.

Жень, можешь поглядеть ответы и встречные вопросы

https://github.com/WLM1ke/PortfolioOptimizer/blob/master/src/portfolio/settings.py
https://github.com/WLM1ke/PortfolioOptimizer/blob/master/src/portfolio/getter/local_cpi.py
https://github.com/WLM1ke/PortfolioOptimizer/blob/master/src/portfolio/getter/local_dividends.py

from poptimizer_old.

Прокомментировал в:

• 94c7f55
• c3b3542
• 2b92af7

Мне кажется нужно побыстрее делать #44,
но не обязательно с классом, можно и внутри существующи функций сначала подчистить + грохнуть вызов caller() - это очень странная конструкция.

Есть умеренная каша с файловой системой, но термпимо, я прокомментировал.

Хорошо бы, если похожий код не будет повторяться между модулями, как общее пожелание.

from poptimizer_old.

также не очень понял как и зачем настроен 100 coverage

from poptimizer_old.

Покрытие настраивается в этом файле.
По поводу твоих комментариев возникли дополнительные вопросы:

• fecbe41
• 634e3fd
• 334f535

from poptimizer_old.

По поводу твоих комментариев возникли дополнительные вопросы:
fecbe41
634e3fd
334f535

прокомментировал в коммитах

from poptimizer_old.

механизм storage.LocalFile предлагаю взять за основу хранения/доступа к данным
основная идея - разделить уровень доступа к файлу и логические операции по его обновлению.
может наткнуться на 'уникальность' структуры фреймов, но надеюсь они стандартные

from poptimizer_old.

Предлагаю local_cpi.py в этом ишью рефакторить, жду изменений от тебя

from poptimizer_old.

Мне как-то не очень нравится эта идея:

• Функцию is_updated(self) практически для всех данных прийдется переопределять
• Функцию read_dataframe(self) нужно как-то передавать конвертор с верхнего уровня

И получается, что только save_dataframe(self, df) претендует на какую-то универсальность на уровне "доступа к файлу" все остальное будет переопределяться в увязке с уровнем "логических операций по его обновлению".

Есть ли смысл в таком разделении?

from poptimizer_old.

Переписал local_cpi

from poptimizer_old.