Skip to content

Improve retry configuration and remove yfinance dependency - #4

Merged
shackmann merged 3 commits into
mainfrom
stefan/work2
Aug 1, 2026
Merged

Improve retry configuration and remove yfinance dependency#4
shackmann merged 3 commits into
mainfrom
stefan/work2

Conversation

@shackmann

@shackmann shackmann commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

Note

Low Risk
Changes are limited to default transport retry behavior and optional notebook packaging; no auth or API contract changes.

Overview
HTTP retry defaults are tuned for DataRobot deployments: max_attempts goes from 10 to 20, and 470 is added to the retriable status codes so the client can wait through custom-model stop/warm-up windows after a deployment restart. The same values appear in RetryConfig, config.sample.yaml, and the README transport example.

The notebooks optional extra no longer pulls in yfinance—only pandas remains. forecast_trading.ipynb (Yahoo Finance / Kelly trading demo) is removed, and tests/test_notebooks.py no longer expects that notebook in the catalog. uv.lock is refreshed to drop the yfinance dependency tree.

Reviewed by Cursor Bugbot for commit 137902a. Bugbot is set up for automated code reviews on this repo. Configure here.

@shackmann
shackmann merged commit 2808b27 into main Aug 1, 2026
2 checks passed
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.

1 participant