Buy and Hold example - monthly cash being added twice?

gretzteam

From the buy and hold blog post:
Buy and Hold

Why is the self.p.monthly_cash added first to self.broker.add, and then again to calculate the target_value for the current order? Isn't this adding the cash twice?

    def notify_timer(self, timer, when, *args, **kwargs):
        # Add the influx of monthly cash to the broker
        self.broker.add_cash(self.p.monthly_cash)

        # buy available cash
        target_value = self.broker.get_value() + self.p.monthly_cash
        self.order_target_value(target=target_value)

run-out

@gretzteam I believe you are adding the cash twice. First using:

self.broker.add_cash(self.p.monthly_cash)

Then here:

self.broker.get_value() + self.p.monthly_cash

get_value will have the first cash addition in it.

gretzteam

@run-out This is what I thought but being new to backtrader I wasn't sure.
I took this code from the documentation here, maybe it needs to get updated?

run-out

@gretzteam No my error, get_value is returning the value of a datas, or if no datas provide then all the market value of the datas. Backtrader's blog is correct. See the comments here