Benchmark data mixed normalization mode fix #8012
Merged
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
Skip the security security initializer for benchmark securities, avoiding the user defined ones (if set).
A user defined security initializer might use a security seeder or set the security market price with historical data and:
We avoid this by not initializing the benchmark security, since it does not need it
Regression algorithms
AlgorithmHistoryDataPoints
decrease:These algorithms use user defined security initializers that set the security market price with historical data. These history requests are skipped for benchmark securities, hence the decrease.
Related Issue
Close #7846
Motivation and Context
Requires Documentation Change
How Has This Been Tested?
Unit test and regression algorithms
Types of changes
Checklist:
bug-<issue#>-<description>
orfeature-<issue#>-<description>