-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Tutorial crashes #1748
Comments
Where are you from? Which country? |
France. I could try through proxies if it can help. |
Hello, I have the same error message, when try to use ticker.info functionality. Even after the last update from 6 Dec (0.2.33) the issue still exisit. I am from Bulgaria and do not use VPN, could you give some info does it possible to fix the issues locally using VPN or other option. |
@NinoNinov
and remove yfinance.cache if it exists |
I prefer they keep cache and debug the error. What is the string? |
2 feedbacks:
|
Sounds easy to fix. #1084 |
The script is from public example. Is this bug general or only happen to me and dmoklaf?
and the issues is:
|
@NinoNinov it's error from old code. Try latest version |
I tried to updated to last version several time but still the same issue, as you can see from my screenshot made several minutes ago. |
@NinoNinov |
@bot-unit many thanks for support! Now it is working, the issie was that I tried to update with comand: |
@NinoNinov I followed your reinstall procedure with the specific flags including |
@dmoklaf the caching is needed for cookies. Since Yahoo started blocking requests without crumbs, we should obtain cookies, then crumb, and only then make requests. |
@dmoklaf Give us the cookie cache database file, I want to understand how error happened. I've updated README to explain where file is. |
Here are the cache files. Let me know if I should perform another test with clean (reset) cache files. Even if small, I had to zip them as GitHub doesn't accept .db files MacOS 14.2 |
I understand the problem but no idea how it happened. Branch |
It works ! Note: the Python documentation is confusing because it states that its default mapping is ISO-compliant. But this is wrong. Using a space is not ISO compliant and is a tolerance stemming from RFC 3339, not supported by all tools. Strict ISO forbids the space, and requires a T. The future for Python may favor strict ISO as the documentation announces default mappings will be removed, and offers as a compensation mapping examples strictly based on ISO (and not based on the RFC 3339 tolerance): So your workaround may be going in the right direction. |
Given this is an edge case, I have lowered priority for merge into |
Hello @dmoklaf for the code below I receive again issue "404 Client Error: Not Found for url:"
S&P Code which is working
|
@NinoNinov I could reproduce your error, and simplified the case to this:
which raises a 404 error. This error is coming from Yahoo and not from this library, as Yahoo declares ATVI (Activision) as a private company: So this is an unrelated error that is not in the scope of this project. |
Many thanks, PyTickerSymbols is not up to date, will scrap NASDAQ data from web. |
Fixed by #1796 |
Describe bug
This tutorial code crashes now:
Simple code that reproduces your problem
See above
Debug log
Bad data proof
No response
yfinance
version0.2.32 (pip installed directly from Github GIT repository)
Python version
3.11.6
Operating system
MacOS Sonoma 14.1.1
The text was updated successfully, but these errors were encountered: