Merge pull request #1085 from c9s/bhwu/grid2/fix-group-id

gx578007 · web-flow · commit b04492a5a7f2 · 2023-03-07T12:01:17.000+08:00
FIX: [grid2] group id should be bound by MaxInt32
diff --git a/pkg/strategy/grid2/strategy.go b/pkg/strategy/grid2/strategy.go
@@ -1642,7 +1642,7 @@ func (s *Strategy) Run(ctx context.Context, _ bbgo.OrderExecutor, session *bbgo.
 	}
 
 	if s.OrderGroupID == 0 {
-		s.OrderGroupID = util.FNV32(instanceID)
+		s.OrderGroupID = util.FNV32(instanceID) % math.MaxInt32
 	}
 
 	if s.AutoRange != nil {
@@ -1869,4 +1869,4 @@ func roundUpMarketQuantity(market types.Market, v fixedpoint.Value, c string) (f
 	}
 
 	return v.Round(prec, fixedpoint.Up), prec
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -1642,7 +1642,7 @@ func (s Strategy) Run(ctx context.Context, _ bbgo.OrderExecutor, session bbgo.`
`1642`	`1642`	`}`
`1643`	`1643`
`1644`	`1644`	`if s.OrderGroupID == 0 {`
`1645`		`- s.OrderGroupID = util.FNV32(instanceID)`
	`1645`	`+ s.OrderGroupID = util.FNV32(instanceID) % math.MaxInt32`
`1646`	`1646`	`}`
`1647`	`1647`
`1648`	`1648`	`if s.AutoRange != nil {`
`@@ -1869,4 +1869,4 @@ func roundUpMarketQuantity(market types.Market, v fixedpoint.Value, c string) (f`
`1869`	`1869`	`}`
`1870`	`1870`
`1871`	`1871`	`return v.Round(prec, fixedpoint.Up), prec`
`1872`		`-}`
	`1872`	`+}`