Commit 772b760dcd4b13b03f001f00000acc5934040381 - golang-github-vbauerster-mpb

using NewThreadSafeMovingAverage by default is overhead NewThreadSafeMovingAverage wrapper isn't needed unless decorator relying on ewma.MovingAverage is shared among multiple bars. If there is need to share a decorator, NewThreadSafeMovingAverage must be used explicitly. Vladimir Bauer 2 years ago

3 changed file(s) with 3 addition(s) and 3 deletion(s). Raw diff Collapse all Expand all

-1

decor/eta.go less more

38	38	} else {
39	39	average = ewma.NewMovingAverage(age)
40	40	}
41		return MovingAverageETA(style, NewThreadSafeMovingAverage(average), nil, wcc...)
	41	return MovingAverageETA(style, average, nil, wcc...)
42	42	}
43	43
44	44	// MovingAverageETA decorator relies on MovingAverage implementation to calculate its average.

-1

decor/moving_average.go less more

69	69
70	70	// NewMedian is fixed last 3 samples median MovingAverage.
71	71	func NewMedian() ewma.MovingAverage {
72		return NewThreadSafeMovingAverage(new(medianWindow))
	72	return new(medianWindow)
73	73	}

-1

decor/speed.go less more

45	45	} else {
46	46	average = ewma.NewMovingAverage(age)
47	47	}
48		return MovingAverageSpeed(unit, format, NewThreadSafeMovingAverage(average), wcc...)
	48	return MovingAverageSpeed(unit, format, average, wcc...)
49	49	}
50	50
51	51	// MovingAverageSpeed decorator relies on MovingAverage implementation