Scouting: Update note scouting for 2023 game

This patch updates the keywords provided by strategy team.

Signed-off-by: Filip Kujawa <filip.j.kujawa@gmail.com>
Change-Id: I473af7d6c229a744a175414994e8552944a82d0e
diff --git a/scouting/webserver/requests/debug/cli/cli_test.py b/scouting/webserver/requests/debug/cli/cli_test.py
index 979ef5a..4763058 100644
--- a/scouting/webserver/requests/debug/cli/cli_test.py
+++ b/scouting/webserver/requests/debug/cli/cli_test.py
@@ -188,10 +188,11 @@
             "notes": "A very inspiring and useful comment",
             "good_driving": True,
             "bad_driving": False,
-            "sketchy_climb": False,
-            "solid_climb": True,
+            "sketchy_pickup": False,
+            "sketchy_placing": True,
             "good_defense": False,
             "bad_defense": False,
+            "easily_defended": False,
         })
         exit_code, _, stderr = run_debug_cli(["-submitNotes", json_path])
         self.assertEqual(exit_code, 0, stderr)
@@ -210,10 +211,11 @@
             Notes: (string) (len=35) "A very inspiring and useful comment",
             GoodDriving: (bool) true,
             BadDriving: (bool) false,
-            SketchyClimb: (bool) false,
-            SolidClimb: (bool) true,
+            SketchyPickup: (bool) false,
+            SketchyPlacing: (bool) true,
             GoodDefense: (bool) false,
-            BadDefense: (bool) false
+            BadDefense: (bool) false,
+            EasilyDefended: (bool) false
             }"""), stdout)
 
     def test_submit_and_request_driver_ranking(self):
diff --git a/scouting/webserver/requests/messages/request_all_notes_response.fbs b/scouting/webserver/requests/messages/request_all_notes_response.fbs
index a69861b..78e662d 100644
--- a/scouting/webserver/requests/messages/request_all_notes_response.fbs
+++ b/scouting/webserver/requests/messages/request_all_notes_response.fbs
@@ -5,10 +5,11 @@
     notes:string (id: 1);
     good_driving:bool (id: 2);
     bad_driving:bool (id: 3);
-    sketchy_climb:bool (id: 4);
-    solid_climb:bool (id: 5);
+    sketchy_pickup:bool (id: 4);
+    sketchy_placing:bool (id: 5);
     good_defense:bool (id: 6);
     bad_defense:bool (id: 7);
+    easily_defended:bool (id: 8);
 }
 
 table RequestAllNotesResponse {
diff --git a/scouting/webserver/requests/messages/submit_notes.fbs b/scouting/webserver/requests/messages/submit_notes.fbs
index 1498e26..27ed472 100644
--- a/scouting/webserver/requests/messages/submit_notes.fbs
+++ b/scouting/webserver/requests/messages/submit_notes.fbs
@@ -5,10 +5,11 @@
     notes:string (id: 1);
     good_driving:bool (id: 2);
     bad_driving:bool (id: 3);
-    sketchy_climb:bool (id: 4);
-    solid_climb:bool (id: 5);
+    sketchy_pickup:bool (id: 4);
+    sketchy_placing:bool (id: 5);
     good_defense:bool (id: 6);
     bad_defense:bool (id: 7);
+    easily_defended:bool (id: 8);
 }
 
 root_type SubmitNotes;
diff --git a/scouting/webserver/requests/requests.go b/scouting/webserver/requests/requests.go
index bf24140..467542a 100644
--- a/scouting/webserver/requests/requests.go
+++ b/scouting/webserver/requests/requests.go
@@ -433,14 +433,15 @@
 	}
 
 	err = handler.db.AddNotes(db.NotesData{
-		TeamNumber:   request.Team(),
-		Notes:        string(request.Notes()),
-		GoodDriving:  bool(request.GoodDriving()),
-		BadDriving:   bool(request.BadDriving()),
-		SketchyClimb: bool(request.SketchyClimb()),
-		SolidClimb:   bool(request.SolidClimb()),
-		GoodDefense:  bool(request.GoodDefense()),
-		BadDefense:   bool(request.BadDefense()),
+		TeamNumber:     request.Team(),
+		Notes:          string(request.Notes()),
+		GoodDriving:    bool(request.GoodDriving()),
+		BadDriving:     bool(request.BadDriving()),
+		SketchyPickup:  bool(request.SketchyPickup()),
+		SketchyPlacing: bool(request.SketchyPlacing()),
+		GoodDefense:    bool(request.GoodDefense()),
+		BadDefense:     bool(request.BadDefense()),
+		EasilyDefended: bool(request.EasilyDefended()),
 	})
 	if err != nil {
 		respondWithError(w, http.StatusInternalServerError, fmt.Sprintf("Failed to insert notes: %v", err))
@@ -686,14 +687,15 @@
 	var response RequestAllNotesResponseT
 	for _, note := range notes {
 		response.NoteList = append(response.NoteList, &request_all_notes_response.NoteT{
-			Team:         note.TeamNumber,
-			Notes:        note.Notes,
-			GoodDriving:  note.GoodDriving,
-			BadDriving:   note.BadDriving,
-			SketchyClimb: note.SketchyClimb,
-			SolidClimb:   note.SolidClimb,
-			GoodDefense:  note.GoodDefense,
-			BadDefense:   note.BadDefense,
+			Team:           note.TeamNumber,
+			Notes:          note.Notes,
+			GoodDriving:    note.GoodDriving,
+			BadDriving:     note.BadDriving,
+			SketchyPickup:  note.SketchyPickup,
+			SketchyPlacing: note.SketchyPlacing,
+			GoodDefense:    note.GoodDefense,
+			BadDefense:     note.BadDefense,
+			EasilyDefended: note.EasilyDefended,
 		})
 	}
 
diff --git a/scouting/webserver/requests/requests_test.go b/scouting/webserver/requests/requests_test.go
index 6df15de..efd770b 100644
--- a/scouting/webserver/requests/requests_test.go
+++ b/scouting/webserver/requests/requests_test.go
@@ -400,14 +400,15 @@
 
 	builder := flatbuffers.NewBuilder(1024)
 	builder.Finish((&submit_notes.SubmitNotesT{
-		Team:         971,
-		Notes:        "Notes",
-		GoodDriving:  true,
-		BadDriving:   false,
-		SketchyClimb: true,
-		SolidClimb:   false,
-		GoodDefense:  true,
-		BadDefense:   false,
+		Team:           971,
+		Notes:          "Notes",
+		GoodDriving:    true,
+		BadDriving:     false,
+		SketchyPickup:  true,
+		SketchyPlacing: false,
+		GoodDefense:    true,
+		BadDefense:     false,
+		EasilyDefended: true,
 	}).Pack(builder))
 
 	_, err := debug.SubmitNotes("http://localhost:8080", builder.FinishedBytes())
@@ -417,14 +418,15 @@
 
 	expected := []db.NotesData{
 		{
-			TeamNumber:   971,
-			Notes:        "Notes",
-			GoodDriving:  true,
-			BadDriving:   false,
-			SketchyClimb: true,
-			SolidClimb:   false,
-			GoodDefense:  true,
-			BadDefense:   false,
+			TeamNumber:     971,
+			Notes:          "Notes",
+			GoodDriving:    true,
+			BadDriving:     false,
+			SketchyPickup:  true,
+			SketchyPlacing: false,
+			GoodDefense:    true,
+			BadDefense:     false,
+			EasilyDefended: true,
 		},
 	}
 
@@ -436,14 +438,15 @@
 func TestRequestNotes(t *testing.T) {
 	database := MockDatabase{
 		notes: []db.NotesData{{
-			TeamNumber:   971,
-			Notes:        "Notes",
-			GoodDriving:  true,
-			BadDriving:   false,
-			SketchyClimb: true,
-			SolidClimb:   false,
-			GoodDefense:  true,
-			BadDefense:   false,
+			TeamNumber:     971,
+			Notes:          "Notes",
+			GoodDriving:    true,
+			BadDriving:     false,
+			SketchyPickup:  true,
+			SketchyPlacing: false,
+			GoodDefense:    true,
+			BadDefense:     false,
+			EasilyDefended: true,
 		}},
 	}
 	scoutingServer := server.NewScoutingServer()
@@ -663,24 +666,26 @@
 	db := MockDatabase{
 		notes: []db.NotesData{
 			{
-				TeamNumber:   971,
-				Notes:        "Notes",
-				GoodDriving:  true,
-				BadDriving:   false,
-				SketchyClimb: true,
-				SolidClimb:   false,
-				GoodDefense:  true,
-				BadDefense:   false,
+				TeamNumber:     971,
+				Notes:          "Notes",
+				GoodDriving:    true,
+				BadDriving:     false,
+				SketchyPickup:  true,
+				SketchyPlacing: false,
+				GoodDefense:    true,
+				BadDefense:     false,
+				EasilyDefended: false,
 			},
 			{
-				TeamNumber:   972,
-				Notes:        "More Notes",
-				GoodDriving:  false,
-				BadDriving:   false,
-				SketchyClimb: false,
-				SolidClimb:   true,
-				GoodDefense:  false,
-				BadDefense:   true,
+				TeamNumber:     972,
+				Notes:          "More Notes",
+				GoodDriving:    false,
+				BadDriving:     false,
+				SketchyPickup:  false,
+				SketchyPlacing: true,
+				GoodDefense:    false,
+				BadDefense:     true,
+				EasilyDefended: false,
 			},
 		},
 	}
@@ -700,24 +705,26 @@
 	expected := request_all_notes_response.RequestAllNotesResponseT{
 		NoteList: []*request_all_notes_response.NoteT{
 			{
-				Team:         971,
-				Notes:        "Notes",
-				GoodDriving:  true,
-				BadDriving:   false,
-				SketchyClimb: true,
-				SolidClimb:   false,
-				GoodDefense:  true,
-				BadDefense:   false,
+				Team:           971,
+				Notes:          "Notes",
+				GoodDriving:    true,
+				BadDriving:     false,
+				SketchyPickup:  true,
+				SketchyPlacing: false,
+				GoodDefense:    true,
+				BadDefense:     false,
+				EasilyDefended: false,
 			},
 			{
-				Team:         972,
-				Notes:        "More Notes",
-				GoodDriving:  false,
-				BadDriving:   false,
-				SketchyClimb: false,
-				SolidClimb:   true,
-				GoodDefense:  false,
-				BadDefense:   true,
+				Team:           972,
+				Notes:          "More Notes",
+				GoodDriving:    false,
+				BadDriving:     false,
+				SketchyPickup:  false,
+				SketchyPlacing: true,
+				GoodDefense:    false,
+				BadDefense:     true,
+				EasilyDefended: false,
 			},
 		},
 	}