-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
some stock id's give an error #25
Comments
Hi @andresrogers, where are you retrieving those IDs from? If those are being retrieved from Anyway, let me know the Investing.com URL of the asserts corresponding to the Thanks! 👍🏻 |
The ones that gave me an error are: investing_id=26490 is META search_assets(query="META", limit=1) investing_id=40654 is TLT search_assets(query="TLT", limit=1) It only gave me that error if I used '04/27/2000' as starting date, didn't get the error for less than 10 years difference to current date. |
Ok @andresrogers let me try and I'll come back to you with a response 👍🏻 But I assume it may have something to do with #24 as it just happens whenever you use too distant dates. |
date |
it gave error because this api doesnt have 2012-05-18 before data this stock |
I thought something of the sort, but I believe the expected behaviour should be to return the earliest possible data without giving an error. |
Thanks for posting that @ramakrishnamekala129, and yes, @andresrogers, the expected behavior is not to fail, so I'll check it and come back with a response ASAP 👍 |
We should be creating a separate issue for this once the new version is released! 😄 |
For some ids, historical_data gives this error:
historical_data(investing_id=40654,from_date=start_date_1, to_date=end_date_1)
also
historical_data(investing_id=26490,from_date=start_date_1, to_date=end_date_1)
for dates:
start_date_1 = '04/27/2000'
end_date_1 = '04/26/2012'
The text was updated successfully, but these errors were encountered: