Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fees and account amount #91

Closed
pennymax opened this issue Dec 29, 2023 · 5 comments
Closed

fees and account amount #91

pennymax opened this issue Dec 29, 2023 · 5 comments

Comments

@pennymax
Copy link

pennymax commented Dec 29, 2023

Hello @edtechre ,

I recently noticed that in the portfolio and metrics calculations, the transaction fees aren't being factored into either the pnl or market_value. This has led to a discrepancy, as the available cash for future investments is not reduced by these fees. This difference becomes particularly significant over thousands of trades, leading to a noticeable divergence from my live trading records.

Could you suggest a way to accurately simulate the scenario where each fee is deducted from the account balance? This would help in creating a more realistic representation of the financial impact these fees have over time. Thank you for your assistance!

@edtechre
Copy link
Owner

edtechre commented Dec 30, 2023

Thank you for this inquiry, @pennymax. Yes, fees are not deducted from your cash balance, right now it is only there to show what fees your account has accrued. However, I can add a config option for this.

@pennymax
Copy link
Author

Thank you for your response, @edtechre! Having the option to accurately account for transaction fees in my portfolio calculations would be immensely beneficial, particularly for conducting long-term backtests.

@tsunamilx
Copy link

Hi @edtechre , any updates on this, this is actually quite important for simulating the real trading result.

@edtechre
Copy link
Owner

Hi @tsunamilx, I will work on supporting this hopefully this week.

@edtechre
Copy link
Owner

A subtract_fees config option has been added as part of the v1.1.32 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants