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

backtest: Small fixes to get_full_chain_info function when running in… #582

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

davidlatte
Copy link
Collaborator

@davidlatte davidlatte commented Oct 16, 2024

… backtest.

Description by Korbit AI

What change is being made?

Add performance logging and improve date handling in the get_chain_full_info function within data_source.py.

Why are these changes being made?

These changes are made to enhance the function's performance tracking by logging the time taken for queries and the total execution time, which aids in debugging and optimization. Additionally, the date handling is improved by ensuring consistent date formatting, which prevents potential errors when accessing option chains.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

Copy link
Contributor

korbit-ai bot commented Oct 16, 2024

You've used up your 5 PR reviews for this month under the Korbit Starter Plan. You'll get 5 more reviews on October 20th, 2024 or you can upgrade to Pro for unlimited PR reviews and enhanced features in your Korbit Console.

@grzesir grzesir merged commit b948b08 into dev Oct 16, 2024
1 check passed
@davidlatte davidlatte deleted the full_chain_fixes branch October 18, 2024 23:06
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