Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description by Korbit AI
What change is being made?
Remove the
include_cash_positions
parameter from strategy setup to use a default configuration for cash handling except indrift_rebalancer
, where it is explicitly set toTrue
.Why are these changes being made?
This change standardizes strategy configuration by removing the redundant
include_cash_positions
parameter, allowing the system's default handling of cash positions unless explicitly needed, like in thedrift_rebalancer
strategy. The updated approach minimizes code duplication and potential configuration errors across different strategy files.