lychiyu/bbgo_origin
1
0
Fork 0
mirror of https://github.com/c9s/bbgo.git synced 2024-11-21 22:43:52 +00:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

add warning logs to pnl cmd

Browse Source
This commit is contained in:
c9s 2022-06-08 14:59:32 +08:00
parent 8c6331073d
commit 99bf914415
No known key found for this signature in database
GPG Key ID: 7385E7E464CB0A54
1 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
pkg/cmd/pnl.go
View File

@ -4,6 +4,7 @@ import (
"context"
"errors"
"fmt"
"sort"
"strings"
"time"
@ -105,7 +106,12 @@ var PnLCmd = &cobra.Command{
if err != nil {
return err
}
_ = withdrawals
sort.Slice(withdrawals, func(i, j int) bool {
a := withdrawals[i].ApplyTime.Time()
b := withdrawals[j].ApplyTime.Time()
return a.Before(b)
})
// we need the backtest klines for the daily prices
backtestService := &service.BacktestService{DB: environ.DatabaseService.DB}
@ -121,8 +127,8 @@ var PnLCmd = &cobra.Command{
trades, err = environ.TradeService.QueryForTradingFeeCurrency(exchange.Name(), symbol, tradingFeeCurrency)
} else {
trades, err = environ.TradeService.Query(service.QueryTradesOptions{
Symbol: symbol,
Limit: limit,
Symbol: symbol,
Limit: limit,
})
}
@ -155,6 +161,9 @@ var PnLCmd = &cobra.Command{
report := calculator.Calculate(symbol, trades, currentPrice)
report.Print()
log.Warnf("note that if you're using cross-exchange arbitrage, the PnL won't be accurate")
log.Warnf("withdrawal and deposits are not considered in the PnL")
return nil
},
}