fix: Build break in onnx_extension

Also deprecate onnx.use_unstructured_input in favour of the INPUT_SERIALIZATION configuration parameter.

rlclientlib/extensions/onnx/include/onnx_extension.h

@@ -1,5 +1,7 @@
 #pragma once
 
+#include "future_compat.h"
+
 namespace reinforcement_learning
 {
 namespace onnx
@@ -17,9 +19,8 @@ namespace name
 // TODO: Explore and expose useful configuration settings here
 const char* const ONNX_OUTPUT_NAME = "onnx.output_name";
 
-const char* const TENSOR_NOTATION_INPUT_SERIALIZATION = "ONNX/TENSOR_NOTATION";
-
 // Unstructured input === TENSOR_NOTATION_INPUT_SERIALIZATION
+RL_DEPRECATED("Use the INPUT_SERIALIZATION configuration mechanism with the TENSOR_NOTATION value instead")
 const char* const ONNX_USE_UNSTRUCTURED_INPUT = "onnx.use_unstructured_input";
 
 }  // namespace name
@@ -30,5 +31,7 @@ namespace reinforcement_learning
 namespace value
 {
 const char* const ONNXRUNTIME_MODEL = "ONNXRUNTIME";
+
+const char* const TENSOR_NOTATION_INPUT_SERIALIZATION = "ONNX/TENSOR_NOTATION";
 }
 }  // namespace reinforcement_learning

rlclientlib/extensions/onnx/src/onnx_extension.cc

@@ -5,6 +5,7 @@
 #include "constants.h"
 #include "err_constants.h"
 #include "factory_resolver.h"
+#include "future_compat.h"
 #include "model_mgmt.h"
 #include "onnx_model.h"
 
@@ -26,7 +27,10 @@ int create_onnx_model(
         << "Output name is not provided in the configuration.";
   }
 
+  RL_IGNORE_DEPRECATED_USAGE_START()
   bool use_unstructured_input = config.get_bool(name::ONNX_USE_UNSTRUCTURED_INPUT, false);
+  RL_IGNORE_DEPRECATED_USAGE_END()
+
   const char* input_serialization = config.get(name::INPUT_SERIALIZATION, value::TENSOR_NOTATION_INPUT_SERIALIZATION);
 
   // TODO: deprecate "USE_UNSUTRUCTURED_INPUT" in favour of "INPUT_SERIALIZATION" types