Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

get_ohlcv 함수의 일자 선택 기능 추가 #68

Open
mr-yoo opened this issue Feb 28, 2022 · 2 comments
Open

get_ohlcv 함수의 일자 선택 기능 추가 #68

mr-yoo opened this issue Feb 28, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@mr-yoo
Copy link
Collaborator

mr-yoo commented Feb 28, 2022

#60 에서 get_ohlcv의 시작일과 종료일을 지정할 수 있는 get_ohlcv_from 함수가 제안됐습니다.
이는 get_ohlcv와 기능적으로 유사하기 때문에 향후 두 코드를 정리해야합니다.


다음 형태로 코드를 사용할 수 있으면 좋을 것 같습니다.

  • 파라미터의 이름은 가칭입니다.

최근 10개 조회

get_ohlcv(ticker="KRW-BTC",  count=10)

특정일까지 10개 조회

get_ohlcv(ticker="KRW-BTC",  count=10, to='2022-02-05')

특정일부터 10개 조회

get_ohlcv(ticker="KRW-BTC",  count=10, from='2020-02-05)

기간조회

get_ohlcv(ticker="KRW-BTC",  from='2020-02-05, to='2020-02-05')

애매모호한 경우 에러!

get_ohlcv(ticker="KRW-BTC",  from='2020-02-05, to='2020-02-05', count=100)
@mr-yoo mr-yoo added the enhancement New feature or request label Feb 28, 2022
@YiruJeon
Copy link
Contributor

YiruJeon commented Mar 2, 2022

"특정일부터 Count개 조회"를 제외한다면 구조가 유사하니 쉽게 합칠수있어요. "특정일부터 count개 조회"는 많이 쓰이지도 않을것도 같고 upbit에서 제공하는 API 구조로 구현하자면 로직이 복잡해질거같은데 어떻게 생각하시는지요?

@mr-yoo
Copy link
Collaborator Author

mr-yoo commented Mar 2, 2022

@YiruJeon 님 의견에 동의합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants