Support income layer start amount#177
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0c5820c054
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if income_layer_start_usd is not None: | ||
| overrides["income_layer_start_usd"] = income_layer_start_usd |
There was a problem hiding this comment.
Scope income layer start overrides to supported profiles
When INCOME_LAYER_START_USD is set for any deployment whose selected profile is not soxl_soxx_trend_income (for example tqqq_growth_income), this unconditional override is merged into the runtime config and then the vendored evaluate_tqqq_growth_income passes it through as **config to build_rebalance_plan, whose signature has no income_layer_start_usd parameter. That makes runtime evaluation fail with an unexpected-keyword TypeError instead of producing a rebalance plan, so this override should be gated to the profile(s) that actually accept it.
Useful? React with 👍 / 👎.
Summary
Tests