Refactor game piece position code & latch in target selector

Move the conversions to where they belong in the superstructure code,
and make the target selector latch the game piece position while it
has a target.

Change-Id: I4cd96fad9f327a241a146024ba38bcf34dfc8564
Signed-off-by: James Kuszmaul <jabukuszmaul+collab@gmail.com>
diff --git a/y2023/constants/simulated_constants_sender.cc b/y2023/constants/simulated_constants_sender.cc
index f18c3c1..8bfc17d 100644
--- a/y2023/constants/simulated_constants_sender.cc
+++ b/y2023/constants/simulated_constants_sender.cc
@@ -5,7 +5,7 @@
 #include "frc971/constants/constants_sender_lib.h"
 
 namespace y2023 {
-void SendSimulationConstants(aos::SimulatedEventLoopFactory *factory, int team,
+bool SendSimulationConstants(aos::SimulatedEventLoopFactory *factory, int team,
                              std::string constants_path) {
   for (const aos::Node *node : factory->nodes()) {
     std::unique_ptr<aos::EventLoop> event_loop =
@@ -13,5 +13,6 @@
     frc971::constants::ConstantSender<Constants, ConstantsList> sender(
         event_loop.get(), constants_path, team, "/constants");
   }
+  return true;
 }
 }  // namespace y2023