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
Tibble display not useful #4
Comments
Agreed. I already include |
Also omitting the pennies when the only difference is in pennies is ... not good. These are prices (and OHLC aren't even computed aggregates). Can we not always show two digits? Also not how my |
Re digits.secs) Yes ive reported that r-lib/pillar#102 Re pennies) Another pillar issue. Aggressive truncation. Try) |
sigfig=6 may still fail on high priced stocks.But I guess this is a new bug :-/ R> res <- riingo_iex_latest("BRK.A")
Error: lexical error: invalid char in json text.
<h1>Not Found</h1><p>The reques
(right here) ------^
R> Also affects |
Frustrating. I can catch |
Hi all, Thanks for the great work @DavisVaughan The reason for the URL handling is that we do not use "." in our ticker methodology. Let me talk to my team and see if we can implement a better JSON-encoded 404 for when "." are used. This should make it consistent. Thank you for the catch. |
Thanks @tiingo for the great support. In the mean time, the dev version catches it with a slightly more informative error: > riingo::riingo_iex_latest("BRK.A")
Error: The ticker name, BRK.A, is invalid or data is currently not available. Check ticker validity with is_supported_ticker().
Tiingo msg) <h1>Not Found</h1><p>The requested URL /iex/BRK.A/prices was not found on this server.</p> Which is still somewhat frustrating because: > is_supported_ticker("BRK.A", "iex")
[1] TRUE IEX supported tickers data come from: |
@DavisVaughan We are able to return a JSON error message on a 404 with a ticker with a ".". For "is_supported_ticker" would you be open to querying a Tiingo endpoint to see if the ticker is available? Holding off pushing the API change for the 404 error depending on your answer to is_supported_ticker |
@tiingo Yea that'd be great |
You should now see a consistent error message if the ticker contains a "." E.g. https://api.tiingo.com/tiingo/daily/brk.a/prices?startDate=2017-1-1 Additionally, you can use the meta endpoint to see if a ticker exists, which is simply: Let me know if this works @DavisVaughan Thanks all |
But ... BRK.A is a valid symbol and known / traded on IEX rfom what I can tell. |
The time should be aware of localtime, and the resolution is simply not useful for pricing data.
The text was updated successfully, but these errors were encountered: