Merge pull request #39 from ByteArena/fix-remove-dump-raw-com

refactor: remove dump-raw-comm

Author: Sven SAULEAU

cmd/ba/main.go

@@ -93,7 +93,6 @@ func makeapp() *cli.App {
 				cli.BoolFlag{Name: "debug", Usage: "Enable debug logging"},
 				cli.BoolFlag{Name: "quiet", Usage: "Decrease verbosity of the output"},
 				cli.BoolFlag{Name: "profile", Usage: "Enable execution profiling"},
-				cli.BoolFlag{Name: "dump-raw-comm", Usage: "Dump all the communication between the agent and the server"},
 				cli.IntFlag{Name: "duration", Usage: "If set, game will stop after this durarion (in seconds)"},
 			},
 			Action: func(c *cli.Context) error {
@@ -110,7 +109,6 @@ func makeapp() *cli.App {
 					IsDebug:         c.Bool("debug"),
 					IsQuiet:         c.Bool("quiet"),
 					ShouldProfile:   c.Bool("profile"),
-					DumpRaw:         c.Bool("dump-raw-comm"),
 					DurationSeconds: c.Int("duration"),
 				}
 

glide.lock

@@ -63,7 +63,6 @@ type TrainActionArguments struct {
 	IsQuiet         bool
 	MapName         string
 	ShouldProfile   bool
-	DumpRaw         bool
 	DurationSeconds int
 }
 
@@ -136,6 +135,7 @@ func TrainAction(args TrainActionArguments) (bool, error) {
 		arenaServerUUID,
 		brokerclient,
 		gameDuration,
+		args.IsDebug,
 	)
 
 	for _, dockerImageName := range args.Agentimages {
@@ -184,8 +184,8 @@ func TrainAction(args TrainActionArguments) (bool, error) {
 				fmt.Printf(HeadsUpColor("[headsup] %s\n"), t.Value)
 
 			case arenaserver.EventRawComm:
-				if args.DumpRaw {
-					fmt.Printf(AgentColor("[agent] %s\n"), t.Value)
+				if args.IsDebug {
+					fmt.Printf(DebugColor("[debug from: %s] %s\n"), t.From, t.Value)
 				}
 
 			case arenaserver.EventClose: