Merge pull request #29 from yubing744/owen/fixbug-tp-sl

feat: ok

Author: yubing744

Makefile

@@ -1,7 +1,7 @@
 .PHONY: clean build unit-test run docker-* tag release
 
 NAME=trading-gpt
-VERSION=0.23.1
+VERSION=0.23.3
 
 clean:
 	rm -rf build/*

pkg/env/exchange/exchange_entity.go

@@ -71,7 +71,7 @@ func (ent *ExchangeEntity) Actions() []*ttypes.ActionDesc {
 			Description: "open long position",
 			Args: []ttypes.ArgmentDesc{
 				{
-					Name:        "stop_loss_trigge_price",
+					Name:        "stop_loss_trigger_price",
 					Description: "Stop-loss trigger price",
 				},
 				{
@@ -254,12 +254,10 @@ func (ent *ExchangeEntity) HandleCommand(ctx context.Context, cmd string, args m
 			}
 		}
 
-		log.Infof("open %s position for signal %v, reason: %s", ent.symbol, side, "")
-
 		opts := make([]interface{}, 0)
 
 		// config stop losss
-		if stopLoss, ok := args["stop_loss_trigger_price"]; ok {
+		if stopLoss, ok := args["stop_loss_trigger_price"]; ok && stopLoss != "" {
 			stopLoss, err := utils.ParseStopLoss(ent.vm, side, closePrice, stopLoss)
 			if err != nil {
 				return errors.Wrapf(err, "the stop loss invalid: %s", stopLoss)
@@ -273,7 +271,7 @@ func (ent *ExchangeEntity) HandleCommand(ctx context.Context, cmd string, args m
 		}
 
 		// config take profix
-		if takeProfix, ok := args["take_profit_trigger_price"]; ok {
+		if takeProfix, ok := args["take_profit_trigger_price"]; ok && takeProfix != "" {
 			takeProfix, err := utils.ParseTakeProfit(ent.vm, side, closePrice, takeProfix)
 			if err != nil {
 				return errors.Wrapf(err, "the take profit invalid: %s", takeProfix)
@@ -286,6 +284,8 @@ func (ent *ExchangeEntity) HandleCommand(ctx context.Context, cmd string, args m
 			}
 		}
 
+		log.Infof("open %s position for signal %v, options: %v", ent.symbol, side, opts)
+
 		if cmd == "open_long_position" || cmd == "open_short_position" {
 			err := ent.OpenPosition(ctx, side, closePrice, opts...)
 			if err != nil {