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.

Sign up for GitHub

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

Jump to bottom

Intraday interval support #334

Closed
felixsanz opened this issue Nov 19, 2021 · 1 comment
Closed

Intraday interval support #334

felixsanz opened this issue Nov 19, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@felixsanz
Copy link

python's yfinance library admits valid intervals: 1m,2m,5m,15m,30m,60m,90m,1h,1d,5d,1wk,1mo,3mo. But this historical() function only admits "1d", "1wk", "1mo". It is possible to have intraday intervals?
@felixsanz felixsanz added the enhancement New feature or request label Nov 19, 2021
@gadicc
Copy link
Owner

gadicc commented Nov 21, 2021

Hey, yahoo-finance2 historical() module uses Yahoo's historical API under the hood, which indeed only supports "1d", "1wk", "1mo" intervals. (Ref: pilwon/node-yahoo-finance#76)

I had a look at yfinance, and it looks like their history module uses Yahoo's chart API (vs their "history" API), to get that data. This is an approach we're exploring too, and we hope to have a dev release very soon.

You can follow the relevant issues, particularly #328. You can leave this issue open and we'll update here too.

gadicc added a commit that referenced this issue Nov 21, 2021
@gadicc
feat(chart): initial release as "_chart" (#239, #328, #334)
92b90b1
@gadicc gadicc mentioned this issue Nov 23, 2021
Open
gadicc pushed a commit that referenced this issue Dec 21, 2021
@semantic-release-bot
chore(release): 2.1.0 [skip ci]
cca0707 
# [2.1.0](v2.0.1...v2.1.0) (2021-12-21)

### Bug Fixes

* **chart:** more query tests, intervals, edge cases ([#336](#336)) ([6b95d7e](6b95d7e))
* **package:** have semantic-release recognize version branches ([a89d895](a89d895))
* **quote:** equity: allow underlyingSymbol.  LDN.MI test ([#363](#363)) ([817410b](817410b))

### Features

* **chart:** { return: "array" } (default) + test fix ([#336](#336)) ([1ac66c3](1ac66c3))
* **chart:** initial release as "_chart" ([#239](#239), [#328](#328), [#334](#334)) ([92b90b1](92b90b1))
@felixsanz felixsanz closed this as not planned Won't fix, can't repro, duplicate, stale Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gadicc @felixsanz