Merge pull request #1345 from xucong053/bugfix

fix: failed to load overall pick_order strategy in parameters_setting
This commit is contained in:
bbx-winner
2022-06-08 17:42:12 +08:00
committed by GitHub
3 changed files with 5 additions and 5 deletions

View File

@@ -12,7 +12,7 @@ import (
)
type TParamsConfig struct {
PickOrder iteratorPickOrder `json:"strategy,omitempty" yaml:"strategy,omitempty"` // overall pick-order strategy
PickOrder iteratorPickOrder `json:"pick_order,omitempty" yaml:"pick_order,omitempty"` // overall pick-order strategy
Strategies map[string]iteratorStrategy `json:"strategies,omitempty" yaml:"strategies,omitempty"` // individual strategies for each parameters
Limit int `json:"limit,omitempty" yaml:"limit,omitempty"`
}
@@ -69,7 +69,7 @@ func newParametersIterator(parameters map[string]Parameters, config *TParamsConf
for paramName := range parameters {
// check parameter individual pick order strategy
strategy, ok := config.Strategies[paramName]
if !ok {
if !ok || strategy.PickOrder == "" {
// default to overall pick order strategy
strategy.PickOrder = config.PickOrder
}