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

feature: backtest report - #2 state recorder #603

Merged
merged 19 commits into from May 11, 2022
Merged

Conversation

c9s
Copy link
Owner

@c9s c9s commented May 9, 2022

  • Use reflect to scan strategy instance fields and record known object types into CSV files
  • Strategy State Recorder
    • Position Recorder
    • Equity Curve Recorder
    • Stop Order Recorder
    • Profit / Loss Recorder

@bbgokarma-bot
Copy link

Welcome back! @c9s, This pull request may get 225 BBG.

@c9s c9s changed the title feature: backtest report - #2 position monitor WIP: feature: backtest report - #2 position monitor May 9, 2022
@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 276 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 532 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 591 BBG

@c9s c9s changed the title WIP: feature: backtest report - #2 position monitor feature: backtest report - #2 state recorder May 10, 2022
@zenixls2
Copy link
Collaborator

test fail

@c9s c9s force-pushed the feature/backtest-report branch from 9b7da78 to 7b17b1a Compare May 10, 2022 05:31
@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 598 BBG

@c9s
Copy link
Owner Author

c9s commented May 10, 2022

fixed and pushed

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 644 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 650 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 663 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 732 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 736 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 809 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 852 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 860 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 915 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 922 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 935 BBG

@c9s c9s merged commit 88cbafe into main May 11, 2022
@c9s c9s deleted the feature/backtest-report branch May 11, 2022 10:56
@bbgokarma-bot
Copy link

Hi @c9s,

Well done! 940 BBG has been sent to your polygon wallet. Please check the following tx:

https://polygonscan.com/tx/0x9b4f2a16cb1cd7c3d232431cb3c94959ad80bcb91d693bcc9a0bd659e465f467

Thank you for your contribution!

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

Successfully merging this pull request may close these issues.

None yet

3 participants