Feature/on time perf

This indicator is pretty involved - the comments should be a decent guide
Note for future reference:  I think @lossyrob  said yesterday that specifying `with AggregatesByAll` is superfluous now, but I may be wrong on that.  We should probably go remove that from all the indicators at some point, if that's the case.