Ori.livneh has submitted this change and it was merged.
Change subject: coal: pass a materialized list to numpy.median
......................................................................
coal: pass a materialized list to numpy.median
numpy.median() doesn't like generator inputs.
Change-Id: I9ed218ed27acc32897239778617fec78433e542f
---
M modules/webperf/files/coal
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Ori.livneh: Verified; Looks good to me, approved
diff --git a/modules/webperf/files/coal b/modules/webperf/files/coal
index 642f131..b4ad7dd 100755
--- a/modules/webperf/files/coal
+++ b/modules/webperf/files/coal
@@ -198,7 +198,8 @@
def current_value(self):
"""Returns a copy of the list of items in the window."""
self.prune()
- return self.aggregate(value for timestamp, value in self.heap)
+ values = [value for timestamp, value in self.heap]
+ return self.aggregate(values)
if __name__ == '__main__':
--
To view, visit https://gerrit.wikimedia.org/r/204674
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9ed218ed27acc32897239778617fec78433e542f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ori.livneh <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
