Skip to content
This repository has been archived by the owner on Dec 27, 2018. It is now read-only.

cleanup Ticket dataframe construction #20

Closed
epogrebnyak opened this issue Mar 6, 2018 · 5 comments
Closed

cleanup Ticket dataframe construction #20

epogrebnyak opened this issue Mar 6, 2018 · 5 comments
Assignees
@WLM1ke

Comments

@epogrebnyak
Copy link
Collaborator

https://github.com/WLM1ke/PortfolioOptimizer/blob/8d55dd398f50aed09053b818463f901d7a7c80f4/src/portfolio/loader_history.py#L110-L125
@WLM1ke
Copy link
Owner

WLM1ke commented Mar 6, 2018

Объеденил и поправил тесты. К сожалению в метод мало чего можно запихнуть: только выбор колонок и преобразования объемов к числам. Все остальное оставил в функции - иначе будет неправильно работать.

https://github.com/WLM1ke/PortfolioOptimizer/blob/0e29241afc24f1c2f68d08371ba9e74f751663f3/src/portfolio/loader_history.py#L105-L111

@epogrebnyak
Copy link
Collaborator Author

Уже хорошо, так понятнее становится.

@WLM1ke
Copy link
Owner

WLM1ke commented Mar 6, 2018

Закрывать можно?

@epogrebnyak
Copy link
Collaborator Author

Закрою сам, одну идею хочу попробовать.

@epogrebnyak
Copy link
Collaborator Author

Закрываем через b689a48

@WLM1ke WLM1ke added this to the First ver. 1.0 milestone Mar 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@WLM1ke WLM1ke

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@epogrebnyak @WLM1ke