search

mhallsmoore / qstrader

QuantStart.com - QSTrader backtesting simulation engine.
https://www.quantstart.com/qstrader/
MIT License
2.79k stars 842 forks source link

SimulatedBroker Issue #376

Closed rricco closed 4 months ago

rricco commented 2 years ago

I have noted that the ABC class Broker contains an abstract method named 'get_portfolio_total_non_cash_equity' that is not contemplated in the subclass SimulatedBroker(Broker), however, the Exception NotImplementedError is not raised. Does anybody know the reason?

juliettejames commented 4 months ago

Thanks,

These methods were found to be superfluous and have been removed in QSTrader 0.2.6