Skip to content

Update ch06_09_FirstScreen.py#11

Closed
yeonhodev wants to merge 3 commits into
INVESTAR:masterfrom
yeonhodev:master
Closed

Update ch06_09_FirstScreen.py#11
yeonhodev wants to merge 3 commits into
INVESTAR:masterfrom
yeonhodev:master

Conversation

@yeonhodev

Copy link
Copy Markdown

Changed the old import statment to access the old API with the new mplfinance package installed

Changed the old import statment to access the old API with the new mplfinance package installed
Changed the old import statment to access the old API with the new mplfinance package installed
Changed the old import statment to access the old API with the new mplfinance package installed
@yeonhodev

Copy link
Copy Markdown
Author

Changed the old import statment to access the old API with the new mplfinance package installed

@INVESTAR

INVESTAR commented Jan 1, 2021

Copy link
Copy Markdown
Owner

의견 주셔서 감사합니다.

구 mpl_finance 패키지의 from mpl_finance import candlestick_ohlc 구문을
신 mplfinance 패키지의 from mplfinance.original_flavor import candlestick_ohlc 구문으로 변경하는 것도 좋지만
책에 나와 있는 코드와 다르게 변경할 경우, 초보 독자들이 다소 혼란스러워 할 수 있을 것 같습니다.

따라서, from mpl_finance import candlestick_ohlc 구문은 그대로 두고
from mplfinance.original_flavor import candlestick_ohlc 구문은 주석으로 추가해 두는 게 좋겠습니다.

참고로, 책에는 195페이지에 아래와 같이 관련 내용이 설명되어 있습니다.
만일 매번 실행될 때마다 표시되는 폐기 경고 메시지가 눈에 거슬린다면 pip install --upgrade mplfinance 명령으로 엠피엘파이낸스로 업그레이드한 후, from mpl_finance import candlestick_ohlc 구문을 from mplfinance.original_flavor import candlestick_ohlc 구문으로 변경함으로써 기존 코드는 그대로 유지한 채 엠피엘파이낸스 패키지를 이용해 경고 메시지 없이 실행할 수 있다.

@INVESTAR INVESTAR closed this Jan 1, 2021
@yeonhodev

Copy link
Copy Markdown
Author

네 동의 합니다. 추가 설명에 감사 드립니다! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants