bbgo icon indicating copy to clipboard operation
bbgo copied to clipboard

WIP: FEATURE: implement stats tracker

Open c9s opened this issue 2 years ago • 4 comments

  • rename Add() to AddProfit() or AddTrade() so that we can apply interface here

c9s avatar Aug 31 '23 05:08 c9s

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

bbgokarma-bot avatar Aug 31 '23 05:08 bbgokarma-bot

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (3dca9aa) 21.28% compared to head (4a3b0bc) 21.28%. Report is 110 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1296   +/-   ##
=======================================
  Coverage   21.28%   21.28%           
=======================================
  Files         596      596           
  Lines       43933    43933           
=======================================
  Hits         9349     9349           
  Misses      33907    33907           
  Partials      677      677           
Files Coverage Δ
pkg/strategy/xmaker/strategy.go 2.64% <0.00%> (ø)
pkg/types/profit.go 0.00% <0.00%> (ø)
pkg/types/trade_stats.go 19.10% <0.00%> (ø)
pkg/bbgo/order_executor_general.go 16.08% <0.00%> (ø)
pkg/strategy/drift/strategy.go 3.39% <0.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3dca9aa...4a3b0bc. Read the comment docs.

codecov[bot] avatar Aug 31 '23 05:08 codecov[bot]

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

bbgokarma-bot avatar Jan 03 '24 09:01 bbgokarma-bot

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jan 24 '24 07:01 CLAassistant