search

santoshlite / EigenLedger

An Open Source Portfolio Backtesting Engine for Everyone | 面向所有人的开源投资组合回测引擎
https://eigenledger.gitbook.io/documentation
Apache License 2.0
950 stars 128 forks source link

trading_year_days removal #40

Closed BrendanGlancy closed 3 years ago

BrendanGlancy commented 3 years ago

With the update to QuantStats on October 4, qs.stats.volatility doesn't take a trading_year_days variable.

This would be my first pull request; sorry if I messed it up. Just something I noticed when trying to use

santoshlite commented 3 years ago

Thanks for this contribution! Looks fine to me :)