From 5114e97eb6201f381ded886b2d360524b5189a75 Mon Sep 17 00:00:00 2001 From: Stanislaw Halik Date: Thu, 28 Mar 2019 08:37:28 +0100 Subject: logic/pipeline: kill clamp_value() --- logic/pipeline.cpp | 20 +------------------- logic/pipeline.hpp | 1 - 2 files changed, 1 insertion(+), 20 deletions(-) diff --git a/logic/pipeline.cpp b/logic/pipeline.cpp index 130b7eff..2b3d9313 100644 --- a/logic/pipeline.cpp +++ b/logic/pipeline.cpp @@ -287,22 +287,6 @@ void pipeline::maybe_set_center_pose(const Pose& value, bool own_center_logic) } } -Pose pipeline::clamp_value(Pose value) const -{ - // hatire, udp, and freepie trackers can mess up here - for (unsigned i = 3; i < 6; i++) - { - value(i) = std::fmod(value(i), 360); - - const double x = value(i); - if (std::fabs(x) - 1e-2 > 180) - value(i) = std::fmod(x + std::copysign(180, x), 360) - std::copysign(180, x); - value(i) = clamp(x, -180, 180); - } - - return value; -} - Pose pipeline::apply_center(Pose value) const { // this is incorrect but people like it @@ -406,12 +390,10 @@ void pipeline::logic() nan_check(newpose, raw, value); - value = clamp_value(value); - { maybe_enable_center_on_tracking_started(); maybe_set_center_pose(value, own_center_logic); - value = clamp_value(apply_center(value)); + value = apply_center(value); // "corrected" - after various transformations to account for camera position logger.write_pose(value); diff --git a/logic/pipeline.hpp b/logic/pipeline.hpp index fa32ec1e..28fe20f5 100644 --- a/logic/pipeline.hpp +++ b/logic/pipeline.hpp @@ -108,7 +108,6 @@ class OTR_LOGIC_EXPORT pipeline : private QThread void run() override; bool maybe_enable_center_on_tracking_started(); void maybe_set_center_pose(const Pose& value, bool own_center_logic); - Pose clamp_value(Pose value) const; Pose apply_center(Pose value) const; std::tuple get_selected_axis_values(const Pose& newpose) const; Pose maybe_apply_filter(const Pose& value) const; -- cgit v1.2.3