commit a1381924d4a012170e65f3cfddb4d043efc889b0
author Austin Schuh <austin.linux@gmail.com> Sun Feb 26 22:27:05 2023 -0800
committer Austin Schuh <austin.linux@gmail.com> Mon Feb 27 23:11:24 2023 -0800
tree 1a157b89fc7b33d4c1de2b89b53458a5c95ceff9
parent 988a54082328545450409acdd503536deac2208f

Generate arm points into the .cc file rather than the header

This reduces rebuild times significantly when updating setpoints since
the setpoints are pulled in when linking rather than needing to be
compiled into superstructure_lib.  Saves probably 30 seconds on the
build for a negligable runtime penalty.

Change-Id: Ib73ce90201542bf51ebb1c062d269269ae3d8a5a
Signed-off-by: Austin Schuh <austin.linux@gmail.com>

y2023/control_loops/python/graph_codegen.py

diff --git a/y2023/control_loops/python/graph_codegen.py b/y2023/control_loops/python/graph_codegen.py
index 632f112..6f3bc0d 100644
--- a/y2023/control_loops/python/graph_codegen.py
+++ b/y2023/control_loops/python/graph_codegen.py
@@ -152,11 +152,12 @@
         h_file.append("")
         h_file.append("constexpr uint32_t %s() { return %d; }" %
                       (index_function_name(key), index))
-        h_file.append("inline ::Eigen::Matrix<double, 3, 1> %sPoint() {" % key)
-        h_file.append(
+        h_file.append("::Eigen::Matrix<double, 3, 1> %sPoint();" % key)
+        cc_file.append("::Eigen::Matrix<double, 3, 1> %sPoint() {" % key)
+        cc_file.append(
             "  return (::Eigen::Matrix<double, 3, 1>() << %f, %f, %f).finished();"
             % (point[0], point[1], point[2]))
-        h_file.append("}")
+        cc_file.append("}")
 
     front_points = [
         index_function_name(point[1]) + "()"