cvxportfolio icon indicating copy to clipboard operation
cvxportfolio copied to clipboard

Added weight_expr function to MultipleReturnsForecasts class. Removed…

Open tyler-abbot opened this issue 6 years ago • 1 comments

… * from definition of control variable in MultiPeriodOpt.get_trades because was not working before and this quantity will never be of variable number (always a single tuple).

These changes were necessary in order to get the multiperiod optimization example notebook to run.

tyler-abbot avatar Jan 09 '19 11:01 tyler-abbot

Coverage Status

Coverage decreased (-0.05%) to 82.407% when pulling 44d44b86bf7973b9c493659dcdc963417b48aeb7 on tyler-abbot:master into 152188adac72c1bce85f1aa526e85b4f611cf497 on cvxgrp:master.

coveralls avatar Jan 10 '19 08:01 coveralls

Thank you for this, I guess this got lost in the past few years of under-maintenance of the repo. Currently https://github.com/cvxgrp/cvxportfolio/blob/master/examples/MultiPeriodTCostOptimization.ipynb works with the MultiPeriod class and the original notebooks are broken because a few of the original libraries broke (e.g., quandl), so I guess the problem you were targeting has been solved. We are working fast to finalize the user interface.

enzbus avatar Apr 11 '23 11:04 enzbus