Skip to content
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

#75 all tests from class historical tests are failing #77

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

jabacrack
Copy link

Switch to chart api for historical data.
fix issues with time zones.
Code logic based on https://github.com/ranaroussi/yfinance

Unfortunately this branch also contains new functionality with Security profiles, it required for my inner project.

@jabacrack jabacrack closed this Sep 23, 2024
@jabacrack jabacrack deleted the #75_All_Tests_from_Class_HistoricalTests_are_failing branch September 23, 2024 19:36
@yarmoliq
Copy link

Does this fix work by any chance?

@jabacrack jabacrack restored the #75_All_Tests_from_Class_HistoricalTests_are_failing branch September 24, 2024 05:03
@jabacrack jabacrack deleted the #75_All_Tests_from_Class_HistoricalTests_are_failing branch September 24, 2024 05:04
@jabacrack
Copy link
Author

@yarmoliq
It work well, with some circumstances: request one ticker at once, timeout 1 sec between requests.

@jabacrack jabacrack restored the #75_All_Tests_from_Class_HistoricalTests_are_failing branch September 24, 2024 05:11
@jabacrack jabacrack reopened this Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants