From 649fb10c8e3da2a35c2aa4f77cbcb4a6887e4b78 Mon Sep 17 00:00:00 2001 From: Eddie Hebert Date: Tue, 22 Jan 2013 20:40:37 -0500 Subject: [PATCH] Creates a new averages container for each call to get_averages. Ideally, we'd want to avoid multiple object creation, but currently not playing well with generator chain. --- zipline/transforms/mavg.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/zipline/transforms/mavg.py b/zipline/transforms/mavg.py index d5045988..3dbba340 100644 --- a/zipline/transforms/mavg.py +++ b/zipline/transforms/mavg.py @@ -108,9 +108,6 @@ class MovingAverageEventWindow(EventWindow): # fields. self.fields = fields self.totals = defaultdict(float) - # Container for averages - # So that we don't create a new object on each event. - self.averages_container = Averages() # Subclass customization for adding new events. def handle_add(self, event): @@ -145,7 +142,7 @@ class MovingAverageEventWindow(EventWindow): """ Return an ndict of all our tracked averages. """ - out = self.averages_container + out = Averages() for field in self.fields: out.__dict__[field] = self.average(field) return out