Ok, though I think, we should resolve that cirularity in a later patch.

fawce <notifications@github.com> writes:

> In zipline/data/loader.py:
>
>>          bmark_fp.write(msgpack.dumps(benchmark_data))
>> +
>> +
>> +def get_benchmark_filename(symbol):
>> +    return "%s_benchmark.msgpack" % symbol
>> +
>> +
>> +def load_market_data(bm_symbol='^GSPC'):
>> +    from zipline.finance.trading import DailyReturn
>
> There is a circularity between loader using DailyReturn and trading using
> loader's methods for loading files, so I'm postponing the loading of Daily
> return.
>
> —
> Reply to this email directly or view it on GitHub.