-
-
Notifications
You must be signed in to change notification settings - Fork 225
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
Feature: querying of minute-resolution intraday data with yahoo api #381
Conversation
Sorry it took me so long to reply. This looks really good! The only potential change I can think of is that Yahoo supports more than 1-minute intraday data. The valid intraday values are: 1m, 2m, 5m, 15m, 30m, 60m, 90m, 1h It would be nice to support all of those. It doesn't have to be done in this PR though. I can do it after merging. What do you think? Also, do you know if you can get more history for the lower resolution intraday data? |
Thanks @joshuaulrich for your feedback!
Ah cool, I wasn't aware of this. Yes, I agree with you that supporting all of those would be indeed nice. However, I am not familiar if one can get more history of the lower resolution intraday data. If this is the case, the "history cutoff"-logic would need to be adapted accordingly: https://github.com/joshuaulrich/quantmod/pull/381/files#diff-0d1f6c1913e17876f7f890fdb8db9d0109d6806720090032749c669bdeb27dfcR290 I have no preference regarding including the other intraday-resolution options into this PR or do it after merging - as you like! |
### Changes in 0.4.25 (2023-08-21) 1. Fix `getQuote.yahoo()` for API changes. Thanks to Ethan B. Smith for the report and patch! Also add error message for users in GDPR countries, since we cannot automatically consent to GDPR and the request fails without consent. [#392](joshuaulrich/quantmod#392) [#393](joshuaulrich/quantmod#393) [#395](joshuaulrich/quantmod#395) 1. Fix `getQuote.yahoo()` when the user only requested metrics that do not have have a value for 'regularMarketTime'. Set the value to NA in these cases so the output remains the same regardless of whether the endpoint returns a 'regularMarketTime' or not. Thanks to @mehdiMBH for the report! [#255](joshuaulrich/quantmod#255) 1. Add fields to `getQuote.yahoo()` that are returned when no fields are explicitly requested. Thanks to @Courvoisier13 for the report! [#335](joshuaulrich/quantmod#335) 1. Add intraday endpoint to `getSymbols.yahoo()`. Thanks to @kapsner for the report and patch! Also allow suppressing the warning if more than 7 days of data are requested (@eddelbuettel). [#351](joshuaulrich/quantmod#351) [#381](joshuaulrich/quantmod#381) [#399](joshuaulrich/quantmod#399) 1. Add warning if `getSymbols()` is called with tickers that are reserved words because accessing them requires back-quotes (e.g. ``NA``). [#401](joshuaulrich/quantmod#401) 1. Fix `allReturns()` when 'subset' is specified. Thanks to @Panagis1980 for the report! [#402](joshuaulrich/quantmod#402) ### Changes in 0.4.24 (2023-07-17) 1. Fix `getSymbols.oanda()` URL. Thanks to @macray76 for the report. [#387](joshuaulrich/quantmod#387) ### Changes in 0.4.23 (2023-06-14) 1. Fix `getQuote.yahoo()` error. Thanks to Ethan B. Smith for the report and patch! [#382](joshuaulrich/quantmod#382) [#383](joshuaulrich/quantmod#383) 1. Add `name` argument to `add_TA()`. Thanks to @SamoPP for the suggestion! [#377](joshuaulrich/quantmod#377) [#205](joshuaulrich/quantmod#205)
Yahoo API allows to query intraday data for last 7 days. This PR implements querying of intraday data with yahoo API.
Addresses #351
Test with