Change clamp function to accept double parameters for improved precision

Author: barnasm1

src/core/include/openvino/core/preprocess/preprocess_steps.hpp

@@ -36,7 +36,7 @@ class OPENVINO_API PreProcessSteps final {
     /// \brief Default destructor
     ~PreProcessSteps();
 
-    PreProcessSteps& clamp(float min_value, float max_value);
+    PreProcessSteps& clamp(double min_value, double max_value);
 
     /// \brief Add convert element type preprocess operation
     ///

src/core/src/preprocess/pre_post_process.cpp

@@ -8,9 +8,8 @@
 #include "function_guard.hpp"
 #include "layout_utils.hpp"
 #include "openvino/core/model.hpp"
+#include "openvino/opsets/opset1.hpp"
 #include "preprocess_impls.hpp"
-#include "openvino/opsets/opset1.hpp" 
-#include "preprocess_steps.hpp"
 
 namespace ov {
 namespace preprocess {
@@ -269,7 +268,7 @@ InputTensorInfo& InputTensorInfo::set_from(const ov::Tensor& runtime_tensor) {
 PreProcessSteps::PreProcessSteps() : m_impl(std::unique_ptr<PreProcessStepsImpl>(new PreProcessStepsImpl())) {}
 PreProcessSteps::~PreProcessSteps() = default;
 
-PreProcessSteps& PreProcessSteps::clamp(float min_value, float max_value) {
+PreProcessSteps& PreProcessSteps::clamp(double min_value, double max_value) {
     m_impl->add_clamp(min_value, max_value);
     return *this;
 }

src/core/src/preprocess/preprocess_steps_impl.cpp

@@ -13,7 +13,6 @@
 #include "openvino/opsets/opset8.hpp"
 #include "openvino/util/common_util.hpp"
 #include "transformations/rt_info/preprocessing_attribute.hpp"
-#include "openvino/opsets/opset1.hpp" 
 
 namespace ov {
 namespace preprocess {
@@ -100,9 +99,9 @@ void PreStepsList::add_scale_impl(const std::vector<float>& values) {
         "scale " + vector_to_string(values));
 }
 
-void PreStepsList::add_clamp(float min_value, float max_value) {
+void PreStepsList::add_clamp(double min_value, double max_value) {
     std::string name = "clamp(min " + std::to_string(min_value) + ", max " + std::to_string(max_value) + ")";
-    
+
     m_actions.emplace_back(
         [min_value, max_value](const std::vector<Output<Node>>& nodes,
                                const std::shared_ptr<Model>& function,
@@ -111,10 +110,11 @@ void PreStepsList::add_clamp(float min_value, float max_value) {
                             "Can't apply clamp to multi-plane input. Suggesting to convert current image to "
                             "RGB/BGR color format using 'PreProcessSteps::convert_color'");
 
-            const auto& node = nodes[0];
+            const auto& node = nodes.front();
             auto element_type = node.get_element_type();
             OPENVINO_ASSERT(element_type.is_real(),
-                            "Clamp preprocessing can be applied to 'float' inputs. Consider using 'convert_element_type' before clamping. Current type is: ",
+                            "Clamp preprocessing can be applied to 'double' inputs. Consider using "
+                            "'convert_element_type' before clamping. Current type is: ",
                             element_type);
 
             auto clamp_op = std::make_shared<ov::op::v0::Clamp>(node, min_value, max_value);
@@ -123,7 +123,6 @@ void PreStepsList::add_clamp(float min_value, float max_value) {
         name);
 }
 
-
 void PreStepsList::add_mean_impl(const std::vector<float>& values) {
     m_actions.emplace_back(
         [values](const std::vector<Output<Node>>& nodes,

src/core/src/preprocess/preprocess_steps_impl.hpp

@@ -157,7 +157,7 @@ struct InternalPreprocessAction {
 class PreStepsList {
 public:
     void add_scale_impl(const std::vector<float>& values);
-    void add_clamp(float min_value, float max_value);
+    void add_clamp(double min_value, double max_value);
     void add_mean_impl(const std::vector<float>& values);
     void add_pad_impl(const std::vector<int>& pads_begin,
                       const std::vector<int>& pads_end,