Debug logs

baranovmv · Mar 28, 2024 · 7b49f3a · 7b49f3a
1 parent fddcaa2
commit 7b49f3a
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/src/internal_modules/roc_audio/freq_estimator.cpp b/src/internal_modules/roc_audio/freq_estimator.cpp
@@ -10,6 +10,7 @@
 #include "roc_core/log.h"
 #include "roc_core/panic.h"
 #include "roc_core/time.h"
+#include <fstream>
 
 namespace roc {
 namespace audio {
@@ -107,9 +108,18 @@ float FreqEstimator::freq_coeff() const {
 }
 
 void FreqEstimator::update(packet::stream_timestamp_t current) {
+    static std::ofstream fout("/tmp/fe.log", std::ios::out);
+
     double filtered;
 
     if (run_decimators_(current, filtered)) {
+        fout << core::timestamp(core::ClockUnix)
+            << ", " << filtered
+            << ", " << target_
+            << ", " << (filtered - target_) * config_.P
+            << ", " << accum_ * config_.I
+            << std::endl;
+        fout.flush();
         coeff_ = run_controller_(filtered);
     }
 }
@@ -214,5 +224,9 @@ static const char* fe_profile_to_str(FreqEstimatorProfile profile) {
     return "<invalid>";
 }
 
+// TODO:
+// * overshoot -- почему аккумулятор должен быть равен площади ошибки?
+// * stable точно норм определяется аккумулятором?
+
 } // namespace audio
 } // namespace roc
diff --git a/src/internal_modules/roc_rtp/link_meter.cpp b/src/internal_modules/roc_rtp/link_meter.cpp
@@ -161,8 +161,8 @@ void LinkMeter::update_jitter_(const packet::Packet& packet) {
         metrics_.min_jitter = (core::nanoseconds_t)packet_jitter_stats_.mov_min();
         metrics_.jitter = mean_jitter();
 
-        fout << packet.rtp()->stream_timestamp
-             << ", " << packet.udp()->enqueue_ts
+        fout << packet.udp()->enqueue_ts
+             << ", " << packet.rtp()->stream_timestamp
              << ", " << (double)std::abs(d_enq_ns - d_s_ns) / core::Millisecond
              << ", " << packet_jitter_stats_.mov_max()
              << ", " << packet_jitter_stats_.mov_min()