-
Notifications
You must be signed in to change notification settings - Fork 275
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
list index out of range #199
Comments
+1, experienced the same issue |
Just now |
Fixed it. Can someone review this PR #200 ? In the meantime, you can use this fork https://github.com/thirumalairajr/finviz @TZimm55 @sw8989 |
Thank you thank you. I appreciate the promptness. I will try to figure out how to use a fork. |
You can uninstall your local finviz dependency and install it from the fork
Once the PR is merged, you can switch back to the main repo. |
Ahh, thank you very much, that worked! : ) That is what I needed. |
@thirumalairajr unfortunately a new related error popped up.
File "/opt/homebrew/lib/python3.11/site-packages/finviz/main_func.py", line 37, in get_stock Any idea what is the issue this time ? |
@thirumalairajr hi, seems like finviz made another small update or something. Line 37 I have made the following change and it seems to work for me! @sw8989 Not sure if you're able to edit the "main_func.py" as such, else I think we can wait for @thirumalairajr to make the change. Let me know if it helps/works |
thanks @ahloytan fixed the issue based on your solution !
|
still experiencing this issue. @MahanyaKochhar Im not sure entirely why but using your fork messed up some numpy math in another area of my project. Specifically when I have your fork installed, a matrix multiplication results in all NaNs instead of the expected values. |
I am getting error on example code, too.
|
I am also facing the same issue C:\Users\asraghu\vcp_screener.github.io-main>python -m vcp_screener |
Ok here's what I did to get it working. Thank you @RiverRo
|
As of today 2023-10-13, I tried to run:
Error received:
Additional Info:
The text was updated successfully, but these errors were encountered: