Skip to content

Conversation

rodrigo-brito
Copy link
Owner

  • Include histogram of returns
  • Limit dataframe in each iteration
  • Change avg price calculation

@rodrigo-brito rodrigo-brito force-pushed the feature/backtest-refactor branch from 9689e3e to 2de1998 Compare July 8, 2023 18:11
@rodrigo-brito rodrigo-brito force-pushed the feature/backtest-refactor branch from 2de1998 to 23e01f8 Compare July 8, 2023 18:18
@codecov-commenter
Copy link

Codecov Report

Patch coverage: 78.64% and project coverage change: +0.92 🎉

Comparison is base (0043d08) 40.40% compared to head (23e01f8) 41.32%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #276      +/-   ##
==========================================
+ Coverage   40.40%   41.32%   +0.92%     
==========================================
  Files          19       19              
  Lines        2896     2911      +15     
==========================================
+ Hits         1170     1203      +33     
+ Misses       1650     1636      -14     
+ Partials       76       72       -4     
Impacted Files Coverage Δ
model/order.go 100.00% <ø> (ø)
plot/chart.go 39.69% <42.85%> (-0.37%) ⬇️
order/controller.go 54.10% <77.50%> (+3.24%) ⬆️
model/model.go 73.58% <100.00%> (+13.98%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@rodrigo-brito rodrigo-brito merged commit b3cd202 into main Jul 8, 2023
@rodrigo-brito rodrigo-brito deleted the feature/backtest-refactor branch July 8, 2023 19:06
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.

2 participants