Results 25 comments of Kieran Mackle

To add to this, `psar.psarbear` and `psar.psarbull` are identical. I believe this might be another case of the issue above, where the same variable is being implicitly referenced, leading to...

Duplicate of Issue #7.

I am currently busy with integrating the Binance API, so it may be a while before I can fully integrate the TD Ameritrade API. However, I am happy to create...

@regholl Please use the template on the version 0.7 development branch, [here](https://github.com/kieran-mackle/AutoTrader/tree/v0.7-dev/templates/newbroker).

Sorry about that, I've since merged the branch into main. The new link is [here](https://github.com/kieran-mackle/AutoTrader/tree/main/templates/newbroker).

@regholl It is a template for integrating any new broker/exchange into AutoTrader. You will need to fill it out to suit the TDA API. See the other [brokers](https://github.com/kieran-mackle/AutoTrader/tree/main/autotrader/brokers) which have...

Very good suggestions! The commissions scheme is something I've been wanting to improve for a little while now, and what you have suggested is a great start. Commissions are currently...

@Salyut9 Version 0.7 now allows specifying `maker_commission` and `taker_commission` arguments in the [`AutoTrader.virtual_account_config`](https://autotrader.readthedocs.io/en/latest/core/AutoTrader.html#virtual-account-configuration) method. Additionally, trading costs will be calculated differently for market vs. limit orders. Make sure to read...

Hey Lewis, the `KeyError` above is popping up because you haven't specified the indicator `type` as `STOCHASTIC` (the key you have named 'STOCHASTIC' is just the string used to [label...

I can't seem to replicate the issue (tried the macd demo strategy [active in the demo runfile](https://github.com/kieran-mackle/autotrader-demo)), and partial trades are allowed by default. Looking into the code a bit...