search

quantopian / alphalens

Performance analysis of predictive (alpha) stock factors
http://quantopian.github.io/alphalens
Apache License 2.0
3.26k stars 1.13k forks source link

Clean up backward compat API #268

Closed twiecki closed 6 years ago

twiecki commented 6 years ago

I think we have kept the backwards compatible API around long enough. can remove in the next release.

mmargenot commented 6 years ago

I think we should do this as a secondary release. Put Pyfolio input and other features in a main one and patch out the backwards compatible API in a quick follow-up. Does this make sense? I'd like to still have the backwards-compatible support on Quantopian Research in conjunction with Pyfolio input.

luca-s commented 6 years ago

@twiecki and I talked about this today and we had the same thought as you. Let's have the next release first, then we can remove the backwards-compatible support

twiecki commented 6 years ago

@luca-s Now that v0.3.0 is out, think we should go ahead on this?

luca-s commented 6 years ago

Maybe we should wait and see that new release is running fine on Quantopian? If we needed to create a new release it would be annoying to re-introduce the backwards compatible layer.

twiecki commented 6 years ago

Good point.

luca-s commented 6 years ago

closed by #286